| Message ID | 20250108163640.1374680-6-ninad@linux.ibm.com (mailing list archive) |
|---|---|
| State | Changes Requested |
| Headers | show |
| Series | DTS updates for system1 BMC | expand |
| Context | Check | Description |
|---|---|---|
| netdev/tree_selection | success | Not a local patch |
On Wed, Jan 08, 2025 at 10:36:33AM -0600, Ninad Palsule wrote: > system1 has 2 transceiver connected through the RGMII interfaces. Added > device tree entry to enable RGMII support. > > ASPEED AST2600 documentation recommends using 'rgmii-rxid' as a > 'phy-mode' for mac0 and mac1 to enable the RX interface delay from the > PHY chip. You appear to if ignored my comment. Please don't do that. If you have no idea about RGMII delays, please say so, so i can help you debug what is wrong. NACK Andrew
Hello Andrew, On 1/8/25 11:03, Andrew Lunn wrote: > On Wed, Jan 08, 2025 at 10:36:33AM -0600, Ninad Palsule wrote: >> system1 has 2 transceiver connected through the RGMII interfaces. Added >> device tree entry to enable RGMII support. >> >> ASPEED AST2600 documentation recommends using 'rgmii-rxid' as a >> 'phy-mode' for mac0 and mac1 to enable the RX interface delay from the >> PHY chip. > You appear to if ignored my comment. Please don't do that. If you have > no idea about RGMII delays, please say so, so i can help you debug > what is wrong. > > NACK I think there is a misunderstanding. I did not ignore your comment. I have contacted ASPEED and asked them to respond. I think Jacky from Aspeed replied to your mail. Regards, Ninad > > Andrew >
On Wed, Jan 08, 2025 at 12:43:07PM -0600, Ninad Palsule wrote: > Hello Andrew, > > > On 1/8/25 11:03, Andrew Lunn wrote: > > On Wed, Jan 08, 2025 at 10:36:33AM -0600, Ninad Palsule wrote: > > > system1 has 2 transceiver connected through the RGMII interfaces. Added > > > device tree entry to enable RGMII support. > > > > > > ASPEED AST2600 documentation recommends using 'rgmii-rxid' as a > > > 'phy-mode' for mac0 and mac1 to enable the RX interface delay from the > > > PHY chip. > > You appear to if ignored my comment. Please don't do that. If you have > > no idea about RGMII delays, please say so, so i can help you debug > > what is wrong. > > > > NACK > > I think there is a misunderstanding. I did not ignore your comment. I have > contacted ASPEED and asked them to respond. I think Jacky from Aspeed > replied to your mail. You did not mention in the cover letter, or the patch. I asked for a detailed explanation in the commit message why it is correct, which you did not do. Now we have more details, it is clear Ethernet support for this board needs to wait until we figure out how to fix the MAC driver. Please either wait with this patchset until that is done, or drop this one patch for the moment and submit it later once the MAC driver is fixed. Andrew
Hi Andrew, On 1/8/25 14:13, Andrew Lunn wrote: > On Wed, Jan 08, 2025 at 12:43:07PM -0600, Ninad Palsule wrote: >> Hello Andrew, >> >> >> On 1/8/25 11:03, Andrew Lunn wrote: >>> On Wed, Jan 08, 2025 at 10:36:33AM -0600, Ninad Palsule wrote: >>>> system1 has 2 transceiver connected through the RGMII interfaces. Added >>>> device tree entry to enable RGMII support. >>>> >>>> ASPEED AST2600 documentation recommends using 'rgmii-rxid' as a >>>> 'phy-mode' for mac0 and mac1 to enable the RX interface delay from the >>>> PHY chip. >>> You appear to if ignored my comment. Please don't do that. If you have >>> no idea about RGMII delays, please say so, so i can help you debug >>> what is wrong. >>> >>> NACK >> I think there is a misunderstanding. I did not ignore your comment. I have >> contacted ASPEED and asked them to respond. I think Jacky from Aspeed >> replied to your mail. > You did not mention in the cover letter, or the patch. I asked for a > detailed explanation in the commit message why it is correct, which > you did not do. Ah, ok. Sorry about that. > > Now we have more details, it is clear Ethernet support for this board > needs to wait until we figure out how to fix the MAC driver. Please > either wait with this patchset until that is done, or drop this one > patch for the moment and submit it later once the MAC driver is fixed. ok, Thanks! Regards, Ninad > > Andrew > >
diff --git a/arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dts b/arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dts index d11a922c9d44..0f4658074709 100644 --- a/arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dts +++ b/arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dts @@ -425,14 +425,42 @@ &lpc_ctrl { memory-region = <&flash_memory>; }; +&mdio0 { + status = "okay"; + + ethphy0: ethernet-phy@0 { + compatible = "ethernet-phy-ieee802.3-c22"; + reg = <0>; + }; +}; + +&mdio2 { + status = "okay"; + + ethphy2: ethernet-phy@0 { + compatible = "ethernet-phy-ieee802.3-c22"; + reg = <0>; + }; +}; + +&mac0 { + status = "okay"; + + phy-mode = "rgmii-rxid"; + phy-handle = <ðphy0>; + + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_rgmii1_default>; +}; + &mac2 { status = "okay"; + + phy-mode = "rgmii"; + phy-handle = <ðphy2>; + pinctrl-names = "default"; - pinctrl-0 = <&pinctrl_rmii3_default>; - clocks = <&syscon ASPEED_CLK_GATE_MAC3CLK>, - <&syscon ASPEED_CLK_MAC3RCLK>; - clock-names = "MACCLK", "RCLK"; - use-ncsi; + pinctrl-0 = <&pinctrl_rgmii3_default>; }; &mac3 {
system1 has 2 transceiver connected through the RGMII interfaces. Added device tree entry to enable RGMII support. ASPEED AST2600 documentation recommends using 'rgmii-rxid' as a 'phy-mode' for mac0 and mac1 to enable the RX interface delay from the PHY chip. Signed-off-by: Ninad Palsule <ninad@linux.ibm.com> --- .../dts/aspeed/aspeed-bmc-ibm-system1.dts | 38 ++++++++++++++++--- 1 file changed, 33 insertions(+), 5 deletions(-)