drm/rockchip: vop: fixup linebuffer mode calc error

linebuffer mode should be LB_YUV_3840X5 when width is bigger than 1280
in yuv mode.

Separate yuv and rgb case makes the scl_vop_cal_lb_mode() logic clearer.

Signed-off-by: Sandy Huang <hjc@rock-chips.com>
Signed-off-by: Heiko Stuebner <heiko@sntech.de>
Link: https://patchwork.freedesktop.org/patch/msgid/1530001004-25036-1-git-send-email-hjc@rock-chips.com

diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_vop.h b/drivers/gpu/drm/rockchip/rockchip_drm_vop.h
index 084acdd0019a8d3d873e7fd8c590ed6de91313be..fcb91041a666d606257e20319023903b9a7f7e3d 100644 (file)
--- a/drivers/gpu/drm/rockchip/rockchip_drm_vop.h
+++ b/drivers/gpu/drm/rockchip/rockchip_drm_vop.h
@@ -331,16 +331,19 @@ static inline int scl_vop_cal_lb_mode(int width, bool is_yuv)
 {
        int lb_mode;
 
-       if (width > 2560)
-               lb_mode = LB_RGB_3840X2;
-       else if (width > 1920)
-               lb_mode = LB_RGB_2560X4;
-       else if (!is_yuv)
-               lb_mode = LB_RGB_1920X5;
-       else if (width > 1280)
-               lb_mode = LB_YUV_3840X5;
-       else
-               lb_mode = LB_YUV_2560X8;
+       if (is_yuv) {
+               if (width > 1280)
+                       lb_mode = LB_YUV_3840X5;
+               else
+                       lb_mode = LB_YUV_2560X8;
+       } else {
+               if (width > 2560)
+                       lb_mode = LB_RGB_3840X2;
+               else if (width > 1920)
+                       lb_mode = LB_RGB_2560X4;
+               else
+                       lb_mode = LB_RGB_1920X5;
+       }
 
        return lb_mode;
 }