From: Philippe Cornu Date: Thu, 25 Jan 2018 10:38:00 +0000 (+0100) Subject: drm/bridge/synopsys: dsi: Fix dsi_host_transfer() return value X-Git-Tag: v4.19~298^2~45^2~507 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=1a1c130cca78c77209cd795282d93297640fb63e;p=platform%2Fkernel%2Flinux-rpi3.git drm/bridge/synopsys: dsi: Fix dsi_host_transfer() return value The dw_mipi_dsi_host_transfer() must return the number of bytes transmitted/received on success instead of 0. Note: As the read feature is not implemented, only the transmitted number of bytes is returned for the moment. Reviewed-by: Brian Norris Signed-off-by: Philippe Cornu Signed-off-by: Archit Taneja Link: https://patchwork.freedesktop.org/patch/msgid/20180125103800.1999-3-philippe.cornu@st.com --- diff --git a/drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c b/drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c index 37d13cca..d098e232 100644 --- a/drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c +++ b/drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c @@ -412,7 +412,16 @@ static ssize_t dw_mipi_dsi_host_transfer(struct mipi_dsi_host *host, dw_mipi_message_config(dsi, msg); - return dw_mipi_dsi_write(dsi, &packet); + ret = dw_mipi_dsi_write(dsi, &packet); + if (ret) + return ret; + + /* + * TODO Only transmitted size is returned as actual driver does + * not support dcs/generic reads. Please update return value when + * delivering the read feature. + */ + return packet.size; } static const struct mipi_dsi_host_ops dw_mipi_dsi_host_ops = {