Re: [PATCH i-g-t v3 1/9] PTL: lib/intel_chipset: add pantherlake definition and support

["Taylor, Clinton A" <clinton.a.taylor@intel.com>]

On Fri, 2024-11-15 at 14:16 +0530, Pottumuttu, Sai Teja wrote:
> On 15-11-2024 00:33, Clint Taylor wrote:
> > From: Matt Atwood <matthew.s.atwood@intel.com>
> > 
> > Add support for the pantherlake platform for xe.
> > 
> > Signed-off-by: Matt Atwood <matthew.s.atwood@intel.com>
> > Signed-off-by: Clint Taylor <Clinton.A.Taylor@intel.com>
> 
> I think we can squash this one and the next flatccs patch so that the 
> complete/correct support stays in a single commit.

Good idea. Will make the change in the next version.

-Clint

> 
> 
> Thanks,
> Sai Teja
> 
> > ---
> >   lib/intel_chipset.h     |  2 ++
> >   lib/intel_device_info.c | 12 ++++++++++++
> >   2 files changed, 14 insertions(+)
> > 
> > diff --git a/lib/intel_chipset.h b/lib/intel_chipset.h
> > index 0c04476bf..c93ad9f75 100644
> > --- a/lib/intel_chipset.h
> > +++ b/lib/intel_chipset.h
> > @@ -92,6 +92,8 @@ struct intel_device_info {
> >   	bool is_pontevecchio : 1;
> >   	bool is_lunarlake : 1;
> >   	bool is_battlemage : 1;
> > +	bool is_pantherlake : 1;
> > +
> >   	const struct intel_cmds_info *cmds_info;
> >   	const char *codename;
> >   };
> > diff --git a/lib/intel_device_info.c b/lib/intel_device_info.c
> > index 546b9c65a..69132e91c 100644
> > --- a/lib/intel_device_info.c
> > +++ b/lib/intel_device_info.c
> > @@ -526,6 +526,16 @@ static const struct intel_device_info intel_battlemage_info = {
> >   	.cmds_info = &xe2_cmds_info,
> >   };
> >   
> > +static const struct intel_device_info intel_pantherlake_info = {
> > +	.graphics_ver = 30,
> > +	.graphics_rel = 0,
> > +	.display_ver = 30,
> > +	.has_4tile = true,
> > +	.is_pantherlake = true,
> > +	.codename = "pantherlake",
> > +	.cmds_info = &gen12_pvc_cmds_info,
> > +};
> > +
> >   #define INTEL_PCI_ID_INIT(_id, _info) { \
> >   	.vendor_id = 0x8086, .device_id = (_id), \
> >   	.subvendor_id = PCI_MATCH_ANY, .subdevice_id = PCI_MATCH_ANY, \
> > @@ -642,6 +652,8 @@ static const struct pci_id_match intel_device_match[] = {
> >   
> >   	INTEL_BMG_IDS(INTEL_PCI_ID_INIT, &intel_battlemage_info),
> >   
> > +	INTEL_PTL_IDS(INTEL_PCI_ID_INIT, &intel_pantherlake_info),
> > +
> >   	INTEL_PCI_ID_INIT(PCI_MATCH_ANY, &intel_generic_info),
> >   };
> >