Add missing break statement for dpu_hw_ctl_update_pending_flush_wb(). Otherwise this leads to below build warning.
drivers/gpu/drm/msm/disp/dpu1/dpu_hw_ctl.c:273:2: warning: unannotated fall-through between switch labels default: ^ drivers/gpu/drm/msm/disp/dpu1/dpu_hw_ctl.c:273:2: note: insert 'break;' to avoid fall-through default: ^ break; 1 warning generated.
Fixes: 2e0086d8c61d ("drm/msm/dpu: add changes to support writeback in hw_ctl") Reported-by: kernel test robot lkp@intel.com Signed-off-by: Abhinav Kumar quic_abhinavk@quicinc.com --- drivers/gpu/drm/msm/disp/dpu1/dpu_hw_ctl.c | 1 + 1 file changed, 1 insertion(+)
diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_ctl.c b/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_ctl.c index 254fdf06bb42..c33e7ef611a6 100644 --- a/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_ctl.c +++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_ctl.c @@ -270,6 +270,7 @@ static void dpu_hw_ctl_update_pending_flush_wb(struct dpu_hw_ctl *ctx, case WB_1: case WB_2: ctx->pending_flush_mask |= BIT(WB_IDX); + break; default: break; }
On Mon, 2 May 2022 at 08:39, Abhinav Kumar quic_abhinavk@quicinc.com wrote:
Reviewed-by: Dmitry Baryshkov dmitry.baryshkov@linaro.org
Quoting Abhinav Kumar (2022-05-01 22:39:41)
What commit is this?
Patch contents look sane
Reviewed-by: Stephen Boyd swboyd@chromium.org
dri-devel@lists.freedesktop.org