* [PATCH] arm64: tegra: Fix ethernet phy-mode for Jetson Xavier
@ 2020-05-01 7:27 Jon Hunter
2020-05-06 23:42 ` Sasha Levin
[not found] ` <20200501072756.25348-1-jonathanh-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
0 siblings, 2 replies; 6+ messages in thread
From: Jon Hunter @ 2020-05-01 7:27 UTC (permalink / raw)
To: Thierry Reding
Cc: devicetree-u79uwXL29TY76Z2rM5mHXA,
linux-tegra-u79uwXL29TY76Z2rM5mHXA, Peter Robinson, Jon Hunter,
stable-u79uwXL29TY76Z2rM5mHXA
The 'phy-mode' property is currently defined as 'rgmii' for Jetson
Xavier. This indicates that the RGMII RX and TX delays are set by the
MAC and the internal delays set by the PHY are not used.
If the Marvell PHY driver is enabled, such that it is used and not the
generic PHY, ethernet failures are seen (DHCP is failing to obtain an
IP address) and this is caused because the Marvell PHY driver is
disabling the internal RX and TX delays. For Jetson Xavier the internal
PHY RX and TX delay should be used and so fix this by setting the
'phy-mode' to 'rgmii-id' and not 'rgmii'.
Cc: stable-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Signed-off-by: Jon Hunter <jonathanh-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
---
arch/arm64/boot/dts/nvidia/tegra194-p2888.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/nvidia/tegra194-p2888.dtsi b/arch/arm64/boot/dts/nvidia/tegra194-p2888.dtsi
index 623f7d7d216b..8e3136dfdd62 100644
--- a/arch/arm64/boot/dts/nvidia/tegra194-p2888.dtsi
+++ b/arch/arm64/boot/dts/nvidia/tegra194-p2888.dtsi
@@ -33,7 +33,7 @@
phy-reset-gpios = <&gpio TEGRA194_MAIN_GPIO(G, 5) GPIO_ACTIVE_LOW>;
phy-handle = <&phy>;
- phy-mode = "rgmii";
+ phy-mode = "rgmii-id";
mdio {
#address-cells = <1>;
--
2.17.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] arm64: tegra: Fix ethernet phy-mode for Jetson Xavier
[not found] ` <20200501072756.25348-1-jonathanh-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
@ 2020-05-05 23:12 ` Thierry Reding
2020-05-06 23:42 ` Sasha Levin
2020-05-06 23:42 ` Sasha Levin
2 siblings, 0 replies; 6+ messages in thread
From: Thierry Reding @ 2020-05-05 23:12 UTC (permalink / raw)
To: Jon Hunter
Cc: devicetree-u79uwXL29TY76Z2rM5mHXA,
linux-tegra-u79uwXL29TY76Z2rM5mHXA, Peter Robinson,
stable-u79uwXL29TY76Z2rM5mHXA
[-- Attachment #1: Type: text/plain, Size: 1004 bytes --]
On Fri, May 01, 2020 at 08:27:56AM +0100, Jon Hunter wrote:
> The 'phy-mode' property is currently defined as 'rgmii' for Jetson
> Xavier. This indicates that the RGMII RX and TX delays are set by the
> MAC and the internal delays set by the PHY are not used.
>
> If the Marvell PHY driver is enabled, such that it is used and not the
> generic PHY, ethernet failures are seen (DHCP is failing to obtain an
> IP address) and this is caused because the Marvell PHY driver is
> disabling the internal RX and TX delays. For Jetson Xavier the internal
> PHY RX and TX delay should be used and so fix this by setting the
> 'phy-mode' to 'rgmii-id' and not 'rgmii'.
>
> Cc: stable-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
>
> Signed-off-by: Jon Hunter <jonathanh-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
> ---
> arch/arm64/boot/dts/nvidia/tegra194-p2888.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Added a Fixes: tag and applied to for-5.8/arm64/dt, thanks.
Thierry
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] arm64: tegra: Fix ethernet phy-mode for Jetson Xavier
2020-05-01 7:27 [PATCH] arm64: tegra: Fix ethernet phy-mode for Jetson Xavier Jon Hunter
@ 2020-05-06 23:42 ` Sasha Levin
[not found] ` <20200506234218.4E11D2082E-+nuXSHJNwjE76Z2rM5mHXA@public.gmane.org>
[not found] ` <20200501072756.25348-1-jonathanh-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
1 sibling, 1 reply; 6+ messages in thread
From: Sasha Levin @ 2020-05-06 23:42 UTC (permalink / raw)
To: Sasha Levin, Jon Hunter, Thierry Reding; +Cc: devicetree, linux-tegra, stable
Hi
[This is an automated email]
This commit has been processed because it contains a -stable tag.
The stable tag indicates that it's relevant for the following trees: all
The bot has tested the following trees: v5.6.8, v5.4.36, v4.19.119, v4.14.177, v4.9.220, v4.4.220.
v5.6.8: Build OK!
v5.4.36: Build OK!
v4.19.119: Build OK!
v4.14.177: Failed to apply! Possible dependencies:
5425fb15d8ee ("arm64: tegra: Add Tegra194 chip device tree")
b8656c673a6b ("arm64: tegra: Add device tree for the Tegra194 P2972-0000 board")
f69ce393ec48 ("arm64: tegra: Add GPIO controller on Tegra194")
f89b58ce71a9 ("arm64: tegra: Add ethernet controller on Tegra194")
v4.9.220: Failed to apply! Possible dependencies:
5425fb15d8ee ("arm64: tegra: Add Tegra194 chip device tree")
99575bceebd6 ("arm64: tegra: Add NVIDIA P2771 board support")
b8656c673a6b ("arm64: tegra: Add device tree for the Tegra194 P2972-0000 board")
f69ce393ec48 ("arm64: tegra: Add GPIO controller on Tegra194")
f89b58ce71a9 ("arm64: tegra: Add ethernet controller on Tegra194")
v4.4.220: Failed to apply! Possible dependencies:
0f279ebdf3ce ("arm64: tegra: Add NVIDIA Tegra132 Norrin support")
2cc85bd90337 ("arm64: tegra: Add NVIDIA P2571 board support")
34b4f6d0599e ("arm64: tegra: Add Tegra132 support")
5425fb15d8ee ("arm64: tegra: Add Tegra194 chip device tree")
5d17ba6e638e ("arm64: tegra: Add support for Google Pixel C")
63023e95bec0 ("arm64: tegra: Add NVIDIA P2371 board support")
99575bceebd6 ("arm64: tegra: Add NVIDIA P2771 board support")
b8656c673a6b ("arm64: tegra: Add device tree for the Tegra194 P2972-0000 board")
f69ce393ec48 ("arm64: tegra: Add GPIO controller on Tegra194")
f89b58ce71a9 ("arm64: tegra: Add ethernet controller on Tegra194")
NOTE: The patch will not be queued to stable trees until it is upstream.
How should we proceed with this patch?
--
Thanks
Sasha
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] arm64: tegra: Fix ethernet phy-mode for Jetson Xavier
[not found] ` <20200501072756.25348-1-jonathanh-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2020-05-05 23:12 ` Thierry Reding
2020-05-06 23:42 ` Sasha Levin
@ 2020-05-06 23:42 ` Sasha Levin
2 siblings, 0 replies; 6+ messages in thread
From: Sasha Levin @ 2020-05-06 23:42 UTC (permalink / raw)
To: Sasha Levin, Jon Hunter, Thierry Reding
Cc: devicetree-u79uwXL29TY76Z2rM5mHXA,
linux-tegra-u79uwXL29TY76Z2rM5mHXA,
stable-u79uwXL29TY76Z2rM5mHXA
Hi
[This is an automated email]
This commit has been processed because it contains a -stable tag.
The stable tag indicates that it's relevant for the following trees: all
The bot has tested the following trees: v5.6.8, v5.4.36, v4.19.119, v4.14.177, v4.9.220, v4.4.220.
v5.6.8: Build OK!
v5.4.36: Build OK!
v4.19.119: Build OK!
v4.14.177: Failed to apply! Possible dependencies:
5425fb15d8ee ("arm64: tegra: Add Tegra194 chip device tree")
b8656c673a6b ("arm64: tegra: Add device tree for the Tegra194 P2972-0000 board")
f69ce393ec48 ("arm64: tegra: Add GPIO controller on Tegra194")
f89b58ce71a9 ("arm64: tegra: Add ethernet controller on Tegra194")
v4.9.220: Failed to apply! Possible dependencies:
5425fb15d8ee ("arm64: tegra: Add Tegra194 chip device tree")
99575bceebd6 ("arm64: tegra: Add NVIDIA P2771 board support")
b8656c673a6b ("arm64: tegra: Add device tree for the Tegra194 P2972-0000 board")
f69ce393ec48 ("arm64: tegra: Add GPIO controller on Tegra194")
f89b58ce71a9 ("arm64: tegra: Add ethernet controller on Tegra194")
v4.4.220: Failed to apply! Possible dependencies:
0f279ebdf3ce ("arm64: tegra: Add NVIDIA Tegra132 Norrin support")
2cc85bd90337 ("arm64: tegra: Add NVIDIA P2571 board support")
34b4f6d0599e ("arm64: tegra: Add Tegra132 support")
5425fb15d8ee ("arm64: tegra: Add Tegra194 chip device tree")
5d17ba6e638e ("arm64: tegra: Add support for Google Pixel C")
63023e95bec0 ("arm64: tegra: Add NVIDIA P2371 board support")
99575bceebd6 ("arm64: tegra: Add NVIDIA P2771 board support")
b8656c673a6b ("arm64: tegra: Add device tree for the Tegra194 P2972-0000 board")
f69ce393ec48 ("arm64: tegra: Add GPIO controller on Tegra194")
f89b58ce71a9 ("arm64: tegra: Add ethernet controller on Tegra194")
NOTE: The patch will not be queued to stable trees until it is upstream.
How should we proceed with this patch?
--
Thanks
Sasha
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] arm64: tegra: Fix ethernet phy-mode for Jetson Xavier
[not found] ` <20200501072756.25348-1-jonathanh-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2020-05-05 23:12 ` Thierry Reding
@ 2020-05-06 23:42 ` Sasha Levin
2020-05-06 23:42 ` Sasha Levin
2 siblings, 0 replies; 6+ messages in thread
From: Sasha Levin @ 2020-05-06 23:42 UTC (permalink / raw)
To: Sasha Levin, Jon Hunter, Thierry Reding
Cc: devicetree-u79uwXL29TY76Z2rM5mHXA,
linux-tegra-u79uwXL29TY76Z2rM5mHXA,
stable-u79uwXL29TY76Z2rM5mHXA
Hi
[This is an automated email]
This commit has been processed because it contains a -stable tag.
The stable tag indicates that it's relevant for the following trees: all
The bot has tested the following trees: v5.6.8, v5.4.36, v4.19.119, v4.14.177, v4.9.220, v4.4.220.
v5.6.8: Build OK!
v5.4.36: Build OK!
v4.19.119: Build OK!
v4.14.177: Failed to apply! Possible dependencies:
5425fb15d8ee ("arm64: tegra: Add Tegra194 chip device tree")
b8656c673a6b ("arm64: tegra: Add device tree for the Tegra194 P2972-0000 board")
f69ce393ec48 ("arm64: tegra: Add GPIO controller on Tegra194")
f89b58ce71a9 ("arm64: tegra: Add ethernet controller on Tegra194")
v4.9.220: Failed to apply! Possible dependencies:
5425fb15d8ee ("arm64: tegra: Add Tegra194 chip device tree")
99575bceebd6 ("arm64: tegra: Add NVIDIA P2771 board support")
b8656c673a6b ("arm64: tegra: Add device tree for the Tegra194 P2972-0000 board")
f69ce393ec48 ("arm64: tegra: Add GPIO controller on Tegra194")
f89b58ce71a9 ("arm64: tegra: Add ethernet controller on Tegra194")
v4.4.220: Failed to apply! Possible dependencies:
0f279ebdf3ce ("arm64: tegra: Add NVIDIA Tegra132 Norrin support")
2cc85bd90337 ("arm64: tegra: Add NVIDIA P2571 board support")
34b4f6d0599e ("arm64: tegra: Add Tegra132 support")
5425fb15d8ee ("arm64: tegra: Add Tegra194 chip device tree")
5d17ba6e638e ("arm64: tegra: Add support for Google Pixel C")
63023e95bec0 ("arm64: tegra: Add NVIDIA P2371 board support")
99575bceebd6 ("arm64: tegra: Add NVIDIA P2771 board support")
b8656c673a6b ("arm64: tegra: Add device tree for the Tegra194 P2972-0000 board")
f69ce393ec48 ("arm64: tegra: Add GPIO controller on Tegra194")
f89b58ce71a9 ("arm64: tegra: Add ethernet controller on Tegra194")
NOTE: The patch will not be queued to stable trees until it is upstream.
How should we proceed with this patch?
--
Thanks
Sasha
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] arm64: tegra: Fix ethernet phy-mode for Jetson Xavier
[not found] ` <20200506234218.4E11D2082E-+nuXSHJNwjE76Z2rM5mHXA@public.gmane.org>
@ 2020-05-07 10:05 ` Thierry Reding
0 siblings, 0 replies; 6+ messages in thread
From: Thierry Reding @ 2020-05-07 10:05 UTC (permalink / raw)
To: Sasha Levin
Cc: Jon Hunter, devicetree-u79uwXL29TY76Z2rM5mHXA,
linux-tegra-u79uwXL29TY76Z2rM5mHXA,
stable-u79uwXL29TY76Z2rM5mHXA
[-- Attachment #1: Type: text/plain, Size: 2230 bytes --]
On Wed, May 06, 2020 at 11:42:17PM +0000, Sasha Levin wrote:
> Hi
>
> [This is an automated email]
>
> This commit has been processed because it contains a -stable tag.
> The stable tag indicates that it's relevant for the following trees: all
>
> The bot has tested the following trees: v5.6.8, v5.4.36, v4.19.119, v4.14.177, v4.9.220, v4.4.220.
>
> v5.6.8: Build OK!
> v5.4.36: Build OK!
> v4.19.119: Build OK!
> v4.14.177: Failed to apply! Possible dependencies:
> 5425fb15d8ee ("arm64: tegra: Add Tegra194 chip device tree")
> b8656c673a6b ("arm64: tegra: Add device tree for the Tegra194 P2972-0000 board")
> f69ce393ec48 ("arm64: tegra: Add GPIO controller on Tegra194")
> f89b58ce71a9 ("arm64: tegra: Add ethernet controller on Tegra194")
>
> v4.9.220: Failed to apply! Possible dependencies:
> 5425fb15d8ee ("arm64: tegra: Add Tegra194 chip device tree")
> 99575bceebd6 ("arm64: tegra: Add NVIDIA P2771 board support")
> b8656c673a6b ("arm64: tegra: Add device tree for the Tegra194 P2972-0000 board")
> f69ce393ec48 ("arm64: tegra: Add GPIO controller on Tegra194")
> f89b58ce71a9 ("arm64: tegra: Add ethernet controller on Tegra194")
>
> v4.4.220: Failed to apply! Possible dependencies:
> 0f279ebdf3ce ("arm64: tegra: Add NVIDIA Tegra132 Norrin support")
> 2cc85bd90337 ("arm64: tegra: Add NVIDIA P2571 board support")
> 34b4f6d0599e ("arm64: tegra: Add Tegra132 support")
> 5425fb15d8ee ("arm64: tegra: Add Tegra194 chip device tree")
> 5d17ba6e638e ("arm64: tegra: Add support for Google Pixel C")
> 63023e95bec0 ("arm64: tegra: Add NVIDIA P2371 board support")
> 99575bceebd6 ("arm64: tegra: Add NVIDIA P2771 board support")
> b8656c673a6b ("arm64: tegra: Add device tree for the Tegra194 P2972-0000 board")
> f69ce393ec48 ("arm64: tegra: Add GPIO controller on Tegra194")
> f89b58ce71a9 ("arm64: tegra: Add ethernet controller on Tegra194")
>
>
> NOTE: The patch will not be queued to stable trees until it is upstream.
>
> How should we proceed with this patch?
Tegra194 support was merged into v4.17, so it doesn't make backport this
to any stable kernels prior to that.
Thierry
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2020-05-07 10:05 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-05-01 7:27 [PATCH] arm64: tegra: Fix ethernet phy-mode for Jetson Xavier Jon Hunter
2020-05-06 23:42 ` Sasha Levin
[not found] ` <20200506234218.4E11D2082E-+nuXSHJNwjE76Z2rM5mHXA@public.gmane.org>
2020-05-07 10:05 ` Thierry Reding
[not found] ` <20200501072756.25348-1-jonathanh-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2020-05-05 23:12 ` Thierry Reding
2020-05-06 23:42 ` Sasha Levin
2020-05-06 23:42 ` Sasha Levin
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).