Re: [PATCH 1/2] drm: Mark up legacy/dri1 drivers with DRM_LEGACY

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Aug 04, 2016 at 09:32:01AM +0200, David Herrmann wrote:
> Hi
> 
> On Wed, Aug 3, 2016 at 9:11 PM, Daniel Vetter <daniel.vetter@xxxxxxxx> wrote:
> > It's super confusing that new drivers need to be marked with
> > DRIVER_MODESET when really it means DRIVER_MODERN. Much better to
> > invert the meaning and rename it to something that's suitably
> > off-putting.
> >
> > Since there's over 100 places using DRIVER_MODESET we need to roll out
> > this change without a flag day.
> >
> > v2: Update docs.
> >
> > Signed-off-by: Daniel Vetter <daniel.vetter@xxxxxxxxx>
> > ---
> >  Documentation/gpu/drm-internals.rst | 9 ++++++---
> >  drivers/gpu/drm/i810/i810_drv.c     | 4 +---
> >  drivers/gpu/drm/mga/mga_drv.c       | 2 +-
> >  drivers/gpu/drm/r128/r128_drv.c     | 2 +-
> >  drivers/gpu/drm/savage/savage_drv.c | 2 +-
> >  drivers/gpu/drm/sis/sis_drv.c       | 2 +-
> >  drivers/gpu/drm/tdfx/tdfx_drv.c     | 1 +
> >  drivers/gpu/drm/via/via_drv.c       | 2 +-
> >  include/drm/drmP.h                  | 1 +
> >  9 files changed, 14 insertions(+), 11 deletions(-)
> 
> Reviewed-by: David Herrmann <dh.herrmann@xxxxxxxxx>
> 
> Also, how about this:

config DRM_LEGACY
	bool "Enable legacy drivers (DANGEROUS)"
	depends on DRM
	depends on BROKEN
	help
	  ...

Then everyone else gets depends on DRM_LEGACY. The benefit is that we
get the chance to add a useful help message and hopefully explains the
situation more cleary to devs.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux