Re: [PATCH 1/1] ARM:dts:aspeed: Initial device tree for AMD Onyx Platform

[Supreeth Venkatesh <supvenka@xxxxxxx>]

On 2/26/24 00:55, Andrew Jeffery wrote:
> Caution: This message originated from an External Source. Use proper caution when opening attachments, clicking links, or responding.
>
>
> On Tue, 2024-01-09 at 21:35 -0600, Supreeth Venkatesh wrote:
> > This patch adds initial device tree and makefile updates for
> > AMD Onyx platform.
> >
> > AMD Onyx platform is an AMD customer reference board with an Aspeed
> > ast2600 BMC manufactured by AMD.
> > It describes I2c devices, Fans, Kcs devices, Uarts, Mac, LEDs, etc.
> > present on AMD Onyx platform.
> >
> > Signed-off-by: Supreeth Venkatesh <supreeth.venkatesh@xxxxxxx>
> > ---
> >   arch/arm/boot/dts/aspeed/Makefile             |  1 +
> >   .../boot/dts/aspeed/aspeed-bmc-amd-onyx.dts   | 98 +++++++++++++++++++
> >   2 files changed, 99 insertions(+)
> >   create mode 100644 arch/arm/boot/dts/aspeed/aspeed-bmc-amd-onyx.dts
> >
> > diff --git a/arch/arm/boot/dts/aspeed/Makefile b/arch/arm/boot/dts/aspeed/Makefile
> > index fb9cc95f1b60..2b27d377aae2 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-mtmitchell.dtb \
> >        aspeed-bmc-arm-stardragon4800-rep2.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..a7056cd29553
> > --- /dev/null
> > +++ b/arch/arm/boot/dts/aspeed/aspeed-bmc-amd-onyx.dts
> > @@ -0,0 +1,98 @@
> > +// SPDX-License-Identifier: GPL-2.0+
> > +// Copyright (c) 2021 - 2024 AMD Inc.
> > +// Author: Supreeth Venkatesh <supreeth.venkatesh@xxxxxxx>
> > +
> > +/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 = &uart5;
> > +               bootargs = "console=ttyS4,115200 earlyprintk vmalloc=512MB";
> Why `vmalloc=512MB`? Can you add a comment explaining the need for
> that?
>
> That said, setting bootargs in the devicetree is generally discouraged.
Thank you. will remove in v2.
> > +       };
> > +
> > +       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 = <&ethphy0>;
> > +
> > +       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";
> > +};
> How are you managing the host console? Enabling UART1 for that on the
> BMC piques my interest :)
UART1 [BMC side] <-> UART 4 [Host side].
> Apologies for the delay in reviewing.
No problem. Thanks for reviewing. Sending v2 shortly.
> Andrew