All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] ARM:dts:aspeed Add Inspur on5263m5 BMC
@ 2019-01-17  6:56 John Wang
  2019-01-17  7:34 ` Joel Stanley
  2019-02-12 22:07 ` Vijay Khemka
  0 siblings, 2 replies; 7+ messages in thread
From: John Wang @ 2019-01-17  6:56 UTC (permalink / raw)
  To: andrew, joel; +Cc: openbmc, linux-aspeed

Add initial version of device tree file for on5263m5 ast2500bmc

Signed-off-by: John Wang <wangzqbj@inspur.com>
---
 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@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@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@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@4e {
+	        compatible = "ti,tmp421";
+		reg = <0x4e>;
+	};
+
+	tmp112@48 {
+		compatible = "ti,tmp112";
+		reg = <0x48>;
+	};
+	
+	eeprom@54 {
+		compatible = "atmel,24c64";
+    		reg = <0x54>; 
+		pagesize = <32>;
+	};
+};
+
+&i2c7 {
+	status = "okay";
+
+	adm1278@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@0 {
+   		reg = <0x00>;
+   		aspeed,fan-tach-ch = /bits/ 8 <0x00 0x01>;  
+ 	};
+
+ 	fan@1 {
+   		reg = <0x01>;
+   		aspeed,fan-tach-ch = /bits/ 8 <0x02 0x03>;
+ 	};
+};
+
+&adc {
+	status = "okay";
+};
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [PATCH] ARM:dts:aspeed Add Inspur on5263m5 BMC
  2019-01-17  6:56 [PATCH] ARM:dts:aspeed Add Inspur on5263m5 BMC John Wang
@ 2019-01-17  7:34 ` Joel Stanley
  2019-02-12 22:07 ` Vijay Khemka
  1 sibling, 0 replies; 7+ messages in thread
From: Joel Stanley @ 2019-01-17  7:34 UTC (permalink / raw)
  To: John Wang; +Cc: Andrew Jeffery, OpenBMC Maillist, linux-aspeed

On Thu, 17 Jan 2019 at 17:56, John Wang <wangzqbj@inspur.com> wrote:
>
> Add initial version of device tree file for on5263m5 ast2500bmc
>
> Signed-off-by: John Wang <wangzqbj@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@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@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@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@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@4e {
> +               compatible = "ti,tmp421";
> +               reg = <0x4e>;
> +       };
> +
> +       tmp112@48 {
> +               compatible = "ti,tmp112";
> +               reg = <0x48>;
> +       };
> +
> +       eeprom@54 {
> +               compatible = "atmel,24c64";
> +               reg = <0x54>;
> +               pagesize = <32>;
> +       };
> +};
> +
> +&i2c7 {
> +       status = "okay";
> +
> +       adm1278@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@0 {
> +               reg = <0x00>;
> +               aspeed,fan-tach-ch = /bits/ 8 <0x00 0x01>;
> +       };
> +
> +       fan@1 {
> +               reg = <0x01>;
> +               aspeed,fan-tach-ch = /bits/ 8 <0x02 0x03>;
> +       };
> +};
> +
> +&adc {
> +       status = "okay";
> +};
> --
> 2.7.4
>

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] ARM:dts:aspeed Add Inspur on5263m5 BMC
  2019-01-17  6:56 [PATCH] ARM:dts:aspeed Add Inspur on5263m5 BMC John Wang
  2019-01-17  7:34 ` Joel Stanley
@ 2019-02-12 22:07 ` Vijay Khemka
  2019-02-13  0:53   ` John Wang
  2019-02-14  2:45   ` Joel Stanley
  1 sibling, 2 replies; 7+ messages in thread
From: Vijay Khemka @ 2019-02-12 22:07 UTC (permalink / raw)
  To: John Wang, andrew, joel; +Cc: openbmc, linux-aspeed



On 2/12/19, 1:58 PM, "Linux-aspeed on behalf of John Wang" <linux-aspeed-bounces+vijaykhemka=fb.com@lists.ozlabs.org on behalf of wangzqbj@inspur.com> wrote:

    Add initial version of device tree file for on5263m5 ast2500bmc
    
    Signed-off-by: John Wang <wangzqbj@inspur.com>
    ---
     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@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@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@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@4e {
    +	        compatible = "ti,tmp421";
    +		reg = <0x4e>;
    +	};
    +
    +	tmp112@48 {
    +		compatible = "ti,tmp112";
    +		reg = <0x48>;
    +	};
    +	
    +	eeprom@54 {
    +		compatible = "atmel,24c64";
    +    		reg = <0x54>; 
    +		pagesize = <32>;
    +	};
    +};
    +
    +&i2c7 {
    +	status = "okay";
    +
    +	adm1278@11 {
    +		  compatible = "adi,adm1278";
    +		  reg = <0x11>;
    +	};
    +};
    +
    +&gfx {
    +	status = "okay";
    +};
    +
    +&pinctrl {
    +	aspeed,external-nodes = <&gfx &lhc>;
    +};
This pinctrl definition is not required as it is already defined in aspeed-g5.dtsi.
    +
    +&pwm_tacho {
    +	status = "okay";
    +	pinctrl-names = "default";
    +	pinctrl-0 = <&pinctrl_pwm0_default &pinctrl_pwm1_default>;
    +
    +	fan@0 {
    +   		reg = <0x00>;
    +   		aspeed,fan-tach-ch = /bits/ 8 <0x00 0x01>;  
    + 	};
    +
    + 	fan@1 {
    +   		reg = <0x01>;
    +   		aspeed,fan-tach-ch = /bits/ 8 <0x02 0x03>;
    + 	};
    +};
    +
    +&adc {
    +	status = "okay";
    +};
    -- 
    2.7.4
    
    


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] ARM:dts:aspeed Add Inspur on5263m5 BMC
  2019-02-12 22:07 ` Vijay Khemka
@ 2019-02-13  0:53   ` John Wang
  2019-02-14  2:45   ` Joel Stanley
  1 sibling, 0 replies; 7+ messages in thread
From: John Wang @ 2019-02-13  0:53 UTC (permalink / raw)
  To: Vijay Khemka; +Cc: andrew, joel, openbmc, linux-aspeed

On Wed, Feb 13, 2019 at 6:11 AM Vijay Khemka <vijaykhemka@fb.com> wrote:
>
>
>
> On 2/12/19, 1:58 PM, "Linux-aspeed on behalf of John Wang" <linux-aspeed-bounces+vijaykhemka=fb.com@lists.ozlabs.org on behalf of wangzqbj@inspur.com> wrote:
>
>     Add initial version of device tree file for on5263m5 ast2500bmc
>
>     Signed-off-by: John Wang <wangzqbj@inspur.com>
>     ---
>      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@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@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@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@4e {
>     +           compatible = "ti,tmp421";
>     +           reg = <0x4e>;
>     +   };
>     +
>     +   tmp112@48 {
>     +           compatible = "ti,tmp112";
>     +           reg = <0x48>;
>     +   };
>     +
>     +   eeprom@54 {
>     +           compatible = "atmel,24c64";
>     +                   reg = <0x54>;
>     +           pagesize = <32>;
>     +   };
>     +};
>     +
>     +&i2c7 {
>     +   status = "okay";
>     +
>     +   adm1278@11 {
>     +             compatible = "adi,adm1278";
>     +             reg = <0x11>;
>     +   };
>     +};
>     +
>     +&gfx {
>     +   status = "okay";
>     +};
>     +
>     +&pinctrl {
>     +   aspeed,external-nodes = <&gfx &lhc>;
>     +};
> This pinctrl definition is not required as it is already defined in aspeed-g5.dtsi.
Thanks,Vijay. Next patch,I would drop the definition.
>     +
>     +&pwm_tacho {
>     +   status = "okay";
>     +   pinctrl-names = "default";
>     +   pinctrl-0 = <&pinctrl_pwm0_default &pinctrl_pwm1_default>;
>     +
>     +   fan@0 {
>     +                   reg = <0x00>;
>     +                   aspeed,fan-tach-ch = /bits/ 8 <0x00 0x01>;
>     +   };
>     +
>     +   fan@1 {
>     +                   reg = <0x01>;
>     +                   aspeed,fan-tach-ch = /bits/ 8 <0x02 0x03>;
>     +   };
>     +};
>     +
>     +&adc {
>     +   status = "okay";
>     +};
>     --
>     2.7.4
>
>
>

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] ARM:dts:aspeed Add Inspur on5263m5 BMC
  2019-02-12 22:07 ` Vijay Khemka
  2019-02-13  0:53   ` John Wang
@ 2019-02-14  2:45   ` Joel Stanley
  1 sibling, 0 replies; 7+ messages in thread
From: Joel Stanley @ 2019-02-14  2:45 UTC (permalink / raw)
  To: Vijay Khemka; +Cc: John Wang, andrew, openbmc, linux-aspeed

On Wed, 13 Feb 2019 at 08:40, Vijay Khemka <vijaykhemka@fb.com> wrote:

> On 2/12/19, 1:58 PM, "Linux-aspeed on behalf of John Wang" <linux-aspeed-bounces+vijaykhemka=fb.com@lists.ozlabs.org on behalf of wangzqbj@inspur.com> wrote:
>
>     Add initial version of device tree file for on5263m5 ast2500bmc
>
>     Signed-off-by: John Wang <wangzqbj@inspur.com>
>     ---
>      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

>     +&pinctrl {
>     +   aspeed,external-nodes = <&gfx &lhc>;
>     +};
> This pinctrl definition is not required as it is already defined in aspeed-g5.dtsi.

I've already applied the patch and sent the pull request. Please
follow up with a patch to clean this up.

Cheers,

Joel

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] ARM:dts:aspeed Add Inspur on5263m5 BMC
  2018-12-21  6:25 John Wang
@ 2019-01-17  2:11 ` Andrew Jeffery
  0 siblings, 0 replies; 7+ messages in thread
From: Andrew Jeffery @ 2019-01-17  2:11 UTC (permalink / raw)
  To: John Wang, joel; +Cc: openbmc

Hi John,

On Fri, 21 Dec 2018, at 16:55, John Wang wrote:
> Add initial version of device tree file for on5263m5 ast2500bmc
> 
> Signed-off-by: John Wang <wangzqbj@inspur.com>
> ---
>  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..dff1085
> --- /dev/null
> +++ b/arch/arm/boot/dts/aspeed-bmc-inspur-on5263m5.dts
> @@ -0,0 +1,146 @@
> +// SPDX-License-Identifier: GPL-2.0
> +// Copyright (c) 2017 Intel Corporation

Shouldn't this be Inspur?

> +/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@7f000000 {
> +			no-map;
> +			reg = <0x7f000000 0x01000000>;

This needs to be in the SDRAM portion of the address-space. With 512MiB of RAM and
a 16MiB VGA framebuffer  the address should be 0x9f000000. At the moment you're
scribbling on the AHB-to-LPC+ bridge mapping.

Cheers,

Andrew

> +		};
> +	};
> +	
> +	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@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@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@4e {
> +	        compatible = "ti,tmp421";
> +		reg = <0x4e>;
> +	};
> +
> +	tmp112@48 {
> +		compatible = "ti,tmp112";
> +		reg = <0x48>;
> +	};
> +	
> +	eeprom@54 {
> +		compatible = "atmel,24c64";
> +    		reg = <0x54>; 
> +		pagesize = <32>;
> +	};
> +};
> +
> +&i2c7 {
> +	status = "okay";
> +
> +	adm1278@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@0 {
> +   		reg = <0x00>;
> +   		aspeed,fan-tach-ch = /bits/ 8 <0x00 0x01>;  
> + 	};
> +
> + 	fan@1 {
> +   		reg = <0x01>;
> +   		aspeed,fan-tach-ch = /bits/ 8 <0x02 0x03>;
> + 	};
> +};
> +
> +&adc {
> +	status = "okay";
> +};
> -- 
> 2.7.4
> 

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [PATCH] ARM:dts:aspeed Add Inspur on5263m5 BMC
@ 2018-12-21  6:25 John Wang
  2019-01-17  2:11 ` Andrew Jeffery
  0 siblings, 1 reply; 7+ messages in thread
From: John Wang @ 2018-12-21  6:25 UTC (permalink / raw)
  To: joel; +Cc: openbmc

Add initial version of device tree file for on5263m5 ast2500bmc

Signed-off-by: John Wang <wangzqbj@inspur.com>
---
 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..dff1085
--- /dev/null
+++ b/arch/arm/boot/dts/aspeed-bmc-inspur-on5263m5.dts
@@ -0,0 +1,146 @@
+// SPDX-License-Identifier: GPL-2.0
+// Copyright (c) 2017 Intel 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@7f000000 {
+			no-map;
+			reg = <0x7f000000 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@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@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@4e {
+	        compatible = "ti,tmp421";
+		reg = <0x4e>;
+	};
+
+	tmp112@48 {
+		compatible = "ti,tmp112";
+		reg = <0x48>;
+	};
+	
+	eeprom@54 {
+		compatible = "atmel,24c64";
+    		reg = <0x54>; 
+		pagesize = <32>;
+	};
+};
+
+&i2c7 {
+	status = "okay";
+
+	adm1278@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@0 {
+   		reg = <0x00>;
+   		aspeed,fan-tach-ch = /bits/ 8 <0x00 0x01>;  
+ 	};
+
+ 	fan@1 {
+   		reg = <0x01>;
+   		aspeed,fan-tach-ch = /bits/ 8 <0x02 0x03>;
+ 	};
+};
+
+&adc {
+	status = "okay";
+};
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2019-02-14  2:46 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-01-17  6:56 [PATCH] ARM:dts:aspeed Add Inspur on5263m5 BMC John Wang
2019-01-17  7:34 ` Joel Stanley
2019-02-12 22:07 ` Vijay Khemka
2019-02-13  0:53   ` John Wang
2019-02-14  2:45   ` Joel Stanley
  -- strict thread matches above, loose matches on Subject: below --
2018-12-21  6:25 John Wang
2019-01-17  2:11 ` Andrew Jeffery

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.