Hi Linus

On Wed, May 22, 2019 at 09:21:53AM +0200, Linus Walleij wrote:
> This adds a new DRM driver for the ST-Ericsson Multi Channel
> Display Engine, MCDE display controller.

This driver breaks build in drm-misc-next at the moment.

Following patch fixed both a build warning and a build error.
Could you take a look and get it fixed properly.

If relevant - then patch is:
Signed-off-by: Sam Ravnborg <s...@ravnborg.org>

        Sam

diff --git a/drivers/gpu/drm/mcde/mcde_display.c 
b/drivers/gpu/drm/mcde/mcde_display.c
index 17dc46d554b0..f3ef108a41d9 100644
--- a/drivers/gpu/drm/mcde/mcde_display.c
+++ b/drivers/gpu/drm/mcde/mcde_display.c
@@ -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 = drm_format_plane_cpp(format, 0);
+       int cpp = fb->format->cpp[0];
        int formatter_cpp;
        struct drm_format_name_buf tmp;
        u32 formatter_frame;
diff --git a/drivers/gpu/drm/mcde/mcde_dsi.c b/drivers/gpu/drm/mcde/mcde_dsi.c
index 456b43482448..07f7090d08b3 100644
--- a/drivers/gpu/drm/mcde/mcde_dsi.c
+++ b/drivers/gpu/drm/mcde/mcde_dsi.c
@@ -191,7 +191,7 @@ static ssize_t mcde_dsi_host_transfer(struct mipi_dsi_host 
*host,
        }
 
        dev_dbg(d->dev,
-               "message to channel %d, %u bytes",
+               "message to channel %d, %zd bytes",
                msg->channel,
                txlen);

        Sam
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to