[PATCH 01/35] drm: Give drm_agp_clear drm_legacy_ prefix
Daniel Vetter
daniel at ffwll.ch
Wed Apr 27 06:41:23 UTC 2016
On Tue, Apr 26, 2016 at 10:09:16PM +0100, Chris Wilson wrote:
> On Tue, Apr 26, 2016 at 07:29:34PM +0200, Daniel Vetter wrote:
> > It has a DRIVER_MODESET check to sure make it's not creating havoc
> > for drm drivers. Make that clear in the name too.
>
> The pattern is that any extern that starts
>
> if (drm_core_check_feature(MODESET)) return
>
> becomes drm_legacy_ ? With the apparent caveat of KMS_LEGACY_CONTEXT.
Yeah that's what I'm aiming for. Gives you a clear signal that as soon as
you spot drm_legacy_* you don't have to bother jumping into the function
at all. We've done the same with headers, splitting things into legacy
internal headers.
> > v2: Move misplaced hunk, spotted by 0day and Thierry.
> >
> > Cc: Thierry Reding <thierry.reding at gmail.com>
> > Signed-off-by: Daniel Vetter <daniel.vetter at intel.com>
> Reviewed-by: Chris Wilson <chris at chris-wilson.co.uk>
Thanks for the review, will apply.
-Daniel
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
More information about the dri-devel
mailing list