Message ID | 20250318041224.1693323-2-Rajaganesh.Rathinasabapathi@amd.com (mailing list archive) |
---|---|
State | New |
Headers | show |
Series | [v3,1/2] dt-bindings: arm: aspeed: Add AMD Onyx BMC compatible | expand |
On Mon, Mar 17, 2025 at 11:12:24PM -0500, Rajaganesh Rathinasabapathi wrote: > +/ { > + model = "AMD Onyx BMC"; > + compatible = "amd,onyx-bmc", "aspeed,ast2600"; > + > + aliases { > + serial0 = &uart1; > + serial4 = &uart5; > + }; > + > + chosen { > + stdout-path = serial4:115200n8; > + }; > + > + memory@80000000 { > + device_type = "memory"; > + reg = <0x80000000 0x80000000>; > + }; > + > +}; <form letter> This is a friendly reminder during the review process. It seems my or other reviewer's previous comments were not fully addressed. Maybe the feedback got lost between the quotes, maybe you just forgot to apply it. Please go back to the previous discussion and either implement all requested changes or keep discussing them. Thank you. </form letter> Best regards, Krzysztof
> +&mac3 { > + status = "okay"; > + phy-mode = "rgmii"; Does the PCB have extra long clock lines to insert the 2ns RGMII delay? Or are you another victim of aspeeds broken MAC/SCU driver? Andrew
On 3/18/25 14:15, Krzysztof Kozlowski wrote: > Caution: This message originated from an External Source. Use proper caution when opening attachments, clicking links, or responding. > > > On Mon, Mar 17, 2025 at 11:12:24PM -0500, Rajaganesh Rathinasabapathi wrote: >> +/ { >> + model = "AMD Onyx BMC"; >> + compatible = "amd,onyx-bmc", "aspeed,ast2600"; >> + >> + aliases { >> + serial0 = &uart1; >> + serial4 = &uart5; >> + }; >> + >> + chosen { >> + stdout-path = serial4:115200n8; >> + }; >> + >> + memory@80000000 { >> + device_type = "memory"; >> + reg = <0x80000000 0x80000000>; >> + }; >> + >> +}; > > <form letter> > This is a friendly reminder during the review process. > > It seems my or other reviewer's previous comments were not fully > addressed. Maybe the feedback got lost between the quotes, maybe you > just forgot to apply it. Please go back to the previous discussion and > either implement all requested changes or keep discussing them. > > Thank you. > </form letter> > > Best regards, > Krzysztof > There were two comments. One was to remove bootargs, this is addressed in v3. Another comment was to add compatibles for amd,onyx-bmc, it part of earlier commit (patch v3 1/2). Thanks, Raja
On 3/18/25 19:38, Andrew Lunn wrote: > Caution: This message originated from an External Source. Use proper caution when opening attachments, clicking links, or responding. > > >> +&mac3 { >> + status = "okay"; >> + phy-mode = "rgmii"; > > Does the PCB have extra long clock lines to insert the 2ns RGMII > delay? Or are you another victim of aspeeds broken MAC/SCU driver? > > Andrew We're following Aspeed SDK and referred other dts based on ast2600. I assume Aspeed has moved clock delays to u-boot. Link: https://lore.kernel.org/lkml/SG2PR06MB23150B3673E58737ABB08D51E6139@SG2PR06MB2315.apcprd06.prod.outlook.com/
On Tue, Mar 18, 2025 at 10:18:46PM +0530, Rajaganesh Rathinasabapathi wrote: > On 3/18/25 19:38, Andrew Lunn wrote: > > Caution: This message originated from an External Source. Use proper caution when opening attachments, clicking links, or responding. > > > > > >> +&mac3 { > >> + status = "okay"; > >> + phy-mode = "rgmii"; > > > > Does the PCB have extra long clock lines to insert the 2ns RGMII > > delay? Or are you another victim of aspeeds broken MAC/SCU driver? > > > > Andrew > We're following Aspeed SDK and referred other dts based on ast2600. Which are all broken. At the moment, you are joining NVIDIA and IBM waiting for Aspeed to sort out this mess. Maybe you can apply some pressure... Andrew
On 18/03/2025 17:44, Rajaganesh Rathinasabapathi wrote: >> >> <form letter> >> This is a friendly reminder during the review process. >> >> It seems my or other reviewer's previous comments were not fully >> addressed. Maybe the feedback got lost between the quotes, maybe you >> just forgot to apply it. Please go back to the previous discussion and >> either implement all requested changes or keep discussing them. >> >> Thank you. >> </form letter> >> >> Best regards, >> Krzysztof >> > > There were two comments. One was to remove bootargs, this is addressed in v3. > Another comment was to add compatibles for amd,onyx-bmc, it part of earlier commit (patch v3 1/2). No, there were more. Best regards, Krzysztof
diff --git a/arch/arm/boot/dts/aspeed/Makefile b/arch/arm/boot/dts/aspeed/Makefile index 2e5f4833a073..1e6a130377b8 100644 --- a/arch/arm/boot/dts/aspeed/Makefile +++ b/arch/arm/boot/dts/aspeed/Makefile @@ -5,6 +5,7 @@ dtb-$(CONFIG_ARCH_ASPEED) += \ aspeed-ast2600-evb.dtb \ aspeed-bmc-amd-daytonax.dtb \ aspeed-bmc-amd-ethanolx.dtb \ + aspeed-bmc-amd-onyx.dtb \ aspeed-bmc-ampere-mtjade.dtb \ aspeed-bmc-ampere-mtjefferson.dtb \ aspeed-bmc-ampere-mtmitchell.dtb \ diff --git a/arch/arm/boot/dts/aspeed/aspeed-bmc-amd-onyx.dts b/arch/arm/boot/dts/aspeed/aspeed-bmc-amd-onyx.dts new file mode 100644 index 000000000000..6f3334995398 --- /dev/null +++ b/arch/arm/boot/dts/aspeed/aspeed-bmc-amd-onyx.dts @@ -0,0 +1,102 @@ +// SPDX-License-Identifier: GPL-2.0+ +// Copyright (c) 2021 - 2024 AMD Inc. +// Author: Supreeth Venkatesh <supreeth.venkatesh@amd.com> + +/dts-v1/; + +#include "aspeed-g6.dtsi" +#include <dt-bindings/gpio/aspeed-gpio.h> + +/ { + model = "AMD Onyx BMC"; + compatible = "amd,onyx-bmc", "aspeed,ast2600"; + + aliases { + serial0 = &uart1; + serial4 = &uart5; + }; + + chosen { + stdout-path = serial4:115200n8; + }; + + memory@80000000 { + device_type = "memory"; + reg = <0x80000000 0x80000000>; + }; + +}; + +&mdio0 { + status = "okay"; + ethphy0: ethernet-phy@0 { + compatible = "ethernet-phy-ieee802.3-c22"; + reg = <0>; + }; +}; + +&mac3 { + status = "okay"; + phy-mode = "rgmii"; + phy-handle = <ðphy0>; + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_rgmii4_default>; +}; + +&fmc { + status = "okay"; + flash@0 { + compatible = "jedec,spi-nor"; + status = "okay"; + #include "openbmc-flash-layout-128.dtsi" + }; +}; + +//Host Console +&uart1 { + status = "okay"; +}; + +//BMC Console +&uart5 { + status = "okay"; +}; + +&gpio0 { +gpio-line-names = + /*A0-A7*/ "","","","","","","","", + /*B0-B7*/ "","","","","MON_POST_COMPLETE","P0_PRESENT_L","","", + /*C0-C7*/ "","","","","","","","", + /*D0-D7*/ "","","","","","","","", + /*E0-E7*/ "","","","","","","","", + /*F0-F7*/ "","","","","","","","", + /*G0-G7*/ "","","","","","","","", + /*H0-H7*/ "","ASSERT_WARM_RST_BTN_L","ASSERT_SOC_RST_BTN_L","","","","","", + /*I0-I7*/ "","","","","","","","P0_I3C_APML_ALERT_L", + /*J0-J7*/ "","","","","","","","", + /*K0-K7*/ "","","","","","","","", + /*L0-L7*/ "","","","","","","","", + /*M0-M7*/ "","","","","","","","", + /*N0-N7*/ "","","","","","","PSP_SOFT_FUSE_NOTIFY","ASSERT_BMC_READY", + /*O0-O7*/ "","","HDT_SEL","HDT_XTRIG5","HDT_XTRIG6","JTAG_TRST_N","","", + /*P0-P7*/ "MON_RST_BTN_L","ASSERT_RST_BTN_L","MON_PWR_BTN_L","ASSERT_PWR_BTN_L", + "HPM_FPGA_LOCKOUT","ASSERT_NMI_BTN_L","MON_PWR_GOOD","", + /*Q0-Q7*/ "","","HDT_DBREQ_L","","BIOS_SPD_MUX_CTRL_RELEASED_L","","","", + /*R0-R7*/ "","","","","","","","", + /*S0-S7*/ "","","","","","","P0_DIMM_AF_ERROR","P0_DIMM_GL_ERROR", + /*T0-T7*/ "","","","","","","","", + /*U0-U7*/ "","","","","","","","", + /*V0-V7*/ "","","","","","","","", + /*W0-W7*/ "","","","","","","","", + /*X0-X7*/ "","","","","","","","", + /*Y0-Y7*/ "","","","","","","","", + /*Z0-Z7*/ "","","","","","","",""; +}; + +&i2c7 { + status = "okay"; + mbeeprom@50 { + compatible = "atmel,24c256"; + reg = <0x50>; + }; +};