On Thu, Jan 25, 2018 at 11:37:59AM +0100, Philippe Cornu wrote:
> The dcs/generic dsi read feature is not yet implemented so it
> is important to warn the host_transfer() caller in case of
> read operation requests.
> 
> Signed-off-by: Philippe Cornu <philippe.co...@st.com>

Awesome, thanks.

Reviewed-by: Brian Norris <briannor...@chromium.org>

> ---
>  drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c 
> b/drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c
> index daec7881be6d..72ecaeb40822 100644
> --- a/drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c
> +++ b/drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c
> @@ -405,6 +405,12 @@ static ssize_t dw_mipi_dsi_host_transfer(struct 
> mipi_dsi_host *host,
>       struct mipi_dsi_packet packet;
>       int ret;
>  
> +     if (msg->rx_buf || msg->rx_len) {
> +             /* TODO dw drv improvements: implement read feature */
> +             dev_warn(dsi->dev, "read operations not yet implemented\n");
> +             return -EINVAL;
> +     }
> +
>       ret = mipi_dsi_create_packet(&packet, msg);
>       if (ret) {
>               dev_err(dsi->dev, "failed to create packet: %d\n", ret);
> -- 
> 2.15.1
> 
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to