[PATCH v2] ARM: dts: aspeed: Add TYAN S7106 BMC machine

Joel Stanley joel at jms.id.au
Fri Oct 8 15:59:28 AEDT 2021


On Fri, 1 Oct 2021 at 15:11, Oskar Senft <osk at google.com> wrote:
>
> Thanks for the reviews!
>
> Is there anything else I'll need to do to get this into upstream, or
> is this just a matter of waiting?

I thought I had applied it, but I am mistaken.

It's now applied to the aspeed tree for v5.16. I made one small fix to
the makefile, where we expect .dtb but you added .dts.

It will appear in the next linux-next, and in v5.16 once the merge window opens.

Cheers,

Joel

>
> Thanks
> Oskar.
>
>
> On Tue, Sep 14, 2021 at 4:59 AM Joel Stanley <joel at jms.id.au> wrote:
> >
> > On Tue, 14 Sept 2021 at 08:56, Joel Stanley <joel at jms.id.au> wrote:
> > >
> > > On Thu, 9 Sept 2021 at 00:49, Oskar Senft <osk at google.com> wrote:
> > > >
> > > > The TYAN S7106 is a server platform with an ASPEED AST2500 BMC.
> > >
> > > Reviewed-by: Joel Stanley <joel at jms.id.au>
> > >
> > > jk, would you like to take a look before I apply?
> >
> > My bad! b4 found a r-b email from you when I grabbed the patch. I
> > never received it though!
> >
> > Your tag is added. I wonder how much of your email I'm not getting.
> >
> > Cheers,
> >
> > Joel
> >
> > >
> > > >
> > > > Signed-off-by: Oskar Senft <osk at google.com>
> > > > ---
> > > >  arch/arm/boot/dts/Makefile                  |   3 +-
> > > >  arch/arm/boot/dts/aspeed-bmc-tyan-s7106.dts | 488 ++++++++++++++++++++
> > > >  2 files changed, 490 insertions(+), 1 deletion(-)
> > > >  create mode 100644 arch/arm/boot/dts/aspeed-bmc-tyan-s7106.dts
> > > >
> > > > diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
> > > > index 7e0934180724..522bf78e896b 100644
> > > > --- a/arch/arm/boot/dts/Makefile
> > > > +++ b/arch/arm/boot/dts/Makefile
> > > > @@ -1497,4 +1497,5 @@ dtb-$(CONFIG_ARCH_ASPEED) += \
> > > >         aspeed-bmc-opp-zaius.dtb \
> > > >         aspeed-bmc-portwell-neptune.dtb \
> > > >         aspeed-bmc-quanta-q71l.dtb \
> > > > -       aspeed-bmc-supermicro-x11spi.dtb
> > > > +       aspeed-bmc-supermicro-x11spi.dtb \
> > > > +       aspeed-bmc-tyan-s7106.dts
> > > > diff --git a/arch/arm/boot/dts/aspeed-bmc-tyan-s7106.dts b/arch/arm/boot/dts/aspeed-bmc-tyan-s7106.dts
> > > > new file mode 100644
> > > > index 000000000000..68f332ee1886
> > > > --- /dev/null
> > > > +++ b/arch/arm/boot/dts/aspeed-bmc-tyan-s7106.dts
> > > > @@ -0,0 +1,488 @@
> > > > +// SPDX-License-Identifier: GPL-2.0+
> > > > +/dts-v1/;
> > > > +
> > > > +#include "aspeed-g5.dtsi"
> > > > +#include <dt-bindings/gpio/aspeed-gpio.h>
> > > > +
> > > > +/ {
> > > > +       model = "Tyan S7106 BMC";
> > > > +       compatible = "tyan,s7106-bmc", "aspeed,ast2500";
> > > > +
> > > > +       chosen {
> > > > +               stdout-path = &uart5;
> > > > +               bootargs = "console=ttyS4,115200 earlycon";
> > > > +       };
> > > > +
> > > > +       memory at 80000000 {
> > > > +               device_type = "memory";
> > > > +               reg = <0x80000000 0x20000000>;
> > > > +       };
> > > > +
> > > > +       reserved-memory {
> > > > +               #address-cells = <1>;
> > > > +               #size-cells = <1>;
> > > > +               ranges;
> > > > +
> > > > +               p2a_memory: region at 987f0000 {
> > > > +                       no-map;
> > > > +                       reg = <0x987f0000 0x00010000>; /* 64KB */
> > > > +               };
> > > > +
> > > > +               vga_memory: framebuffer at 9f000000 {
> > > > +                       no-map;
> > > > +                       reg = <0x9f000000 0x01000000>; /* 16M */
> > > > +               };
> > > > +
> > > > +               gfx_memory: framebuffer {
> > > > +                       size = <0x01000000>; /* 16M */
> > > > +                       alignment = <0x01000000>;
> > > > +                       compatible = "shared-dma-pool";
> > > > +                       reusable;
> > > > +               };
> > > > +       };
> > > > +
> > > > +       leds {
> > > > +               compatible = "gpio-leds";
> > > > +
> > > > +               identify {
> > > > +                       gpios = <&gpio ASPEED_GPIO(A, 2) GPIO_ACTIVE_LOW>;
> > > > +               };
> > > > +
> > > > +               heartbeat {
> > > > +                       gpios = <&gpio ASPEED_GPIO(E, 7) GPIO_ACTIVE_LOW>;
> > > > +               };
> > > > +       };
> > > > +
> > > > +       iio-hwmon {
> > > > +               compatible = "iio-hwmon";
> > > > +               io-channels = <&adc 0>, <&adc 1>, <&adc 2>, <&adc 3>,
> > > > +                       <&adc 4>, <&adc 5>, <&adc 6>, <&adc 7>,
> > > > +                       <&adc 8>, <&adc 9>, <&adc 10>, <&adc 11>,
> > > > +                       <&adc 12>, <&adc 13>, <&adc 14>;
> > > > +       };
> > > > +
> > > > +       iio-hwmon-battery {
> > > > +               compatible = "iio-hwmon";
> > > > +               io-channels = <&adc 15>;
> > > > +       };
> > > > +};
> > > > +
> > > > +&fmc {
> > > > +       status = "okay";
> > > > +       flash at 0 {
> > > > +               label = "bmc";
> > > > +               status = "okay";
> > > > +               m25p,fast-read;
> > > > +#include "openbmc-flash-layout.dtsi"
> > > > +       };
> > > > +};
> > > > +
> > > > +&spi1 {
> > > > +       status = "okay";
> > > > +       pinctrl-names = "default";
> > > > +       pinctrl-0 = <&pinctrl_spi1_default>;
> > > > +
> > > > +       flash at 0 {
> > > > +               status = "okay";
> > > > +               label = "pnor";
> > > > +               m25p,fast-read;
> > > > +       };
> > > > +};
> > > > +
> > > > +&uart1 {
> > > > +       /* Rear RS-232 connector */
> > > > +       status = "okay";
> > > > +       pinctrl-names = "default";
> > > > +       pinctrl-0 = <&pinctrl_txd1_default
> > > > +                       &pinctrl_rxd1_default>;
> > > > +};
> > > > +
> > > > +&uart2 {
> > > > +       /* RS-232 connector on header */
> > > > +       status = "okay";
> > > > +       pinctrl-names = "default";
> > > > +       pinctrl-0 = <&pinctrl_txd2_default
> > > > +                       &pinctrl_rxd2_default>;
> > > > +};
> > > > +
> > > > +&uart3 {
> > > > +       /* Alternative to vuart to internally connect (route) to uart1
> > > > +        * when vuart cannot be used due to BIOS limitations.
> > > > +        */
> > > > +       status = "okay";
> > > > +};
> > > > +
> > > > +&uart4 {
> > > > +       /* Alternative to vuart to internally connect (route) to the
> > > > +        * external port usually used by uart1 when vuart cannot be
> > > > +        * used due to BIOS limitations.
> > > > +        */
> > > > +       status = "okay";
> > > > +};
> > > > +
> > > > +&uart5 {
> > > > +       /* BMC "debug" (console) UART; connected to RS-232 connector
> > > > +        * on header; selectable via jumpers as alternative to uart2
> > > > +        */
> > > > +       status = "okay";
> > > > +};
> > > > +
> > > > +&vuart {
> > > > +       status = "okay";
> > > > +};
> > > > +
> > > > +&lpc_ctrl {
> > > > +       status = "okay";
> > > > +};
> > > > +
> > > > +&p2a {
> > > > +       status = "okay";
> > > > +       memory-region = <&p2a_memory>;
> > > > +};
> > > > +
> > > > +&lpc_snoop {
> > > > +       status = "okay";
> > > > +       snoop-ports = <0x80>;
> > > > +};
> > > > +
> > > > +&adc {
> > > > +       status = "okay";
> > > > +};
> > > > +
> > > > +&vhub {
> > > > +       status = "okay";
> > > > +};
> > > > +
> > > > +&pwm_tacho {
> > > > +       status = "okay";
> > > > +       pinctrl-names = "default";
> > > > +       pinctrl-0 = <&pinctrl_pwm0_default
> > > > +                       &pinctrl_pwm1_default
> > > > +                       &pinctrl_pwm3_default
> > > > +                       &pinctrl_pwm4_default>;
> > > > +
> > > > +       /* CPU fan #0 */
> > > > +       fan at 0 {
> > > > +               reg = <0x00>;
> > > > +               aspeed,fan-tach-ch = /bits/ 8 <0x00>;
> > > > +       };
> > > > +
> > > > +       /* CPU fan #1 */
> > > > +       fan at 1 {
> > > > +               reg = <0x01>;
> > > > +               aspeed,fan-tach-ch = /bits/ 8 <0x01>;
> > > > +       };
> > > > +
> > > > +       /* PWM group for chassis fans #1, #2, #3 and #4  */
> > > > +       fan at 2 {
> > > > +               reg = <0x03>;
> > > > +               aspeed,fan-tach-ch = /bits/ 8 <0x02>;
> > > > +       };
> > > > +
> > > > +       fan at 3 {
> > > > +               reg = <0x03>;
> > > > +               aspeed,fan-tach-ch = /bits/ 8 <0x03>;
> > > > +       };
> > > > +
> > > > +       fan at 4 {
> > > > +               reg = <0x03>;
> > > > +               aspeed,fan-tach-ch = /bits/ 8 <0x04>;
> > > > +       };
> > > > +
> > > > +       fan at 5 {
> > > > +               reg = <0x03>;
> > > > +               aspeed,fan-tach-ch = /bits/ 8 <0x05>;
> > > > +       };
> > > > +
> > > > +       /* PWM group for chassis fans #5 and #6  */
> > > > +       fan at 6 {
> > > > +               reg = <0x04>;
> > > > +               aspeed,fan-tach-ch = /bits/ 8 <0x06>;
> > > > +       };
> > > > +
> > > > +       fan at 7 {
> > > > +               reg = <0x04>;
> > > > +               aspeed,fan-tach-ch = /bits/ 8 <0x07>;
> > > > +       };
> > > > +};
> > > > +
> > > > +&i2c0 {
> > > > +       status = "okay";
> > > > +
> > > > +       /* Hardware monitor with temperature sensors */
> > > > +       nct7802 at 28 {
> > > > +               compatible = "nuvoton,nct7802";
> > > > +               reg = <0x28>;
> > > > +       };
> > > > +
> > > > +       /* Also connected to:
> > > > +        * - IPMB pin header
> > > > +        * - CPU #0 memory error LED @ 0x3A
> > > > +        * - CPU #1 memory error LED @ 0x3C
> > > > +        */
> > > > +};
> > > > +
> > > > +&i2c1 {
> > > > +       /* Directly connected to PCH SMBUS #0 */
> > > > +       status = "okay";
> > > > +};
> > > > +
> > > > +&i2c2 {
> > > > +       status = "okay";
> > > > +
> > > > +       /* BMC EEPROM, incl. mainboard FRU */
> > > > +       eeprom at 50 {
> > > > +               compatible = "atmel,24c256";
> > > > +               reg = <0x50>;
> > > > +       };
> > > > +
> > > > +       /* Also connected to:
> > > > +        * - fan header
> > > > +        * - mini-SAS HD connector
> > > > +        * - SSATA SGPIO
> > > > +        * - via switch (BMC_SMB3_PCH_IE_SML3_EN, active low)
> > > > +        *   to PCH SMBUS #3
> > > > +        */
> > > > +};
> > > > +
> > > > +&i2c3 {
> > > > +       status = "okay";
> > > > +
> > > > +       /* PSU1 FRU @ 0xA0 */
> > > > +       eeprom at 50 {
> > > > +               compatible = "atmel,24c02";
> > > > +               reg = <0x50>;
> > > > +       };
> > > > +
> > > > +       /* PSU2 FRU @ 0xA2 */
> > > > +       eeprom at 51 {
> > > > +               compatible = "atmel,24c02";
> > > > +               reg = <0x51>;
> > > > +       };
> > > > +
> > > > +       /* PSU1 @ 0xB0 */
> > > > +       power-supply at 58 {
> > > > +               compatible = "pmbus";
> > > > +               reg = <0x58>;
> > > > +       };
> > > > +
> > > > +       /* PSU2 @ 0xB2 */
> > > > +       power-supply at 59 {
> > > > +               compatible = "pmbus";
> > > > +               reg = <0x59>;
> > > > +       };
> > > > +
> > > > +       /* Also connected to:
> > > > +        * - PCH SMBUS #1
> > > > +        */
> > > > +};
> > > > +
> > > > +&i2c4 {
> > > > +       status = "okay";
> > > > +
> > > > +       /* Connected to:
> > > > +        * - PCH SMBUS #2
> > > > +        */
> > > > +
> > > > +       /* Connected via switch to:
> > > > +        * - CPU #0 channels ABC VDDQ @ 0x80
> > > > +        * - CPU #0 channels DEF VDDQ @ 0x81
> > > > +        * - CPU #1 channels ABC VDDQ @ 0x82
> > > > +        * - CPU #1 channels DEF VDDQ @ 0x83
> > > > +        * - CPU #0 VCCIO & VMCP @ 0x52
> > > > +        * - CPU #1 VCCIO & VMCP @ 0x53
> > > > +        * - CPU #0 VCCIN @ 0xC0
> > > > +        * - CPU #0 VSA @ 0xC2
> > > > +        * - CPU #1 VCCIN @ 0xC4
> > > > +        * - CPU #1 VSA @ 0xC6
> > > > +        * - J110
> > > > +        */
> > > > +};
> > > > +
> > > > +&i2c5 {
> > > > +       status = "okay";
> > > > +
> > > > +       /* Connected via switch (PCH_BMC_SMB_SW_P) to:
> > > > +        * - mainboard FRU @ 0xAE
> > > > +        * - XDP connector
> > > > +        * - ME debug header
> > > > +        * - clock buffer @ 0xD8
> > > > +        * - i2c4 via switch (PCH_VR_SMBUS_SW_P; controlled by PCH)
> > > > +        * - PCH SMBUS
> > > > +        */
> > > > +};
> > > > +
> > > > +&i2c6 {
> > > > +       status = "okay";
> > > > +
> > > > +       /* Connected via switch (BMC_PE_SMB_EN_1_N) to
> > > > +        * bus mux (selector BMC_PE_SMB_SW_BIT[1..0]) to:
> > > > +        * - 0,0: PCIE slot 1, SMB #1
> > > > +        * - 0,1: PCIE slot 1, SMB #2
> > > > +        * - 1,0: PCIE slot 2, SMB #1
> > > > +        * - 1,1: PCIE slot 2, SMB #2
> > > > +        */
> > > > +
> > > > +       /* Connected via switch (BMC_PE_SMB_EN_2_N) to
> > > > +        * bus mux (selector BMC_PE_SMB_SW_BIT[1..0]) to:
> > > > +        * - 0,0: OCP0 (A) SMB
> > > > +        * - 0,1: OCP0 (C) SMB
> > > > +        * - 1,0: OCP1 (A) SMB
> > > > +        * - 1,1: NC
> > > > +        */
> > > > +};
> > > > +
> > > > +&i2c7 {
> > > > +       status = "okay";
> > > > +
> > > > +       /* Connected to:
> > > > +        * - PCH SMBUS #4
> > > > +        */
> > > > +};
> > > > +
> > > > +&i2c8 {
> > > > +       status = "okay";
> > > > +
> > > > +       /* Not connected */
> > > > +};
> > > > +
> > > > +&mac0 {
> > > > +       status = "okay";
> > > > +       use-ncsi;
> > > > +       pinctrl-names = "default";
> > > > +       pinctrl-0 = <&pinctrl_rmii1_default>;
> > > > +};
> > > > +
> > > > +&mac1 {
> > > > +       status = "okay";
> > > > +       pinctrl-names = "default";
> > > > +       pinctrl-0 = <&pinctrl_rgmii2_default &pinctrl_mdio2_default>;
> > > > +};
> > > > +
> > > > +&ibt {
> > > > +       status = "okay";
> > > > +};
> > > > +
> > > > +&kcs1 {
> > > > +       status = "okay";
> > > > +       aspeed,lpc-io-reg = <0xca8>;
> > > > +};
> > > > +
> > > > +&kcs3 {
> > > > +       status = "okay";
> > > > +       aspeed,lpc-io-reg = <0xca2>;
> > > > +};
> > > > +
> > > > +/* Enable BMC VGA output to show an early (pre-BIOS) boot screen */
> > > > +&gfx {
> > > > +       status = "okay";
> > > > +       memory-region = <&gfx_memory>;
> > > > +};
> > > > +
> > > > +/* We're following the GPIO naming as defined at
> > > > + * https://github.com/openbmc/docs/blob/master/designs/device-tree-gpio-naming.md.
> > > > + *
> > > > + * Notes on led-identify and id-button:
> > > > + * - A physical button is connected to id-button which
> > > > + *   triggers the clock on a D flip-flop. The /Q output of the
> > > > + *   flip-flop drives its D input.
> > > > + * - The flip-flop's Q output drives led-identify which is
> > > > + *   connected to LEDs.
> > > > + * - With that, every button press toggles the LED between on and off.
> > > > + *
> > > > + * Notes on power-, reset- and nmi- button and control:
> > > > + * - The -button signals can be used to monitor physical buttons.
> > > > + * - The -control signals can be used to actuate the specific
> > > > + *   operation.
> > > > + * - In hardware, the -button signals are connected to the -control
> > > > + *   signals through drivers with the -control signals being
> > > > + *   protected through diodes.
> > > > + */
> > > > +&gpio {
> > > > +       status = "okay";
> > > > +       gpio-line-names =
> > > > +       /*A0*/          "",
> > > > +       /*A1*/          "",
> > > > +       /*A2*/          "led-identify", /* in/out: BMC_IDLED_ON_N */
> > > > +       /*A3*/          "",
> > > > +       /*A4*/          "",
> > > > +       /*A5*/          "",
> > > > +       /*A6*/          "",
> > > > +       /*A7*/          "",
> > > > +       /*B0-B7*/       "","","","","","","","",
> > > > +       /*C0*/          "",
> > > > +       /*C1*/          "",
> > > > +       /*C2*/          "",
> > > > +       /*C3*/          "",
> > > > +       /*C4*/          "id-button", /* in/out: BMC_IDBTN_IN_OUT_N */
> > > > +       /*C5*/          "post-complete", /* in: FM_BIOS_POST_CMPLT_N */
> > > > +       /*C6*/          "",
> > > > +       /*C7*/          "",
> > > > +       /*D0*/          "",
> > > > +       /*D1*/          "",
> > > > +       /*D2*/          "power-chassis-good", /* in: SYS_PWROK_BUF */
> > > > +       /*D3*/          "platform-reset", /* in: SYS_PLTRST_N */
> > > > +       /*D4*/          "",
> > > > +       /*D5*/          "",
> > > > +       /*D6*/          "",
> > > > +       /*D7*/          "",
> > > > +       /*E0*/          "power-button", /* in: BMC_PWBTN_IN_N */
> > > > +       /*E1*/          "power-chassis-control", /* out: BMC_PWRBTN_OUT_N */
> > > > +       /*E2*/          "reset-button", /* in: BMC_RSTBTN_IN_N */
> > > > +       /*E3*/          "reset-control", /* out: BMC_RSTBTN_OUT_N */
> > > > +       /*E4*/          "nmi-button", /* in: BMC_NMIBTN_IN_N */
> > > > +       /*E5*/          "nmi-control", /* out: BMC_NMIBTN_OUT_N */
> > > > +       /*E6*/          "",
> > > > +       /*E7*/          "led-heartbeat", /* out: BMC_HEARTBRAT_LED_N */
> > > > +       /*F0*/          "",
> > > > +       /*F1*/          "clear-cmos-control", /* out: BMC_CLR_CMOS_N */
> > > > +       /*F2*/          "",
> > > > +       /*F3*/          "",
> > > > +       /*F4*/          "led-fault", /* out: AST_HW_FAULT_N */
> > > > +       /*F5*/          "",
> > > > +       /*F6*/          "",
> > > > +       /*F7*/          "",
> > > > +       /*G0*/          "BMC_PE_SMB_EN_1_N", /* out */
> > > > +       /*G1*/          "BMC_PE_SMB_EN_2_N", /* out */
> > > > +       /*G2*/          "",
> > > > +       /*G3*/          "",
> > > > +       /*G4*/          "",
> > > > +       /*G5*/          "",
> > > > +       /*G6*/          "",
> > > > +       /*G7*/          "",
> > > > +       /*H0-H7*/       "","","","","","","","",
> > > > +       /*I0-I7*/       "","","","","","","","",
> > > > +       /*J0-J7*/       "","","","","","","","",
> > > > +       /*K0-K7*/       "","","","","","","","",
> > > > +       /*L0-L7*/       "","","","","","","","",
> > > > +       /*M0-M7*/       "","","","","","","","",
> > > > +       /*N0-N7*/       "","","","","","","","",
> > > > +       /*O0-O7*/       "","","","","","","","",
> > > > +       /*P0-P7*/       "","","","","","","","",
> > > > +       /*Q0*/          "",
> > > > +       /*Q1*/          "",
> > > > +       /*Q2*/          "",
> > > > +       /*Q3*/          "",
> > > > +       /*Q4*/          "BMC_PE_SMB_SW_BIT0", /* out */
> > > > +       /*Q5*/          "BMC_PE_SMB_SW_BIT1", /* out */
> > > > +       /*Q6*/          "",
> > > > +       /*Q7*/          "",
> > > > +       /*R0-R7*/       "","","","","","","","",
> > > > +       /*S0-S7*/       "","","","","","","","",
> > > > +       /*T0-T7*/       "","","","","","","","",
> > > > +       /*U0-U7*/       "","","","","","","","",
> > > > +       /*V0-V7*/       "","","","","","","","",
> > > > +       /*W0-W7*/       "","","","","","","","",
> > > > +       /*X0-X7*/       "","","","","","","","",
> > > > +       /*Y0-Y7*/       "","","","","","","","",
> > > > +       /*Z0-Z7*/       "","","","","","","","",
> > > > +       /*AA0*/         "",
> > > > +       /*AA1*/         "",
> > > > +       /*AA2*/         "",
> > > > +       /*AA3*/         "BMC_SMB3_PCH_IE_SML3_EN", /* out */
> > > > +       /*AA4*/         "",
> > > > +       /*AA5*/         "",
> > > > +       /*AA6*/         "",
> > > > +       /*AA7*/         "",
> > > > +       /*AB0-AB7*/     "","","","","","","","";
> > > > +};
> > > > --
> > > > 2.33.0.309.g3052b89438-goog
> > > >


More information about the Linux-aspeed mailing list