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

["Pottumuttu, Sai Teja" <sai.teja.pottumuttu@intel.com>]

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.


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),
>   };
>