[PATCH i-g-t 1/3] lib/i915/perf: include i915_pciid_local.h
Kamil Konieczny
kamil.konieczny at linux.intel.com
Tue Jan 9 12:37:13 UTC 2024
Hi Chaitanya,
On 2024-01-09 at 11:39:41 +0530, Chaitanya Kumar Borah wrote:
> perf.c uses PCI ID macros not found in kernel header i915_pciid.h.
> Therefore include i915_pciid_local.h instead of i915_pciid.h so that
-------------------------------------- ^^^^^^^^^^
This is somewhat misleading because _local.h will include i915_pciids.h
imho s/instead of/and/
Or other way to put it:
Therefore include i915_pciid_local.h (which will include i915_pciid.h)
so that
> we can move these macros to the local file. That way we don't lose
> them when i915_pciid.h is synced with kernel.
>
> Signed-off-by: Chaitanya Kumar Borah <chaitanya.kumar.borah at intel.com>
> Cc: Matt Roper <matthew.d.roper at intel.com>
> Cc: Juha-Pekka Heikkila <juhapekka.heikkila at gmail.com>
> Cc: Kamil Konieczny <kamil.konieczny at linux.intel.com>
> Cc: Lionel Landwerlin <lionel.g.landwerlin at intel.com>
> Cc: Niranjana Vishwanathapura <niranjana.vishwanathapura at intel.com>
>
> ---
> lib/i915/perf.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/lib/i915/perf.c b/lib/i915/perf.c
> index ddadb53b6..2006dd4ad 100644
> --- a/lib/i915/perf.c
> +++ b/lib/i915/perf.c
> @@ -37,7 +37,7 @@
>
> #include <i915_drm.h>
>
> -#include "i915_pciids.h"
^^^^^^^^^^^^^^^^^^^^^^^^^^^
Consider adding here comment like:
/* i915_pciids.h will be included by i915_pciids_local.h */
or maybe even just leave that include here.
Regards,
Kamil
> +#include "i915_pciids_local.h"
>
> #include "intel_chipset.h"
> #include "perf.h"
> --
> 2.25.1
>
More information about the igt-dev
mailing list