[Intel-gfx] [PATCH 5/5] drm/i915/guc: Capture list clean up - 4
Teres Alexis, Alan Previn
alan.previn.teres.alexis at intel.com
Tue Apr 25 19:10:20 UTC 2023
On Thu, 2023-04-06 at 15:26 -0700, John.C.Harrison at Intel.com wrote:
> From: John Harrison <John.C.Harrison at Intel.com>
>
> Don't use GEN9 as a prefix for register lists that contain all GEN8
> registers.
alan:snip
alan: This patch as a stand-along looks good, so I'll provide the RB but take note of the comment below
that should be reflected by decision on the review comments of patch #1 so this patch might
change from GEN9_foo-from-patch-1 to GEN8_foo-from-patch-1.
Reviewed-by: Alan Previn <alan.previn.teres.alexis at intel.com>
> -/* GEN9 - Global */
> +/* GEN8 - Global */
> static const struct __guc_mmio_reg_descr default_global_regs[] = {
> COMMON_BASE_GLOBAL,
> - COMMON_GEN9BASE_GLOBAL,
> - GEN9_GLOBAL,
> + COMMON_GEN8BASE_GLOBAL,
> + GEN8_GLOBAL,
alan: see patch comment about "COMMON_GLOBAL" vs "GLOBAL" confusion.
More information about the Intel-gfx
mailing list