Re: [PATCH] drm/xe/lnl: Drop force_probe requirement

["Thomas Hellström" <thomas.hellstrom@linux.intel.com>]

On Fri, 2024-08-23 at 10:30 -0400, Rodrigo Vivi wrote:
> On Thu, Aug 22, 2024 at 03:46:15PM -0700, Lucas De Marchi wrote:
> > Lunar Lake has been usable for a while in a desktop setup. Bugs are
> > sporadically showing up in CI, but being promptly fixed. Nothing
> > very
> > concerning.
> > 
> > All the uapi changes related to fundamental platform usage have
> > been
> > finalized.
> > 
> > Remove the force_probe requirement and enable the platform by
> > default.
> > 
> > Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com>
> > Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
> > Cc: Jani Nikula <jani.nikula@intel.com>
> > Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
> 
> Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>

Reviewed-by: Thomas Hellström <thomas.hellstrom@linux.intel.com>

> 
> > ---
> > 
> > Last concerning bug[1] should now be fixed since [2] is merged.
> > Mesa CI
> > was hitting it frequently, which was badly impacting their
> > execution.
> > 
> > [1] https://gitlab.freedesktop.org/drm/xe/kernel/-/issues/1705
> > [2]
> > https://lore.kernel.org/intel-xe/20240821193842.352557-1-rodrigo.vivi@intel.com/T/#t
> > 
> > We should probably wait a few more days for this forward-looking
> > patch
> > before merging it though.
> > 
> >  drivers/gpu/drm/xe/xe_pci.c | 1 -
> >  1 file changed, 1 deletion(-)
> > 
> > diff --git a/drivers/gpu/drm/xe/xe_pci.c
> > b/drivers/gpu/drm/xe/xe_pci.c
> > index 0c2342988650..fa57996f9c37 100644
> > --- a/drivers/gpu/drm/xe/xe_pci.c
> > +++ b/drivers/gpu/drm/xe/xe_pci.c
> > @@ -342,7 +342,6 @@ static const struct xe_device_desc mtl_desc = {
> >  static const struct xe_device_desc lnl_desc = {
> >  	PLATFORM(LUNARLAKE),
> >  	.has_display = true,
> > -	.require_force_probe = true,
> >  };
> >  
> >  static const struct xe_device_desc bmg_desc = {
> > -- 
> > 2.43.0
> >