[PATCH 32/36] drm/mcde: use bpp instead of cpp for drm_format_info

Sandy Huang hjc at rock-chips.com
Mon Sep 23 12:53:46 UTC 2019


cpp[BytePerPlane] can't describe the 10bit data format correctly,
So we use bpp[BitPerPlane] to instead cpp.

Signed-off-by: Sandy Huang <hjc at rock-chips.com>
---
 drivers/gpu/drm/mcde/mcde_display.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/mcde/mcde_display.c b/drivers/gpu/drm/mcde/mcde_display.c
index 751454a..da2a3db 100644
--- a/drivers/gpu/drm/mcde/mcde_display.c
+++ b/drivers/gpu/drm/mcde/mcde_display.c
@@ -168,7 +168,7 @@ static int mcde_display_check(struct drm_simple_display_pipe *pipe,
 		 * There's no pitch register, the mode's hdisplay
 		 * controls this.
 		 */
-		if (fb->pitches[0] != mode->hdisplay * fb->format->cpp[0]) {
+		if (fb->pitches[0] != mode->hdisplay * fb->format->bpp[0] / 8) {
 			DRM_DEBUG_KMS("can't handle pitches\n");
 			return -EINVAL;
 		}
@@ -823,7 +823,7 @@ static void mcde_display_enable(struct drm_simple_display_pipe *pipe,
 	u32 formatter_ppl = mode->hdisplay; /* pixels per line */
 	u32 formatter_lpf = mode->vdisplay; /* lines per frame */
 	int pkt_size, fifo_wtrmrk;
-	int cpp = fb->format->cpp[0];
+	int cpp = fb->format->bpp[0] / 8;
 	int formatter_cpp;
 	struct drm_format_name_buf tmp;
 	u32 formatter_frame;
-- 
2.7.4





More information about the dri-devel mailing list