Commit 81a1d1f5 authored by Gustavo Sousa's avatar Gustavo Sousa Committed by Matt Roper
Browse files

drm/i915/dmc_wl: Show description string for enable_dmc_wl 



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: default avatarGustavo Sousa <gustavo.sousa@intel.com>
Reviewed-by: default avatarDnyaneshwar Bhadane <dnyaneshwar.bhadane@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20241219221429.109668-3-gustavo.sousa@intel.com


Signed-off-by: default avatarMatt Roper <matthew.d.roper@intel.com>
parent 47ad9092

drivers/gpu/drm/i915/display/intel_dmc_wl.c

+16 −2
Original line number Diff line number Diff line
@@ -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)