[Intel-gfx] [PATCH v3 1/2] drm/i915: Use of BARs names instead of numbers

Jani Nikula jani.nikula at intel.com
Wed Aug 10 11:31:22 UTC 2022


On Wed, 10 Aug 2022, Andrzej Hajda <andrzej.hajda at intel.com> wrote:
> On 05.08.2022 17:59, Piorkowski, Piotr wrote:
>> +/* PCI BARs */
>> +#define GTTMMADR_BAR				0
>> +#define GEN2_GTTMMADR_BAR			1
>> +#define GFXMEM_BAR				2
>> +#define GTT_APERTURE_BAR			GFXMEM_BAR
>> +#define GEN12_LMEM_BAR				GFXMEM_BAR
>
> In INTEL_GVT_PCI_BAR_GTTMMIO we have BAR in prefix, here in suffix.
> I am not sure which onel is better, just pointing out slight incosistency.

Thought about it too, but decided this one's trivial to change if
needed. The main change is switching to macros.

> Anyway:
> Reviewed-by: Andrzej Hajda <andrzej.hajda at intel.com>

Thanks, though I already pushed this. :)

BR,
Jani.

-- 
Jani Nikula, Intel Open Source Graphics Center


More information about the dri-devel mailing list