drm/i915/dmc: use has_dmc_id_fw() instead of poking dmc->dmc_info directly

This will help in follow-up changes.

Cc: Imre Deak <imre.deak@intel.com>
Reviewed-by: Imre Deak <imre.deak@intel.com>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20230301122944.1298929-2-jani.nikula@intel.com

diff --git a/drivers/gpu/drm/i915/display/intel_dmc.c b/drivers/gpu/drm/i915/display/intel_dmc.c
index ab4fded..599fb92 100644 (file)
--- a/drivers/gpu/drm/i915/display/intel_dmc.c
+++ b/drivers/gpu/drm/i915/display/intel_dmc.c
@@ -1098,12 +1098,12 @@ static int intel_dmc_debugfs_status_show(struct seq_file *m, void *unused)
        seq_printf(m, "Pipe A fw needed: %s\n",
                   str_yes_no(GRAPHICS_VER(i915) >= 12));
        seq_printf(m, "Pipe A fw loaded: %s\n",
-                  str_yes_no(dmc->dmc_info[DMC_FW_PIPEA].payload));
+                  str_yes_no(has_dmc_id_fw(i915, DMC_FW_PIPEA)));
        seq_printf(m, "Pipe B fw needed: %s\n",
                   str_yes_no(IS_ALDERLAKE_P(i915) ||
                              DISPLAY_VER(i915) >= 14));
        seq_printf(m, "Pipe B fw loaded: %s\n",
-                  str_yes_no(dmc->dmc_info[DMC_FW_PIPEB].payload));
+                  str_yes_no(has_dmc_id_fw(i915, DMC_FW_PIPEB)));
 
        if (!intel_dmc_has_payload(i915))
                goto out;