* [PATCH 1/1] phy: tegra: xusb: Explicitly configure HS_DISCON_LEVEL to 0x7
@ 2025-12-12 3:21 Wayne Chang
2025-12-24 7:32 ` Vinod Koul
0 siblings, 1 reply; 2+ messages in thread
From: Wayne Chang @ 2025-12-12 3:21 UTC (permalink / raw)
To: jckuo, vkoul, kishon, thierry.reding, jonathanh
Cc: waynec, haotienh, linux-phy, linux-tegra, linux-kernel, stable
The USB2 Bias Pad Control register manages analog parameters for signal
detection. Previously, the HS_DISCON_LEVEL relied on hardware reset
values, which may lead to the detection failure.
Explicitly configure HS_DISCON_LEVEL to 0x7. This ensures the disconnect
threshold is sufficient to guarantee reliable detection.
Fixes: bbf711682cd5 ("phy: tegra: xusb: Add Tegra186 support")
Cc: stable@vger.kernel.org
Signed-off-by: Wayne Chang <waynec@nvidia.com>
---
drivers/phy/tegra/xusb-tegra186.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/phy/tegra/xusb-tegra186.c b/drivers/phy/tegra/xusb-tegra186.c
index e818f6c3980e..bec9616c4a2e 100644
--- a/drivers/phy/tegra/xusb-tegra186.c
+++ b/drivers/phy/tegra/xusb-tegra186.c
@@ -84,6 +84,7 @@
#define XUSB_PADCTL_USB2_BIAS_PAD_CTL0 0x284
#define BIAS_PAD_PD BIT(11)
#define HS_SQUELCH_LEVEL(x) (((x) & 0x7) << 0)
+#define HS_DISCON_LEVEL(x) (((x) & 0x7) << 3)
#define XUSB_PADCTL_USB2_BIAS_PAD_CTL1 0x288
#define USB2_TRK_START_TIMER(x) (((x) & 0x7f) << 12)
@@ -623,6 +624,8 @@ static void tegra186_utmi_bias_pad_power_on(struct tegra_xusb_padctl *padctl)
value &= ~BIAS_PAD_PD;
value &= ~HS_SQUELCH_LEVEL(~0);
value |= HS_SQUELCH_LEVEL(priv->calib.hs_squelch);
+ value &= ~HS_DISCON_LEVEL(~0);
+ value |= HS_DISCON_LEVEL(0x7);
padctl_writel(padctl, value, XUSB_PADCTL_USB2_BIAS_PAD_CTL0);
udelay(1);
--
2.25.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH 1/1] phy: tegra: xusb: Explicitly configure HS_DISCON_LEVEL to 0x7
2025-12-12 3:21 [PATCH 1/1] phy: tegra: xusb: Explicitly configure HS_DISCON_LEVEL to 0x7 Wayne Chang
@ 2025-12-24 7:32 ` Vinod Koul
0 siblings, 0 replies; 2+ messages in thread
From: Vinod Koul @ 2025-12-24 7:32 UTC (permalink / raw)
To: jckuo, kishon, thierry.reding, jonathanh, Wayne Chang
Cc: haotienh, linux-phy, linux-tegra, linux-kernel, stable
On Fri, 12 Dec 2025 11:21:16 +0800, Wayne Chang wrote:
> The USB2 Bias Pad Control register manages analog parameters for signal
> detection. Previously, the HS_DISCON_LEVEL relied on hardware reset
> values, which may lead to the detection failure.
>
> Explicitly configure HS_DISCON_LEVEL to 0x7. This ensures the disconnect
> threshold is sufficient to guarantee reliable detection.
>
> [...]
Applied, thanks!
[1/1] phy: tegra: xusb: Explicitly configure HS_DISCON_LEVEL to 0x7
commit: b246caa68037aa495390a60d080acaeb84f45fff
Best regards,
--
~Vinod
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2025-12-24 7:32 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-12-12 3:21 [PATCH 1/1] phy: tegra: xusb: Explicitly configure HS_DISCON_LEVEL to 0x7 Wayne Chang
2025-12-24 7:32 ` Vinod Koul
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox