Re: [PATCH v3 7/7] drm/bridge: ti-sn65dsi86: Add mystery delay to enable()

2018-08-27 Thread Sean Paul
On Mon, Aug 27, 2018 at 07:40:05AM +0530, spa...@codeaurora.org wrote: > On 2018-08-14 03:00, Sean Paul wrote: > > From: Sean Paul > > > > This patch adds a 70ms mystery delay to the bridge driver in enable. > > By experimentation, it seems like it can go anywhere up until we > > initiate

Re: [PATCH v3 7/7] drm/bridge: ti-sn65dsi86: Add mystery delay to enable()

2018-08-26 Thread spanda
On 2018-08-14 03:00, Sean Paul wrote: From: Sean Paul This patch adds a 70ms mystery delay to the bridge driver in enable. By experimentation, it seems like it can go anywhere up until we initiate semi-auto link training. If we don't have the delay, link training fails. I tried to root cause

Re: [PATCH v3 7/7] drm/bridge: ti-sn65dsi86: Add mystery delay to enable()

2018-08-26 Thread Sean Paul
On Sun, Aug 26, 2018, 1:44 AM wrote: > On 2018-08-23 19:55, Sean Paul wrote: > > On Tue, Aug 14, 2018 at 10:01:45AM -0400, Sean Paul wrote: > >> On Tue, Aug 14, 2018 at 10:00:33AM -0400, Sean Paul wrote: > >> > On Tue, Aug 14, 2018 at 04:59:31PM +0530, spa...@codeaurora.org > wrote: > >> > > On

Re: [PATCH v3 7/7] drm/bridge: ti-sn65dsi86: Add mystery delay to enable()

2018-08-25 Thread spanda
On 2018-08-23 19:55, Sean Paul wrote: On Tue, Aug 14, 2018 at 10:01:45AM -0400, Sean Paul wrote: On Tue, Aug 14, 2018 at 10:00:33AM -0400, Sean Paul wrote: > On Tue, Aug 14, 2018 at 04:59:31PM +0530, spa...@codeaurora.org wrote: > > On 2018-08-14 03:00, Sean Paul wrote: > > > From: Sean Paul >

Re: [PATCH v3 7/7] drm/bridge: ti-sn65dsi86: Add mystery delay to enable()

2018-08-23 Thread Sean Paul
On Tue, Aug 14, 2018 at 10:01:45AM -0400, Sean Paul wrote: > On Tue, Aug 14, 2018 at 10:00:33AM -0400, Sean Paul wrote: > > On Tue, Aug 14, 2018 at 04:59:31PM +0530, spa...@codeaurora.org wrote: > > > On 2018-08-14 03:00, Sean Paul wrote: > > > > From: Sean Paul > > > > > > > > This patch adds a

Re: [PATCH v3 7/7] drm/bridge: ti-sn65dsi86: Add mystery delay to enable()

2018-08-14 Thread Sean Paul
On Tue, Aug 14, 2018 at 04:59:31PM +0530, spa...@codeaurora.org wrote: > On 2018-08-14 03:00, Sean Paul wrote: > > From: Sean Paul > > > > This patch adds a 70ms mystery delay to the bridge driver in enable. > > By experimentation, it seems like it can go anywhere up until we > > initiate

Re: [PATCH v3 7/7] drm/bridge: ti-sn65dsi86: Add mystery delay to enable()

2018-08-14 Thread Sean Paul
On Tue, Aug 14, 2018 at 10:00:33AM -0400, Sean Paul wrote: > On Tue, Aug 14, 2018 at 04:59:31PM +0530, spa...@codeaurora.org wrote: > > On 2018-08-14 03:00, Sean Paul wrote: > > > From: Sean Paul > > > > > > This patch adds a 70ms mystery delay to the bridge driver in enable. > > > By

Re: [PATCH v3 7/7] drm/bridge: ti-sn65dsi86: Add mystery delay to enable()

2018-08-14 Thread spanda
On 2018-08-14 03:00, Sean Paul wrote: From: Sean Paul This patch adds a 70ms mystery delay to the bridge driver in enable. By experimentation, it seems like it can go anywhere up until we initiate semi-auto link training. If we don't have the delay, link training fails. I tried to root cause

[PATCH v3 7/7] drm/bridge: ti-sn65dsi86: Add mystery delay to enable()

2018-08-13 Thread Sean Paul
From: Sean Paul This patch adds a 70ms mystery delay to the bridge driver in enable. By experimentation, it seems like it can go anywhere up until we initiate semi-auto link training. If we don't have the delay, link training fails. I tried to root cause this as best I could, but neither the