[PATCH u-boot v2019.04-aspeed-openbmc v2] ARM: dts: aspeed: add Ampere's Mt. Mitchell BMC

[Quan Nguyen]

Dear reviewers,

Thank you a lot for the reviews and comments so far on the patch.
In fact, I'm still scratching my head and confused if this patch still 
needs some improvement or I just need to wait for it to get merged?

Thank you,
- Quan

On 03/01/2023 15:28, Quan Nguyen wrote:
> Just a gentle ping on the patch.
> Thank you and Happy New Year,
> - Quan
> 
> On 08/12/2022 13:57, Quan Nguyen wrote:
>> Add initial device tree for Aspeed AST2600-based Mt. Mitchell
>> BMC found on AmpereOne's reference platform.
>>
>> Signed-off-by: Quan Nguyen <quan at os.amperecomputing.com>
>> Reviewed-by: Zev Weiss <zweiss at equinix.com>
>> ---
>> v2:
>> - Fix tab issue                                        [Zev Weiss]
>> ---
>>   arch/arm/dts/Makefile               |   1 +
>>   arch/arm/dts/ast2600-mtmitchell.dts | 114 ++++++++++++++++++++++++++++
>>   2 files changed, 115 insertions(+)
>>   create mode 100644 arch/arm/dts/ast2600-mtmitchell.dts
>>
>> diff --git a/arch/arm/dts/Makefile b/arch/arm/dts/Makefile
>> index 6c34b83336..b6f9df21fc 100755
>> --- a/arch/arm/dts/Makefile
>> +++ b/arch/arm/dts/Makefile
>> @@ -687,6 +687,7 @@ dtb-$(CONFIG_ARCH_ASPEED) += \
>>       ast2600-dcscm.dtb \
>>       ast2600-fpga.dtb \
>>       ast2600-intel.dtb \
>> +    ast2600-mtmitchell.dtb \
>>       ast2600-ncsi.dtb \
>>       ast2600-p10bmc.dtb \
>>       ast2600-pfr.dtb \
>> diff --git a/arch/arm/dts/ast2600-mtmitchell.dts 
>> b/arch/arm/dts/ast2600-mtmitchell.dts
>> new file mode 100644
>> index 0000000000..ad95be43da
>> --- /dev/null
>> +++ b/arch/arm/dts/ast2600-mtmitchell.dts
>> @@ -0,0 +1,114 @@
>> +// SPDX-License-Identifier: GPL-2.0-only
>> +// Copyright (c) 2022, Ampere Computing LLC
>> +
>> +/dts-v1/;
>> +
>> +#include "ast2600-u-boot.dtsi"
>> +
>> +/ {
>> +    model = "Ampere Mt.Mitchell BMC";
>> +    compatible = "ampere,mtmitchell-bmc", "aspeed,ast2600";
>> +
>> +    memory {
>> +        device_type = "memory";
>> +        reg = <0x80000000 0x40000000>;
>> +    };
>> +
>> +    chosen {
>> +        stdout-path = &uart5;
>> +    };
>> +
>> +    aliases {
>> +        spi0 = &fmc;
>> +        ethernet0 = &mac0;
>> +    };
>> +
>> +    cpus {
>> +        cpu at 0 {
>> +            clock-frequency = <800000000>;
>> +        };
>> +        cpu at 1 {
>> +            clock-frequency = <800000000>;
>> +        };
>> +    };
>> +};
>> +
>> +&uart5 {
>> +    u-boot,dm-pre-reloc;
>> +    status = "okay";
>> +};
>> +
>> +&sdrammc {
>> +    clock-frequency = <400000000>;
>> +};
>> +
>> +&wdt1 {
>> +    status = "okay";
>> +};
>> +
>> +&wdt2 {
>> +    status = "okay";
>> +};
>> +
>> +&wdt3 {
>> +    status = "okay";
>> +};
>> +
>> +&mdio {
>> +    status = "okay";
>> +    pinctrl-names = "default";
>> +    pinctrl-0 = <&pinctrl_mdio1_default &pinctrl_mdio2_default
>> +             &pinctrl_mdio3_default &pinctrl_mdio4_default>;
>> +    #address-cells = <1>;
>> +    #size-cells = <0>;
>> +    ethphy0: ethernet-phy at 0 {
>> +        reg = <0>;
>> +    };
>> +};
>> +
>> +&mac0 {
>> +    status = "okay";
>> +    phy-mode = "rgmii-rxid";
>> +    phy-handle = <&ethphy0>;
>> +    pinctrl-names = "default";
>> +    pinctrl-0 = <&pinctrl_rgmii1_default>;
>> +};
>> +
>> +&fmc {
>> +    status = "okay";
>> +
>> +    pinctrl-names = "default";
>> +    pinctrl-0 = <&pinctrl_fmcquad_default>;
>> +
>> +    flash at 0 {
>> +        compatible = "spi-flash", "sst,w25q256";
>> +        status = "okay";
>> +        spi-max-frequency = <50000000>;
>> +        spi-tx-bus-width = <4>;
>> +        spi-rx-bus-width = <4>;
>> +    };
>> +
>> +    flash at 1 {
>> +        compatible = "spi-flash", "sst,w25q256";
>> +        status = "okay";
>> +        spi-max-frequency = <50000000>;
>> +        spi-tx-bus-width = <4>;
>> +        spi-rx-bus-width = <4>;
>> +    };
>> +};
>> +
>> +&scu {
>> +    mac0-clk-delay = <0x10 0x0a
>> +              0x10 0x10
>> +              0x10 0x10>;
>> +};
>> +
>> +&hace {
>> +    u-boot,dm-pre-reloc;
>> +    status = "okay";
>> +};
>> +
>> +&acry {
>> +    u-boot,dm-pre-reloc;
>> +    status = "okay";
>> +};