<div dir="ltr">yes, it is wrong.<div>it supposed to be </div><div><br></div><div>if(err <= 0) {</div><div>          if(err == 0)</div><div><br></div><div>will resend a modified version<br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jul 15, 2016 at 2:50 AM, Emil Velikov <span dir="ltr"><<a href="mailto:emil.l.velikov@gmail.com" target="_blank">emil.l.velikov@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 13 July 2016 at 17:44, Vinay Simha BN <<a href="mailto:simhavcs@gmail.com">simhavcs@gmail.com</a>> wrote:<br>
<br>
> +int mipi_dsi_dcs_get_display_brightness(struct mipi_dsi_device *dsi,<br>
> +                                       u16 *brightness)<br>
> +{<br>
> +       ssize_t err;<br>
> +<br>
> +       err = mipi_dsi_dcs_read(dsi, MIPI_DCS_GET_DISPLAY_BRIGHTNESS,<br>
> +                               brightness, sizeof(*brightness));<br>
> +       if (err < 0) {<br>
> +               if (err == 0)<br>
</span>Something looks fishy here. This can never be true, can it ?<br>
<span class="HOEnZb"><font color="#888888"><br>
 -Emil<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature">Regards,<br><br>Vinay Simha.B.N.<br><br></div>
</div></div></div>