From: Maxime Ripard <mrip...@kernel.org>

LBM allocations need a different size depending on the line length,
format, etc.

This can get tricky, and fail. Let's add some more prints to ease the
debugging when it does.

Signed-off-by: Maxime Ripard <mrip...@kernel.org>
Signed-off-by: Dave Stevenson <dave.steven...@raspberrypi.com>
---
 drivers/gpu/drm/vc4/vc4_plane.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/drivers/gpu/drm/vc4/vc4_plane.c b/drivers/gpu/drm/vc4/vc4_plane.c
index e4fcef11cf6f..16197c4b4b33 100644
--- a/drivers/gpu/drm/vc4/vc4_plane.c
+++ b/drivers/gpu/drm/vc4/vc4_plane.c
@@ -741,6 +741,7 @@ static int vc4_plane_allocate_lbm(struct drm_plane_state 
*state)
 {
        struct drm_device *drm = state->plane->dev;
        struct vc4_dev *vc4 = to_vc4_dev(drm);
+       struct drm_plane *plane = state->plane;
        struct vc4_plane_state *vc4_state = to_vc4_plane_state(state);
        unsigned long irqflags;
        u32 lbm_size;
@@ -749,6 +750,9 @@ static int vc4_plane_allocate_lbm(struct drm_plane_state 
*state)
        if (!lbm_size)
                return 0;
 
+       drm_dbg_driver(drm, "[PLANE:%d:%s] LBM Allocation Size: %u\n",
+                      plane->base.id, plane->name, lbm_size);
+
        if (WARN_ON(!vc4_state->lbm_offset))
                return -EINVAL;
 
-- 
2.34.1

Reply via email to