[PATCH] ARM:dts:aspeed Add Inspur on5263m5 BMC
Joel Stanley
joel at jms.id.au
Thu Jan 17 18:34:34 AEDT 2019
On Thu, 17 Jan 2019 at 17:56, John Wang <wangzqbj at inspur.com> wrote:
>
> Add initial version of device tree file for on5263m5 ast2500bmc
>
> Signed-off-by: John Wang <wangzqbj at inspur.com>
The changes look good. It's convention to add a changelog to your
patch when you resend it, just below the ---. For example:
---
v2: FIx reserved memory address and correct copyright
You should also send the patch with v2 (or v3, or v4, etc) in the
subject line. Git format-patch can do this for you with the -v flag:
git format-patch -v 2 --to=openbmc at lists.ozlabs.org -1
Keep this in mind for next time.
Your patch has some bad whitespace. I get this warnings when applying:
Applying: ARM:dts:aspeed Add Inspur on5263m5 BMC
.git/rebase-apply/patch:42: trailing whitespace.
.git/rebase-apply/patch:45: trailing whitespace.
.git/rebase-apply/patch:114: trailing whitespace.
.git/rebase-apply/patch:117: trailing whitespace, space before tab in indent.
reg = <0x54>;
.git/rebase-apply/patch:145: space before tab in indent.
reg = <0x00>;
warning: squelched 6 whitespace errors
warning: 11 lines add whitespace errors.
You can use a tool called checkpatch.pl to check that your patches are
okay before sending:
./scripts/checkpatch.pl 0001-ARM-dts-aspeed-Add-Inspur-on5263m5-BMC.patch
Can you please do that and send a v3?
Thanks!
Joel
> arch/arm/boot/dts/aspeed-bmc-inspur-on5263m5.dts | 146 +++++++++++++++++++++++
> 1 file changed, 146 insertions(+)
> create mode 100644 arch/arm/boot/dts/aspeed-bmc-inspur-on5263m5.dts
>
> diff --git a/arch/arm/boot/dts/aspeed-bmc-inspur-on5263m5.dts b/arch/arm/boot/dts/aspeed-bmc-inspur-on5263m5.dts
> new file mode 100644
> index 0000000..064ae30
> --- /dev/null
> +++ b/arch/arm/boot/dts/aspeed-bmc-inspur-on5263m5.dts
> @@ -0,0 +1,146 @@
> +// SPDX-License-Identifier: GPL-2.0
> +// Copyright (c) 2018 Inspur Corporation
> +/dts-v1/;
> +
> +#include "aspeed-g5.dtsi"
> +#include <dt-bindings/gpio/aspeed-gpio.h>
> +
> +/ {
> + model = "ON5263M5 BMC";
> + compatible = "inspur,on5263m5-bmc", "aspeed,ast2500";
> +
> + chosen {
> + stdout-path = &uart5;
> + bootargs = "earlyprintk";
> + };
> +
> + memory {
> + reg = <0x80000000 0x20000000>;
> + };
> +
> + reserved-memory {
> + #address-cells = <1>;
> + #size-cells = <1>;
> + ranges;
> +
> + vga_memory: framebuffer at 9f000000 {
> + no-map;
> + reg = <0x9f000000 0x01000000>;
> + };
> + };
> +
> + leds {
> + compatible = "gpio-leds";
> +
> + bmc_alive {
> + label = "bmc_alive";
> + gpios = <&gpio ASPEED_GPIO(I, 1) GPIO_ACTIVE_LOW>;
> + linux,default-trigger = "timer";
> + };
> + };
> +
> + iio-hwmon {
> + compatible = "iio-hwmon";
> + io-channels = <&adc 0>, <&adc 1>, <&adc 2>, <&adc 3>,
> + <&adc 4>, <&adc 5>, <&adc 6>, <&adc 7>;
> + };
> +
> +};
> +
> +&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";
> + m25p,fast-read;
> + label = "pnor";
> + };
> +};
> +
> +&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>;
> +};
> +
> +&i2c6 {
> + status = "okay";
> +
> + tmp421 at 4e {
> + compatible = "ti,tmp421";
> + reg = <0x4e>;
> + };
> +
> + tmp112 at 48 {
> + compatible = "ti,tmp112";
> + reg = <0x48>;
> + };
> +
> + eeprom at 54 {
> + compatible = "atmel,24c64";
> + reg = <0x54>;
> + pagesize = <32>;
> + };
> +};
> +
> +&i2c7 {
> + status = "okay";
> +
> + adm1278 at 11 {
> + compatible = "adi,adm1278";
> + reg = <0x11>;
> + };
> +};
> +
> +&gfx {
> + status = "okay";
> +};
> +
> +&pinctrl {
> + aspeed,external-nodes = <&gfx &lhc>;
> +};
> +
> +&pwm_tacho {
> + status = "okay";
> + pinctrl-names = "default";
> + pinctrl-0 = <&pinctrl_pwm0_default &pinctrl_pwm1_default>;
> +
> + fan at 0 {
> + reg = <0x00>;
> + aspeed,fan-tach-ch = /bits/ 8 <0x00 0x01>;
> + };
> +
> + fan at 1 {
> + reg = <0x01>;
> + aspeed,fan-tach-ch = /bits/ 8 <0x02 0x03>;
> + };
> +};
> +
> +&adc {
> + status = "okay";
> +};
> --
> 2.7.4
>
More information about the openbmc
mailing list