We already provide the value resulting from sanitization of
enable_dmc_wl in dmesg, however the reader will need to either have the
meanings memorized or look them up in the parameter's documentation.
Let's make things easier by providing a short human-readable name for
the parameter in dmesg.

Signed-off-by: Gustavo Sousa <gustavo.so...@intel.com>
---
 drivers/gpu/drm/i915/display/intel_dmc_wl.c | 18 ++++++++++++++++--
 1 file changed, 16 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/i915/display/intel_dmc_wl.c 
b/drivers/gpu/drm/i915/display/intel_dmc_wl.c
index cff841521ca0..2315c6318b51 100644
--- a/drivers/gpu/drm/i915/display/intel_dmc_wl.c
+++ b/drivers/gpu/drm/i915/display/intel_dmc_wl.c
@@ -279,6 +279,8 @@ static bool __intel_dmc_wl_supported(struct intel_display 
*display)
 
 static void intel_dmc_wl_sanitize_param(struct intel_display *display)
 {
+       const char *desc;
+
        if (!HAS_DMC_WAKELOCK(display)) {
                display->params.enable_dmc_wl = ENABLE_DMC_WL_DISABLED;
        } else if (display->params.enable_dmc_wl < 0) {
@@ -294,8 +296,20 @@ static void intel_dmc_wl_sanitize_param(struct 
intel_display *display)
                    display->params.enable_dmc_wl < 0 ||
                    display->params.enable_dmc_wl >= ENABLE_DMC_WL_MAX);
 
-       drm_dbg_kms(display->drm, "Sanitized enable_dmc_wl value: %d\n",
-                   display->params.enable_dmc_wl);
+       switch (display->params.enable_dmc_wl) {
+       case ENABLE_DMC_WL_DISABLED:
+               desc = "disabled";
+               break;
+       case ENABLE_DMC_WL_ENABLED:
+               desc = "enabled";
+               break;
+       default:
+               desc = "unknown";
+               break;
+       }
+
+       drm_dbg_kms(display->drm, "Sanitized enable_dmc_wl value: %d (%s)\n",
+                   display->params.enable_dmc_wl, desc);
 }
 
 void intel_dmc_wl_init(struct intel_display *display)
-- 
2.47.1

Reply via email to