Re: [PATCH 1/1] ARM:dts:aspeed: Initial device tree for AMD EthanolX
Hi Joel
On 7/22/20 12:57 AM, Joel Stanley wrote:
> [CAUTION: External Email]
>
> On Mon, 20 Jul 2020 at 16:02, Supreeth Venkatesh
> wrote:
>>
>> Initial introduction of AMD EthanolX platform equipped with an
>> Aspeed ast2500 BMC manufactured by AMD.
>>
>> AMD EthanolX platform is an AMD customer reference board with an
>> Aspeed ast2500 BMC manufactured by AMD.
>> This adds AMD EthanolX device tree file including the flash layout
>> used by EthanolX BMC machines.
>>
>> This also adds an entry of AMD EthanolX device tree file in Makefile.
>>
>> Signed-off-by: Supreeth Venkatesh
>
> Reviewed-by: Joel Stanley
Thanks for reviewing it.
>
> Looks good. One question about the licence.
>
>
>> +++ b/arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts
>> @@ -0,0 +1,209 @@
>> +// SPDX-License-Identifier: Apache-2.0
>> +// Copyright (c) 2020 AMD Inc.
>
> Can you have a read of the licence rules and add a preferred licence.
> The rules are here:
>
>
> https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.kernel.org%2Fdoc%2Fhtml%2Flatest%2Fprocess%2Flicense-rules.html&data=02%7C01%7Csupreeth.venkatesh%40amd.com%7C24e9c801e7e84cb2dd9008d82e042e60%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637309942787036878&sdata=yHUh9uKH%2BYLpLI%2BPFVTJO%2FdXfQpRtSa2t3Hg5r3MGrI%3D&reserved=0
>
> This very hacky one liner will give you an idea of common licences
> used by device trees:
>
> $ git grep -h SPDX -- arch/arm/boot/dts/ | cut -c3- |sort -b | uniq -c
> | sort -hr
> 579 SPDX-License-Identifier: GPL-2.0
> 305 SPDX-License-Identifier: GPL-2.0-only
> 222 SPDX-License-Identifier: GPL-2.0-or-later
> 188 SPDX-License-Identifier: (GPL-2.0+ OR MIT)
> 91 SPDX-License-Identifier: GPL-2.0+
> 72 SPDX-License-Identifier: (GPL-2.0 OR MIT)
> 57 SPDX-License-Identifier: GPL-2.0+ OR MIT
> 46 SPDX-License-Identifier: GPL-2.0-or-later OR MIT
> 38 SPDX-License-Identifier: GPL-2.0 OR X11
> 29 SPDX-License-Identifier: GPL-2.0 OR MIT
> 19 SPDX-License-Identifier: (GPL-2.0+ OR BSD-3-Clause)
> 16 SPDX-License-Identifier: GPL-2.0-only */
> 6 SPDX-License-Identifier: ISC
> 5 SPDX-License-Identifier: (GPL-2.0 OR BSD-3-Clause)
> 4 SPDX-License-Identifier: (GPL-2.0+ OR X11)
> 4 SPDX-License-Identifier: (GPL-2.0 or MIT)
> 4 SPDX-License-Identifier: GPL-2.0 */
> 3 SPDX-License-Identifier: (GPL-2.0-or-later OR BSD-2-Clause)
> 2 SPDX-License-Identifier: (GPL-2.0-or-later OR BSD-3-Clause) */
> 2 SPDX-License-Identifier: GPL-2.0-or-later */
> 2 SPDX-License-Identifier: GPL-2.0+ OR BSD-3-Clause
> 2 SPDX-License-Identifier: (GPL-2.0+)
> 2 SPDX-License-Identifier: (GPL-2.0)
> 2 SPDX-License-Identifier: GPL-2.0
> 1 SPDX-License-Identifier: (GPL-2.0+ OR BSD-3-Clause) */
> 1 SPDX-License-Identifier: GPL-2.0+
> 1 SPDX-License-Identifier: GPL-2.0+
> 1 SPDX-License-Identifier: BSD-3-Clause
>
I will modify in v2 of the patch and send it out.
>
>> +// Author: Supreeth Venkatesh
>> +/dts-v1/;
>> +
>> +#include "aspeed-g5.dtsi"
>> +#include
>> +
>> +/ {
>> + model = "AMD EthanolX BMC";
>> + compatible = "amd,ethanolx-bmc", "aspeed,ast2500";
>> +
>> + memory@8000 {
>> + reg = <0x8000 0x2000>;
>> + };
>> + aliases {
>> + serial0 = &uart1;
>> + serial4 = &uart5;
>> + };
>> + chosen {
>> + stdout-path = &uart5;
>> + bootargs = "console=ttyS4,115200 earlyprintk";
>> + };
>> + leds {
>> + compatible = "gpio-leds";
>> +
>> + fault {
>> + gpios = <&gpio ASPEED_GPIO(A, 2) GPIO_ACTIVE_LOW>;
>> + };
>> +
>> + identify {
>> + gpios = <&gpio ASPEED_GPIO(A, 3) GPIO_ACTIVE_LOW>;
>> + };
>> + };
>> + iio-hwmon {
>> + compatible = "iio-hwmon";
>> + io-channels = <&adc 0>, <&adc 1>, <&adc 2>, <&adc 3>, <&adc
>> 4>;
>> + };
>> +};
>> +
>> +&fmc {
>> + status = "okay";
>> + flash@0 {
>> + status = "okay";
>> + m25p,fast-read;
>> + #include "openbmc-flash-layout.dtsi"
>> + };
>> +};
>> +
>> +
>> +&mac0 {
>> + status = "okay";
>> +
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&pinctrl_rmii1_default>;
>> + clocks = <&syscon ASPEED_CLK_GATE_MAC1CLK>,
>> +<&syscon ASPEED_CLK_MAC1RCLK>;
>> + clock-names = "MACCLK", "RCLK";
>> +};
>> +
>> +&uart1 {
>> + //Host Console
>> + status = "okay";
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&pinctrl_txd1_default
>> +&pinctrl_rxd1_default>;
>> +};
>> +
>> +&uart5 {
>> + //BMC Console
>> + status = "okay";
>> +};
>> +
>> +&adc {
>> + status = "okay";
>> +
>> + pinctrl-names = "defa
Re: [PATCH 1/1] ARM:dts:aspeed: Initial device tree for AMD EthanolX
On Mon, 20 Jul 2020 at 16:02, Supreeth Venkatesh
wrote:
>
> Initial introduction of AMD EthanolX platform equipped with an
> Aspeed ast2500 BMC manufactured by AMD.
>
> AMD EthanolX platform is an AMD customer reference board with an
> Aspeed ast2500 BMC manufactured by AMD.
> This adds AMD EthanolX device tree file including the flash layout
> used by EthanolX BMC machines.
>
> This also adds an entry of AMD EthanolX device tree file in Makefile.
>
> Signed-off-by: Supreeth Venkatesh
Reviewed-by: Joel Stanley
Looks good. One question about the licence.
> +++ b/arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts
> @@ -0,0 +1,209 @@
> +// SPDX-License-Identifier: Apache-2.0
> +// Copyright (c) 2020 AMD Inc.
Can you have a read of the licence rules and add a preferred licence.
The rules are here:
https://www.kernel.org/doc/html/latest/process/license-rules.html
This very hacky one liner will give you an idea of common licences
used by device trees:
$ git grep -h SPDX -- arch/arm/boot/dts/ | cut -c3- |sort -b | uniq -c
| sort -hr
579 SPDX-License-Identifier: GPL-2.0
305 SPDX-License-Identifier: GPL-2.0-only
222 SPDX-License-Identifier: GPL-2.0-or-later
188 SPDX-License-Identifier: (GPL-2.0+ OR MIT)
91 SPDX-License-Identifier: GPL-2.0+
72 SPDX-License-Identifier: (GPL-2.0 OR MIT)
57 SPDX-License-Identifier: GPL-2.0+ OR MIT
46 SPDX-License-Identifier: GPL-2.0-or-later OR MIT
38 SPDX-License-Identifier: GPL-2.0 OR X11
29 SPDX-License-Identifier: GPL-2.0 OR MIT
19 SPDX-License-Identifier: (GPL-2.0+ OR BSD-3-Clause)
16 SPDX-License-Identifier: GPL-2.0-only */
6 SPDX-License-Identifier: ISC
5 SPDX-License-Identifier: (GPL-2.0 OR BSD-3-Clause)
4 SPDX-License-Identifier: (GPL-2.0+ OR X11)
4 SPDX-License-Identifier: (GPL-2.0 or MIT)
4 SPDX-License-Identifier: GPL-2.0 */
3 SPDX-License-Identifier: (GPL-2.0-or-later OR BSD-2-Clause)
2 SPDX-License-Identifier: (GPL-2.0-or-later OR BSD-3-Clause) */
2 SPDX-License-Identifier: GPL-2.0-or-later */
2 SPDX-License-Identifier: GPL-2.0+ OR BSD-3-Clause
2 SPDX-License-Identifier: (GPL-2.0+)
2 SPDX-License-Identifier: (GPL-2.0)
2 SPDX-License-Identifier: GPL-2.0
1 SPDX-License-Identifier: (GPL-2.0+ OR BSD-3-Clause) */
1 SPDX-License-Identifier: GPL-2.0+
1 SPDX-License-Identifier: GPL-2.0+
1 SPDX-License-Identifier: BSD-3-Clause
> +// Author: Supreeth Venkatesh
> +/dts-v1/;
> +
> +#include "aspeed-g5.dtsi"
> +#include
> +
> +/ {
> + model = "AMD EthanolX BMC";
> + compatible = "amd,ethanolx-bmc", "aspeed,ast2500";
> +
> + memory@8000 {
> + reg = <0x8000 0x2000>;
> + };
> + aliases {
> + serial0 = &uart1;
> + serial4 = &uart5;
> + };
> + chosen {
> + stdout-path = &uart5;
> + bootargs = "console=ttyS4,115200 earlyprintk";
> + };
> + leds {
> + compatible = "gpio-leds";
> +
> + fault {
> + gpios = <&gpio ASPEED_GPIO(A, 2) GPIO_ACTIVE_LOW>;
> + };
> +
> + identify {
> + gpios = <&gpio ASPEED_GPIO(A, 3) GPIO_ACTIVE_LOW>;
> + };
> + };
> + iio-hwmon {
> + compatible = "iio-hwmon";
> + io-channels = <&adc 0>, <&adc 1>, <&adc 2>, <&adc 3>, <&adc
> 4>;
> + };
> +};
> +
> +&fmc {
> + status = "okay";
> + flash@0 {
> + status = "okay";
> + m25p,fast-read;
> + #include "openbmc-flash-layout.dtsi"
> + };
> +};
> +
> +
> +&mac0 {
> + status = "okay";
> +
> + pinctrl-names = "default";
> + pinctrl-0 = <&pinctrl_rmii1_default>;
> + clocks = <&syscon ASPEED_CLK_GATE_MAC1CLK>,
> +<&syscon ASPEED_CLK_MAC1RCLK>;
> + clock-names = "MACCLK", "RCLK";
> +};
> +
> +&uart1 {
> + //Host Console
> + status = "okay";
> + pinctrl-names = "default";
> + pinctrl-0 = <&pinctrl_txd1_default
> +&pinctrl_rxd1_default>;
> +};
> +
> +&uart5 {
> + //BMC Console
> + status = "okay";
> +};
> +
> +&adc {
> + status = "okay";
> +
> + pinctrl-names = "default";
> + pinctrl-0 = <&pinctrl_adc0_default
> +&pinctrl_adc1_default
> +&pinctrl_adc2_default
> +&pinctrl_adc3_default
> +&pinctrl_adc4_default>;
> +};
> +
> +// Thermal Sensors
> +&i2c7 {
> + status = "okay";
> +
> + lm75a@48 {
> + compatible = "national,lm75a";
> + reg = <0x48>;
> + };
> +
> + lm75a@49 {
> + compatible = "national,lm75a";
> + reg = <0x49>;
> + };
> +
> + lm75a@4a {
> + compatible = "national,lm75a";
> +
[PATCH 1/1] ARM:dts:aspeed: Initial device tree for AMD EthanolX
Initial introduction of AMD EthanolX platform equipped with an
Aspeed ast2500 BMC manufactured by AMD.
AMD EthanolX platform is an AMD customer reference board with an
Aspeed ast2500 BMC manufactured by AMD.
This adds AMD EthanolX device tree file including the flash layout
used by EthanolX BMC machines.
This also adds an entry of AMD EthanolX device tree file in Makefile.
Signed-off-by: Supreeth Venkatesh
---
arch/arm/boot/dts/Makefile| 1 +
arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts | 209 ++
2 files changed, 210 insertions(+)
create mode 100644 arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts
diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
index d6dfdf73e66b..55ed881d2ebc 100644
--- a/arch/arm/boot/dts/Makefile
+++ b/arch/arm/boot/dts/Makefile
@@ -1281,6 +1281,7 @@ dtb-$(CONFIG_ARCH_ZX) += zx296702-ad1.dtb
dtb-$(CONFIG_ARCH_ASPEED) += \
aspeed-ast2500-evb.dtb \
aspeed-ast2600-evb.dtb \
+ aspeed-bmc-amd-ethanolx.dtb \
aspeed-bmc-arm-centriq2400-rep.dtb \
aspeed-bmc-arm-stardragon4800-rep2.dtb \
aspeed-bmc-facebook-cmm.dtb \
diff --git a/arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts
b/arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts
new file mode 100644
index ..dcabdfc08172
--- /dev/null
+++ b/arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts
@@ -0,0 +1,209 @@
+// SPDX-License-Identifier: Apache-2.0
+// Copyright (c) 2020 AMD Inc.
+// Author: Supreeth Venkatesh
+/dts-v1/;
+
+#include "aspeed-g5.dtsi"
+#include
+
+/ {
+ model = "AMD EthanolX BMC";
+ compatible = "amd,ethanolx-bmc", "aspeed,ast2500";
+
+ memory@8000 {
+ reg = <0x8000 0x2000>;
+ };
+ aliases {
+ serial0 = &uart1;
+ serial4 = &uart5;
+ };
+ chosen {
+ stdout-path = &uart5;
+ bootargs = "console=ttyS4,115200 earlyprintk";
+ };
+ leds {
+ compatible = "gpio-leds";
+
+ fault {
+ gpios = <&gpio ASPEED_GPIO(A, 2) GPIO_ACTIVE_LOW>;
+ };
+
+ identify {
+ gpios = <&gpio ASPEED_GPIO(A, 3) GPIO_ACTIVE_LOW>;
+ };
+ };
+ iio-hwmon {
+ compatible = "iio-hwmon";
+ io-channels = <&adc 0>, <&adc 1>, <&adc 2>, <&adc 3>, <&adc 4>;
+ };
+};
+
+&fmc {
+ status = "okay";
+ flash@0 {
+ status = "okay";
+ m25p,fast-read;
+ #include "openbmc-flash-layout.dtsi"
+ };
+};
+
+
+&mac0 {
+ status = "okay";
+
+ pinctrl-names = "default";
+ pinctrl-0 = <&pinctrl_rmii1_default>;
+ clocks = <&syscon ASPEED_CLK_GATE_MAC1CLK>,
+<&syscon ASPEED_CLK_MAC1RCLK>;
+ clock-names = "MACCLK", "RCLK";
+};
+
+&uart1 {
+ //Host Console
+ status = "okay";
+ pinctrl-names = "default";
+ pinctrl-0 = <&pinctrl_txd1_default
+&pinctrl_rxd1_default>;
+};
+
+&uart5 {
+ //BMC Console
+ status = "okay";
+};
+
+&adc {
+ status = "okay";
+
+ pinctrl-names = "default";
+ pinctrl-0 = <&pinctrl_adc0_default
+&pinctrl_adc1_default
+&pinctrl_adc2_default
+&pinctrl_adc3_default
+&pinctrl_adc4_default>;
+};
+
+// Thermal Sensors
+&i2c7 {
+ status = "okay";
+
+ lm75a@48 {
+ compatible = "national,lm75a";
+ reg = <0x48>;
+ };
+
+ lm75a@49 {
+ compatible = "national,lm75a";
+ reg = <0x49>;
+ };
+
+ lm75a@4a {
+ compatible = "national,lm75a";
+ reg = <0x4a>;
+ };
+
+ lm75a@4b {
+ compatible = "national,lm75a";
+ reg = <0x4b>;
+ };
+
+ lm75a@4c {
+ compatible = "national,lm75a";
+ reg = <0x4c>;
+ };
+
+ lm75a@4d {
+ compatible = "national,lm75a";
+ reg = <0x4d>;
+ };
+
+ lm75a@4e {
+ compatible = "national,lm75a";
+ reg = <0x4e>;
+ };
+
+ lm75a@4f {
+ compatible = "national,lm75a";
+ reg = <0x4f>;
+ };
+};
+
+&kcs1 {
+ status = "okay";
+ kcs_addr = <0x60>;
+};
+
+&kcs2 {
+ status = "okay";
+ kcs_addr = <0x62>;
+};
+
+&kcs4 {
+ status = "okay";
+ kcs_addr = <0x97DE>;
+};
+
+&lpc_snoop {
+ status = "okay";
+ snoop-ports = <0x80>;
+};
+
+&lpc_ctrl {
+ //Enable lpc clock
+ status = "okay";
+};
+
+&pwm_tacho {
+ status = "okay";
+ pinctrl-names = "default";
+ pinctrl-0 = <&pinctrl_pwm0_default
+ &pinctrl_pwm1_default
+ &pinctrl_pwm2_default
+ &pinctrl_pwm3_default
+ &pinctrl_pwm4_default
+ &pinctrl_pwm5_default
+ &pinctrl_
