From 9e8f2710ea678f7c0956f168e6236bf3a7f6281f Mon Sep 17 00:00:00 2001 From: Sandeep Gangadharaiah Date: Mon, 16 May 2022 15:40:52 -0700 Subject: [PATCH] disp: msm: dp: change display status log level Some of the display status changes such as re-enable or re-disable are not critical errors and can be ignored. This change downgrades such errors as warnings. Change-Id: I6800e534fc7fb825a472f37bb78a928e2e78d63a Signed-off-by: Sandeep Gangadharaiah --- msm/dp/dp_display.c | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/msm/dp/dp_display.c b/msm/dp/dp_display.c index 6eb0d252ca..b622f7a61c 100644 --- a/msm/dp/dp_display.c +++ b/msm/dp/dp_display.c @@ -41,6 +41,11 @@ dp_display_state_name(dp->state)); \ SDE_EVT32_EXTERNAL(dp->state); } +#define dp_display_state_warn(x) { \ + DP_WARN("%s: state (0x%x): %s\n", x, dp->state, \ + dp_display_state_name(dp->state)); \ + SDE_EVT32_EXTERNAL(dp->state); } + #define dp_display_state_log(x) { \ DP_DEBUG("%s: state (0x%x): %s\n", x, dp->state, \ dp_display_state_name(dp->state)); \ @@ -1140,7 +1145,7 @@ static int dp_display_host_ready(struct dp_display_private *dp) static void dp_display_host_unready(struct dp_display_private *dp) { if (!dp_display_state_is(DP_STATE_INITIALIZED)) { - dp_display_state_show("[not initialized]"); + dp_display_state_warn("[not initialized]"); return; } @@ -2390,10 +2395,10 @@ static int dp_display_prepare(struct dp_display *dp_display, void *panel) /* * If DP_STATE_ENABLED, there is nothing left to do. - * However, this should not happen ideally. So, log this. + * This would happen during MST flow. So, log this. */ if (dp_display_state_is(DP_STATE_ENABLED)) { - dp_display_state_show("[already enabled]"); + dp_display_state_warn("[already enabled]"); goto end; }