Re: [PATCH v2 3/4] drm/dsi: Implement dcs set/get display brightness

From: Emil Velikov
Date: Thu Jul 14 2016 - 17:20:53 EST


On 13 July 2016 at 17:44, Vinay Simha BN <simhavcs@xxxxxxxxx> wrote:

> +int mipi_dsi_dcs_get_display_brightness(struct mipi_dsi_device *dsi,
> + u16 *brightness)
> +{
> + ssize_t err;
> +
> + err = mipi_dsi_dcs_read(dsi, MIPI_DCS_GET_DISPLAY_BRIGHTNESS,
> + brightness, sizeof(*brightness));
> + if (err < 0) {
> + if (err == 0)
Something looks fishy here. This can never be true, can it ?

-Emil