[Intel-gfx] [PATCH] drm/i915: Don't select BROKEN

[Chris Wilson]

Quoting Daniel Vetter (2019-11-05 20:58:25)
> On Tue, Nov 5, 2019 at 9:38 PM Chris Wilson <chris at chris-wilson.co.uk> wrote:
> >
> > Quoting Daniel Vetter (2019-11-05 19:38:29)
> > > It's broken.
> > >
> > > Reported-by: Stephen Rothwell <sfr at canb.auug.org.au>
> > > References: https://lists.freedesktop.org/archives/dri-devel/2019-November/242625.html
> > > Signed-off-by: Daniel Vetter <daniel.vetter at intel.com>
> > > ---
> > > Note: Probably best to apply this directly onto drm-next to avoid
> > > having drm-next dropped from linux-next until the next pull request.
> > > -Daniel
> > > ---
> > >  drivers/gpu/drm/i915/Kconfig.debug | 1 -
> > >  1 file changed, 1 deletion(-)
> > >
> > > diff --git a/drivers/gpu/drm/i915/Kconfig.debug b/drivers/gpu/drm/i915/Kconfig.debug
> > > index ef123eb29168..d2ba8f7e5e50 100644
> > > --- a/drivers/gpu/drm/i915/Kconfig.debug
> > > +++ b/drivers/gpu/drm/i915/Kconfig.debug
> > > @@ -44,7 +44,6 @@ config DRM_I915_DEBUG
> > >         select DRM_I915_SELFTEST
> > >         select DRM_I915_DEBUG_RUNTIME_PM
> > >         select DRM_I915_DEBUG_MMIO
> > > -       select BROKEN # for prototype uAPI
> >
> > You have to replace it with another secret bool as you cannot otherwise
> > enable CONFIG_BROKEN in .config.
> 
> Or this:
> 
> diff --git a/init/Kconfig b/init/Kconfig
> index b4daad2bac23..4dbea1b9e6bb 100644
> --- a/init/Kconfig
> +++ b/init/Kconfig
> @@ -75,6 +75,7 @@ menu "General setup"
> 
>  config BROKEN
>         bool
> +       default y
> 
>  config BROKEN_ON_SMP
>         bool
> 
> Either way it needs to be in topic/core-for-CI, not in any official
> tree. Because if you allow autobuilders to enable CONFIG_BROKEN, no
> matter how well hidden, they'll all break. You can also just revert my
> patch that Dave pushed to drm-next (to get us back into the linux-next
> club).

Fwiw, I think the revert in core-for-CI is reasonable, as that gives
devs the ability to toggle on the hidden menus, while at the same time
requiring them to have the minimal debug setup.
-Chris