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

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

On Mon, Jul 07, 2025 at 05:34:52PM -0400, Rodrigo Vivi wrote:
> On Mon, Jul 07, 2025 at 01:19:59PM -0700, Matt Atwood wrote:
> > Panther Lake has proven to be stable through testing and use.
> > 
> > Remove the force_probe requirement and enable the platform by default.
> > 
> > Signed-off-by: Matt Atwood <matthew.s.atwood@intel.com>
> > ---
> > Panther Lake is running internal CI for the moment, but running upstream
> > branches. Tests show all greens and work in progress to get the machines
> > approved for moving to the upstream CI.
> 
> I was happy with the internal CI views that I saw last Friday.
> And I was almost acking this patch to move it through, but it
> looks we got some regression there that is blocking fw loading.
> So, let's keep the force_probe protection in place while the issue
> is investigated and getting fixed.

I take it back. Yesterday view was not a bad, but just a delayed view
with pending runs. All the runs completed and we have a green solid view.

I'm just pushed this patch to drm-xe-next. Let's continuously monitor
this green view and get these machines to the public CI asap.

Thanks,
Rodrigo.

> 
> > 
> >  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 e9c5d3150efc..ffd6ad569b7c 100644
> > --- a/drivers/gpu/drm/xe/xe_pci.c
> > +++ b/drivers/gpu/drm/xe/xe_pci.c
> > @@ -341,7 +341,6 @@ static const struct xe_device_desc ptl_desc = {
> >  	.has_display = true,
> >  	.has_sriov = true,
> >  	.max_gt_per_tile = 2,
> > -	.require_force_probe = true,
> >  	.needs_scratch = true,
> >  };
> >  
> > -- 
> > 2.49.0
> >