<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jul 15, 2016 at 2:48 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:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><span class="">On 13 July 2016 at 19:58, John Stultz <<a href="mailto:john.stultz@linaro.org">john.stultz@linaro.org</a>> wrote:<br>
> On Wed, Jul 13, 2016 at 9:44 AM, Vinay Simha BN <<a href="mailto:simhavcs@gmail.com">simhavcs@gmail.com</a>> wrote:<br>
>> Add support for the JDI LT070ME05000 WUXGA DSI panel used in<br>
>> Nexus 7 2013 devices.<br>
>><br>
>> Programming sequence for the panel is was originally found in the<br>
>> android-msm-flo-3.4-lollipop-release branch from:<br>
>>     <a href="https://android.googlesource.com/kernel/msm.git" rel="noreferrer" target="_blank">https://android.googlesource.com/kernel/msm.git</a><br>
>><br>
>> And video mode setting is from dsi-panel-jdi-dualmipi1-video.dtsi<br>
>> file in:<br>
>>     git://<a href="http://codeaurora.org/kernel/msm-3.10.git" rel="noreferrer" target="_blank">codeaurora.org/kernel/msm-3.10.git</a>  LNX.LA.3.6_rb1.27<br>
>><br>
>> Cc: Archit Taneja <<a href="mailto:archit.taneja@gmail.com">archit.taneja@gmail.com</a>><br>
>> Cc: Rob Clark <<a href="mailto:robdclark@gmail.com">robdclark@gmail.com</a>><br>
>> Cc: Sumit Semwal <<a href="mailto:sumit.semwal@linaro.org">sumit.semwal@linaro.org</a>><br>
>> Cc: John Stultz <<a href="mailto:john.stultz@linaro.org">john.stultz@linaro.org</a>><br>
>> Cc: Emil Velikov <<a href="mailto:emil.l.velikov@gmail.com">emil.l.velikov@gmail.com</a>><br>
>> Cc: Thierry Reding <<a href="mailto:thierry.reding@gmail.com">thierry.reding@gmail.com</a>><br>
>> Cc: David Airlie <<a href="mailto:airlied@linux.ie">airlied@linux.ie</a>><br>
>> Signed-off-by: Sumit Semwal <<a href="mailto:sumit.semwal@linaro.org">sumit.semwal@linaro.org</a>><br>
>> Signed-off-by: John Stultz <<a href="mailto:john.stultz@linaro.org">john.stultz@linaro.org</a>><br>
>> Signed-off-by: Vinay Simha BN <<a href="mailto:simhavcs@gmail.com">simhavcs@gmail.com</a>><br>
><br>
> Just fyi, I've re-integrated this patch set into my flo-WIP branch and<br>
> its working well.<br>
><br>
> I dunno if its of any use, but:<br>
> Tested-by: John Stultz <<a href="mailto:john.stultz@linaro.org">john.stultz@linaro.org</a>><br>
><br>
</span>It always is. Thank you!<br>
<br>
Vinay, thanks for the patience and I hope you grok the reason behind<br>
the requested changes.<br></blockquote><div>yes.</div><div><br class="">premature returns should be avoided. it leads to the caller failure, intermediate funcs also will not get executed (gpios, regulator disable).</div><div>In this case even the dsi fails, it should be panel(dsi write) failures rather than the bridge/interface. </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">
<br>
The patch is<br>
Reviewed-by: Emil Velikov <<a href="mailto:emil.l.velikov@gmail.com">emil.l.velikov@gmail.com</a>><br>
<br>
Regards,<br>
Emil<br>
</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>