RE: [PATCH v5 6/9] drm/msm/dp: wait for hpd high before any sink interaction

2022-03-25 Thread Sankeerth Billakanti
; > > ; Kuogee Hsieh (QUIC) > > > ; Andy Gross ; > > > bjorn.anders...@linaro.org; Rob Herring ; > > > krzk...@kernel.org; Sean Paul ; David Airlie > > > ; Daniel Vetter ; Thierry Reding > > > ; Sam Ravnborg ; > > > dmitry.barysh...@linaro.org; quic_vproddut

Re: [PATCH v5 6/9] drm/msm/dp: wait for hpd high before any sink interaction

2022-03-25 Thread Doug Anderson
linaro.org; Rob Herring ; > > krzk...@kernel.org; Sean Paul ; David Airlie > > ; Daniel Vetter ; Thierry Reding > > ; Sam Ravnborg ; > > dmitry.barysh...@linaro.org; quic_vproddut > > Subject: Re: [PATCH v5 6/9] drm/msm/dp: wait for hpd high before any sink > >

RE: [PATCH v5 6/9] drm/msm/dp: wait for hpd high before any sink interaction

2022-03-25 Thread Sankeerth Billakanti (QUIC)
try.barysh...@linaro.org; quic_vproddut > Subject: Re: [PATCH v5 6/9] drm/msm/dp: wait for hpd high before any sink > interaction > > Hi, > > On Fri, Mar 18, 2022 at 4:27 PM Stephen Boyd > wrote: > > > > > > Pushing hpd state checking into aux transacti

Re: [PATCH v5 6/9] drm/msm/dp: wait for hpd high before any sink interaction

2022-03-18 Thread Doug Anderson
Hi, On Fri, Mar 18, 2022 at 4:27 PM Stephen Boyd wrote: > > > > Pushing hpd state checking into aux > > > transactions looks like the wrong direction. Also, as I said up above I > > > am concerned that even checking the GPIO won't work and we need some way > > > to ask the bridge if HPD is

Re: [PATCH v5 6/9] drm/msm/dp: wait for hpd high before any sink interaction

2022-03-18 Thread Stephen Boyd
Quoting Doug Anderson (2022-03-18 14:58:55) > Hi, > > On Fri, Mar 18, 2022 at 1:17 PM Stephen Boyd wrote: > > > > > > > + ret = dp_catalog_aux_wait_for_hpd_connect_state(aux->catalog); > > > > > > > > Why are we making aux transactions when hpd isn't asserted? Can we only > > > > register

Re: [PATCH v5 6/9] drm/msm/dp: wait for hpd high before any sink interaction

2022-03-18 Thread Doug Anderson
Hi, On Fri, Mar 18, 2022 at 1:17 PM Stephen Boyd wrote: > > > > > + ret = dp_catalog_aux_wait_for_hpd_connect_state(aux->catalog); > > > > > > Why are we making aux transactions when hpd isn't asserted? Can we only > > > register the aux device once we know that state is "connected"? I'm >

Re: [PATCH v5 6/9] drm/msm/dp: wait for hpd high before any sink interaction

2022-03-18 Thread Stephen Boyd
Quoting Doug Anderson (2022-03-18 09:24:17) > Hi, > > On Thu, Mar 17, 2022 at 6:19 PM Stephen Boyd wrote: > > > > Quoting Sankeerth Billakanti (2022-03-16 10:35:51) > > > The source device should ensure the sink is ready before > > > proceeding to read the sink capability or performing

Re: [PATCH v5 6/9] drm/msm/dp: wait for hpd high before any sink interaction

2022-03-18 Thread Doug Anderson
Hi, On Thu, Mar 17, 2022 at 6:19 PM Stephen Boyd wrote: > > Quoting Sankeerth Billakanti (2022-03-16 10:35:51) > > The source device should ensure the sink is ready before > > proceeding to read the sink capability or performing any aux transactions. > > The sink will indicate its

Re: [PATCH v5 6/9] drm/msm/dp: wait for hpd high before any sink interaction

2022-03-17 Thread Stephen Boyd
Quoting Sankeerth Billakanti (2022-03-16 10:35:51) > The source device should ensure the sink is ready before > proceeding to read the sink capability or performing any aux transactions. > The sink will indicate its readiness by asserting the HPD line. > > The eDP sink requires