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 <sam@xxxxxxxxxxxx> 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@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
