From d65c12ca5a124754b87818847a8199444654e629 Mon Sep 17 00:00:00 2001 From: Veera Sundaram Sankaran Date: Wed, 10 Aug 2022 11:33:26 -0700 Subject: [PATCH] disp: msm: sde: add check to avoid NULL WB output fb Change the debug message to error during the writeback encoder validate for wb output buffer. The output buffer can be NULL only during disable frame and all other frames need to have a valid output buffer. Change-Id: I4d6fecfeaf863e56fe25e17ab1200849003b3309 Signed-off-by: Veera Sundaram Sankaran --- msm/sde/sde_encoder_phys_wb.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/msm/sde/sde_encoder_phys_wb.c b/msm/sde/sde_encoder_phys_wb.c index 44e713a5eb..1a1807334c 100644 --- a/msm/sde/sde_encoder_phys_wb.c +++ b/msm/sde/sde_encoder_phys_wb.c @@ -1024,8 +1024,8 @@ static int sde_encoder_phys_wb_atomic_check(struct sde_encoder_phys *phys_enc, /* bypass check if commit with no framebuffer */ fb = sde_wb_connector_state_get_output_fb(conn_state); if (!fb) { - SDE_DEBUG("[enc:%d wb:%d] no out fb\n", DRMID(phys_enc->parent), WBID(wb_enc)); - return 0; + SDE_ERROR("[enc:%d wb:%d] no out fb\n", DRMID(phys_enc->parent), WBID(wb_enc)); + return -EINVAL; } fmt = sde_get_sde_format_ext(fb->format->format, fb->modifier);