Re: [PATCH] drm/xe/ptl: Add a new PTL PCI ID

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

On Thu, Oct 01, 2026 at 08:41:59AM -0400, Rodrigo Vivi wrote:
> On Fri, Sep 25, 2026 at 09:06:14PM -0400, Rodrigo Vivi wrote:
> > On Wed, Sep 23, 2026 at 10:11:05AM -0700, Violet Monti wrote:
> > > On Tue, Sep 22, 2026 at 06:21:21PM -0400, Rodrigo Vivi wrote:
> > > > On Tue, Sep 22, 2026 at 09:56:08AM -0700, Violet Monti wrote:
> > > > > A new PCI ID for PTL.
> > > > > 
> > > > > Bspec: 72574
> > > > > 
> > > > > Signed-off-by: Matt Atwood <matthew.s.atwood@intel.com>
> > > > > Signed-off-by: Violet Monti <violet.monti@intel.com>
> > > > 
> > > > I honestly believe it would be better and faster if you had just reviewed
> > > > the version that Matt sent yesterday instead of resending it.
> > > 
> > > The reason for my resubmission was to rerun the CI results after a KUnit
> > > failure unrelated to the PCI ID change. Matt is out for this week, and
> > > delegated this patch to me to review and handle in his absence. Since I
> > > am not the owner of the original submission, I was not able to rerun the
> > > tests using the resubmit.
> > > 
> > > Is there a preferred method for re-testing patches that have been
> > > delegated, such as situations like this?
> > 
> > 
> > Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
> > and pushed. Thanks for the patch and diligence.
> 
> Matt, please accept my apologies, but I just noticed that I pushed the patch
> with the wrong authorship.

I fixed the authorship and force-pushed the branch.

> 
> Violet, next time you are handling someone's else patch you sign it off, but
> never ever change the authorship of it if you haven't modified the content.
> 
> > 
> > > 
> > > Violet
> > > > 
> > > > > ---
> > > > >  include/drm/intel/pciids.h | 1 +
> > > > >  1 file changed, 1 insertion(+)
> > > > > 
> > > > > diff --git a/include/drm/intel/pciids.h b/include/drm/intel/pciids.h
> > > > > index dff389b56eb3..32c88a690ad8 100644
> > > > > --- a/include/drm/intel/pciids.h
> > > > > +++ b/include/drm/intel/pciids.h
> > > > > @@ -880,6 +880,7 @@
> > > > >  	MACRO__(0xB08F, ## __VA_ARGS__), \
> > > > >  	MACRO__(0xB090, ## __VA_ARGS__), \
> > > > >  	MACRO__(0xB0A0, ## __VA_ARGS__), \
> > > > > +	MACRO__(0xB0A1, ## __VA_ARGS__), \
> > > > >  	MACRO__(0xB0B0, ## __VA_ARGS__)
> > > > >  
> > > > >  /* WCL */
> > > > > -- 
> > > > > 2.43.0
> > > > >