diff --git a/msm/dp/dp_display.c b/msm/dp/dp_display.c index 3fef95b5..4a83a2c3 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; } @@ -2371,10 +2376,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; }