Messages in this thread | | | Date | Fri, 19 Dec 2014 11:10:31 -0800 | From | Steve Longerbeam <> | Subject | Re: [PATCH v2 3/7] drm_modes: add videomode_from_drm_display_mode |
| |
On 12/19/2014 03:03 AM, Philipp Zabel wrote: > > +EXPORT_SYMBOL_GPL(videomode_from_drm_display_mode); > Is it ok for drm_modes to export a function that doesn't start with > drm_ ? We could just rename this to drm_display_mode_to_videomode if > necessary. I can fix it up as I apply it, but I'd like to know which is > preferred.
Yeah, drm_display_mode_to_videomode() is probably better, makes it more clear it's part of the DRM kernel interfaces.
Steve
| |