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

[Rodrigo Vivi <rodrigo.vivi@intel.com>]

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>

> ---
> 
> 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
>