drm/i915/guc: make 'guc_hw_reg_state' static as it isn't exported

bjdooks-ct · jnikula · commit 81d0223832f9 · 2026-01-12T13:10:36.000+02:00
The guc_hw_reg_state array is not exported, so make it static. Fixes the following sparse warning: drivers/gpu/drm/i915/i915_gpu_error.c:692:3: warning: symbol 'guc_hw_reg_state' was not declared. Should it be static? Fixes: ba391a1 ("drm/i915/guc: Include the GuC registers in the error state") Signed-off-by: Ben Dooks <ben.dooks@codethink.co.uk> Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com> Link: https://patch.msgid.link/20260108201202.59250-2-rodrigo.vivi@intel.com Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com> (cherry picked from commit 701c474) Signed-off-by: Jani Nikula <jani.nikula@intel.com>
diff --git a/drivers/gpu/drm/i915/i915_gpu_error.c b/drivers/gpu/drm/i915/i915_gpu_error.c
@@ -686,7 +686,7 @@ static void err_print_guc_ctb(struct drm_i915_error_state_buf *m,
 }
 
 /* This list includes registers that are useful in debugging GuC hangs. */
-const struct {
+static const struct {
 	u32 start;
 	u32 count;
 } guc_hw_reg_state[] = {

Original file line number	Diff line number	Diff line change
`@@ -686,7 +686,7 @@ static void err_print_guc_ctb(struct drm_i915_error_state_buf *m,`
`686`	`686`	`}`
`687`	`687`
`688`	`688`	`/* This list includes registers that are useful in debugging GuC hangs. */`
`689`		`-const struct {`
	`689`	`+static const struct {`
`690`	`690`	`u32 start;`
`691`	`691`	`u32 count;`
`692`	`692`	`} guc_hw_reg_state[] = {`