[PATCH] lib: sync i915_pciids.h with kernel

[Borah, Chaitanya Kumar]

Hello,

> -----Original Message-----
> From: Kamil Konieczny <kamil.konieczny at linux.intel.com>
> Sent: Monday, December 18, 2023 9:49 PM
> To: igt-dev at lists.freedesktop.org
> Cc: Borah, Chaitanya Kumar <chaitanya.kumar.borah at intel.com>; Juha-Pekka
> Heikkila <juhapekka.heikkila at gmail.com>; Garg, Nemesa
> <nemesa.garg at intel.com>
> Subject: Re: [PATCH] lib: sync i915_pciids.h with kernel
> 
> Hi,
> 
> On 2023-12-18 at 10:14:34 +0000, Borah, Chaitanya Kumar wrote:
> > Hello Kamil,
> >
> > > -----Original Message-----
> > > From: Kamil Konieczny <kamil.konieczny at linux.intel.com>
> > > Sent: Thursday, December 14, 2023 8:06 PM
> > > To: igt-dev at lists.freedesktop.org
> > > Cc: Borah, Chaitanya Kumar <chaitanya.kumar.borah at intel.com>;
> > > Juha-Pekka Heikkila <juhapekka.heikkila at gmail.com>; Garg, Nemesa
> > > <nemesa.garg at intel.com>
> > > Subject: Re: [PATCH] lib: sync i915_pciids.h with kernel
> > >
> > > Hi,
> > >
> > > On 2023-12-14 at 04:48:27 +0000, Borah, Chaitanya Kumar wrote:
> > > > Hello JP,
> > > >
> > > > > -----Original Message-----
> > > > > From: Juha-Pekka Heikkila <juhapekka.heikkila at gmail.com>
> > > > > Sent: Wednesday, December 13, 2023 6:47 PM
> > > > > To: Borah, Chaitanya Kumar <chaitanya.kumar.borah at intel.com>;
> > > > > igt- dev at lists.freedesktop.org
> > > > > Cc: Garg, Nemesa <nemesa.garg at intel.com>
> > > > > Subject: Re: [PATCH] lib: sync i915_pciids.h with kernel
> > > > >
> > > > > This is not correct. This does not synchronize with kernel commit.
> > > > > i915_pciids.h is supposed to be copied as is from kernel.
> > > >
> > > > This has crossed my mind but there are a lot of changes[1]
> > > > unrelated to
> > > ARL-S if we just copy the file.
> > > > Therefore, it made me a bit uncomfortable. Should I go ahead anyway?
> > > >
> > > > At least for PVC device ids, we might have to take an exception.
> > > > Or may be
> > > they did not belong there in the first place.
> > > >
> > >
> > > You may add only ARL-S but then write in subject what you did as it
> > > is not syncing, for example:
> > >
> > > [PATCH] lib/i915_pciids: Add ARL-S
> > >
> > > and describe why you added only that. Write also which commit (hash
> > > and
> > > subject) you used (working link could also help).
> > >
> > > One more thing: why not adding all without removing PVC ids?
> > >
> >
> > Actually, I will go ahead and do that. I will post a patch soon.
> >

+Matt, Lionel

Spoke to soon! We are running into build errors for MTL PCI IDs as the kernel does not define the macros

INTEL_MTL_P_GT2_IDS
INTEL_MTL_P_GT3_IDS

I am not sure how to handle this now. Make an exception for MTL too? Or Should these changes be added back to the kernel header?


../lib/i915/perf.c: In function ‘is_mtl_gt2’:
../lib/i915/perf.c:216:3: error: implicit declaration of function ‘INTEL_MTL_M_IDS’; did you mean ‘INTEL_MTL_IDS’? [-Werror=implicit-function-declaration]
  216 |   INTEL_MTL_M_IDS(NULL),
      |   ^~~~~~~~~~~~~~~
      |   INTEL_MTL_IDS
../lib/i915/perf.c:216:3: warning: nested extern declaration of ‘INTEL_MTL_M_IDS’ [-Wnested-externs]
../lib/i915/perf.c:216:3: error: initializer element is not constant
../lib/i915/perf.c:216:3: note: (near initialization for ‘devids[0]’)
../lib/i915/perf.c:217:3: error: implicit declaration of function ‘INTEL_MTL_P_GT2_IDS’; did you mean ‘INTEL_CFL_U_GT2_IDS’? [-Werror=implicit-function-declaration]
  217 |   INTEL_MTL_P_GT2_IDS(NULL),
      |   ^~~~~~~~~~~~~~~~~~~
      |   INTEL_CFL_U_GT2_IDS
../lib/i915/perf.c:217:3: warning: nested extern declaration of ‘INTEL_MTL_P_GT2_IDS’ [-Wnested-externs]
../lib/i915/perf.c:217:3: error: initializer element is not constant
../lib/i915/perf.c:217:3: note: (near initialization for ‘devids[1]’)
../lib/i915/perf.c: In function ‘is_mtl_gt3’:
../lib/i915/perf.c:234:3: error: implicit declaration of function ‘INTEL_MTL_P_GT3_IDS’; did you mean ‘INTEL_WHL_U_GT3_IDS’? [-Werror=implicit-function-declaration]
  234 |   INTEL_MTL_P_GT3_IDS(NULL),
      |   ^~~~~~~~~~~~~~~~~~~
      |   INTEL_WHL_U_GT3_IDS
../lib/i915/perf.c:234:3: warning: nested extern declaration of ‘INTEL_MTL_P_GT3_IDS’ [-Wnested-externs]
../lib/i915/perf.c:234:3: error: initializer element is not constant
../lib/i915/perf.c:234:3: note: (near initialization for ‘devids[0]’)

> > Regards
> >
> > Chaitanya
> 
> Last thing, please add i-g-t in subject after PATCH, so it will be:
> 
> [PATCH i-g-t] describe here your patch
> 

Ack.

Regards

Chaitanya

> Regards,
> Kamil
> 
> [...cut...]