LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
* [PATCH] drm/msm/dp: Shorten SETUP timeout
@ 2021-10-05  2:37 Bjorn Andersson
  2021-10-05 18:45 ` Stephen Boyd
  0 siblings, 1 reply; 17+ messages in thread
From: Bjorn Andersson @ 2021-10-05  2:37 UTC (permalink / raw)
  To: Rob Clark, Sean Paul, David Airlie, Daniel Vetter, Kuogee Hsieh,
	Stephen Boyd, Dmitry Baryshkov, Abhinav Kumar
  Cc: linux-arm-msm, dri-devel, freedreno, linux-kernel, Sankeerth Billakanti

Found in the middle of a patch from Sankeerth was the reduction of the
INIT_SETUP timeout from 10s to 100ms. Upon INIT_SETUP timeout the host
is initalized and HPD interrupt start to be serviced, so in the case of
eDP this reduction improves the user experience dramatically - i.e.
removes 9.9s of bland screen time at boot.

Suggested-by: Sankeerth Billakanti <sbillaka@codeaurora.org>
Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
---
 drivers/gpu/drm/msm/dp/dp_display.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/msm/dp/dp_display.c b/drivers/gpu/drm/msm/dp/dp_display.c
index 21b9c1de4ecb..46d9f3eb6d13 100644
--- a/drivers/gpu/drm/msm/dp/dp_display.c
+++ b/drivers/gpu/drm/msm/dp/dp_display.c
@@ -1438,7 +1438,7 @@ void msm_dp_irq_postinstall(struct msm_dp *dp_display)
 
 	dp_hpd_event_setup(dp);
 
-	dp_add_event(dp, EV_HPD_INIT_SETUP, 0, 100);
+	dp_add_event(dp, EV_HPD_INIT_SETUP, 0, 1);
 }
 
 void msm_dp_debugfs_init(struct msm_dp *dp_display, struct drm_minor *minor)
-- 
2.29.2


^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH] drm/msm/dp: Shorten SETUP timeout
  2021-10-05  2:37 [PATCH] drm/msm/dp: Shorten SETUP timeout Bjorn Andersson
@ 2021-10-05 18:45 ` Stephen Boyd
  2021-10-05 21:40   ` Bjorn Andersson
  0 siblings, 1 reply; 17+ messages in thread
From: Stephen Boyd @ 2021-10-05 18:45 UTC (permalink / raw)
  To: Abhinav Kumar, Bjorn Andersson, Daniel Vetter, David Airlie,
	Dmitry Baryshkov, Kuogee Hsieh, Rob Clark, Sean Paul
  Cc: linux-arm-msm, dri-devel, freedreno, linux-kernel, Sankeerth Billakanti

Quoting Bjorn Andersson (2021-10-04 19:37:50)
> Found in the middle of a patch from Sankeerth was the reduction of the
> INIT_SETUP timeout from 10s to 100ms. Upon INIT_SETUP timeout the host
> is initalized and HPD interrupt start to be serviced, so in the case of
> eDP this reduction improves the user experience dramatically - i.e.
> removes 9.9s of bland screen time at boot.
>
> Suggested-by: Sankeerth Billakanti <sbillaka@codeaurora.org>
> Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
> ---

Any Fixes tag? BTW, the delay design is pretty convoluted. I had to go
re-read the code a couple times to understand that it's waiting 100ms
times the 'delay' number. Whaaaaat?

Reviewed-by: Stephen Boyd <swboyd@chromium.org>

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH] drm/msm/dp: Shorten SETUP timeout
  2021-10-05 18:45 ` Stephen Boyd
@ 2021-10-05 21:40   ` Bjorn Andersson
  2021-10-05 22:36     ` Stephen Boyd
  0 siblings, 1 reply; 17+ messages in thread
From: Bjorn Andersson @ 2021-10-05 21:40 UTC (permalink / raw)
  To: Stephen Boyd
  Cc: Abhinav Kumar, Daniel Vetter, David Airlie, Dmitry Baryshkov,
	Kuogee Hsieh, Rob Clark, Sean Paul, linux-arm-msm, dri-devel,
	freedreno, linux-kernel, Sankeerth Billakanti

On Tue 05 Oct 11:45 PDT 2021, Stephen Boyd wrote:

> Quoting Bjorn Andersson (2021-10-04 19:37:50)
> > Found in the middle of a patch from Sankeerth was the reduction of the
> > INIT_SETUP timeout from 10s to 100ms. Upon INIT_SETUP timeout the host
> > is initalized and HPD interrupt start to be serviced, so in the case of
> > eDP this reduction improves the user experience dramatically - i.e.
> > removes 9.9s of bland screen time at boot.
> >
> > Suggested-by: Sankeerth Billakanti <sbillaka@codeaurora.org>
> > Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
> > ---
> 
> Any Fixes tag? BTW, the delay design is pretty convoluted. I had to go
> re-read the code a couple times to understand that it's waiting 100ms
> times the 'delay' number. Whaaaaat?
> 

I assume you're happy with the current 10s delay on the current
devices, so I don't think we should push for this to be backported.
I have no need for it to be backported on my side at least.

> Reviewed-by: Stephen Boyd <swboyd@chromium.org>

Thanks,
Bjorn

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH] drm/msm/dp: Shorten SETUP timeout
  2021-10-05 21:40   ` Bjorn Andersson
@ 2021-10-05 22:36     ` Stephen Boyd
  2021-10-05 23:04       ` khsieh
  0 siblings, 1 reply; 17+ messages in thread
From: Stephen Boyd @ 2021-10-05 22:36 UTC (permalink / raw)
  To: Bjorn Andersson
  Cc: Abhinav Kumar, Daniel Vetter, David Airlie, Dmitry Baryshkov,
	Kuogee Hsieh, Rob Clark, Sean Paul, linux-arm-msm, dri-devel,
	freedreno, linux-kernel, Sankeerth Billakanti

Quoting Bjorn Andersson (2021-10-05 14:40:38)
> On Tue 05 Oct 11:45 PDT 2021, Stephen Boyd wrote:
>
> > Quoting Bjorn Andersson (2021-10-04 19:37:50)
> > > Found in the middle of a patch from Sankeerth was the reduction of the
> > > INIT_SETUP timeout from 10s to 100ms. Upon INIT_SETUP timeout the host
> > > is initalized and HPD interrupt start to be serviced, so in the case of
> > > eDP this reduction improves the user experience dramatically - i.e.
> > > removes 9.9s of bland screen time at boot.
> > >
> > > Suggested-by: Sankeerth Billakanti <sbillaka@codeaurora.org>
> > > Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
> > > ---
> >
> > Any Fixes tag? BTW, the delay design is pretty convoluted. I had to go
> > re-read the code a couple times to understand that it's waiting 100ms
> > times the 'delay' number. Whaaaaat?
> >
>
> I assume you're happy with the current 10s delay on the current
> devices, so I don't think we should push for this to be backported.
> I have no need for it to be backported on my side at least.
>

Sure. Fixes tag != backported to stable trees but it is close.

> > Reviewed-by: Stephen Boyd <swboyd@chromium.org>
>

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH] drm/msm/dp: Shorten SETUP timeout
  2021-10-05 22:36     ` Stephen Boyd
@ 2021-10-05 23:04       ` khsieh
  2021-10-05 23:35         ` Stephen Boyd
  2021-10-06  2:10         ` Bjorn Andersson
  0 siblings, 2 replies; 17+ messages in thread
From: khsieh @ 2021-10-05 23:04 UTC (permalink / raw)
  To: Stephen Boyd
  Cc: Bjorn Andersson, Abhinav Kumar, Daniel Vetter, David Airlie,
	Dmitry Baryshkov, Rob Clark, Sean Paul, linux-arm-msm, dri-devel,
	freedreno, linux-kernel, Sankeerth Billakanti

On 2021-10-05 15:36, Stephen Boyd wrote:
> Quoting Bjorn Andersson (2021-10-05 14:40:38)
>> On Tue 05 Oct 11:45 PDT 2021, Stephen Boyd wrote:
>> 
>> > Quoting Bjorn Andersson (2021-10-04 19:37:50)
>> > > Found in the middle of a patch from Sankeerth was the reduction of the
>> > > INIT_SETUP timeout from 10s to 100ms. Upon INIT_SETUP timeout the host
>> > > is initalized and HPD interrupt start to be serviced, so in the case of
>> > > eDP this reduction improves the user experience dramatically - i.e.
>> > > removes 9.9s of bland screen time at boot.
>> > >
>> > > Suggested-by: Sankeerth Billakanti <sbillaka@codeaurora.org>
>> > > Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
>> > > ---
>> >
>> > Any Fixes tag? BTW, the delay design is pretty convoluted. I had to go
>> > re-read the code a couple times to understand that it's waiting 100ms
>> > times the 'delay' number. Whaaaaat?
>> >
>> 
>> I assume you're happy with the current 10s delay on the current
>> devices, so I don't think we should push for this to be backported.
>> I have no need for it to be backported on my side at least.
>> 
> 
> Sure. Fixes tag != backported to stable trees but it is close.
> 
>> > Reviewed-by: Stephen Boyd <swboyd@chromium.org>
>> 
   dp_add_event(dp, EV_HPD_INIT_SETUP, 0, 1); <== to 100ms

This patch will prevent usb3 from working due to dp driver initialize 
phy earlier than usb3 which cause timeout error at power up usb3 phy 
when both edp and dp are enabled.
I had prepared a patch (drm/msm/dp: do not initialize combo phy until 
plugin interrupt) to fix this problem.
Unfortunately, my patch is depend on Bjorn's patch (PATCH v3 3/5] 
drm/msm/dp: Support up to 3 DP controllers).
I will submit my patch for review once Bjorn's patches merged in.
Therefore I would think this patch should go after both Bjorn's patches 
and my patch.




^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH] drm/msm/dp: Shorten SETUP timeout
  2021-10-05 23:04       ` khsieh
@ 2021-10-05 23:35         ` Stephen Boyd
  2021-10-06  2:10         ` Bjorn Andersson
  1 sibling, 0 replies; 17+ messages in thread
From: Stephen Boyd @ 2021-10-05 23:35 UTC (permalink / raw)
  To: khsieh
  Cc: Bjorn Andersson, Abhinav Kumar, Daniel Vetter, David Airlie,
	Dmitry Baryshkov, Rob Clark, Sean Paul, linux-arm-msm, dri-devel,
	freedreno, linux-kernel, Sankeerth Billakanti

Quoting khsieh@codeaurora.org (2021-10-05 16:04:40)
> On 2021-10-05 15:36, Stephen Boyd wrote:
> > Quoting Bjorn Andersson (2021-10-05 14:40:38)
> >> On Tue 05 Oct 11:45 PDT 2021, Stephen Boyd wrote:
> >>
> >> > Quoting Bjorn Andersson (2021-10-04 19:37:50)
> >> > > Found in the middle of a patch from Sankeerth was the reduction of the
> >> > > INIT_SETUP timeout from 10s to 100ms. Upon INIT_SETUP timeout the host
> >> > > is initalized and HPD interrupt start to be serviced, so in the case of
> >> > > eDP this reduction improves the user experience dramatically - i.e.
> >> > > removes 9.9s of bland screen time at boot.
> >> > >
> >> > > Suggested-by: Sankeerth Billakanti <sbillaka@codeaurora.org>
> >> > > Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
> >> > > ---
> >> >
> >> > Any Fixes tag? BTW, the delay design is pretty convoluted. I had to go
> >> > re-read the code a couple times to understand that it's waiting 100ms
> >> > times the 'delay' number. Whaaaaat?
> >> >
> >>
> >> I assume you're happy with the current 10s delay on the current
> >> devices, so I don't think we should push for this to be backported.
> >> I have no need for it to be backported on my side at least.
> >>
> >
> > Sure. Fixes tag != backported to stable trees but it is close.
> >
> >> > Reviewed-by: Stephen Boyd <swboyd@chromium.org>
> >>
>    dp_add_event(dp, EV_HPD_INIT_SETUP, 0, 1); <== to 100ms
>
> This patch will prevent usb3 from working due to dp driver initialize
> phy earlier than usb3 which cause timeout error at power up usb3 phy
> when both edp and dp are enabled.

That sounds pretty bad.

> I had prepared a patch (drm/msm/dp: do not initialize combo phy until
> plugin interrupt) to fix this problem.

Great! When were you planning to report this problem on the list?

> Unfortunately, my patch is depend on Bjorn's patch (PATCH v3 3/5]
> drm/msm/dp: Support up to 3 DP controllers).
> I will submit my patch for review once Bjorn's patches merged in.
> Therefore I would think this patch should go after both Bjorn's patches
> and my patch.
>

Why can't you send it now? Point to the other patch series as a
dependency.

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH] drm/msm/dp: Shorten SETUP timeout
  2021-10-05 23:04       ` khsieh
  2021-10-05 23:35         ` Stephen Boyd
@ 2021-10-06  2:10         ` Bjorn Andersson
  2021-10-06 15:37           ` khsieh
  1 sibling, 1 reply; 17+ messages in thread
From: Bjorn Andersson @ 2021-10-06  2:10 UTC (permalink / raw)
  To: khsieh
  Cc: Stephen Boyd, Abhinav Kumar, Daniel Vetter, David Airlie,
	Dmitry Baryshkov, Rob Clark, Sean Paul, linux-arm-msm, dri-devel,
	freedreno, linux-kernel, Sankeerth Billakanti

On Tue 05 Oct 16:04 PDT 2021, khsieh@codeaurora.org wrote:

> On 2021-10-05 15:36, Stephen Boyd wrote:
> > Quoting Bjorn Andersson (2021-10-05 14:40:38)
> > > On Tue 05 Oct 11:45 PDT 2021, Stephen Boyd wrote:
> > > 
> > > > Quoting Bjorn Andersson (2021-10-04 19:37:50)
> > > > > Found in the middle of a patch from Sankeerth was the reduction of the
> > > > > INIT_SETUP timeout from 10s to 100ms. Upon INIT_SETUP timeout the host
> > > > > is initalized and HPD interrupt start to be serviced, so in the case of
> > > > > eDP this reduction improves the user experience dramatically - i.e.
> > > > > removes 9.9s of bland screen time at boot.
> > > > >
> > > > > Suggested-by: Sankeerth Billakanti <sbillaka@codeaurora.org>
> > > > > Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
> > > > > ---
> > > >
> > > > Any Fixes tag? BTW, the delay design is pretty convoluted. I had to go
> > > > re-read the code a couple times to understand that it's waiting 100ms
> > > > times the 'delay' number. Whaaaaat?
> > > >
> > > 
> > > I assume you're happy with the current 10s delay on the current
> > > devices, so I don't think we should push for this to be backported.
> > > I have no need for it to be backported on my side at least.
> > > 
> > 
> > Sure. Fixes tag != backported to stable trees but it is close.
> > 
> > > > Reviewed-by: Stephen Boyd <swboyd@chromium.org>
> > > 
>   dp_add_event(dp, EV_HPD_INIT_SETUP, 0, 1); <== to 100ms
> 
> This patch will prevent usb3 from working due to dp driver initialize phy
> earlier than usb3 which cause timeout error at power up usb3 phy when both
> edp and dp are enabled.

Can you please help me understand what you mean here, I use this on my
sc8180x with both eDP and USB-C/DP right now. What is it that doesn't
work? Or am I just lucky in some race condition?

Thanks,
Bjorn

> I had prepared a patch (drm/msm/dp: do not initialize combo phy until plugin
> interrupt) to fix this problem.
> Unfortunately, my patch is depend on Bjorn's patch (PATCH v3 3/5]
> drm/msm/dp: Support up to 3 DP controllers).
> I will submit my patch for review once Bjorn's patches merged in.
> Therefore I would think this patch should go after both Bjorn's patches and
> my patch.
> 
> 
> 

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH] drm/msm/dp: Shorten SETUP timeout
  2021-10-06  2:10         ` Bjorn Andersson
@ 2021-10-06 15:37           ` khsieh
  2021-10-06 17:31             ` Bjorn Andersson
  0 siblings, 1 reply; 17+ messages in thread
From: khsieh @ 2021-10-06 15:37 UTC (permalink / raw)
  To: Bjorn Andersson
  Cc: Stephen Boyd, Abhinav Kumar, Daniel Vetter, David Airlie,
	Dmitry Baryshkov, Rob Clark, Sean Paul, linux-arm-msm, dri-devel,
	freedreno, linux-kernel, Sankeerth Billakanti

On 2021-10-05 19:10, Bjorn Andersson wrote:
> On Tue 05 Oct 16:04 PDT 2021, khsieh@codeaurora.org wrote:
> 
>> On 2021-10-05 15:36, Stephen Boyd wrote:
>> > Quoting Bjorn Andersson (2021-10-05 14:40:38)
>> > > On Tue 05 Oct 11:45 PDT 2021, Stephen Boyd wrote:
>> > >
>> > > > Quoting Bjorn Andersson (2021-10-04 19:37:50)
>> > > > > Found in the middle of a patch from Sankeerth was the reduction of the
>> > > > > INIT_SETUP timeout from 10s to 100ms. Upon INIT_SETUP timeout the host
>> > > > > is initalized and HPD interrupt start to be serviced, so in the case of
>> > > > > eDP this reduction improves the user experience dramatically - i.e.
>> > > > > removes 9.9s of bland screen time at boot.
>> > > > >
>> > > > > Suggested-by: Sankeerth Billakanti <sbillaka@codeaurora.org>
>> > > > > Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
>> > > > > ---
>> > > >
>> > > > Any Fixes tag? BTW, the delay design is pretty convoluted. I had to go
>> > > > re-read the code a couple times to understand that it's waiting 100ms
>> > > > times the 'delay' number. Whaaaaat?
>> > > >
>> > >
>> > > I assume you're happy with the current 10s delay on the current
>> > > devices, so I don't think we should push for this to be backported.
>> > > I have no need for it to be backported on my side at least.
>> > >
>> >
>> > Sure. Fixes tag != backported to stable trees but it is close.
>> >
>> > > > Reviewed-by: Stephen Boyd <swboyd@chromium.org>
>> > >
>>   dp_add_event(dp, EV_HPD_INIT_SETUP, 0, 1); <== to 100ms
>> 
>> This patch will prevent usb3 from working due to dp driver initialize 
>> phy
>> earlier than usb3 which cause timeout error at power up usb3 phy when 
>> both
>> edp and dp are enabled.
> 
> Can you please help me understand what you mean here, I use this on my
> sc8180x with both eDP and USB-C/DP right now. What is it that doesn't
> work? Or am I just lucky in some race condition?
> 
> Thanks,
> Bjorn
> 
The problem is seen at sc7280.
Apple dongle have both  hdmi and usb port.
plug Apple dongle into type-c, then plug DP into apple's hdmi port and 
usb mouse into apple's usb port.
If edp enabled at this time, then usb mouse will not work due to timeout 
at phy power up.

>> I had prepared a patch (drm/msm/dp: do not initialize combo phy until 
>> plugin
>> interrupt) to fix this problem.
>> Unfortunately, my patch is depend on Bjorn's patch (PATCH v3 3/5]
>> drm/msm/dp: Support up to 3 DP controllers).
>> I will submit my patch for review once Bjorn's patches merged in.
>> Therefore I would think this patch should go after both Bjorn's 
>> patches and
>> my patch.
>> 
>> 
>> 

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH] drm/msm/dp: Shorten SETUP timeout
  2021-10-06 15:37           ` khsieh
@ 2021-10-06 17:31             ` Bjorn Andersson
  2021-10-07 19:51               ` khsieh
  0 siblings, 1 reply; 17+ messages in thread
From: Bjorn Andersson @ 2021-10-06 17:31 UTC (permalink / raw)
  To: khsieh
  Cc: Stephen Boyd, Abhinav Kumar, Daniel Vetter, David Airlie,
	Dmitry Baryshkov, Rob Clark, Sean Paul, linux-arm-msm, dri-devel,
	freedreno, linux-kernel, Sankeerth Billakanti

On Wed 06 Oct 08:37 PDT 2021, khsieh@codeaurora.org wrote:

> On 2021-10-05 19:10, Bjorn Andersson wrote:
> > On Tue 05 Oct 16:04 PDT 2021, khsieh@codeaurora.org wrote:
> > 
> > > On 2021-10-05 15:36, Stephen Boyd wrote:
> > > > Quoting Bjorn Andersson (2021-10-05 14:40:38)
> > > > > On Tue 05 Oct 11:45 PDT 2021, Stephen Boyd wrote:
> > > > >
> > > > > > Quoting Bjorn Andersson (2021-10-04 19:37:50)
> > > > > > > Found in the middle of a patch from Sankeerth was the reduction of the
> > > > > > > INIT_SETUP timeout from 10s to 100ms. Upon INIT_SETUP timeout the host
> > > > > > > is initalized and HPD interrupt start to be serviced, so in the case of
> > > > > > > eDP this reduction improves the user experience dramatically - i.e.
> > > > > > > removes 9.9s of bland screen time at boot.
> > > > > > >
> > > > > > > Suggested-by: Sankeerth Billakanti <sbillaka@codeaurora.org>
> > > > > > > Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
> > > > > > > ---
> > > > > >
> > > > > > Any Fixes tag? BTW, the delay design is pretty convoluted. I had to go
> > > > > > re-read the code a couple times to understand that it's waiting 100ms
> > > > > > times the 'delay' number. Whaaaaat?
> > > > > >
> > > > >
> > > > > I assume you're happy with the current 10s delay on the current
> > > > > devices, so I don't think we should push for this to be backported.
> > > > > I have no need for it to be backported on my side at least.
> > > > >
> > > >
> > > > Sure. Fixes tag != backported to stable trees but it is close.
> > > >
> > > > > > Reviewed-by: Stephen Boyd <swboyd@chromium.org>
> > > > >
> > >   dp_add_event(dp, EV_HPD_INIT_SETUP, 0, 1); <== to 100ms
> > > 
> > > This patch will prevent usb3 from working due to dp driver
> > > initialize phy
> > > earlier than usb3 which cause timeout error at power up usb3 phy
> > > when both
> > > edp and dp are enabled.
> > 
> > Can you please help me understand what you mean here, I use this on my
> > sc8180x with both eDP and USB-C/DP right now. What is it that doesn't
> > work? Or am I just lucky in some race condition?
> > 
> > Thanks,
> > Bjorn
> > 
> The problem is seen at sc7280.
> Apple dongle have both  hdmi and usb port.
> plug Apple dongle into type-c, then plug DP into apple's hdmi port and usb
> mouse into apple's usb port.
> If edp enabled at this time, then usb mouse will not work due to timeout at
> phy power up.
> 

Okay, so you're saying that if the DP driver invokes phy_power_on()
before the USB driver does, USB initialization fails (or at least USB
doesn't work)?

Sounds like something we need to work out in the QMP phy driver. Do you
have any more details about what's going wrong.


Also, I've seen various references to said "Apple dongle", do you have a
link to the exact one you're testing with so I can pick one up for
testing purposes as well?

Regards,
Bjorn

> > > I had prepared a patch (drm/msm/dp: do not initialize combo phy
> > > until plugin
> > > interrupt) to fix this problem.
> > > Unfortunately, my patch is depend on Bjorn's patch (PATCH v3 3/5]
> > > drm/msm/dp: Support up to 3 DP controllers).
> > > I will submit my patch for review once Bjorn's patches merged in.
> > > Therefore I would think this patch should go after both Bjorn's
> > > patches and
> > > my patch.
> > > 
> > > 
> > > 

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH] drm/msm/dp: Shorten SETUP timeout
  2021-10-06 17:31             ` Bjorn Andersson
@ 2021-10-07 19:51               ` khsieh
  2021-10-07 20:06                 ` Bjorn Andersson
  0 siblings, 1 reply; 17+ messages in thread
From: khsieh @ 2021-10-07 19:51 UTC (permalink / raw)
  To: Bjorn Andersson
  Cc: Stephen Boyd, Abhinav Kumar, Daniel Vetter, David Airlie,
	Dmitry Baryshkov, Rob Clark, Sean Paul, linux-arm-msm, dri-devel,
	freedreno, linux-kernel, Sankeerth Billakanti

On 2021-10-06 10:31, Bjorn Andersson wrote:
> On Wed 06 Oct 08:37 PDT 2021, khsieh@codeaurora.org wrote:
> 
>> On 2021-10-05 19:10, Bjorn Andersson wrote:
>> > On Tue 05 Oct 16:04 PDT 2021, khsieh@codeaurora.org wrote:
>> >
>> > > On 2021-10-05 15:36, Stephen Boyd wrote:
>> > > > Quoting Bjorn Andersson (2021-10-05 14:40:38)
>> > > > > On Tue 05 Oct 11:45 PDT 2021, Stephen Boyd wrote:
>> > > > >
>> > > > > > Quoting Bjorn Andersson (2021-10-04 19:37:50)
>> > > > > > > Found in the middle of a patch from Sankeerth was the reduction of the
>> > > > > > > INIT_SETUP timeout from 10s to 100ms. Upon INIT_SETUP timeout the host
>> > > > > > > is initalized and HPD interrupt start to be serviced, so in the case of
>> > > > > > > eDP this reduction improves the user experience dramatically - i.e.
>> > > > > > > removes 9.9s of bland screen time at boot.
>> > > > > > >
>> > > > > > > Suggested-by: Sankeerth Billakanti <sbillaka@codeaurora.org>
>> > > > > > > Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
>> > > > > > > ---
>> > > > > >
>> > > > > > Any Fixes tag? BTW, the delay design is pretty convoluted. I had to go
>> > > > > > re-read the code a couple times to understand that it's waiting 100ms
>> > > > > > times the 'delay' number. Whaaaaat?
>> > > > > >
>> > > > >
>> > > > > I assume you're happy with the current 10s delay on the current
>> > > > > devices, so I don't think we should push for this to be backported.
>> > > > > I have no need for it to be backported on my side at least.
>> > > > >
>> > > >
>> > > > Sure. Fixes tag != backported to stable trees but it is close.
>> > > >
>> > > > > > Reviewed-by: Stephen Boyd <swboyd@chromium.org>
>> > > > >
>> > >   dp_add_event(dp, EV_HPD_INIT_SETUP, 0, 1); <== to 100ms
>> > >
>> > > This patch will prevent usb3 from working due to dp driver
>> > > initialize phy
>> > > earlier than usb3 which cause timeout error at power up usb3 phy
>> > > when both
>> > > edp and dp are enabled.
>> >
>> > Can you please help me understand what you mean here, I use this on my
>> > sc8180x with both eDP and USB-C/DP right now. What is it that doesn't
>> > work? Or am I just lucky in some race condition?
>> >
>> > Thanks,
>> > Bjorn
>> >
>> The problem is seen at sc7280.
>> Apple dongle have both  hdmi and usb port.
>> plug Apple dongle into type-c, then plug DP into apple's hdmi port and 
>> usb
>> mouse into apple's usb port.
>> If edp enabled at this time, then usb mouse will not work due to 
>> timeout at
>> phy power up.
>> 
> 
> Okay, so you're saying that if the DP driver invokes phy_power_on()
> before the USB driver does, USB initialization fails (or at least USB
> doesn't work)?

if dp driver call qcom_qmp_phy_init() before usb3 call 
qcom_qmp_phy_init(),
usb3 driver will timeout at readl_poll_timeout(status, val, (val & mask) 
== ready, 10, PHY_INIT_COMPLETE_TIMEOUT) of qcom_qmp_phy_power_on().
> 
> Sounds like something we need to work out in the QMP phy driver. Do you
> have any more details about what's going wrong.
> 
> 
> Also, I've seen various references to said "Apple dongle", do you have 
> a
> link to the exact one you're testing with so I can pick one up for
> testing purposes as well?

Apple A2119 hdmi+usb dongle.
https://www.amazon.com/Apple-USB-C-Digital-Multiport-Adapter/dp/B07WF96FY5/ref=sr_1_2?dchild=1&keywords=apple+a2119&qid=1633636227&sr=8-2

> 
> Regards,
> Bjorn
> 
>> > > I had prepared a patch (drm/msm/dp: do not initialize combo phy
>> > > until plugin
>> > > interrupt) to fix this problem.
>> > > Unfortunately, my patch is depend on Bjorn's patch (PATCH v3 3/5]
>> > > drm/msm/dp: Support up to 3 DP controllers).
>> > > I will submit my patch for review once Bjorn's patches merged in.
>> > > Therefore I would think this patch should go after both Bjorn's
>> > > patches and
>> > > my patch.
>> > >
>> > >
>> > >

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH] drm/msm/dp: Shorten SETUP timeout
  2021-10-07 19:51               ` khsieh
@ 2021-10-07 20:06                 ` Bjorn Andersson
  2021-10-07 20:28                   ` khsieh
  0 siblings, 1 reply; 17+ messages in thread
From: Bjorn Andersson @ 2021-10-07 20:06 UTC (permalink / raw)
  To: khsieh
  Cc: Stephen Boyd, Abhinav Kumar, Daniel Vetter, David Airlie,
	Dmitry Baryshkov, Rob Clark, Sean Paul, linux-arm-msm, dri-devel,
	freedreno, linux-kernel, Sankeerth Billakanti

On Thu 07 Oct 12:51 PDT 2021, khsieh@codeaurora.org wrote:

> On 2021-10-06 10:31, Bjorn Andersson wrote:
> > On Wed 06 Oct 08:37 PDT 2021, khsieh@codeaurora.org wrote:
> > 
> > > On 2021-10-05 19:10, Bjorn Andersson wrote:
> > > > On Tue 05 Oct 16:04 PDT 2021, khsieh@codeaurora.org wrote:
> > > >
> > > > > On 2021-10-05 15:36, Stephen Boyd wrote:
> > > > > > Quoting Bjorn Andersson (2021-10-05 14:40:38)
> > > > > > > On Tue 05 Oct 11:45 PDT 2021, Stephen Boyd wrote:
> > > > > > >
> > > > > > > > Quoting Bjorn Andersson (2021-10-04 19:37:50)
> > > > > > > > > Found in the middle of a patch from Sankeerth was the reduction of the
> > > > > > > > > INIT_SETUP timeout from 10s to 100ms. Upon INIT_SETUP timeout the host
> > > > > > > > > is initalized and HPD interrupt start to be serviced, so in the case of
> > > > > > > > > eDP this reduction improves the user experience dramatically - i.e.
> > > > > > > > > removes 9.9s of bland screen time at boot.
> > > > > > > > >
> > > > > > > > > Suggested-by: Sankeerth Billakanti <sbillaka@codeaurora.org>
> > > > > > > > > Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
> > > > > > > > > ---
> > > > > > > >
> > > > > > > > Any Fixes tag? BTW, the delay design is pretty convoluted. I had to go
> > > > > > > > re-read the code a couple times to understand that it's waiting 100ms
> > > > > > > > times the 'delay' number. Whaaaaat?
> > > > > > > >
> > > > > > >
> > > > > > > I assume you're happy with the current 10s delay on the current
> > > > > > > devices, so I don't think we should push for this to be backported.
> > > > > > > I have no need for it to be backported on my side at least.
> > > > > > >
> > > > > >
> > > > > > Sure. Fixes tag != backported to stable trees but it is close.
> > > > > >
> > > > > > > > Reviewed-by: Stephen Boyd <swboyd@chromium.org>
> > > > > > >
> > > > >   dp_add_event(dp, EV_HPD_INIT_SETUP, 0, 1); <== to 100ms
> > > > >
> > > > > This patch will prevent usb3 from working due to dp driver
> > > > > initialize phy
> > > > > earlier than usb3 which cause timeout error at power up usb3 phy
> > > > > when both
> > > > > edp and dp are enabled.
> > > >
> > > > Can you please help me understand what you mean here, I use this on my
> > > > sc8180x with both eDP and USB-C/DP right now. What is it that doesn't
> > > > work? Or am I just lucky in some race condition?
> > > >
> > > > Thanks,
> > > > Bjorn
> > > >
> > > The problem is seen at sc7280.
> > > Apple dongle have both  hdmi and usb port.
> > > plug Apple dongle into type-c, then plug DP into apple's hdmi port
> > > and usb
> > > mouse into apple's usb port.
> > > If edp enabled at this time, then usb mouse will not work due to
> > > timeout at
> > > phy power up.
> > > 
> > 
> > Okay, so you're saying that if the DP driver invokes phy_power_on()
> > before the USB driver does, USB initialization fails (or at least USB
> > doesn't work)?
> 
> if dp driver call qcom_qmp_phy_init() before usb3 call qcom_qmp_phy_init(),
> usb3 driver will timeout at readl_poll_timeout(status, val, (val & mask) ==
> ready, 10, PHY_INIT_COMPLETE_TIMEOUT) of qcom_qmp_phy_power_on().

Thanks, I will try to reproduce this on my side. So the 10 seconds here
is strictly to give good enough time for the dwc3 driver to probe...

Any idea why you're saying that this is specific to sc7280, what changed
from sc7180?

> > 
> > Sounds like something we need to work out in the QMP phy driver. Do you
> > have any more details about what's going wrong.
> > 
> > 
> > Also, I've seen various references to said "Apple dongle", do you have a
> > link to the exact one you're testing with so I can pick one up for
> > testing purposes as well?
> 
> Apple A2119 hdmi+usb dongle.
> https://www.amazon.com/Apple-USB-C-Digital-Multiport-Adapter/dp/B07WF96FY5/ref=sr_1_2?dchild=1&keywords=apple+a2119&qid=1633636227&sr=8-2
> 

Thanks,
Bjorn

> > 
> > Regards,
> > Bjorn
> > 
> > > > > I had prepared a patch (drm/msm/dp: do not initialize combo phy
> > > > > until plugin
> > > > > interrupt) to fix this problem.
> > > > > Unfortunately, my patch is depend on Bjorn's patch (PATCH v3 3/5]
> > > > > drm/msm/dp: Support up to 3 DP controllers).
> > > > > I will submit my patch for review once Bjorn's patches merged in.
> > > > > Therefore I would think this patch should go after both Bjorn's
> > > > > patches and
> > > > > my patch.
> > > > >
> > > > >
> > > > >

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH] drm/msm/dp: Shorten SETUP timeout
  2021-10-07 20:06                 ` Bjorn Andersson
@ 2021-10-07 20:28                   ` khsieh
  2021-10-07 22:34                     ` Stephen Boyd
  0 siblings, 1 reply; 17+ messages in thread
From: khsieh @ 2021-10-07 20:28 UTC (permalink / raw)
  To: Bjorn Andersson
  Cc: Stephen Boyd, Abhinav Kumar, Daniel Vetter, David Airlie,
	Dmitry Baryshkov, Rob Clark, Sean Paul, linux-arm-msm, dri-devel,
	freedreno, linux-kernel, Sankeerth Billakanti

On 2021-10-07 13:06, Bjorn Andersson wrote:
> On Thu 07 Oct 12:51 PDT 2021, khsieh@codeaurora.org wrote:
> 
>> On 2021-10-06 10:31, Bjorn Andersson wrote:
>> > On Wed 06 Oct 08:37 PDT 2021, khsieh@codeaurora.org wrote:
>> >
>> > > On 2021-10-05 19:10, Bjorn Andersson wrote:
>> > > > On Tue 05 Oct 16:04 PDT 2021, khsieh@codeaurora.org wrote:
>> > > >
>> > > > > On 2021-10-05 15:36, Stephen Boyd wrote:
>> > > > > > Quoting Bjorn Andersson (2021-10-05 14:40:38)
>> > > > > > > On Tue 05 Oct 11:45 PDT 2021, Stephen Boyd wrote:
>> > > > > > >
>> > > > > > > > Quoting Bjorn Andersson (2021-10-04 19:37:50)
>> > > > > > > > > Found in the middle of a patch from Sankeerth was the reduction of the
>> > > > > > > > > INIT_SETUP timeout from 10s to 100ms. Upon INIT_SETUP timeout the host
>> > > > > > > > > is initalized and HPD interrupt start to be serviced, so in the case of
>> > > > > > > > > eDP this reduction improves the user experience dramatically - i.e.
>> > > > > > > > > removes 9.9s of bland screen time at boot.
>> > > > > > > > >
>> > > > > > > > > Suggested-by: Sankeerth Billakanti <sbillaka@codeaurora.org>
>> > > > > > > > > Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
>> > > > > > > > > ---
>> > > > > > > >
>> > > > > > > > Any Fixes tag? BTW, the delay design is pretty convoluted. I had to go
>> > > > > > > > re-read the code a couple times to understand that it's waiting 100ms
>> > > > > > > > times the 'delay' number. Whaaaaat?
>> > > > > > > >
>> > > > > > >
>> > > > > > > I assume you're happy with the current 10s delay on the current
>> > > > > > > devices, so I don't think we should push for this to be backported.
>> > > > > > > I have no need for it to be backported on my side at least.
>> > > > > > >
>> > > > > >
>> > > > > > Sure. Fixes tag != backported to stable trees but it is close.
>> > > > > >
>> > > > > > > > Reviewed-by: Stephen Boyd <swboyd@chromium.org>
>> > > > > > >
>> > > > >   dp_add_event(dp, EV_HPD_INIT_SETUP, 0, 1); <== to 100ms
>> > > > >
>> > > > > This patch will prevent usb3 from working due to dp driver
>> > > > > initialize phy
>> > > > > earlier than usb3 which cause timeout error at power up usb3 phy
>> > > > > when both
>> > > > > edp and dp are enabled.
>> > > >
>> > > > Can you please help me understand what you mean here, I use this on my
>> > > > sc8180x with both eDP and USB-C/DP right now. What is it that doesn't
>> > > > work? Or am I just lucky in some race condition?
>> > > >
>> > > > Thanks,
>> > > > Bjorn
>> > > >
>> > > The problem is seen at sc7280.
>> > > Apple dongle have both  hdmi and usb port.
>> > > plug Apple dongle into type-c, then plug DP into apple's hdmi port
>> > > and usb
>> > > mouse into apple's usb port.
>> > > If edp enabled at this time, then usb mouse will not work due to
>> > > timeout at
>> > > phy power up.
>> > >
>> >
>> > Okay, so you're saying that if the DP driver invokes phy_power_on()
>> > before the USB driver does, USB initialization fails (or at least USB
>> > doesn't work)?
>> 
>> if dp driver call qcom_qmp_phy_init() before usb3 call 
>> qcom_qmp_phy_init(),
>> usb3 driver will timeout at readl_poll_timeout(status, val, (val & 
>> mask) ==
>> ready, 10, PHY_INIT_COMPLETE_TIMEOUT) of qcom_qmp_phy_power_on().
> 
> Thanks, I will try to reproduce this on my side. So the 10 seconds here
> is strictly to give good enough time for the dwc3 driver to probe...
> 
> Any idea why you're saying that this is specific to sc7280, what 
> changed
> from sc7180?

I did not have sc7180 with edp before so that i am not sure it will 
happen on sc7180 or not.
The usb3 does not work when both edp and dp enabled I just seen at 
sc7280.
Current at sc7280 EC is not boot up correctly when system power up.
I have to manual reboot EC from linux kernel shell before DP/usb3 can 
work.
I am not sure this contribute to this problem or not.


> 
>> >
>> > Sounds like something we need to work out in the QMP phy driver. Do you
>> > have any more details about what's going wrong.
>> >
>> >
>> > Also, I've seen various references to said "Apple dongle", do you have a
>> > link to the exact one you're testing with so I can pick one up for
>> > testing purposes as well?
>> 
>> Apple A2119 hdmi+usb dongle.
>> https://www.amazon.com/Apple-USB-C-Digital-Multiport-Adapter/dp/B07WF96FY5/ref=sr_1_2?dchild=1&keywords=apple+a2119&qid=1633636227&sr=8-2
>> 
> 
> Thanks,
> Bjorn
> 
>> >
>> > Regards,
>> > Bjorn
>> >
>> > > > > I had prepared a patch (drm/msm/dp: do not initialize combo phy
>> > > > > until plugin
>> > > > > interrupt) to fix this problem.
>> > > > > Unfortunately, my patch is depend on Bjorn's patch (PATCH v3 3/5]
>> > > > > drm/msm/dp: Support up to 3 DP controllers).
>> > > > > I will submit my patch for review once Bjorn's patches merged in.
>> > > > > Therefore I would think this patch should go after both Bjorn's
>> > > > > patches and
>> > > > > my patch.
>> > > > >
>> > > > >
>> > > > >

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH] drm/msm/dp: Shorten SETUP timeout
  2021-10-07 20:28                   ` khsieh
@ 2021-10-07 22:34                     ` Stephen Boyd
  2021-10-08 16:07                       ` khsieh
  0 siblings, 1 reply; 17+ messages in thread
From: Stephen Boyd @ 2021-10-07 22:34 UTC (permalink / raw)
  To: Bjorn Andersson, khsieh
  Cc: Abhinav Kumar, Daniel Vetter, David Airlie, Dmitry Baryshkov,
	Rob Clark, Sean Paul, linux-arm-msm, dri-devel, freedreno,
	linux-kernel, Sankeerth Billakanti

Quoting khsieh@codeaurora.org (2021-10-07 13:28:12)
> On 2021-10-07 13:06, Bjorn Andersson wrote:
> > On Thu 07 Oct 12:51 PDT 2021, khsieh@codeaurora.org wrote:
> >
> >> On 2021-10-06 10:31, Bjorn Andersson wrote:
> >> > On Wed 06 Oct 08:37 PDT 2021, khsieh@codeaurora.org wrote:
> >> >
> >> > > On 2021-10-05 19:10, Bjorn Andersson wrote:
> >> > > > On Tue 05 Oct 16:04 PDT 2021, khsieh@codeaurora.org wrote:
> >> > > >
> >> > > > > On 2021-10-05 15:36, Stephen Boyd wrote:
> >> > > > > > Quoting Bjorn Andersson (2021-10-05 14:40:38)
> >> > > > > > > On Tue 05 Oct 11:45 PDT 2021, Stephen Boyd wrote:
> >> > > > > > >
> >> > > > > > > > Quoting Bjorn Andersson (2021-10-04 19:37:50)
> >> > > > > > > > > Found in the middle of a patch from Sankeerth was the reduction of the
> >> > > > > > > > > INIT_SETUP timeout from 10s to 100ms. Upon INIT_SETUP timeout the host
> >> > > > > > > > > is initalized and HPD interrupt start to be serviced, so in the case of
> >> > > > > > > > > eDP this reduction improves the user experience dramatically - i.e.
> >> > > > > > > > > removes 9.9s of bland screen time at boot.
> >> > > > > > > > >
> >> > > > > > > > > Suggested-by: Sankeerth Billakanti <sbillaka@codeaurora.org>
> >> > > > > > > > > Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
> >> > > > > > > > > ---
> >> > > > > > > >
> >> > > > > > > > Any Fixes tag? BTW, the delay design is pretty convoluted. I had to go
> >> > > > > > > > re-read the code a couple times to understand that it's waiting 100ms
> >> > > > > > > > times the 'delay' number. Whaaaaat?
> >> > > > > > > >
> >> > > > > > >
> >> > > > > > > I assume you're happy with the current 10s delay on the current
> >> > > > > > > devices, so I don't think we should push for this to be backported.
> >> > > > > > > I have no need for it to be backported on my side at least.
> >> > > > > > >
> >> > > > > >
> >> > > > > > Sure. Fixes tag != backported to stable trees but it is close.
> >> > > > > >
> >> > > > > > > > Reviewed-by: Stephen Boyd <swboyd@chromium.org>
> >> > > > > > >
> >> > > > >   dp_add_event(dp, EV_HPD_INIT_SETUP, 0, 1); <== to 100ms
> >> > > > >
> >> > > > > This patch will prevent usb3 from working due to dp driver
> >> > > > > initialize phy
> >> > > > > earlier than usb3 which cause timeout error at power up usb3 phy
> >> > > > > when both
> >> > > > > edp and dp are enabled.
> >> > > >
> >> > > > Can you please help me understand what you mean here, I use this on my
> >> > > > sc8180x with both eDP and USB-C/DP right now. What is it that doesn't
> >> > > > work? Or am I just lucky in some race condition?
> >> > > >
> >> > > > Thanks,
> >> > > > Bjorn
> >> > > >
> >> > > The problem is seen at sc7280.
> >> > > Apple dongle have both  hdmi and usb port.
> >> > > plug Apple dongle into type-c, then plug DP into apple's hdmi port
> >> > > and usb
> >> > > mouse into apple's usb port.
> >> > > If edp enabled at this time, then usb mouse will not work due to
> >> > > timeout at
> >> > > phy power up.
> >> > >
> >> >
> >> > Okay, so you're saying that if the DP driver invokes phy_power_on()
> >> > before the USB driver does, USB initialization fails (or at least USB
> >> > doesn't work)?
> >>
> >> if dp driver call qcom_qmp_phy_init() before usb3 call
> >> qcom_qmp_phy_init(),
> >> usb3 driver will timeout at readl_poll_timeout(status, val, (val &
> >> mask) ==
> >> ready, 10, PHY_INIT_COMPLETE_TIMEOUT) of qcom_qmp_phy_power_on().
> >
> > Thanks, I will try to reproduce this on my side. So the 10 seconds here
> > is strictly to give good enough time for the dwc3 driver to probe...
> >
> > Any idea why you're saying that this is specific to sc7280, what
> > changed
> > from sc7180?
>
> I did not have sc7180 with edp before so that i am not sure it will
> happen on sc7180 or not.
> The usb3 does not work when both edp and dp enabled I just seen at
> sc7280.
> Current at sc7280 EC is not boot up correctly when system power up.
> I have to manual reboot EC from linux kernel shell before DP/usb3 can
> work.
> I am not sure this contribute to this problem or not.
>

Can you make the usb driver into a module and only load that module
later in boot after the DP driver calls qcom_qmp_phy_init()? That would
be an easy way to move usb probe after DP probe and expose this problem.

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH] drm/msm/dp: Shorten SETUP timeout
  2021-10-07 22:34                     ` Stephen Boyd
@ 2021-10-08 16:07                       ` khsieh
  2021-10-08 16:44                         ` Bjorn Andersson
  0 siblings, 1 reply; 17+ messages in thread
From: khsieh @ 2021-10-08 16:07 UTC (permalink / raw)
  To: Stephen Boyd
  Cc: Bjorn Andersson, Abhinav Kumar, Daniel Vetter, David Airlie,
	Dmitry Baryshkov, Rob Clark, Sean Paul, linux-arm-msm, dri-devel,
	freedreno, linux-kernel, Sankeerth Billakanti

On 2021-10-07 15:34, Stephen Boyd wrote:
> Quoting khsieh@codeaurora.org (2021-10-07 13:28:12)
>> On 2021-10-07 13:06, Bjorn Andersson wrote:
>> > On Thu 07 Oct 12:51 PDT 2021, khsieh@codeaurora.org wrote:
>> >
>> >> On 2021-10-06 10:31, Bjorn Andersson wrote:
>> >> > On Wed 06 Oct 08:37 PDT 2021, khsieh@codeaurora.org wrote:
>> >> >
>> >> > > On 2021-10-05 19:10, Bjorn Andersson wrote:
>> >> > > > On Tue 05 Oct 16:04 PDT 2021, khsieh@codeaurora.org wrote:
>> >> > > >
>> >> > > > > On 2021-10-05 15:36, Stephen Boyd wrote:
>> >> > > > > > Quoting Bjorn Andersson (2021-10-05 14:40:38)
>> >> > > > > > > On Tue 05 Oct 11:45 PDT 2021, Stephen Boyd wrote:
>> >> > > > > > >
>> >> > > > > > > > Quoting Bjorn Andersson (2021-10-04 19:37:50)
>> >> > > > > > > > > Found in the middle of a patch from Sankeerth was the reduction of the
>> >> > > > > > > > > INIT_SETUP timeout from 10s to 100ms. Upon INIT_SETUP timeout the host
>> >> > > > > > > > > is initalized and HPD interrupt start to be serviced, so in the case of
>> >> > > > > > > > > eDP this reduction improves the user experience dramatically - i.e.
>> >> > > > > > > > > removes 9.9s of bland screen time at boot.
>> >> > > > > > > > >
>> >> > > > > > > > > Suggested-by: Sankeerth Billakanti <sbillaka@codeaurora.org>
>> >> > > > > > > > > Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
>> >> > > > > > > > > ---
>> >> > > > > > > >
>> >> > > > > > > > Any Fixes tag? BTW, the delay design is pretty convoluted. I had to go
>> >> > > > > > > > re-read the code a couple times to understand that it's waiting 100ms
>> >> > > > > > > > times the 'delay' number. Whaaaaat?
>> >> > > > > > > >
>> >> > > > > > >
>> >> > > > > > > I assume you're happy with the current 10s delay on the current
>> >> > > > > > > devices, so I don't think we should push for this to be backported.
>> >> > > > > > > I have no need for it to be backported on my side at least.
>> >> > > > > > >
>> >> > > > > >
>> >> > > > > > Sure. Fixes tag != backported to stable trees but it is close.
>> >> > > > > >
>> >> > > > > > > > Reviewed-by: Stephen Boyd <swboyd@chromium.org>
>> >> > > > > > >
>> >> > > > >   dp_add_event(dp, EV_HPD_INIT_SETUP, 0, 1); <== to 100ms
>> >> > > > >
>> >> > > > > This patch will prevent usb3 from working due to dp driver
>> >> > > > > initialize phy
>> >> > > > > earlier than usb3 which cause timeout error at power up usb3 phy
>> >> > > > > when both
>> >> > > > > edp and dp are enabled.
>> >> > > >
>> >> > > > Can you please help me understand what you mean here, I use this on my
>> >> > > > sc8180x with both eDP and USB-C/DP right now. What is it that doesn't
>> >> > > > work? Or am I just lucky in some race condition?
>> >> > > >
>> >> > > > Thanks,
>> >> > > > Bjorn
>> >> > > >
>> >> > > The problem is seen at sc7280.
>> >> > > Apple dongle have both  hdmi and usb port.
>> >> > > plug Apple dongle into type-c, then plug DP into apple's hdmi port
>> >> > > and usb
>> >> > > mouse into apple's usb port.
>> >> > > If edp enabled at this time, then usb mouse will not work due to
>> >> > > timeout at
>> >> > > phy power up.
>> >> > >
>> >> >
>> >> > Okay, so you're saying that if the DP driver invokes phy_power_on()
>> >> > before the USB driver does, USB initialization fails (or at least USB
>> >> > doesn't work)?
>> >>
>> >> if dp driver call qcom_qmp_phy_init() before usb3 call
>> >> qcom_qmp_phy_init(),
>> >> usb3 driver will timeout at readl_poll_timeout(status, val, (val &
>> >> mask) ==
>> >> ready, 10, PHY_INIT_COMPLETE_TIMEOUT) of qcom_qmp_phy_power_on().
>> >
>> > Thanks, I will try to reproduce this on my side. So the 10 seconds here
>> > is strictly to give good enough time for the dwc3 driver to probe...
>> >
>> > Any idea why you're saying that this is specific to sc7280, what
>> > changed
>> > from sc7180?
>> 
>> I did not have sc7180 with edp before so that i am not sure it will
>> happen on sc7180 or not.
>> The usb3 does not work when both edp and dp enabled I just seen at
>> sc7280.
>> Current at sc7280 EC is not boot up correctly when system power up.
>> I have to manual reboot EC from linux kernel shell before DP/usb3 can
>> work.
>> I am not sure this contribute to this problem or not.
>> 
> 
> Can you make the usb driver into a module and only load that module
> later in boot after the DP driver calls qcom_qmp_phy_init()? That would
> be an easy way to move usb probe after DP probe and expose this 
> problem.

we need usb calls qcom_qmp_phy_init() before dp.

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH] drm/msm/dp: Shorten SETUP timeout
  2021-10-08 16:07                       ` khsieh
@ 2021-10-08 16:44                         ` Bjorn Andersson
  2021-10-11 15:24                           ` khsieh
  0 siblings, 1 reply; 17+ messages in thread
From: Bjorn Andersson @ 2021-10-08 16:44 UTC (permalink / raw)
  To: khsieh
  Cc: Stephen Boyd, Abhinav Kumar, Daniel Vetter, David Airlie,
	Dmitry Baryshkov, Rob Clark, Sean Paul, linux-arm-msm, dri-devel,
	freedreno, linux-kernel, Sankeerth Billakanti

On Fri 08 Oct 09:07 PDT 2021, khsieh@codeaurora.org wrote:

> On 2021-10-07 15:34, Stephen Boyd wrote:
> > Quoting khsieh@codeaurora.org (2021-10-07 13:28:12)
> > > On 2021-10-07 13:06, Bjorn Andersson wrote:
> > > > On Thu 07 Oct 12:51 PDT 2021, khsieh@codeaurora.org wrote:
> > > >
> > > >> On 2021-10-06 10:31, Bjorn Andersson wrote:
> > > >> > On Wed 06 Oct 08:37 PDT 2021, khsieh@codeaurora.org wrote:
> > > >> >
> > > >> > > On 2021-10-05 19:10, Bjorn Andersson wrote:
> > > >> > > > On Tue 05 Oct 16:04 PDT 2021, khsieh@codeaurora.org wrote:
> > > >> > > >
> > > >> > > > > On 2021-10-05 15:36, Stephen Boyd wrote:
> > > >> > > > > > Quoting Bjorn Andersson (2021-10-05 14:40:38)
> > > >> > > > > > > On Tue 05 Oct 11:45 PDT 2021, Stephen Boyd wrote:
> > > >> > > > > > >
> > > >> > > > > > > > Quoting Bjorn Andersson (2021-10-04 19:37:50)
> > > >> > > > > > > > > Found in the middle of a patch from Sankeerth was the reduction of the
> > > >> > > > > > > > > INIT_SETUP timeout from 10s to 100ms. Upon INIT_SETUP timeout the host
> > > >> > > > > > > > > is initalized and HPD interrupt start to be serviced, so in the case of
> > > >> > > > > > > > > eDP this reduction improves the user experience dramatically - i.e.
> > > >> > > > > > > > > removes 9.9s of bland screen time at boot.
> > > >> > > > > > > > >
> > > >> > > > > > > > > Suggested-by: Sankeerth Billakanti <sbillaka@codeaurora.org>
> > > >> > > > > > > > > Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
> > > >> > > > > > > > > ---
> > > >> > > > > > > >
> > > >> > > > > > > > Any Fixes tag? BTW, the delay design is pretty convoluted. I had to go
> > > >> > > > > > > > re-read the code a couple times to understand that it's waiting 100ms
> > > >> > > > > > > > times the 'delay' number. Whaaaaat?
> > > >> > > > > > > >
> > > >> > > > > > >
> > > >> > > > > > > I assume you're happy with the current 10s delay on the current
> > > >> > > > > > > devices, so I don't think we should push for this to be backported.
> > > >> > > > > > > I have no need for it to be backported on my side at least.
> > > >> > > > > > >
> > > >> > > > > >
> > > >> > > > > > Sure. Fixes tag != backported to stable trees but it is close.
> > > >> > > > > >
> > > >> > > > > > > > Reviewed-by: Stephen Boyd <swboyd@chromium.org>
> > > >> > > > > > >
> > > >> > > > >   dp_add_event(dp, EV_HPD_INIT_SETUP, 0, 1); <== to 100ms
> > > >> > > > >
> > > >> > > > > This patch will prevent usb3 from working due to dp driver
> > > >> > > > > initialize phy
> > > >> > > > > earlier than usb3 which cause timeout error at power up usb3 phy
> > > >> > > > > when both
> > > >> > > > > edp and dp are enabled.
> > > >> > > >
> > > >> > > > Can you please help me understand what you mean here, I use this on my
> > > >> > > > sc8180x with both eDP and USB-C/DP right now. What is it that doesn't
> > > >> > > > work? Or am I just lucky in some race condition?
> > > >> > > >
> > > >> > > > Thanks,
> > > >> > > > Bjorn
> > > >> > > >
> > > >> > > The problem is seen at sc7280.
> > > >> > > Apple dongle have both  hdmi and usb port.
> > > >> > > plug Apple dongle into type-c, then plug DP into apple's hdmi port
> > > >> > > and usb
> > > >> > > mouse into apple's usb port.
> > > >> > > If edp enabled at this time, then usb mouse will not work due to
> > > >> > > timeout at
> > > >> > > phy power up.
> > > >> > >
> > > >> >
> > > >> > Okay, so you're saying that if the DP driver invokes phy_power_on()
> > > >> > before the USB driver does, USB initialization fails (or at least USB
> > > >> > doesn't work)?
> > > >>
> > > >> if dp driver call qcom_qmp_phy_init() before usb3 call
> > > >> qcom_qmp_phy_init(),
> > > >> usb3 driver will timeout at readl_poll_timeout(status, val, (val &
> > > >> mask) ==
> > > >> ready, 10, PHY_INIT_COMPLETE_TIMEOUT) of qcom_qmp_phy_power_on().
> > > >
> > > > Thanks, I will try to reproduce this on my side. So the 10 seconds here
> > > > is strictly to give good enough time for the dwc3 driver to probe...
> > > >
> > > > Any idea why you're saying that this is specific to sc7280, what
> > > > changed
> > > > from sc7180?
> > > 
> > > I did not have sc7180 with edp before so that i am not sure it will
> > > happen on sc7180 or not.
> > > The usb3 does not work when both edp and dp enabled I just seen at
> > > sc7280.
> > > Current at sc7280 EC is not boot up correctly when system power up.
> > > I have to manual reboot EC from linux kernel shell before DP/usb3 can
> > > work.
> > > I am not sure this contribute to this problem or not.
> > > 
> > 
> > Can you make the usb driver into a module and only load that module
> > later in boot after the DP driver calls qcom_qmp_phy_init()? That would
> > be an easy way to move usb probe after DP probe and expose this problem.
> 
> we need usb calls qcom_qmp_phy_init() before dp.

Why?

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH] drm/msm/dp: Shorten SETUP timeout
  2021-10-08 16:44                         ` Bjorn Andersson
@ 2021-10-11 15:24                           ` khsieh
  2021-10-11 17:30                             ` Bjorn Andersson
  0 siblings, 1 reply; 17+ messages in thread
From: khsieh @ 2021-10-11 15:24 UTC (permalink / raw)
  To: Bjorn Andersson
  Cc: Stephen Boyd, Abhinav Kumar, Daniel Vetter, David Airlie,
	Dmitry Baryshkov, Rob Clark, Sean Paul, linux-arm-msm, dri-devel,
	freedreno, linux-kernel, Sankeerth Billakanti

On 2021-10-08 09:44, Bjorn Andersson wrote:
> On Fri 08 Oct 09:07 PDT 2021, khsieh@codeaurora.org wrote:
> 
>> On 2021-10-07 15:34, Stephen Boyd wrote:
>> > Quoting khsieh@codeaurora.org (2021-10-07 13:28:12)
>> > > On 2021-10-07 13:06, Bjorn Andersson wrote:
>> > > > On Thu 07 Oct 12:51 PDT 2021, khsieh@codeaurora.org wrote:
>> > > >
>> > > >> On 2021-10-06 10:31, Bjorn Andersson wrote:
>> > > >> > On Wed 06 Oct 08:37 PDT 2021, khsieh@codeaurora.org wrote:
>> > > >> >
>> > > >> > > On 2021-10-05 19:10, Bjorn Andersson wrote:
>> > > >> > > > On Tue 05 Oct 16:04 PDT 2021, khsieh@codeaurora.org wrote:
>> > > >> > > >
>> > > >> > > > > On 2021-10-05 15:36, Stephen Boyd wrote:
>> > > >> > > > > > Quoting Bjorn Andersson (2021-10-05 14:40:38)
>> > > >> > > > > > > On Tue 05 Oct 11:45 PDT 2021, Stephen Boyd wrote:
>> > > >> > > > > > >
>> > > >> > > > > > > > Quoting Bjorn Andersson (2021-10-04 19:37:50)
>> > > >> > > > > > > > > Found in the middle of a patch from Sankeerth was the reduction of the
>> > > >> > > > > > > > > INIT_SETUP timeout from 10s to 100ms. Upon INIT_SETUP timeout the host
>> > > >> > > > > > > > > is initalized and HPD interrupt start to be serviced, so in the case of
>> > > >> > > > > > > > > eDP this reduction improves the user experience dramatically - i.e.
>> > > >> > > > > > > > > removes 9.9s of bland screen time at boot.
>> > > >> > > > > > > > >
>> > > >> > > > > > > > > Suggested-by: Sankeerth Billakanti <sbillaka@codeaurora.org>
>> > > >> > > > > > > > > Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
>> > > >> > > > > > > > > ---
>> > > >> > > > > > > >
>> > > >> > > > > > > > Any Fixes tag? BTW, the delay design is pretty convoluted. I had to go
>> > > >> > > > > > > > re-read the code a couple times to understand that it's waiting 100ms
>> > > >> > > > > > > > times the 'delay' number. Whaaaaat?
>> > > >> > > > > > > >
>> > > >> > > > > > >
>> > > >> > > > > > > I assume you're happy with the current 10s delay on the current
>> > > >> > > > > > > devices, so I don't think we should push for this to be backported.
>> > > >> > > > > > > I have no need for it to be backported on my side at least.
>> > > >> > > > > > >
>> > > >> > > > > >
>> > > >> > > > > > Sure. Fixes tag != backported to stable trees but it is close.
>> > > >> > > > > >
>> > > >> > > > > > > > Reviewed-by: Stephen Boyd <swboyd@chromium.org>
>> > > >> > > > > > >
>> > > >> > > > >   dp_add_event(dp, EV_HPD_INIT_SETUP, 0, 1); <== to 100ms
>> > > >> > > > >
>> > > >> > > > > This patch will prevent usb3 from working due to dp driver
>> > > >> > > > > initialize phy
>> > > >> > > > > earlier than usb3 which cause timeout error at power up usb3 phy
>> > > >> > > > > when both
>> > > >> > > > > edp and dp are enabled.
>> > > >> > > >
>> > > >> > > > Can you please help me understand what you mean here, I use this on my
>> > > >> > > > sc8180x with both eDP and USB-C/DP right now. What is it that doesn't
>> > > >> > > > work? Or am I just lucky in some race condition?
>> > > >> > > >
>> > > >> > > > Thanks,
>> > > >> > > > Bjorn
>> > > >> > > >
>> > > >> > > The problem is seen at sc7280.
>> > > >> > > Apple dongle have both  hdmi and usb port.
>> > > >> > > plug Apple dongle into type-c, then plug DP into apple's hdmi port
>> > > >> > > and usb
>> > > >> > > mouse into apple's usb port.
>> > > >> > > If edp enabled at this time, then usb mouse will not work due to
>> > > >> > > timeout at
>> > > >> > > phy power up.
>> > > >> > >
>> > > >> >
>> > > >> > Okay, so you're saying that if the DP driver invokes phy_power_on()
>> > > >> > before the USB driver does, USB initialization fails (or at least USB
>> > > >> > doesn't work)?
>> > > >>
>> > > >> if dp driver call qcom_qmp_phy_init() before usb3 call
>> > > >> qcom_qmp_phy_init(),
>> > > >> usb3 driver will timeout at readl_poll_timeout(status, val, (val &
>> > > >> mask) ==
>> > > >> ready, 10, PHY_INIT_COMPLETE_TIMEOUT) of qcom_qmp_phy_power_on().
>> > > >
>> > > > Thanks, I will try to reproduce this on my side. So the 10 seconds here
>> > > > is strictly to give good enough time for the dwc3 driver to probe...
>> > > >
>> > > > Any idea why you're saying that this is specific to sc7280, what
>> > > > changed
>> > > > from sc7180?
>> > >
>> > > I did not have sc7180 with edp before so that i am not sure it will
>> > > happen on sc7180 or not.
>> > > The usb3 does not work when both edp and dp enabled I just seen at
>> > > sc7280.
>> > > Current at sc7280 EC is not boot up correctly when system power up.
>> > > I have to manual reboot EC from linux kernel shell before DP/usb3 can
>> > > work.
>> > > I am not sure this contribute to this problem or not.
>> > >
>> >
>> > Can you make the usb driver into a module and only load that module
>> > later in boot after the DP driver calls qcom_qmp_phy_init()? That would
>> > be an easy way to move usb probe after DP probe and expose this problem.
>> 
>> we need usb calls qcom_qmp_phy_init() before dp.
> 
> Why?
I do not know the details.
But I did see below scenario,

if dp driver call qcom_qmp_phy_init() before usb3 call
qcom_qmp_phy_init(),
usb3 driver will timeout at readl_poll_timeout(status, val, (val &
mask) ==
ready, 10, PHY_INIT_COMPLETE_TIMEOUT) of qcom_qmp_phy_power_on().

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH] drm/msm/dp: Shorten SETUP timeout
  2021-10-11 15:24                           ` khsieh
@ 2021-10-11 17:30                             ` Bjorn Andersson
  0 siblings, 0 replies; 17+ messages in thread
From: Bjorn Andersson @ 2021-10-11 17:30 UTC (permalink / raw)
  To: khsieh
  Cc: Stephen Boyd, Abhinav Kumar, Daniel Vetter, David Airlie,
	Dmitry Baryshkov, Rob Clark, Sean Paul, linux-arm-msm, dri-devel,
	freedreno, linux-kernel, Sankeerth Billakanti

On Mon 11 Oct 08:24 PDT 2021, khsieh@codeaurora.org wrote:

> On 2021-10-08 09:44, Bjorn Andersson wrote:
> > On Fri 08 Oct 09:07 PDT 2021, khsieh@codeaurora.org wrote:
> > 
> > > On 2021-10-07 15:34, Stephen Boyd wrote:
> > > > Quoting khsieh@codeaurora.org (2021-10-07 13:28:12)
> > > > > On 2021-10-07 13:06, Bjorn Andersson wrote:
> > > > > > On Thu 07 Oct 12:51 PDT 2021, khsieh@codeaurora.org wrote:
> > > > > >
> > > > > >> On 2021-10-06 10:31, Bjorn Andersson wrote:
> > > > > >> > On Wed 06 Oct 08:37 PDT 2021, khsieh@codeaurora.org wrote:
> > > > > >> >
> > > > > >> > > On 2021-10-05 19:10, Bjorn Andersson wrote:
> > > > > >> > > > On Tue 05 Oct 16:04 PDT 2021, khsieh@codeaurora.org wrote:
> > > > > >> > > >
> > > > > >> > > > > On 2021-10-05 15:36, Stephen Boyd wrote:
> > > > > >> > > > > > Quoting Bjorn Andersson (2021-10-05 14:40:38)
> > > > > >> > > > > > > On Tue 05 Oct 11:45 PDT 2021, Stephen Boyd wrote:
> > > > > >> > > > > > >
> > > > > >> > > > > > > > Quoting Bjorn Andersson (2021-10-04 19:37:50)
> > > > > >> > > > > > > > > Found in the middle of a patch from Sankeerth was the reduction of the
> > > > > >> > > > > > > > > INIT_SETUP timeout from 10s to 100ms. Upon INIT_SETUP timeout the host
> > > > > >> > > > > > > > > is initalized and HPD interrupt start to be serviced, so in the case of
> > > > > >> > > > > > > > > eDP this reduction improves the user experience dramatically - i.e.
> > > > > >> > > > > > > > > removes 9.9s of bland screen time at boot.
> > > > > >> > > > > > > > >
> > > > > >> > > > > > > > > Suggested-by: Sankeerth Billakanti <sbillaka@codeaurora.org>
> > > > > >> > > > > > > > > Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
> > > > > >> > > > > > > > > ---
> > > > > >> > > > > > > >
> > > > > >> > > > > > > > Any Fixes tag? BTW, the delay design is pretty convoluted. I had to go
> > > > > >> > > > > > > > re-read the code a couple times to understand that it's waiting 100ms
> > > > > >> > > > > > > > times the 'delay' number. Whaaaaat?
> > > > > >> > > > > > > >
> > > > > >> > > > > > >
> > > > > >> > > > > > > I assume you're happy with the current 10s delay on the current
> > > > > >> > > > > > > devices, so I don't think we should push for this to be backported.
> > > > > >> > > > > > > I have no need for it to be backported on my side at least.
> > > > > >> > > > > > >
> > > > > >> > > > > >
> > > > > >> > > > > > Sure. Fixes tag != backported to stable trees but it is close.
> > > > > >> > > > > >
> > > > > >> > > > > > > > Reviewed-by: Stephen Boyd <swboyd@chromium.org>
> > > > > >> > > > > > >
> > > > > >> > > > >   dp_add_event(dp, EV_HPD_INIT_SETUP, 0, 1); <== to 100ms
> > > > > >> > > > >
> > > > > >> > > > > This patch will prevent usb3 from working due to dp driver
> > > > > >> > > > > initialize phy
> > > > > >> > > > > earlier than usb3 which cause timeout error at power up usb3 phy
> > > > > >> > > > > when both
> > > > > >> > > > > edp and dp are enabled.
> > > > > >> > > >
> > > > > >> > > > Can you please help me understand what you mean here, I use this on my
> > > > > >> > > > sc8180x with both eDP and USB-C/DP right now. What is it that doesn't
> > > > > >> > > > work? Or am I just lucky in some race condition?
> > > > > >> > > >
> > > > > >> > > > Thanks,
> > > > > >> > > > Bjorn
> > > > > >> > > >
> > > > > >> > > The problem is seen at sc7280.
> > > > > >> > > Apple dongle have both  hdmi and usb port.
> > > > > >> > > plug Apple dongle into type-c, then plug DP into apple's hdmi port
> > > > > >> > > and usb
> > > > > >> > > mouse into apple's usb port.
> > > > > >> > > If edp enabled at this time, then usb mouse will not work due to
> > > > > >> > > timeout at
> > > > > >> > > phy power up.
> > > > > >> > >
> > > > > >> >
> > > > > >> > Okay, so you're saying that if the DP driver invokes phy_power_on()
> > > > > >> > before the USB driver does, USB initialization fails (or at least USB
> > > > > >> > doesn't work)?
> > > > > >>
> > > > > >> if dp driver call qcom_qmp_phy_init() before usb3 call
> > > > > >> qcom_qmp_phy_init(),
> > > > > >> usb3 driver will timeout at readl_poll_timeout(status, val, (val &
> > > > > >> mask) ==
> > > > > >> ready, 10, PHY_INIT_COMPLETE_TIMEOUT) of qcom_qmp_phy_power_on().
> > > > > >
> > > > > > Thanks, I will try to reproduce this on my side. So the 10 seconds here
> > > > > > is strictly to give good enough time for the dwc3 driver to probe...
> > > > > >
> > > > > > Any idea why you're saying that this is specific to sc7280, what
> > > > > > changed
> > > > > > from sc7180?
> > > > >
> > > > > I did not have sc7180 with edp before so that i am not sure it will
> > > > > happen on sc7180 or not.
> > > > > The usb3 does not work when both edp and dp enabled I just seen at
> > > > > sc7280.
> > > > > Current at sc7280 EC is not boot up correctly when system power up.
> > > > > I have to manual reboot EC from linux kernel shell before DP/usb3 can
> > > > > work.
> > > > > I am not sure this contribute to this problem or not.
> > > > >
> > > >
> > > > Can you make the usb driver into a module and only load that module
> > > > later in boot after the DP driver calls qcom_qmp_phy_init()? That would
> > > > be an easy way to move usb probe after DP probe and expose this problem.
> > > 
> > > we need usb calls qcom_qmp_phy_init() before dp.
> > 
> > Why?
> I do not know the details.
> But I did see below scenario,
> 
> if dp driver call qcom_qmp_phy_init() before usb3 call
> qcom_qmp_phy_init(),
> usb3 driver will timeout at readl_poll_timeout(status, val, (val &
> mask) ==
> ready, 10, PHY_INIT_COMPLETE_TIMEOUT) of qcom_qmp_phy_power_on().

Sounds like a bug in the QMP driver, something that could easily be
reproduced by waiting for DP to be up before we bring up USB - either by
chance, or following Stephen's suggestion on purpose.

Someone will have to look into this, so that we don't need a 10 second
workaround in the (e)DP driver.

Regards,
Bjorn

^ permalink raw reply	[flat|nested] 17+ messages in thread

end of thread, other threads:[~2021-10-11 17:28 UTC | newest]

Thread overview: 17+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-10-05  2:37 [PATCH] drm/msm/dp: Shorten SETUP timeout Bjorn Andersson
2021-10-05 18:45 ` Stephen Boyd
2021-10-05 21:40   ` Bjorn Andersson
2021-10-05 22:36     ` Stephen Boyd
2021-10-05 23:04       ` khsieh
2021-10-05 23:35         ` Stephen Boyd
2021-10-06  2:10         ` Bjorn Andersson
2021-10-06 15:37           ` khsieh
2021-10-06 17:31             ` Bjorn Andersson
2021-10-07 19:51               ` khsieh
2021-10-07 20:06                 ` Bjorn Andersson
2021-10-07 20:28                   ` khsieh
2021-10-07 22:34                     ` Stephen Boyd
2021-10-08 16:07                       ` khsieh
2021-10-08 16:44                         ` Bjorn Andersson
2021-10-11 15:24                           ` khsieh
2021-10-11 17:30                             ` Bjorn Andersson

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).