[PATCH v8 4/4] drm/panel: Add JDI LT070ME05000 WUXGA DSI Panel

[Vinay Simha]

On Fri, Jul 15, 2016 at 2:48 AM, Emil Velikov <emil.l.velikov at gmail.com>
wrote:

> On 13 July 2016 at 19:58, John Stultz <john.stultz at linaro.org> wrote:
> > On Wed, Jul 13, 2016 at 9:44 AM, Vinay Simha BN <simhavcs at gmail.com>
> wrote:
> >> Add support for the JDI LT070ME05000 WUXGA DSI panel used in
> >> Nexus 7 2013 devices.
> >>
> >> Programming sequence for the panel is was originally found in the
> >> android-msm-flo-3.4-lollipop-release branch from:
> >>     https://android.googlesource.com/kernel/msm.git
> >>
> >> And video mode setting is from dsi-panel-jdi-dualmipi1-video.dtsi
> >> file in:
> >>     git://codeaurora.org/kernel/msm-3.10.git  LNX.LA.3.6_rb1.27
> >>
> >> Cc: Archit Taneja <archit.taneja at gmail.com>
> >> Cc: Rob Clark <robdclark at gmail.com>
> >> Cc: Sumit Semwal <sumit.semwal at linaro.org>
> >> Cc: John Stultz <john.stultz at linaro.org>
> >> Cc: Emil Velikov <emil.l.velikov at gmail.com>
> >> Cc: Thierry Reding <thierry.reding at gmail.com>
> >> Cc: David Airlie <airlied at linux.ie>
> >> Signed-off-by: Sumit Semwal <sumit.semwal at linaro.org>
> >> Signed-off-by: John Stultz <john.stultz at linaro.org>
> >> Signed-off-by: Vinay Simha BN <simhavcs at gmail.com>
> >
> > Just fyi, I've re-integrated this patch set into my flo-WIP branch and
> > its working well.
> >
> > I dunno if its of any use, but:
> > Tested-by: John Stultz <john.stultz at linaro.org>
> >
> It always is. Thank you!
>
> Vinay, thanks for the patience and I hope you grok the reason behind
> the requested changes.
>
yes.

premature returns should be avoided. it leads to the caller failure,
intermediate funcs also will not get executed (gpios, regulator disable).
In this case even the dsi fails, it should be panel(dsi write) failures
rather than the bridge/interface.

>
> The patch is
> Reviewed-by: Emil Velikov <emil.l.velikov at gmail.com>
>
> Regards,
> Emil
>



-- 
Regards,

Vinay Simha.B.N.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160718/c7432098/attachment-0001.html>