FW: [PATCH linux dev-4.13 v1] ARM: dts: aspeed: Add Inventec Lanyang BMC
YangBrianC.W 楊嘉偉 TAO
yang.brianc.w at inventec.com
Mon Apr 16 21:09:54 AEST 2018
Hi Andrew,
2018-04-16 15:04 GMT+08:00 Chen.KenYY 陳永營 TAO <Chen.KenYY at inventec.com>:
> FYR
>
> -----Original Message-----
> From: openbmc [mailto:openbmc-bounces+chen.kenyy=inventec.com at lists.ozlabs.org] On Behalf Of Andrew Jeffery
> Sent: Friday, April 13, 2018 9:58 AM
> To: openbmc at lists.ozlabs.org
> Subject: Re: [PATCH linux dev-4.13 v1] ARM: dts: aspeed: Add Inventec Lanyang BMC
>
> Hi Brian,
>
> On Wed, 11 Apr 2018, at 17:12, Brian Yang wrote:
>> The Inventec Lanyang is Power 9 platform with ast2500 BMC.
>>
>> Tested-by: Brian Yang <yang.brianc.w at inventec.com>
>> Signed-off-by: Brian Yang <yang.brianc.w at inventec.com>
>>
>> ---
>>
>> v0->v1
>> -Add test information in commit
>> ---
>> arch/arm/boot/dts/Makefile | 1 +
>> arch/arm/boot/dts/aspeed-bmc-opp-lanyang.dts | 331
>> +++++++++++++++++++++++++++
>> 2 files changed, 332 insertions(+)
>> create mode 100644 arch/arm/boot/dts/aspeed-bmc-opp-lanyang.dts
>>
>> diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
>> index 5cfee25..d1882f0 100644
>> --- a/arch/arm/boot/dts/Makefile
>> +++ b/arch/arm/boot/dts/Makefile
>> @@ -1057,6 +1057,7 @@ dtb-$(CONFIG_ARCH_ASPEED) += \
>> aspeed-ast2500-evb.dtb \
>> aspeed-bmc-arm-centriq2400-rep.dtb \
>> aspeed-bmc-intel-s2600wf.dtb \
>> + aspeed-bmc-opp-lanyang.dtb \
>> aspeed-bmc-opp-palmetto.dtb \
>> aspeed-bmc-opp-romulus.dtb \
>> aspeed-bmc-opp-witherspoon.dtb \
>> diff --git a/arch/arm/boot/dts/aspeed-bmc-opp-lanyang.dts b/arch/arm/
>> boot/dts/aspeed-bmc-opp-lanyang.dts
>> new file mode 100644
>> index 0000000..6241730
>> --- /dev/null
>> +++ b/arch/arm/boot/dts/aspeed-bmc-opp-lanyang.dts
>> @@ -0,0 +1,331 @@
>> +// SPDX-License-Identifier: GPL-2.0+
>> +// Copyright (c) 2018 Inventec Corporation /dts-v1/;
>> +
>> +#include "aspeed-g5.dtsi"
>> +#include <dt-bindings/gpio/aspeed-gpio.h>
>> +
>> +/ {
>> + model = "Lanyang BMC";
>> + compatible = "inventec,lanyang-bmc", "aspeed,ast2500";
>> +
>> + chosen {
>> + stdout-path = &uart5;
>> + bootargs = "console=ttyS4,115200 earlyprintk";
>> + };
>> +
>> + memory {
>> + reg = <0x80000000 0x40000000>;
>> + };
>> +
>> + reserved-memory {
>> + #address-cells = <1>;
>> + #size-cells = <1>;
>> + ranges;
>> +
>> + flash_memory: region at 98000000 {
>> + no-map;
>> + reg = <0x98000000 0x04000000>; /* 64M */
>> + };
>> + };
>> +
>> + leds {
>> + compatible = "gpio-leds";
>> +
>> + sys_boot_status {
>> + label = "System_boot_status";
>> + gpios = <&gpio ASPEED_GPIO(B, 6) GPIO_ACTIVE_LOW>;
>> + };
>> +
>> + attention {
>> + label = "Attention_locator";
>> + gpios = <&gpio ASPEED_GPIO(B, 7) GPIO_ACTIVE_HIGH>;
>> + };
>> +
>> + plt_fault {
>> + label = "Platform_fault";
>> + gpios = <&gpio ASPEED_GPIO(B, 1) GPIO_ACTIVE_HIGH>;
>> + };
>> +
>> + hdd_fault {
>> + label = "Onboard_drive_fault";
>> + gpios = <&gpio ASPEED_GPIO(B, 3) GPIO_ACTIVE_HIGH>;
>> + };
>> + bmc_err {
>> + lable = "BMC_fault";
>> + gpios = <&gpio ASPEED_GPIO(H, 6) GPIO_ACTIVE_HIGH>;
>> + };
>> +
>> + sys_err {
>> + lable = "Sys_fault";
>> + gpios = <&gpio ASPEED_GPIO(H, 7) GPIO_ACTIVE_HIGH>;
>> + };
>> + };
>> +
>> + fsi: gpio-fsi {
>> + compatible = "fsi-master-gpio", "fsi-master";
>> + #address-cells = <2>;
>> + #size-cells = <0>;
>> +
>> + clock-gpios = <&gpio ASPEED_GPIO(J, 0) GPIO_ACTIVE_HIGH>;
>> + data-gpios = <&gpio ASPEED_GPIO(J, 1) GPIO_ACTIVE_HIGH>;
>> + trans-gpios = <&gpio ASPEED_GPIO(D, 5) GPIO_ACTIVE_HIGH>;
>> + enable-gpios = <&gpio ASPEED_GPIO(D, 0) GPIO_ACTIVE_HIGH>;
>> + mux-gpios = <&gpio ASPEED_GPIO(H, 2) GPIO_ACTIVE_HIGH>;
>> + };
>> +
>> + 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 13>, <&adc 14>, <&adc 15>;
>> + };
>> +
>> + iio-hwmon-battery {
>> + compatible = "iio-hwmon";
>> + io-channels = <&adc 12>;
>> + };
>> +};
>> +
>> +#include "ibm-power9-cfam.dtsi"
>> +
>> +&pwm_tacho {
>> + status = "okay";
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&pinctrl_pwm0_default &pinctrl_pwm1_default
>> + &pinctrl_pwm2_default &pinctrl_pwm3_default>;
>> +
>> + fan at 0 {
>> + reg = <0x00>;
>> + aspeed,fan-tach-ch = /bits/ 8 <0x00>;
>> + };
>> +
>> + fan at 1 {
>> + reg = <0x01>;
>> + aspeed,fan-tach-ch = /bits/ 8 <0x01>;
>> + };
>> +
>> + fan at 2 {
>> + reg = <0x02>;
>> + aspeed,fan-tach-ch = /bits/ 8 <0x02>;
>> + };
>> +
>> + fan at 3 {
>> + reg = <0x03>;
>> + aspeed,fan-tach-ch = /bits/ 8 <0x03>;
>> + };
>> +};
>> +
>> +&fmc {
>> + status = "okay";
>> + flash at 0 {
>> + status = "okay";
>> + m25p,fast-read;
>> + label = "bmc";
>> +#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;
>> + };
>> +};
>> +
>> +&spi2 {
>> + status = "okay";
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&pinctrl_spi2ck_default
>> + &pinctrl_spi2cs0_default
>> + &pinctrl_spi2cs1_default
>> + &pinctrl_spi2miso_default
>> + &pinctrl_spi2mosi_default>;
>> +
>> + flash at 0 {
>> + status = "okay";
>> + };
>> +};
>> +
>> +&uart1 {
>> + status = "okay";
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&pinctrl_txd1_default
>> + &pinctrl_rxd1_default>;
>> +};
>> +
>> +&lpc_ctrl {
>> + status = "okay";
>> + memory-region = <&flash_memory>;
>> + flash = <&spi1>;
>> +};
>> +
>> +&lpc_snoop {
>> + status = "okay";
>> + snoop-ports = <0x80>;
>> +};
>> +
>> +&mbox {
>> + status = "okay";
>> +};
>> +
>> +&uart5 {
>> + status = "okay";
>> +};
>> +
>> +&mac0 {
>> + status = "okay";
>> +
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&pinctrl_rmii1_default>;
>> + use-ncsi;
>> +};
>> +
>> +&mac1 {
>> + status = "okay";
>> +
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&pinctrl_rgmii2_default &pinctrl_mdio2_default>; };
>> +
>> +&i2c0 {
>> + status = "okay";
>> +
>> + eeprom at 55 {
>> + compatible = "atmel,24c64";
>> + reg = <0x55>;
>> + pagesize = <32>;
>> + };
>> +
>> + rtc at 68 {
>> + compatible = "nxp,pcf8523";
>> + reg = <0x68>;
>> + };
>> +
>> + tmp75 at 48 {
>> + compatible = "ti,tmp75";
>> + reg = <0x48>;
>> + };
>> +};
>> +
>> +&i2c1 {
>> + status = "okay";
>> +};
>> +
>> +&i2c2 {
>> + status = "okay";
>> +};
>> +
>> +&i2c3 {
>> + status = "okay";
>> +};
>> +
>> +&i2c4 {
>> + status = "okay";
>> +};
>> +
>> +&i2c5 {
>> + status = "okay";
>> +};
>> +
>> +&i2c6 {
>> + status = "okay";
>> +};
>> +
>> +&i2c7 {
>> + status = "okay";
>> +};
>> +
>> +&i2c8 {
>> + status = "okay";
>> +};
>> +
>> +&i2c9 {
>> + status = "okay";
>> +};
>> +
>> +&i2c10 {
>> + status = "okay";
>> +};
>> +
>> +&i2c11 {
>> + status = "okay";
>> +};
>
> Do we need to enable all of the above i2c buses? By comparison, Zaius enables i2c0, i2c1, i2c4, i2c7 and i2c8.
Thank you for your review.
We need to enable all of the above i2c buses because we want to verify
the i2c bus is fine by i2ctool.
Because we use the above i2c buses on Lanyang platform.
>
>> +
>> +&vuart {
>> + status = "okay";
>> +};
>> +
>> +&gfx {
>> + status = "okay";
>> +};
>> +
>> +&pinctrl {
>> + aspeed,external-nodes = <&gfx &lhc>; };
>> +
>> +&gpio {
>> + pin_gpio_b0 {
>> + gpio-hog;
>> + gpios = <ASPEED_GPIO(B, 0) GPIO_ACTIVE_HIGH>;
>> + output-high;
>> + line-name = "BMC_HDD1_PWR_EN";
>> + };
>> +
>> + pin_gpio_b5 {
>> + gpio-hog;
>> + gpios = <ASPEED_GPIO(B, 5) GPIO_ACTIVE_HIGH>;
>> + input;
>> + line-name = "BMC_USB1_OCI2";
>> + };
>> +
>> + pin_gpio_h5 {
>> + gpio-hog;
>> + gpios = <ASPEED_GPIO(H, 5) GPIO_ACTIVE_HIGH>;
>> + output-high;
>> + line-name = "BMC_CP0_PERST_ENABLE_R";
>> + };
>> +
>> + pin_gpio_z2 {
>> + gpio-hog;
>> + gpios = <ASPEED_GPIO(Z, 2) GPIO_ACTIVE_HIGH>;
>> + output-high;
>> + line-name = "RST_PCA9546_U177_N";
>> + };
>> +
>> + pin_gpio_aa6 {
>> + gpio-hog;
>> + gpios = <ASPEED_GPIO(AA, 6) GPIO_ACTIVE_HIGH>;
>> + output-high;
>> + line-name = "BMC_CP0_RESET_N";
>> + };
>> +
>> + pin_gpio_aa7 {
>> + gpio-hog;
>> + gpios = <ASPEED_GPIO(AA, 7) GPIO_ACTIVE_HIGH>;
>> + output-high;
>> + line-name = "BMC_TPM_RESET_N";
>> + };
>> +
>> + pin_gpio_ab0 {
>> + gpio-hog;
>> + gpios = <ASPEED_GPIO(AB, 0) GPIO_ACTIVE_LOW>;
>> + output-high;
>> + line-name = "BMC_USB_PWRON_N";
>> + };
>> +};
>> +
>> +&ibt {
>> + status = "okay";
>> +};
>> +
>> +&adc {
>> + status = "okay";
>> +};
>> +
>> --
>> 2.7.4
>>
>
> Looks okay from a pinctrl perspective, so if there is reason to enable all of the i2c buses, then I'll send an Acked-by
>
> Cheers,
>
> Andrew
Thank you
Brian
More information about the openbmc
mailing list