Quoting Michal Wajdeczko (2020-03-30 12:33:38) > We report HuC status in debugfs using register read, but > we missed that on Gen11+ HuC uses different register. > Use correct one. > > While here, correct placement of the colon. > > Signed-off-by: Michal Wajdeczko <michal.wajdeczko@xxxxxxxxx> > Cc: Daniele Ceraolo Spurio <daniele.ceraolospurio@xxxxxxxxx> > Cc: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> > --- > drivers/gpu/drm/i915/gt/uc/intel_huc.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/i915/gt/uc/intel_huc.c b/drivers/gpu/drm/i915/gt/uc/intel_huc.c > index d6097b46600c..3fee65308474 100644 > --- a/drivers/gpu/drm/i915/gt/uc/intel_huc.c > +++ b/drivers/gpu/drm/i915/gt/uc/intel_huc.c > @@ -244,6 +244,6 @@ void intel_huc_load_status(struct intel_huc *huc, struct drm_printer *p) > intel_uc_fw_dump(&huc->fw, p); > > with_intel_runtime_pm(gt->uncore->rpm, wakeref) > - drm_printf(p, "\nHuC status 0x%08x:\n", > - intel_uncore_read(gt->uncore, HUC_STATUS2)); > + drm_printf(p, "HuC status: 0x%08x\n", > + intel_uncore_read(gt->uncore, huc->status.reg)); drivers/gpu/drm/i915/gt/uc/intel_huc.c: huc->status.reg = GEN11_HUC_KERNEL_LOAD_INFO; drivers/gpu/drm/i915/gt/uc/intel_huc.c: huc->status.reg = HUC_STATUS2; drivers/gpu/drm/i915/gt/uc/intel_huc.c: huc->status.reg, drivers/gpu/drm/i915/gt/uc/intel_huc.c: status = intel_uncore_read(gt->uncore, huc->status.reg); Reviewed-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
