On Tue, 22 Nov 2022, Lucas De Marchi <lucas.demarchi@xxxxxxxxx> wrote:
> On Tue, Nov 22, 2022 at 04:16:16PM +0200, Jani Nikula wrote:
>>The default_lists array should be in rodata.
>>
>>Fixes: dce2bd542337 ("drm/i915/guc: Add Gen9 registers for GuC error state capture.")
>>Cc: Alan Previn <alan.previn.teres.alexis@xxxxxxxxx>
>>Cc: Umesh Nerlige Ramappa <umesh.nerlige.ramappa@xxxxxxxxx>
>>Cc: Lucas De Marchi <lucas.demarchi@xxxxxxxxx>
>>Signed-off-by: Jani Nikula <jani.nikula@xxxxxxxxx>
>
>
> Reviewed-by: Lucas De Marchi <lucas.demarchi@xxxxxxxxx>
Thanks, pushed to drm-intel-gt-next.
BR,
Jani.
>
> thanks
> Lucas De Marchi
>
>>---
>> drivers/gpu/drm/i915/gt/uc/intel_guc_capture.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>>diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_capture.c b/drivers/gpu/drm/i915/gt/uc/intel_guc_capture.c
>>index 1d49a7ec0bd8..1c1b85073b4b 100644
>>--- a/drivers/gpu/drm/i915/gt/uc/intel_guc_capture.c
>>+++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_capture.c
>>@@ -170,7 +170,7 @@ static const struct __guc_mmio_reg_descr empty_regs_list[] = {
>> }
>>
>> /* List of lists */
>>-static struct __guc_mmio_reg_descr_group default_lists[] = {
>>+static const struct __guc_mmio_reg_descr_group default_lists[] = {
>> MAKE_REGLIST(default_global_regs, PF, GLOBAL, 0),
>> MAKE_REGLIST(default_rc_class_regs, PF, ENGINE_CLASS, GUC_RENDER_CLASS),
>> MAKE_REGLIST(xe_lpd_rc_inst_regs, PF, ENGINE_INSTANCE, GUC_RENDER_CLASS),
>>--
>>2.34.1
>>
--
Jani Nikula, Intel Open Source Graphics Center
