[PATCH u-boot v2019.04-aspeed-openbmc v5] board: ast2600: Add Inspur SCM V1 board
lintao1890
lintao1890 at 163.com
Tue May 30 16:17:32 AEST 2023
At 2023-05-30 14:00:43, "Joel Stanley" <joel at jms.id.au> wrote:
>On Tue, 30 May 2023 at 03:03, <lintao1890 at 163.com> wrote:
>>
>> From: LinTao <lintao.lc at inspur.com>
>>
>> AST2600-INSPUR is an Inspur SCM V1 board which is
>> equipped with AST2600
>>
>> Signed-off-by: LinTao <lintao.lc at inspur.com>
>> ---
>> V1 -> V2:
>> - add a missing '\'
>> ---
>> V2 -> V3:
>> - use gpio subsystem
>> ---
>> V3 -> V4:
>> - fix style problems
>> ---
>> V4 -> V5:
>> - turn on SCM LED1
>> ---
>> arch/arm/dts/Makefile | 3 +-
>> arch/arm/dts/ast2600-inspur-nf5280m7.dts | 190 +++++++++++++++++++++++
>> arch/arm/mach-aspeed/ast2600/Kconfig | 8 +
>> board/aspeed/ast2600_inspur/Kconfig | 13 ++
>> board/aspeed/ast2600_inspur/Makefile | 1 +
>> board/aspeed/ast2600_inspur/inspur.c | 18 +++
>> 6 files changed, 232 insertions(+), 1 deletion(-)
>> create mode 100644 arch/arm/dts/ast2600-inspur-nf5280m7.dts
>> create mode 100644 board/aspeed/ast2600_inspur/Kconfig
>> create mode 100644 board/aspeed/ast2600_inspur/Makefile
>> create mode 100644 board/aspeed/ast2600_inspur/inspur.c
>>
>> diff --git a/arch/arm/dts/Makefile b/arch/arm/dts/Makefile
>> index 6c34b83336..f62c16b4f1 100755
>> --- a/arch/arm/dts/Makefile
>> +++ b/arch/arm/dts/Makefile
>> @@ -695,7 +695,8 @@ dtb-$(CONFIG_ARCH_ASPEED) += \
>> ast2600-slt.dtb \
>> ast2600-tacoma.dtb \
>> ast2600-intel.dtb \
>> - ast2600-tacoma.dtb
>> + ast2600-tacoma.dtb \
>> + ast2600-inspur-nf5280m7.dtb
>>
>> dtb-$(CONFIG_ARCH_STI) += stih410-b2260.dtb
>>
>> diff --git a/arch/arm/dts/ast2600-inspur-nf5280m7.dts b/arch/arm/dts/ast2600-inspur-nf5280m7.dts
>> new file mode 100644
>> index 0000000000..8210797868
>> --- /dev/null
>> +++ b/arch/arm/dts/ast2600-inspur-nf5280m7.dts
>> @@ -0,0 +1,190 @@
>> +// SPDX-License-Identifier: GPL-2.0+
>> +// Copyright 2023 Inspur Corp.
>> +/dts-v1/;
>> +
>> +#include "ast2600-u-boot.dtsi"
>> +
>> +/ {
>> + model = "Inspur SCM V1";
>> + compatible = "inspur,nf5280m7-bmc", "aspeed,ast2600";
>> +
>> + memory {
>> + device_type = "memory";
>> + reg = <0x80000000 0x40000000>;
>> + };
>> +
>> + chosen {
>> + stdout-path = &uart5;
>> + };
>> +
>> + aliases {
>> + mmc0 = &emmc_slot0;
>> + mmc1 = &sdhci_slot0;
>> + mmc2 = &sdhci_slot1;
>> + spi0 = &fmc;
>> + spi1 = &spi1;
>> + spi2 = &spi2;
>> + ethernet1 = &mac1;
>> + };
>> +
>> + cpus {
>> + cpu at 0 {
>> + clock-frequency = <1200000000>;
>> + };
>> + cpu at 1 {
>> + clock-frequency = <1200000000>;
>> + };
>> + };
>> +};
>> +
>> +&gpio0 {
>> + status = "okay";
>> +};
>> +
>> +&uart1 {
>> + status = "okay";
>> +};
>> +
>> +&uart2 {
>> + status = "okay";
>> +};
>> +
>> +&uart5 {
>> + u-boot,dm-pre-reloc;
>> + status = "okay";
>> +};
>> +
>> +&sdrammc {
>> + clock-frequency = <400000000>;
>> +};
>> +
>> +&mdio {
>> + status = "okay";
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&pinctrl_mdio2_default>;
>> + #address-cells = <1>;
>> + #size-cells = <0>;
>> + ethphy0: ethernet-phy at 0 {
>> + reg = <0>;
>> + };
>> +
>> + ethphy1: ethernet-phy at 1 {
>> + reg = <0>;
>> + };
>> +
>> + ethphy2: ethernet-phy at 2 {
>> + reg = <0>;
>> + };
>> +
>> + ethphy3: ethernet-phy at 3 {
>> + reg = <0>;
>> + };
>> +};
>> +
>> +&mac1 {
>> + status = "okay";
>> + phy-mode = "rgmii";
>> + phy-handle = <ðphy1>;
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&pinctrl_rgmii2_default>;
>> +};
>> +
>> +&fmc {
>> + status = "okay";
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&pinctrl_fmcquad_default>;
>> + flash at 0 {
>> + status = "okay";
>> + spi-max-frequency = <40000000>;
>> + spi-tx-bus-width = <4>;
>> + spi-rx-bus-width = <4>;
>> + };
>> + flash at 1 {
>> + status = "okay";
>> + spi-max-frequency = <40000000>;
>> + spi-tx-bus-width = <4>;
>> + spi-rx-bus-width = <4>;
>> + };
>> +};
>> +
>> +&i2c1 {
>> + status = "okay";
>> + multi-master;
>> +
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&pinctrl_i2c2_default>;
>
>You shouldn't need to add the pinctrl nodes, as they're present in ast2600.dtsi.
>
>> +};
>> +
>> +&i2c4 {
>> + status = "okay";
>> + multi-master;
>> +
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&pinctrl_i2c5_default>;
>> +};
>> +
>> +&i2c5 {
>> + status = "okay";
>> + multi-master;
>> +
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&pinctrl_i2c6_default>;
>> +};
>> +
>> +&i2c6 {
>> + status = "okay";
>> + multi-master;
>> +
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&pinctrl_i2c7_default>;
>> +};
>> +
>> +&i2c7 {
>> + status = "okay";
>> + multi-master;
>> +
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&pinctrl_i2c8_default>;
>> +};
>> +
>> +&i2c8 {
>> + status = "okay";
>> + multi-master;
>> +
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&pinctrl_i2c9_default>;
>> +};
>> +
>> +&i2c9 {
>> + status = "okay";
>> + multi-master;
>> +
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&pinctrl_i2c10_default>;
>> +};
>> +
>> +&i2c12 {
>> + status = "okay";
>> + multi-master;
>> +
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&pinctrl_i2c13_default>;
>> +};
>> +
>> +&i2c13 {
>> + status = "okay";
>> + multi-master;
>> +
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&pinctrl_i2c14_default>;
>> +};
>> +
>> +&hace {
>> + u-boot,dm-pre-reloc;
>> + status = "okay";
>> +};
>> +
>> +&acry {
>> + u-boot,dm-pre-reloc;
>> + status = "disabled";
>> +};
>
>
>> --- /dev/null
>> +++ b/board/aspeed/ast2600_inspur/inspur.c
>
> +int board_late_init(void)
>> +{
>> + /* Turn on SCM LED1 */
>> + gpio_request(ASPEED_GPIO(B, 2), "");
>> + gpio_direction_output(ASPEED_GPIO(B, 2), GPIO_ACTIVE_LOW);
>
>Are you able to do this with a gpio hog in the device tree?
>
>If so, we would prefer that approach as it avoids adding another
>platform to the source tree.
>
>Cheers,
>
>Joel
I will be on the board_ late_init adding other functions ,
this turn on led1 is only the first fill.
I hope my first submission is as simple as possible,
so I added this Turn on Led.
It ( "turn on led1") will be deleted in the future.
so I didn't do this in the device tree .
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.ozlabs.org/pipermail/openbmc/attachments/20230530/75e49b30/attachment.htm>
More information about the openbmc
mailing list