[PATCH] lib/pci: Add new and missing PCI IDs to DG2 platform

Matt Roper matthew.d.roper at intel.com
Tue Mar 26 21:03:54 UTC 2024


On Tue, Mar 26, 2024 at 07:18:43PM +0530, Ravi Kumar Vodapalli wrote:
> New PCI ID is added in Bspec and some are missing in IGT, add them
> to driver

Generally rather than modifying these headers by hand in IGT we just
copy the version of the files out of the kernel source tree and then
create a commit that specifies the specific kernel commit ID that we've
synchronized up to.  For example, see commit 1b32bcdb2 ("lib: sync
i915_pciids.h with kernel").

The Xe driver kernel patch has landed, but the i915 one is still waiting
for CI, so we should probably wait for that to land, then create a
"sync" commit here to copy the latest snapshot of these headers over.


Matt

> 
> Bspec: 44477
> Signed-off-by: Ravi Kumar Vodapalli <ravi.kumar.vodapalli at intel.com>
> ---
>  lib/i915_pciids.h |  4 +++-
>  lib/xe_pciids.h   | 10 ++++++++--
>  2 files changed, 11 insertions(+), 3 deletions(-)
> 
> diff --git a/lib/i915_pciids.h b/lib/i915_pciids.h
> index fcf1849aa..16221ce77 100644
> --- a/lib/i915_pciids.h
> +++ b/lib/i915_pciids.h
> @@ -709,7 +709,9 @@
>  	INTEL_VGA_DEVICE(0x5692, info), \
>  	INTEL_VGA_DEVICE(0x56A0, info), \
>  	INTEL_VGA_DEVICE(0x56A1, info), \
> -	INTEL_VGA_DEVICE(0x56A2, info)
> +	INTEL_VGA_DEVICE(0x56A2, info), \
> +	INTEL_VGA_DEVICE(0x56BE, info), \
> +	INTEL_VGA_DEVICE(0x56BF, info)
>  
>  #define INTEL_DG2_G11_IDS(info) \
>  	INTEL_VGA_DEVICE(0x5693, info), \
> diff --git a/lib/xe_pciids.h b/lib/xe_pciids.h
> index 29f07a00a..77aa412d2 100644
> --- a/lib/xe_pciids.h
> +++ b/lib/xe_pciids.h
> @@ -128,7 +128,9 @@
>  	MACRO__(0x5692, ## __VA_ARGS__),	\
>  	MACRO__(0x56A0, ## __VA_ARGS__),	\
>  	MACRO__(0x56A1, ## __VA_ARGS__),	\
> -	MACRO__(0x56A2, ## __VA_ARGS__)
> +	MACRO__(0x56A2, ## __VA_ARGS__),	\
> +	MACRO__(0x56BE, ## __VA_ARGS__),	\
> +	MACRO__(0x56BF, ## __VA_ARGS__)
>  
>  #define XE_DG2_G11_IDS(MACRO__, ...)		\
>  	MACRO__(0x5693, ## __VA_ARGS__),	\
> @@ -138,7 +140,11 @@
>  	MACRO__(0x56A5, ## __VA_ARGS__),	\
>  	MACRO__(0x56A6, ## __VA_ARGS__),	\
>  	MACRO__(0x56B0, ## __VA_ARGS__),	\
> -	MACRO__(0x56B1, ## __VA_ARGS__)
> +	MACRO__(0x56B1, ## __VA_ARGS__),	\
> +	MACRO__(0x56BA, ## __VA_ARGS__),	\
> +	MACRO__(0x56BB, ## __VA_ARGS__),	\
> +	MACRO__(0x56BC, ## __VA_ARGS__),	\
> +	MACRO__(0x56BD, ## __VA_ARGS__)
>  
>  #define XE_DG2_G12_IDS(MACRO__, ...)		\
>  	MACRO__(0x5696, ## __VA_ARGS__),	\
> -- 
> 2.25.1
> 

-- 
Matt Roper
Graphics Software Engineer
Linux GPU Platform Enablement
Intel Corporation


More information about the igt-dev mailing list