Re: [Intel-gfx] [PATCH] drm/i915/dp: return proper DPRX link training result
Hi, > -Original Message- > From: Intel-gfx On Behalf Of Imre > Deak > Sent: maanantai 30. elokuuta 2021 9.45 > To: Ville Syrjälä > Cc: Lee, Shawn C ; Almahallawy, Khaled > ; intel-gfx@lists.freedesktop.org; Chiou, Cooper > ; Tseng, William > Subject: Re: [Intel-gfx] [PATCH] drm/i915/dp: return proper DPRX link > training result > > On Sat, Aug 21, 2021 at 02:02:03AM +0300, Imre Deak wrote: > > On Sat, Aug 21, 2021 at 01:20:04AM +0300, Ville Syrjälä wrote: > > > On Wed, Aug 18, 2021 at 07:17:12PM +0300, Imre Deak wrote: > > > > On Wed, Aug 18, 2021 at 06:09:43PM +0300, Lee, Shawn C wrote: > > > > > On Tue, 2021-07-07, Lee Shawn C wrote: > > > > > >On Tue, 2021-07-07, Almahallawy, Khaled > > > > > > > wrote: > > > > > >>I believe Imre's LT fallback: > > > > > >>https://github.com/ideak/linux/commits/linktraining-fallback-fix > > > > > >>and > Chrome user space fix: > > > > > >>https://chromium-review.googlesource.com/c/chromium/src/+/3003 > > > > > >>487 should address Chrome concerns for LT failure and LTTPRs > > > > > >> > > > > > > > > > > > >Thanks for comment! The new fallback patch should help on this DPRX > problem. > > > > > >One more thing. If driver did not handle DPRX link train failed > > > > > >properly. > > > > > >It would impact link layer compliance test case in below. > > > > > > > > > > > >400.3.1.3 > > > > > >400.3.1.4 > > > > > >400.3.1.6 > > > > > >400.3.1.12 > > > > > >400.3.1.13 > > > > > >400.3.1.14 > > > > > > > > > > > >Best regards, > > > > > >Shawn > > > > > > > > > > > > > > > > Hi all, before Imre's patch series land on upstream driver. The > > > > > link train failed handling works for LTTPR only. But DPRX does > > > > > not. Could you please consider to have this change as temporary > > > > > solution? > Thanks! > > > > > > > > I sent already fixing this, see > > > > https://lore.kernel.org/intel-gfx/20201027133600.3656665-1-imre.de > > > > a...@intel.com/ > > > > > > > > but it fell through the cracks. Applied now your patch, thanks. > > > > > > We seem to have a tgl that fails consistently at DPRX link training: > > > https://intel-gfx-ci.01.org/tree/drm-tip/fi-tgl-1115g4.html > > > > > > Previously the error went unnoticed. > > > > Yea, didn't notice this. Can't see anything obvious, besides that it's > > a DPCD rev 1.1 monitor, so maybe not compatible with LTTPRs. I follow > > up if I find something. > > Power cycling the machine and replugging the monitor and Type C dongle and > native > DP cable fixed it. So, it could be a flaky connection (unlikely, because > detection > worked), or the port in a stuck state even across warm-reboots. The latter > would > mean the machine wasn't power cycled for a long time and that's likely > because the > pstore kernel didn't boot (grub failed to load it due to a missing initrd). Thanks Imre for checking. > > One possibile root cause is the TypeC PHY getting stuck across warm reboots > issue > (and nothing recovering it besides a power-cycle), that was fixed by > 151ec347b06a2fb ("drm/i915: Force a TypeC PHY disconnect during > suspend/shutdown") > > I'll re-check this after more CI testruns. Ack but now looks indeed green again. > > > > -- > > > Ville Syrjälä > > > Intel
Re: [Intel-gfx] [PATCH] drm/i915/dp: return proper DPRX link training result
On Sat, Aug 21, 2021 at 02:02:03AM +0300, Imre Deak wrote: > On Sat, Aug 21, 2021 at 01:20:04AM +0300, Ville Syrjälä wrote: > > On Wed, Aug 18, 2021 at 07:17:12PM +0300, Imre Deak wrote: > > > On Wed, Aug 18, 2021 at 06:09:43PM +0300, Lee, Shawn C wrote: > > > > On Tue, 2021-07-07, Lee Shawn C wrote: > > > > >On Tue, 2021-07-07, Almahallawy, Khaled > > > > >wrote: > > > > >>I believe Imre's LT fallback: > > > > >>https://github.com/ideak/linux/commits/linktraining-fallback-fix and > > > > >>Chrome user space fix: > > > > >>https://chromium-review.googlesource.com/c/chromium/src/+/3003487 > > > > >>should address Chrome concerns for LT failure and LTTPRs > > > > >> > > > > > > > > > >Thanks for comment! The new fallback patch should help on this DPRX > > > > >problem. > > > > >One more thing. If driver did not handle DPRX link train failed > > > > >properly. > > > > >It would impact link layer compliance test case in below. > > > > > > > > > >400.3.1.3 > > > > >400.3.1.4 > > > > >400.3.1.6 > > > > >400.3.1.12 > > > > >400.3.1.13 > > > > >400.3.1.14 > > > > > > > > > >Best regards, > > > > >Shawn > > > > > > > > > > > > > Hi all, before Imre's patch series land on upstream driver. The link > > > > train failed > > > > handling works for LTTPR only. But DPRX does not. Could you please > > > > consider to have > > > > this change as temporary solution? Thanks! > > > > > > I sent already fixing this, see > > > https://lore.kernel.org/intel-gfx/20201027133600.3656665-1-imre.d...@intel.com/ > > > > > > but it fell through the cracks. Applied now your patch, thanks. > > > > We seem to have a tgl that fails consistently at DPRX link training: > > https://intel-gfx-ci.01.org/tree/drm-tip/fi-tgl-1115g4.html > > > > Previously the error went unnoticed. > > Yea, didn't notice this. Can't see anything obvious, besides that it's a > DPCD rev 1.1 monitor, so maybe not compatible with LTTPRs. I follow up > if I find something. Power cycling the machine and replugging the monitor and Type C dongle and native DP cable fixed it. So, it could be a flaky connection (unlikely, because detection worked), or the port in a stuck state even across warm-reboots. The latter would mean the machine wasn't power cycled for a long time and that's likely because the pstore kernel didn't boot (grub failed to load it due to a missing initrd). One possibile root cause is the TypeC PHY getting stuck across warm reboots issue (and nothing recovering it besides a power-cycle), that was fixed by 151ec347b06a2fb ("drm/i915: Force a TypeC PHY disconnect during suspend/shutdown") I'll re-check this after more CI testruns. > > -- > > Ville Syrjälä > > Intel
Re: [Intel-gfx] [PATCH] drm/i915/dp: return proper DPRX link training result
On Sat, Aug 21, 2021 at 02:02:03AM +0300, Imre Deak wrote: > On Sat, Aug 21, 2021 at 01:20:04AM +0300, Ville Syrjälä wrote: > > On Wed, Aug 18, 2021 at 07:17:12PM +0300, Imre Deak wrote: > > > On Wed, Aug 18, 2021 at 06:09:43PM +0300, Lee, Shawn C wrote: > > > > On Tue, 2021-07-07, Lee Shawn C wrote: > > > > >On Tue, 2021-07-07, Almahallawy, Khaled > > > > >wrote: > > > > >>I believe Imre's LT fallback: > > > > >>https://github.com/ideak/linux/commits/linktraining-fallback-fix and > > > > >>Chrome user space fix: > > > > >>https://chromium-review.googlesource.com/c/chromium/src/+/3003487 > > > > >>should address Chrome concerns for LT failure and LTTPRs > > > > >> > > > > > > > > > >Thanks for comment! The new fallback patch should help on this DPRX > > > > >problem. > > > > >One more thing. If driver did not handle DPRX link train failed > > > > >properly. > > > > >It would impact link layer compliance test case in below. > > > > > > > > > >400.3.1.3 > > > > >400.3.1.4 > > > > >400.3.1.6 > > > > >400.3.1.12 > > > > >400.3.1.13 > > > > >400.3.1.14 > > > > > > > > > >Best regards, > > > > >Shawn > > > > > > > > > > > > > Hi all, before Imre's patch series land on upstream driver. The link > > > > train failed > > > > handling works for LTTPR only. But DPRX does not. Could you please > > > > consider to have > > > > this change as temporary solution? Thanks! > > > > > > I sent already fixing this, see > > > https://lore.kernel.org/intel-gfx/20201027133600.3656665-1-imre.d...@intel.com/ > > > > > > but it fell through the cracks. Applied now your patch, thanks. > > > > We seem to have a tgl that fails consistently at DPRX link training: > > https://intel-gfx-ci.01.org/tree/drm-tip/fi-tgl-1115g4.html > > > > Previously the error went unnoticed. > > Yea, didn't notice this. Can't see anything obvious, besides that it's a > DPCD rev 1.1 monitor, so maybe not compatible with LTTPRs. I follow up > if I find something. I opened this thread exactly to tell that I'm avoiding this patch on this week's pull request targeting 5.14 exactly because I saw something strange with CI_DIF_604 on TGL that doesn't happen without this patch CI_DIF_603. Since I don't know what's going on there I'm also avoiding d7f213c131ad ("drm/i915/dp: Use max params for panels < eDP 1.4") just in case... > > > > > -- > > Ville Syrjälä > > Intel
Re: [Intel-gfx] [PATCH] drm/i915/dp: return proper DPRX link training result
Here is the gitlab issue https://gitlab.freedesktop.org/drm/intel/-/issues/4002 Lakshmi. -Original Message- From: Saarinen, Jani Sent: Monday, August 23, 2021 1:39 AM To: Ville Syrjälä ; Deak, Imre ; Vudum, Lakshminarayana Cc: Lee, Shawn C ; Almahallawy, Khaled ; intel-gfx@lists.freedesktop.org; Chiou, Cooper ; Tseng, William Subject: RE: [Intel-gfx] [PATCH] drm/i915/dp: return proper DPRX link training result Hi, > -Original Message- > From: Intel-gfx On Behalf Of > Ville Syrjälä > Sent: lauantai 21. elokuuta 2021 1.20 > To: Deak, Imre > Cc: Lee, Shawn C ; Almahallawy, Khaled > ; intel-gfx@lists.freedesktop.org; > Chiou, Cooper ; Tseng, William > > Subject: Re: [Intel-gfx] [PATCH] drm/i915/dp: return proper DPRX link > training result > > On Wed, Aug 18, 2021 at 07:17:12PM +0300, Imre Deak wrote: > > On Wed, Aug 18, 2021 at 06:09:43PM +0300, Lee, Shawn C wrote: > > > On Tue, 2021-07-07, Lee Shawn C wrote: > > > >On Tue, 2021-07-07, Almahallawy, Khaled > > > > > wrote: > > > >>I believe Imre's LT fallback: > > > >>https://github.com/ideak/linux/commits/linktraining-fallback-fix > > > >>and Chrome > user space fix: > > > >>https://chromium-review.googlesource.com/c/chromium/src/+/300348 > > > >>7 should address Chrome concerns for LT failure and LTTPRs > > > >> > > > > > > > >Thanks for comment! The new fallback patch should help on this DPRX > > > >problem. > > > >One more thing. If driver did not handle DPRX link train failed properly. > > > >It would impact link layer compliance test case in below. > > > > > > > >400.3.1.3 > > > >400.3.1.4 > > > >400.3.1.6 > > > >400.3.1.12 > > > >400.3.1.13 > > > >400.3.1.14 > > > > > > > >Best regards, > > > >Shawn > > > > > > > > > > Hi all, before Imre's patch series land on upstream driver. The > > > link train failed handling works for LTTPR only. But DPRX does > > > not. Could you please consider to have this change as temporary solution? > > > Thanks! > > > > I sent already fixing this, see > > https://lore.kernel.org/intel-gfx/20201027133600.3656665-1-imre.deak > > @i > > ntel.com/ > > > > but it fell through the cracks. Applied now your patch, thanks. > > We seem to have a tgl that fails consistently at DPRX link training: > https://intel-gfx-ci.01.org/tree/drm-tip/fi-tgl-1115g4.html Yes, @Vudum, Lakshminarayana what is gitlab issue for this? [Lakshmi] https://gitlab.freedesktop.org/drm/intel/-/issues/4002 > > Previously the error went unnoticed. > > -- > Ville Syrjälä > Intel
Re: [Intel-gfx] [PATCH] drm/i915/dp: return proper DPRX link training result
Hi, > -Original Message- > From: Intel-gfx On Behalf Of Ville > Syrjälä > Sent: lauantai 21. elokuuta 2021 1.20 > To: Deak, Imre > Cc: Lee, Shawn C ; Almahallawy, Khaled > ; intel-gfx@lists.freedesktop.org; Chiou, Cooper > ; Tseng, William > Subject: Re: [Intel-gfx] [PATCH] drm/i915/dp: return proper DPRX link > training result > > On Wed, Aug 18, 2021 at 07:17:12PM +0300, Imre Deak wrote: > > On Wed, Aug 18, 2021 at 06:09:43PM +0300, Lee, Shawn C wrote: > > > On Tue, 2021-07-07, Lee Shawn C wrote: > > > >On Tue, 2021-07-07, Almahallawy, Khaled > wrote: > > > >>I believe Imre's LT fallback: > > > >>https://github.com/ideak/linux/commits/linktraining-fallback-fix and > > > >>Chrome > user space fix: > > > >>https://chromium-review.googlesource.com/c/chromium/src/+/3003487 > > > >>should address Chrome concerns for LT failure and LTTPRs > > > >> > > > > > > > >Thanks for comment! The new fallback patch should help on this DPRX > > > >problem. > > > >One more thing. If driver did not handle DPRX link train failed properly. > > > >It would impact link layer compliance test case in below. > > > > > > > >400.3.1.3 > > > >400.3.1.4 > > > >400.3.1.6 > > > >400.3.1.12 > > > >400.3.1.13 > > > >400.3.1.14 > > > > > > > >Best regards, > > > >Shawn > > > > > > > > > > Hi all, before Imre's patch series land on upstream driver. The link > > > train failed handling works for LTTPR only. But DPRX does not. Could > > > you please consider to have this change as temporary solution? Thanks! > > > > I sent already fixing this, see > > https://lore.kernel.org/intel-gfx/20201027133600.3656665-1-imre.deak@i > > ntel.com/ > > > > but it fell through the cracks. Applied now your patch, thanks. > > We seem to have a tgl that fails consistently at DPRX link training: > https://intel-gfx-ci.01.org/tree/drm-tip/fi-tgl-1115g4.html Yes, @Vudum, Lakshminarayana what is gitlab issue for this? > > Previously the error went unnoticed. > > -- > Ville Syrjälä > Intel
Re: [Intel-gfx] [PATCH] drm/i915/dp: return proper DPRX link training result
On Sat, Aug 21, 2021 at 01:20:04AM +0300, Ville Syrjälä wrote: > On Wed, Aug 18, 2021 at 07:17:12PM +0300, Imre Deak wrote: > > On Wed, Aug 18, 2021 at 06:09:43PM +0300, Lee, Shawn C wrote: > > > On Tue, 2021-07-07, Lee Shawn C wrote: > > > >On Tue, 2021-07-07, Almahallawy, Khaled > > > >wrote: > > > >>I believe Imre's LT fallback: > > > >>https://github.com/ideak/linux/commits/linktraining-fallback-fix and > > > >>Chrome user space fix: > > > >>https://chromium-review.googlesource.com/c/chromium/src/+/3003487 > > > >>should address Chrome concerns for LT failure and LTTPRs > > > >> > > > > > > > >Thanks for comment! The new fallback patch should help on this DPRX > > > >problem. > > > >One more thing. If driver did not handle DPRX link train failed properly. > > > >It would impact link layer compliance test case in below. > > > > > > > >400.3.1.3 > > > >400.3.1.4 > > > >400.3.1.6 > > > >400.3.1.12 > > > >400.3.1.13 > > > >400.3.1.14 > > > > > > > >Best regards, > > > >Shawn > > > > > > > > > > Hi all, before Imre's patch series land on upstream driver. The link > > > train failed > > > handling works for LTTPR only. But DPRX does not. Could you please > > > consider to have > > > this change as temporary solution? Thanks! > > > > I sent already fixing this, see > > https://lore.kernel.org/intel-gfx/20201027133600.3656665-1-imre.d...@intel.com/ > > > > but it fell through the cracks. Applied now your patch, thanks. > > We seem to have a tgl that fails consistently at DPRX link training: > https://intel-gfx-ci.01.org/tree/drm-tip/fi-tgl-1115g4.html > > Previously the error went unnoticed. Yea, didn't notice this. Can't see anything obvious, besides that it's a DPCD rev 1.1 monitor, so maybe not compatible with LTTPRs. I follow up if I find something. > > -- > Ville Syrjälä > Intel
Re: [Intel-gfx] [PATCH] drm/i915/dp: return proper DPRX link training result
On Wed, Aug 18, 2021 at 07:17:12PM +0300, Imre Deak wrote: > On Wed, Aug 18, 2021 at 06:09:43PM +0300, Lee, Shawn C wrote: > > On Tue, 2021-07-07, Lee Shawn C wrote: > > >On Tue, 2021-07-07, Almahallawy, Khaled > > >wrote: > > >>I believe Imre's LT fallback: > > >>https://github.com/ideak/linux/commits/linktraining-fallback-fix and > > >>Chrome user space fix: > > >>https://chromium-review.googlesource.com/c/chromium/src/+/3003487 > > >>should address Chrome concerns for LT failure and LTTPRs > > >> > > > > > >Thanks for comment! The new fallback patch should help on this DPRX > > >problem. > > >One more thing. If driver did not handle DPRX link train failed properly. > > >It would impact link layer compliance test case in below. > > > > > >400.3.1.3 > > >400.3.1.4 > > >400.3.1.6 > > >400.3.1.12 > > >400.3.1.13 > > >400.3.1.14 > > > > > >Best regards, > > >Shawn > > > > > > > Hi all, before Imre's patch series land on upstream driver. The link train > > failed > > handling works for LTTPR only. But DPRX does not. Could you please consider > > to have > > this change as temporary solution? Thanks! > > I sent already fixing this, see > https://lore.kernel.org/intel-gfx/20201027133600.3656665-1-imre.d...@intel.com/ > > but it fell through the cracks. Applied now your patch, thanks. We seem to have a tgl that fails consistently at DPRX link training: https://intel-gfx-ci.01.org/tree/drm-tip/fi-tgl-1115g4.html Previously the error went unnoticed. -- Ville Syrjälä Intel
Re: [Intel-gfx] [PATCH] drm/i915/dp: return proper DPRX link training result
On Wed, Aug 18, 2021 at 06:09:43PM +0300, Lee, Shawn C wrote: > On Tue, 2021-07-07, Lee Shawn C wrote: > >On Tue, 2021-07-07, Almahallawy, Khaled wrote: > >>I believe Imre's LT fallback: > >>https://github.com/ideak/linux/commits/linktraining-fallback-fix and > >>Chrome user space fix: > >>https://chromium-review.googlesource.com/c/chromium/src/+/3003487 > >>should address Chrome concerns for LT failure and LTTPRs > >> > > > >Thanks for comment! The new fallback patch should help on this DPRX problem. > >One more thing. If driver did not handle DPRX link train failed properly. > >It would impact link layer compliance test case in below. > > > >400.3.1.3 > >400.3.1.4 > >400.3.1.6 > >400.3.1.12 > >400.3.1.13 > >400.3.1.14 > > > >Best regards, > >Shawn > > > > Hi all, before Imre's patch series land on upstream driver. The link train > failed > handling works for LTTPR only. But DPRX does not. Could you please consider > to have > this change as temporary solution? Thanks! I sent already fixing this, see https://lore.kernel.org/intel-gfx/20201027133600.3656665-1-imre.d...@intel.com/ but it fell through the cracks. Applied now your patch, thanks. > > Best regards, > Shawn > > >>Thanks > >>Khaled > >> > >>On Tue, 2021-07-06 at 23:25 +0800, Lee Shawn C wrote: > >>> After DPRX link training, intel_dp_link_train_phy() did not return > >>> the training result properly. If link training failed, > >>> i915 driver would not run into link train fallback function. > >>> And no hotplug uevent would be received by user space application. > >>> > >>> Fixes: b30edfd8d0b4 ("drm/i915: Switch to LTTPR non-transparent mode > >>> link training") > >>> Cc: Ville Syrjala > >>> Cc: Imre Deak > >>> Cc: Jani Nikula > >>> Cc: Cooper Chiou > >>> Cc: William Tseng > >>> Signed-off-by: Lee Shawn C > >>> --- > >>> drivers/gpu/drm/i915/display/intel_dp_link_training.c | 2 +- > >>> 1 file changed, 1 insertion(+), 1 deletion(-) > >>> > >>> diff --git a/drivers/gpu/drm/i915/display/intel_dp_link_training.c > >>> b/drivers/gpu/drm/i915/display/intel_dp_link_training.c > >>> index 08bceae40aa8..e44788b2c564 100644 > >>> --- a/drivers/gpu/drm/i915/display/intel_dp_link_training.c > >>> +++ b/drivers/gpu/drm/i915/display/intel_dp_link_training.c > >>> @@ -849,7 +849,7 @@ intel_dp_link_train_all_phys(struct intel_dp > >>> *intel_dp, > >>> } > >>> > >>> if (ret) > >>> - intel_dp_link_train_phy(intel_dp, crtc_state, > >>> DP_PHY_DPRX); > >>> + ret = intel_dp_link_train_phy(intel_dp, crtc_state, > >>> DP_PHY_DPRX); > >>> > >>> if (intel_dp->set_idle_link_train) > >>> intel_dp->set_idle_link_train(intel_dp, crtc_state); > >> > >___ > >Intel-gfx mailing list > >Intel-gfx@lists.freedesktop.org > >https://lists.freedesktop.org/mailman/listinfo/intel-gfx
Re: [Intel-gfx] [PATCH] drm/i915/dp: return proper DPRX link training result
On Tue, 2021-07-07, Lee Shawn C wrote: >On Tue, 2021-07-07, Almahallawy, Khaled wrote: >>I believe Imre's LT fallback: >>https://github.com/ideak/linux/commits/linktraining-fallback-fix and Chrome >>user space fix: >>https://chromium-review.googlesource.com/c/chromium/src/+/3003487 >>should address Chrome concerns for LT failure and LTTPRs >> > >Thanks for comment! The new fallback patch should help on this DPRX problem. >One more thing. If driver did not handle DPRX link train failed properly. >It would impact link layer compliance test case in below. > >400.3.1.3 >400.3.1.4 >400.3.1.6 >400.3.1.12 >400.3.1.13 >400.3.1.14 > >Best regards, >Shawn > Hi all, before Imre's patch series land on upstream driver. The link train failed handling works for LTTPR only. But DPRX does not. Could you please consider to have this change as temporary solution? Thanks! Best regards, Shawn >>Thanks >>Khaled >> >>On Tue, 2021-07-06 at 23:25 +0800, Lee Shawn C wrote: >>> After DPRX link training, intel_dp_link_train_phy() did not return >>> the training result properly. If link training failed, >>> i915 driver would not run into link train fallback function. >>> And no hotplug uevent would be received by user space application. >>> >>> Fixes: b30edfd8d0b4 ("drm/i915: Switch to LTTPR non-transparent mode >>> link training") >>> Cc: Ville Syrjala >>> Cc: Imre Deak >>> Cc: Jani Nikula >>> Cc: Cooper Chiou >>> Cc: William Tseng >>> Signed-off-by: Lee Shawn C >>> --- >>> drivers/gpu/drm/i915/display/intel_dp_link_training.c | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> diff --git a/drivers/gpu/drm/i915/display/intel_dp_link_training.c >>> b/drivers/gpu/drm/i915/display/intel_dp_link_training.c >>> index 08bceae40aa8..e44788b2c564 100644 >>> --- a/drivers/gpu/drm/i915/display/intel_dp_link_training.c >>> +++ b/drivers/gpu/drm/i915/display/intel_dp_link_training.c >>> @@ -849,7 +849,7 @@ intel_dp_link_train_all_phys(struct intel_dp >>> *intel_dp, >>> } >>> >>> if (ret) >>> - intel_dp_link_train_phy(intel_dp, crtc_state, >>> DP_PHY_DPRX); >>> + ret = intel_dp_link_train_phy(intel_dp, crtc_state, >>> DP_PHY_DPRX); >>> >>> if (intel_dp->set_idle_link_train) >>> intel_dp->set_idle_link_train(intel_dp, crtc_state); >> >___ >Intel-gfx mailing list >Intel-gfx@lists.freedesktop.org >https://lists.freedesktop.org/mailman/listinfo/intel-gfx
Re: [Intel-gfx] [PATCH] drm/i915/dp: return proper DPRX link training result
On Tue, 2021-07-07, Almahallawy, Khaled wrote: >I believe Imre's LT fallback: >https://github.com/ideak/linux/commits/linktraining-fallback-fix and Chrome >user space fix: >https://chromium-review.googlesource.com/c/chromium/src/+/3003487 >should address Chrome concerns for LT failure and LTTPRs > Thanks for comment! The new fallback patch should help on this DPRX problem. One more thing. If driver did not handle DPRX link train failed properly. It would impact link layer compliance test case in below. 400.3.1.3 400.3.1.4 400.3.1.6 400.3.1.12 400.3.1.13 400.3.1.14 Best regards, Shawn >Thanks >Khaled > >On Tue, 2021-07-06 at 23:25 +0800, Lee Shawn C wrote: >> After DPRX link training, intel_dp_link_train_phy() did not return the >> training result properly. If link training failed, >> i915 driver would not run into link train fallback function. >> And no hotplug uevent would be received by user space application. >> >> Fixes: b30edfd8d0b4 ("drm/i915: Switch to LTTPR non-transparent mode >> link training") >> Cc: Ville Syrjala >> Cc: Imre Deak >> Cc: Jani Nikula >> Cc: Cooper Chiou >> Cc: William Tseng >> Signed-off-by: Lee Shawn C >> --- >> drivers/gpu/drm/i915/display/intel_dp_link_training.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/drivers/gpu/drm/i915/display/intel_dp_link_training.c >> b/drivers/gpu/drm/i915/display/intel_dp_link_training.c >> index 08bceae40aa8..e44788b2c564 100644 >> --- a/drivers/gpu/drm/i915/display/intel_dp_link_training.c >> +++ b/drivers/gpu/drm/i915/display/intel_dp_link_training.c >> @@ -849,7 +849,7 @@ intel_dp_link_train_all_phys(struct intel_dp >> *intel_dp, >> } >> >> if (ret) >> -intel_dp_link_train_phy(intel_dp, crtc_state, >> DP_PHY_DPRX); >> +ret = intel_dp_link_train_phy(intel_dp, crtc_state, >> DP_PHY_DPRX); >> >> if (intel_dp->set_idle_link_train) >> intel_dp->set_idle_link_train(intel_dp, crtc_state); > ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx
Re: [Intel-gfx] [PATCH] drm/i915/dp: return proper DPRX link training result
I believe Imre’s LT fallback: https://github.com/ideak/linux/commits/linktraining-fallback-fix and Chrome user space fix: https://chromium-review.googlesource.com/c/chromium/src/+/3003487 should address Chrome concerns for LT failure and LTTPRs Thanks Khaled On Tue, 2021-07-06 at 23:25 +0800, Lee Shawn C wrote: > After DPRX link training, intel_dp_link_train_phy() did not > return the training result properly. If link training failed, > i915 driver would not run into link train fallback function. > And no hotplug uevent would be received by user space application. > > Fixes: b30edfd8d0b4 ("drm/i915: Switch to LTTPR non-transparent mode > link training") > Cc: Ville Syrjala > Cc: Imre Deak > Cc: Jani Nikula > Cc: Cooper Chiou > Cc: William Tseng > Signed-off-by: Lee Shawn C > --- > drivers/gpu/drm/i915/display/intel_dp_link_training.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/i915/display/intel_dp_link_training.c > b/drivers/gpu/drm/i915/display/intel_dp_link_training.c > index 08bceae40aa8..e44788b2c564 100644 > --- a/drivers/gpu/drm/i915/display/intel_dp_link_training.c > +++ b/drivers/gpu/drm/i915/display/intel_dp_link_training.c > @@ -849,7 +849,7 @@ intel_dp_link_train_all_phys(struct intel_dp > *intel_dp, > } > > if (ret) > - intel_dp_link_train_phy(intel_dp, crtc_state, > DP_PHY_DPRX); > + ret = intel_dp_link_train_phy(intel_dp, crtc_state, > DP_PHY_DPRX); > > if (intel_dp->set_idle_link_train) > intel_dp->set_idle_link_train(intel_dp, crtc_state); ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx
[Intel-gfx] [PATCH] drm/i915/dp: return proper DPRX link training result
After DPRX link training, intel_dp_link_train_phy() did not return the training result properly. If link training failed, i915 driver would not run into link train fallback function. And no hotplug uevent would be received by user space application. Fixes: b30edfd8d0b4 ("drm/i915: Switch to LTTPR non-transparent mode link training") Cc: Ville Syrjala Cc: Imre Deak Cc: Jani Nikula Cc: Cooper Chiou Cc: William Tseng Signed-off-by: Lee Shawn C --- drivers/gpu/drm/i915/display/intel_dp_link_training.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/display/intel_dp_link_training.c b/drivers/gpu/drm/i915/display/intel_dp_link_training.c index 08bceae40aa8..e44788b2c564 100644 --- a/drivers/gpu/drm/i915/display/intel_dp_link_training.c +++ b/drivers/gpu/drm/i915/display/intel_dp_link_training.c @@ -849,7 +849,7 @@ intel_dp_link_train_all_phys(struct intel_dp *intel_dp, } if (ret) - intel_dp_link_train_phy(intel_dp, crtc_state, DP_PHY_DPRX); + ret = intel_dp_link_train_phy(intel_dp, crtc_state, DP_PHY_DPRX); if (intel_dp->set_idle_link_train) intel_dp->set_idle_link_train(intel_dp, crtc_state); -- 2.17.1 ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx