Skip to contentSkip to conversation

ARM: dts: aspeed: Adding Facebook Yosemite V2 BMC

PATCH Superseded manikandan-e 3 participants 8 messages Nov 18, 2019 – Nov 25, 2019

Subsystems (2)
Device tree bindings Arm/Aspeed machine support

v1 of 8 Superseded Latest is v8 →

lore
Share & tools
lore

manikandan-e

author
#0 lore raw

PATCH 1/1 ARM: dts: aspeed: Adding Facebook Yosemite V2 BMC

The Yosemite V2 is a facebook multi-node server
platform that host four OCP server. The BMC
in the Yosemite V2 platorm based on AST2500 SoC.

This patch adds linux device tree entry related to Yosemite V2 specific devices connected to BMC SoC.

Signed-off-by manikandan-e <[email protected]>
---
 .../boot/dts/aspeed-bmc-facebook-yosemitev2.dts    | 170 +++++++++++++++++++++
 1 file changed, 170 insertions(+)
 create mode 100644 arch/arm/boot/dts/aspeed-bmc-facebook-yosemitev2.dts
arch/arm/boot/dts/aspeed-bmc-facebook-yosemitev2.dts new file +170 -0
new file mode 100644
index 0000000..46a285a
@@ -0,0 +1,170 @@
+// SPDX-License-Identifier: GPL-2.0+
+// Copyright (c) 2018 Facebook Inc.
+// Author:
+/dts-v1/;
+
+#include "aspeed-g5.dtsi"
+#include <dt-bindings/gpio/aspeed-gpio.h>
+
+/ {
+	model = "Facebook Yosemitev2 BMC";
+	compatible = "facebook,yosemitev2-bmc", "aspeed,ast2500";
+	aliases {
+		serial0 = &uart1;
+		serial4 = &uart5;
+	};
+	chosen {
+		stdout-path = &uart5;
+		bootargs = "console=ttyS4,115200 earlyprintk";
+	};
+
+	memory at 80000000 {
+		reg = <0x80000000 0x20000000>;
+	};
+
+	iio-hwmon {
+		// VOLATAGE SENSOR
+		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 12> , <&adc 13> , <&adc 14> , <&adc 15> ;
+	};
+};
+
+&fmc {
+	status = "okay";
+	flash at 0 {
+		status = "okay";
+		m25p,fast-read;
+#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";
+	};
+};
+
+&lpc_snoop {
+	status = "okay";
+	snoop-ports = <0x80>;
+};
+
+&lpc_ctrl {
+	// Enable lpc clock
+	status = "okay";
+};
+
+&vuart {
+	// VUART Host Console
+	status = "okay";
+};
+
+&uart1 {
+	// Host Console
+	status = "okay";
+	pinctrl-names = "default";
+	pinctrl-0 = <&pinctrl_txd1_default
+		     &pinctrl_rxd1_default>;
+};
+
+&uart2 {
+	// SoL Host Console
+	status = "okay";
+};
+
+&uart3 {
+	// SoL BMC Console
+	status = "okay";
+};
+
+&uart5 {
+	// BMC Console
+	status = "okay";
+};
+
+&mac0 {
+	status = "okay";
+
+	pinctrl-names = "default";
+	pinctrl-0 = <&pinctrl_rmii1_default>;
+	use-ncsi;
+};
+
+&adc {
+	status = "okay";
+};
+
+&i2c8 {
+	status = "okay";
+	//FRU EEPROM
+	eeprom at 51 {
+		compatible = "atmel,24c64";
+		reg = <0x51>;
+		pagesize = <32>;
+	};
+};
+
+&i2c9 {
+	status = "okay";
+	tmp421 at 4e {
+	//INLET TEMP
+		compatible = "ti,tmp421";
+		reg = <0x4e>;
+	};
+	//OUTLET TEMP
+	tmp421 at 4f {
+		compatible = "ti,tmp421";
+		reg = <0x4f>;
+	};
+};
+
+&i2c10 {
+	status = "okay";
+	//HSC
+	adm1278 at 40 {
+		compatible = "adi,adm1278";
+		reg = <0x40>;
+	};
+};
+
+&i2c11 {
+	status = "okay";
+	//MEZZ_TEMP_SENSOR
+	tmp421 at 1f {
+		compatible = "ti,tmp421";
+		reg = <0x1f>;
+	};
+};
+
+&i2c12 {
+	status = "okay";
+	//MEZZ_FRU
+	eeprom at 51 {
+		compatible = "atmel,24c64";
+		reg = <0x51>;
+		pagesize = <32>;
+	};
+};
+
+&pwm_tacho {
+	status = "okay";
+   //FSC
+	pinctrl-names = "default";
+	pinctrl-0 = <&pinctrl_pwm0_default &pinctrl_pwm1_default>;
+	fan at 0 {
+		reg = <0x00>;
+		aspeed,fan-tach-ch = /bits/ 8 <0x00>;
+	};
+	fan at 1 {
+		reg = <0x01>;
+		aspeed,fan-tach-ch = /bits/ 8 <0x02>;
+	};
+};
Signature · 1 line
2.7.4
#1 lore raw
Quoted 72 lines · manikandan-e

The Yosemite V2 is a facebook multi-node server platform that host four OCP server. The BMC in the Yosemite V2 platorm based on AST2500 SoC.

This patch adds linux device tree entry related to Yosemite V2 specific devices connected to BMC SoC.

Signed-off-by manikandan-e <[email protected]>
---
 .../boot/dts/aspeed-bmc-facebook-yosemitev2.dts    | 170 +++++++++++++++++++++
 1 file changed, 170 insertions(+)
 create mode 100644 arch/arm/boot/dts/aspeed-bmc-facebook-yosemitev2.dts
arch/arm/boot/dts/aspeed-bmc-facebook-yosemitev2.dts new file +52 -0
new file mode 100644
index 0000000..46a285a
@@ -0,0 +1,170 @@
+// SPDX-License-Identifier: GPL-2.0+
+// Copyright (c) 2018 Facebook Inc.
+// Author:
+/dts-v1/;
+
+#include "aspeed-g5.dtsi"
+#include <dt-bindings/gpio/aspeed-gpio.h>
+
+/ {
+	model = "Facebook Yosemitev2 BMC";
+	compatible = "facebook,yosemitev2-bmc", "aspeed,ast2500";
+	aliases {
+		serial0 = &uart1;
+		serial4 = &uart5;
+	};
+	chosen {
+		stdout-path = &uart5;
+		bootargs = "console=ttyS4,115200 earlyprintk";
+	};
+
+	memory@80000000 {
+		reg = <0x80000000 0x20000000>;
+	};
+
+	iio-hwmon {
+		// VOLATAGE SENSOR
+		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 12> , <&adc 13> , <&adc 14> , <&adc 15> ;
+	};
+};
+
+&fmc {
+	status = "okay";
+	flash@0 {
+		status = "okay";
+		m25p,fast-read;
+#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";
+	};

arch/arm/boot/dts/aspeed-bmc-facebook-yosemitev2.dts line 62

Quoted 10 lines
arch/arm/boot/dts/aspeed-bmc-facebook-yosemitev2.dts new file +10 -0
+};
+
+&lpc_snoop {
+	status = "okay";
+	snoop-ports = <0x80>;
+};
+
+&lpc_ctrl {
+	// Enable lpc clock
+	status = "okay";

Something I'm intending to fix in the devicetrees using LPC is to hog the pins in the pinctrl node. You should consider doing the same here.

Quoted 8 lines
arch/arm/boot/dts/aspeed-bmc-facebook-yosemitev2.dts new file +8 -0
+};
+
+&vuart {
+	// VUART Host Console
+	status = "okay";
+};
+
+&uart1 {

arch/arm/boot/dts/aspeed-bmc-facebook-yosemitev2.dts line 80

Quoted 10 lines
arch/arm/boot/dts/aspeed-bmc-facebook-yosemitev2.dts new file +10 -0
+	// Host Console
+	status = "okay";
+	pinctrl-names = "default";
+	pinctrl-0 = <&pinctrl_txd1_default
+		     &pinctrl_rxd1_default>;
+};
+
+&uart2 {
+	// SoL Host Console
+	status = "okay";

Also needs pinctrl configuration.

arch/arm/boot/dts/aspeed-bmc-facebook-yosemitev2.dts line 85

Quoted 5 lines
arch/arm/boot/dts/aspeed-bmc-facebook-yosemitev2.dts new file +5 -0
+};
+
+&uart3 {
+	// SoL BMC Console
+	status = "okay";

Again needs pinctrl.

Quoted 7 lines
arch/arm/boot/dts/aspeed-bmc-facebook-yosemitev2.dts new file +7 -0
+};
+
+&uart5 {
+	// BMC Console
+	status = "okay";
+};
+

arch/arm/boot/dts/aspeed-bmc-facebook-yosemitev2.dts line 102

Quoted 10 lines
arch/arm/boot/dts/aspeed-bmc-facebook-yosemitev2.dts new file +10 -0
+&mac0 {
+	status = "okay";
+
+	pinctrl-names = "default";
+	pinctrl-0 = <&pinctrl_rmii1_default>;
+	use-ncsi;
+};
+
+&adc {
+	status = "okay";

Strongly suggest adding the pinctrl properties here to ensure exclusive access for the ADC pins.

Otherwise it looks reasonable.

Andrew

#2 lore raw

?On 11/19/19, 9:56 PM, "Linux-aspeed on behalf of Andrew Jeffery" <linux-aspeed-bounces+vijaykhemka=fb.com at lists.ozlabs.org on behalf of [email protected]> wrote:

    On Mon, 18 Nov 2019, at 23:07, manikandan-e wrote:
    > The Yosemite V2 is a facebook multi-node server
    > platform that host four OCP server. The BMC
    > in the Yosemite V2 platorm based on AST2500 SoC.
    > 
    > This patch adds linux device tree entry related to
    > Yosemite V2 specific devices connected to BMC SoC.
    > 
    > Signed-off-by: manikandan-e <[email protected]>
    > ---
    >  .../boot/dts/aspeed-bmc-facebook-yosemitev2.dts    | 170 +++++++++++++++++++++
    >  1 file changed, 170 insertions(+)
    >  create mode 100644 arch/arm/boot/dts/aspeed-bmc-facebook-yosemitev2.dts
    > 
    > diff --git a/arch/arm/boot/dts/aspeed-bmc-facebook-yosemitev2.dts 
    > b/arch/arm/boot/dts/aspeed-bmc-facebook-yosemitev2.dts
    > new file mode 100644
    > index 0000000..46a285a
    > --- /dev/null
    > +++ b/arch/arm/boot/dts/aspeed-bmc-facebook-yosemitev2.dts
    > @@ -0,0 +1,170 @@
    > +// SPDX-License-Identifier: GPL-2.0+
    > +// Copyright (c) 2018 Facebook Inc.
    > +// Author:
    > +/dts-v1/;
    > +
    > +#include "aspeed-g5.dtsi"
    > +#include <dt-bindings/gpio/aspeed-gpio.h>
    > +
    > +/ {
    > +	model = "Facebook Yosemitev2 BMC";
    > +	compatible = "facebook,yosemitev2-bmc", "aspeed,ast2500";
    > +	aliases {
    > +		serial0 = &uart1;
    > +		serial4 = &uart5;
    > +	};
    > +	chosen {
    > +		stdout-path = &uart5;
    > +		bootargs = "console=ttyS4,115200 earlyprintk";
    > +	};
    > +
    > +	memory at 80000000 {
    > +		reg = <0x80000000 0x20000000>;
    > +	};
    > +
    > +	iio-hwmon {
    > +		// VOLATAGE SENSOR
    > +		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 12> , <&adc 13> , <&adc 14> , <&adc 15> ;
    > +	};
    > +};
    > +
    > +&fmc {
    > +	status = "okay";
    > +	flash at 0 {
    > +		status = "okay";
    > +		m25p,fast-read;
    > +#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";
    > +	};
    > +};
    > +
    > +&lpc_snoop {
    > +	status = "okay";
    > +	snoop-ports = <0x80>;
    > +};

No lpc in Yosemite so please remove.

arch/arm/boot/dts/aspeed-bmc-facebook-yosemitev2.dts line 62

Quoted 4 lines
arch/arm/boot/dts/aspeed-bmc-facebook-yosemitev2.dts new file +4 -0
+
+&lpc_ctrl {
+	// Enable lpc clock
+	status = "okay";

Same here remove.

    Something I'm intending to fix in the devicetrees using LPC is to hog
    the pins in the pinctrl node. You should consider doing the same here.
    
    > +};
    > +
    > +&vuart {
    > +	// VUART Host Console
    > +	status = "okay";
    > +};

No Vuart.

arch/arm/boot/dts/aspeed-bmc-facebook-yosemitev2.dts line 80

Quoted 12 lines
arch/arm/boot/dts/aspeed-bmc-facebook-yosemitev2.dts new file +12 -0
+
+&uart1 {
+	// Host Console
+	status = "okay";
+	pinctrl-names = "default";
+	pinctrl-0 = <&pinctrl_txd1_default
+		     &pinctrl_rxd1_default>;
+};
+
+&uart2 {
+	// SoL Host Console
+	status = "okay";

uart1-4 are all assigned for 4 multiple hosts so define accordingly.

    Also needs pinctrl configuration.
    
    > +};
    > +
    > +&uart3 {
    > +	// SoL BMC Console
    > +	status = "okay";
    
    Again needs pinctrl.
    
    > +};
    > +
    > +&uart5 {
    > +	// BMC Console
    > +	status = "okay";
    > +};
    > +
    > +&mac0 {
    > +	status = "okay";
    > +
    > +	pinctrl-names = "default";
    > +	pinctrl-0 = <&pinctrl_rmii1_default>;
    > +	use-ncsi;
    > +};
    > +
    > +&adc {
    > +	status = "okay";
    
    Strongly suggest adding the pinctrl properties here to ensure
    exclusive access for the ADC pins.
    
    Otherwise it looks reasonable.
    
    Andrew
author
#3 lore raw

Hi Andrew/Vijay,

Thanks for the review .

The following changes done in dts and tested in Facebook Yosemite V2 BMC platform,
1. LPC feature removed as not supported .
2. VUART feature removed as not supported.
3. Host UART feature removed as not in the current scope.
4. ADC pinctrl details added in dts.

Thanks
Mani.E

Quoted 145 lines · Vijay Khemka

?On 11/19/19, 9:56 PM, "Linux-aspeed on behalf of Andrew Jeffery" <linux-aspeed-bounces+vijaykhemka=fb.com at lists.ozlabs.org on behalf of [email protected]> wrote:

    On Mon, 18 Nov 2019, at 23:07, manikandan-e wrote:
    > The Yosemite V2 is a facebook multi-node server
    > platform that host four OCP server. The BMC
    > in the Yosemite V2 platorm based on AST2500 SoC.
    > 
    > This patch adds linux device tree entry related to
    > Yosemite V2 specific devices connected to BMC SoC.
    > 
    > Signed-off-by: manikandan-e <[email protected]>
    > ---
    >  .../boot/dts/aspeed-bmc-facebook-yosemitev2.dts    | 170 +++++++++++++++++++++
    >  1 file changed, 170 insertions(+)
    >  create mode 100644 arch/arm/boot/dts/aspeed-bmc-facebook-yosemitev2.dts
    > 
    > diff --git a/arch/arm/boot/dts/aspeed-bmc-facebook-yosemitev2.dts 
    > b/arch/arm/boot/dts/aspeed-bmc-facebook-yosemitev2.dts
    > new file mode 100644
    > index 0000000..46a285a
    > --- /dev/null
    > +++ b/arch/arm/boot/dts/aspeed-bmc-facebook-yosemitev2.dts
    > @@ -0,0 +1,170 @@
    > +// SPDX-License-Identifier: GPL-2.0+
    > +// Copyright (c) 2018 Facebook Inc.
    > +// Author:
    > +/dts-v1/;
    > +
    > +#include "aspeed-g5.dtsi"
    > +#include <dt-bindings/gpio/aspeed-gpio.h>
    > +
    > +/ {
    > +	model = "Facebook Yosemitev2 BMC";
    > +	compatible = "facebook,yosemitev2-bmc", "aspeed,ast2500";
    > +	aliases {
    > +		serial0 = &uart1;
    > +		serial4 = &uart5;
    > +	};
    > +	chosen {
    > +		stdout-path = &uart5;
    > +		bootargs = "console=ttyS4,115200 earlyprintk";
    > +	};
    > +
    > +	memory at 80000000 {
    > +		reg = <0x80000000 0x20000000>;
    > +	};
    > +
    > +	iio-hwmon {
    > +		// VOLATAGE SENSOR
    > +		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 12> , <&adc 13> , <&adc 14> , <&adc 15> ;
    > +	};
    > +};
    > +
    > +&fmc {
    > +	status = "okay";
    > +	flash at 0 {
    > +		status = "okay";
    > +		m25p,fast-read;
    > +#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";
    > +	};
    > +};
    > +
    > +&lpc_snoop {
    > +	status = "okay";
    > +	snoop-ports = <0x80>;
    > +};

No lpc in Yosemite so please remove.

Review

Quoted 4 lines
(quoted diff) +4 -0
+
+&lpc_ctrl {
+	// Enable lpc clock
+	status = "okay";

Same here remove.

    Something I'm intending to fix in the devicetrees using LPC is to hog
    the pins in the pinctrl node. You should consider doing the same here.
    
    > +};
    > +
    > +&vuart {
    > +	// VUART Host Console
    > +	status = "okay";
    > +};

No Vuart.

Review

Quoted 12 lines
(quoted diff) +12 -0
+
+&uart1 {
+	// Host Console
+	status = "okay";
+	pinctrl-names = "default";
+	pinctrl-0 = <&pinctrl_txd1_default
+		     &pinctrl_rxd1_default>;
+};
+
+&uart2 {
+	// SoL Host Console
+	status = "okay";

uart1-4 are all assigned for 4 multiple hosts so define accordingly.

    Also needs pinctrl configuration.
    
    > +};
    > +
    > +&uart3 {
    > +	// SoL BMC Console
    > +	status = "okay";
    
    Again needs pinctrl.
    
    > +};
    > +
    > +&uart5 {
    > +	// BMC Console
    > +	status = "okay";
    > +};
    > +
    > +&mac0 {
    > +	status = "okay";
    > +
    > +	pinctrl-names = "default";
    > +	pinctrl-0 = <&pinctrl_rmii1_default>;
    > +	use-ncsi;
    > +};
    > +
    > +&adc {
Quoted 10 lines
    > +	status = "okay";
    
    Strongly suggest adding the pinctrl properties here to ensure
    exclusive access for the ADC pins.
    
    Otherwise it looks reasonable.
    
    Andrew

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-ARM-dts-aspeed-Adding-Facebook-Yosemite-V2-BMC.patch
Type: text/x-diff
Size: 3692 bytes
Desc: not available
URL: <http://lists.ozlabs.org/pipermail/linux-aspeed/attachments/20191121/f190d653/attachment-0001.patch>

#4 lore raw
Quoted 11 lines · Manikandan

Hi Andrew/Vijay,

Thanks for the review .

The following changes done in dts and tested in Facebook Yosemite V2 BMC platform,
1. LPC feature removed as not supported .
2. VUART feature removed as not supported.
3. Host UART feature removed as not in the current scope.
4. ADC pinctrl details added in dts.

Can you please re-send the patch as a v2 and inline to the mail rather than as an attachment?

Cheers,

Andrew

author
#5 lore raw

From 9a17872b5faf2c00ab0b572bac0072e44a3d8b91 Mon Sep 17 00:00:00 2001
From: manikandan-e <[email protected]>
Date: Thu, 21 Nov 2019 11:57:07 +0530
Subject: [PATCH] ARM: dts: aspeed: Adding Facebook Yosemite V2 BMC

The Yosemite V2 is a facebook multi-node server
platform that host four OCP server. The BMC
in the Yosemite V2 platorm based on AST2500 SoC.

This patch adds linux device tree entry related to Yosemite V2 specific devices connected to BMC SoC.

Signed-off-by manikandan-e <[email protected]>
---
 .../boot/dts/aspeed-bmc-facebook-yosemitev2.dts    | 152 +++++++++++++++++++++
 1 file changed, 152 insertions(+)
 create mode 100644 arch/arm/boot/dts/aspeed-bmc-facebook-yosemitev2.dts
arch/arm/boot/dts/aspeed-bmc-facebook-yosemitev2.dts new file +152 -0
new file mode 100644
index 0000000..5f9a2e1
@@ -0,0 +1,152 @@
+// SPDX-License-Identifier: GPL-2.0+
+// Copyright (c) 2018 Facebook Inc.
+/dts-v1/;
+
+#include "aspeed-g5.dtsi"
+#include <dt-bindings/gpio/aspeed-gpio.h>
+
+/ {
+	model = "Facebook Yosemitev2 BMC";
+	compatible = "facebook,yosemitev2-bmc", "aspeed,ast2500";
+	aliases {
+		serial0 = &uart1;
+		serial4 = &uart5;
+	};
+	chosen {
+		stdout-path = &uart5;
+		bootargs = "console=ttyS4,115200 earlyprintk";
+	};
+
+	memory at 80000000 {
+		reg = <0x80000000 0x20000000>;
+	};
+
+	iio-hwmon {
+		// VOLATAGE SENSOR
+		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 12> , <&adc 13> , <&adc 14> , <&adc 15> ;
+	};
+};
+
+&fmc {
+	status = "okay";
+	flash at 0 {
+		status = "okay";
+		m25p,fast-read;
+#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 {
+	// BMC Console
+	status = "okay";
+};
+
+&mac0 {
+	status = "okay";
+	pinctrl-names = "default";
+	pinctrl-0 = <&pinctrl_rmii1_default>;
+	use-ncsi;
+};
+
+&adc {
+	status = "okay";
+	pinctrl-names = "default";
+	pinctrl-0 = <&pinctrl_adc0_default
+		&pinctrl_adc1_default
+		&pinctrl_adc2_default
+		&pinctrl_adc3_default
+		&pinctrl_adc4_default
+		&pinctrl_adc5_default
+		&pinctrl_adc6_default
+		&pinctrl_adc7_default
+		&pinctrl_adc8_default
+		&pinctrl_adc9_default
+		&pinctrl_adc10_default
+		&pinctrl_adc11_default
+		&pinctrl_adc12_default
+		&pinctrl_adc13_default
+		&pinctrl_adc14_default
+		&pinctrl_adc15_default>;
+};
+
+&i2c8 {
+	status = "okay";
+	//FRU EEPROM
+	eeprom at 51 {
+		compatible = "atmel,24c64";
+		reg = <0x51>;
+		pagesize = <32>;
+	};
+};
+
+&i2c9 {
+	status = "okay";
+	tmp421 at 4e {
+	//INLET TEMP
+		compatible = "ti,tmp421";
+		reg = <0x4e>;
+	};
+	//OUTLET TEMP
+	tmp421 at 4f {
+		compatible = "ti,tmp421";
+		reg = <0x4f>;
+	};
+};
+
+&i2c10 {
+	status = "okay";
+	//HSC
+	adm1278 at 40 {
+		compatible = "adi,adm1278";
+		reg = <0x40>;
+	};
+};
+
+&i2c11 {
+	status = "okay";
+	//MEZZ_TEMP_SENSOR
+	tmp421 at 1f {
+		compatible = "ti,tmp421";
+		reg = <0x1f>;
+	};
+};
+
+&i2c12 {
+	status = "okay";
+	//MEZZ_FRU
+	eeprom at 51 {
+		compatible = "atmel,24c64";
+		reg = <0x51>;
+		pagesize = <32>;
+	};
+};
+
+&pwm_tacho {
+	status = "okay";
+	//FSC
+	pinctrl-names = "default";
+	pinctrl-0 = <&pinctrl_pwm0_default &pinctrl_pwm1_default>;
+	fan at 0 {
+		reg = <0x00>;
+		aspeed,fan-tach-ch = /bits/ 8 <0x00>;
+	};
+	fan at 1 {
+		reg = <0x01>;
+		aspeed,fan-tach-ch = /bits/ 8 <0x02>;
+	};
+};
Signature · 1 line
2.7.4
Quoted 21 lines · Andrew Jeffery
Quoted 11 lines · Manikandan

Hi Andrew/Vijay,

Thanks for the review .

The following changes done in dts and tested in Facebook Yosemite V2 BMC platform,
1. LPC feature removed as not supported .
2. VUART feature removed as not supported.
3. Host UART feature removed as not in the current scope.
4. ADC pinctrl details added in dts.

Can you please re-send the patch as a v2 and inline to the mail rather than as an attachment?

Cheers,

Andrew

Acked-by
#6 lore raw
Quoted 28 lines · Manikandan

From 9a17872b5faf2c00ab0b572bac0072e44a3d8b91 Mon Sep 17 00:00:00 2001
From: manikandan-e <[email protected]>
Date: Thu, 21 Nov 2019 11:57:07 +0530
Subject: [PATCH] ARM: dts: aspeed: Adding Facebook Yosemite V2 BMC

The Yosemite V2 is a facebook multi-node server platform that host four OCP server. The BMC in the Yosemite V2 platorm based on AST2500 SoC.

This patch adds linux device tree entry related to Yosemite V2 specific devices connected to BMC SoC.

Signed-off-by manikandan-e <[email protected]>
---
 .../boot/dts/aspeed-bmc-facebook-yosemitev2.dts    | 152 +++++++++++++++++++++
 1 file changed, 152 insertions(+)
 create mode 100644 arch/arm/boot/dts/aspeed-bmc-facebook-yosemitev2.dts
arch/arm/boot/dts/aspeed-bmc-facebook-yosemitev2.dts new file +2 -0
new file mode 100644
index 0000000..5f9a2e1
@@ -0,0 +1,152 @@
+// SPDX-License-Identifier: GPL-2.0+
+// Copyright (c) 2018 Facebook Inc.

arch/arm/boot/dts/aspeed-bmc-facebook-yosemitev2.dts line 12

Quoted 10 lines
arch/arm/boot/dts/aspeed-bmc-facebook-yosemitev2.dts new file +10 -0
+/dts-v1/;
+
+#include "aspeed-g5.dtsi"
+#include <dt-bindings/gpio/aspeed-gpio.h>
+
+/ {
+	model = "Facebook Yosemitev2 BMC";
+	compatible = "facebook,yosemitev2-bmc", "aspeed,ast2500";
+	aliases {
+		serial0 = &uart1;

Is this necessary now that you're not enabling uart1?

But otherwise:

Acked-by Andrew Jeffery <[email protected]>

However, I'm guessing you copy/pasted the patch as we still have the reply text at the bottom. Copy/paste can lead to mangled whitespace or other errors that prevent the patch from being applied cleanly. I recommend getting set up and familiar with git send-email to give your patch the best chance of being applied without trouble. The git format-patch / git send-email pair also take care of iterating on your patch with e.g. the -v switch that injects the version number of the patch in the appropriate places.

Andrew

Quoted 167 lines
arch/arm/boot/dts/aspeed-bmc-facebook-yosemitev2.dts new file +140 -0
+		serial4 = &uart5;
+	};
+	chosen {
+		stdout-path = &uart5;
+		bootargs = "console=ttyS4,115200 earlyprintk";
+	};
+
+	memory@80000000 {
+		reg = <0x80000000 0x20000000>;
+	};
+
+	iio-hwmon {
+		// VOLATAGE SENSOR
+		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 12> , <&adc 13> , <&adc 14> , <&adc 15> ;
+	};
+};
+
+&fmc {
+	status = "okay";
+	flash@0 {
+		status = "okay";
+		m25p,fast-read;
+#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 {
+	// BMC Console
+	status = "okay";
+};
+
+&mac0 {
+	status = "okay";
+	pinctrl-names = "default";
+	pinctrl-0 = <&pinctrl_rmii1_default>;
+	use-ncsi;
+};
+
+&adc {
+	status = "okay";
+	pinctrl-names = "default";
+	pinctrl-0 = <&pinctrl_adc0_default
+		&pinctrl_adc1_default
+		&pinctrl_adc2_default
+		&pinctrl_adc3_default
+		&pinctrl_adc4_default
+		&pinctrl_adc5_default
+		&pinctrl_adc6_default
+		&pinctrl_adc7_default
+		&pinctrl_adc8_default
+		&pinctrl_adc9_default
+		&pinctrl_adc10_default
+		&pinctrl_adc11_default
+		&pinctrl_adc12_default
+		&pinctrl_adc13_default
+		&pinctrl_adc14_default
+		&pinctrl_adc15_default>;
+};
+
+&i2c8 {
+	status = "okay";
+	//FRU EEPROM
+	eeprom@51 {
+		compatible = "atmel,24c64";
+		reg = <0x51>;
+		pagesize = <32>;
+	};
+};
+
+&i2c9 {
+	status = "okay";
+	tmp421@4e {
+	//INLET TEMP
+		compatible = "ti,tmp421";
+		reg = <0x4e>;
+	};
+	//OUTLET TEMP
+	tmp421@4f {
+		compatible = "ti,tmp421";
+		reg = <0x4f>;
+	};
+};
+
+&i2c10 {
+	status = "okay";
+	//HSC
+	adm1278@40 {
+		compatible = "adi,adm1278";
+		reg = <0x40>;
+	};
+};
+
+&i2c11 {
+	status = "okay";
+	//MEZZ_TEMP_SENSOR
+	tmp421@1f {
+		compatible = "ti,tmp421";
+		reg = <0x1f>;
+	};
+};
+
+&i2c12 {
+	status = "okay";
+	//MEZZ_FRU
+	eeprom@51 {
+		compatible = "atmel,24c64";
+		reg = <0x51>;
+		pagesize = <32>;
+	};
+};
+
+&pwm_tacho {
+	status = "okay";
+	//FSC
+	pinctrl-names = "default";
+	pinctrl-0 = <&pinctrl_pwm0_default &pinctrl_pwm1_default>;
+	fan@0 {
+		reg = <0x00>;
+		aspeed,fan-tach-ch = /bits/ 8 <0x00>;
+	};
+	fan@1 {
+		reg = <0x01>;
+		aspeed,fan-tach-ch = /bits/ 8 <0x02>;
+	};
+};
Signature · 1 line
2.7.4
Quoted 21 lines · Andrew Jeffery
Quoted 11 lines · Manikandan

Hi Andrew/Vijay,

Thanks for the review .

The following changes done in dts and tested in Facebook Yosemite V2 BMC platform,
1. LPC feature removed as not supported .
2. VUART feature removed as not supported.
3. Host UART feature removed as not in the current scope.
4. ADC pinctrl details added in dts.

Can you please re-send the patch as a v2 and inline to the mail rather than as an attachment?

Cheers,

Andrew

author
#7 lore raw
Quoted 35 lines · Andrew Jeffery
Quoted 32 lines · Manikandan

From 9a17872b5faf2c00ab0b572bac0072e44a3d8b91 Mon Sep 17 00:00:00 2001
From: manikandan-e <[email protected]>
Date: Thu, 21 Nov 2019 11:57:07 +0530
Subject: [PATCH] ARM: dts: aspeed: Adding Facebook Yosemite V2 BMC

The Yosemite V2 is a facebook multi-node server platform that host four OCP server. The BMC in the Yosemite V2 platorm based on AST2500 SoC.

This patch adds linux device tree entry related to Yosemite V2 specific devices connected to BMC SoC.

Signed-off-by manikandan-e <[email protected]>
---
 .../boot/dts/aspeed-bmc-facebook-yosemitev2.dts    | 152 +++++++++++++++++++++
 1 file changed, 152 insertions(+)
 create mode 100644 arch/arm/boot/dts/aspeed-bmc-facebook-yosemitev2.dts
arch/arm/boot/dts/aspeed-bmc-facebook-yosemitev2.dts new file +6 -0
new file mode 100644
index 0000000..5f9a2e1
@@ -0,0 +1,152 @@
+// SPDX-License-Identifier: GPL-2.0+
+// Copyright (c) 2018 Facebook Inc.
+/dts-v1/;
+
+#include "aspeed-g5.dtsi"
+#include <dt-bindings/gpio/aspeed-gpio.h>
Quoted 8 lines

arch/arm/boot/dts/aspeed-bmc-facebook-yosemitev2.dts line 12

Quoted 6 lines
arch/arm/boot/dts/aspeed-bmc-facebook-yosemitev2.dts new file +6 -0
+
+/ {
+	model = "Facebook Yosemitev2 BMC";
+	compatible = "facebook,yosemitev2-bmc", "aspeed,ast2500";
+	aliases {
+		serial0 = &uart1;

Is this necessary now that you're not enabling uart1?

Removed in dts.

Quoted 14 lines

But otherwise:

Acked-by Andrew Jeffery <[email protected]>

However, I'm guessing you copy/pasted the patch as we still have the reply text at the bottom. Copy/paste can lead to mangled whitespace or other errors that prevent the patch from being applied cleanly. I recommend getting set up and familiar with git send-email to give your patch the best chance of being applied without trouble. The git format-patch / git send-email pair also take care of iterating on your patch with e.g. the -v switch that injects the version number of the patch in the appropriate places.

Quoted 2 lines

Andrew

Thanks for input. I have done patch version based on 'git format-patch' and send patch as v3 using mutt.

Quoted 167 lines
Quoted 167 lines
arch/arm/boot/dts/aspeed-bmc-facebook-yosemitev2.dts new file +140 -0
+		serial4 = &uart5;
+	};
+	chosen {
+		stdout-path = &uart5;
+		bootargs = "console=ttyS4,115200 earlyprintk";
+	};
+
+	memory at 80000000 {
+		reg = <0x80000000 0x20000000>;
+	};
+
+	iio-hwmon {
+		// VOLATAGE SENSOR
+		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 12> , <&adc 13> , <&adc 14> , <&adc 15> ;
+	};
+};
+
+&fmc {
+	status = "okay";
+	flash at 0 {
+		status = "okay";
+		m25p,fast-read;
+#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 {
+	// BMC Console
+	status = "okay";
+};
+
+&mac0 {
+	status = "okay";
+	pinctrl-names = "default";
+	pinctrl-0 = <&pinctrl_rmii1_default>;
+	use-ncsi;
+};
+
+&adc {
+	status = "okay";
+	pinctrl-names = "default";
+	pinctrl-0 = <&pinctrl_adc0_default
+		&pinctrl_adc1_default
+		&pinctrl_adc2_default
+		&pinctrl_adc3_default
+		&pinctrl_adc4_default
+		&pinctrl_adc5_default
+		&pinctrl_adc6_default
+		&pinctrl_adc7_default
+		&pinctrl_adc8_default
+		&pinctrl_adc9_default
+		&pinctrl_adc10_default
+		&pinctrl_adc11_default
+		&pinctrl_adc12_default
+		&pinctrl_adc13_default
+		&pinctrl_adc14_default
+		&pinctrl_adc15_default>;
+};
+
+&i2c8 {
+	status = "okay";
+	//FRU EEPROM
+	eeprom at 51 {
+		compatible = "atmel,24c64";
+		reg = <0x51>;
+		pagesize = <32>;
+	};
+};
+
+&i2c9 {
+	status = "okay";
+	tmp421 at 4e {
+	//INLET TEMP
+		compatible = "ti,tmp421";
+		reg = <0x4e>;
+	};
+	//OUTLET TEMP
+	tmp421 at 4f {
+		compatible = "ti,tmp421";
+		reg = <0x4f>;
+	};
+};
+
+&i2c10 {
+	status = "okay";
+	//HSC
+	adm1278 at 40 {
+		compatible = "adi,adm1278";
+		reg = <0x40>;
+	};
+};
+
+&i2c11 {
+	status = "okay";
+	//MEZZ_TEMP_SENSOR
+	tmp421 at 1f {
+		compatible = "ti,tmp421";
+		reg = <0x1f>;
+	};
+};
+
+&i2c12 {
+	status = "okay";
+	//MEZZ_FRU
+	eeprom at 51 {
+		compatible = "atmel,24c64";
+		reg = <0x51>;
+		pagesize = <32>;
+	};
+};
+
+&pwm_tacho {
+	status = "okay";
+	//FSC
+	pinctrl-names = "default";
+	pinctrl-0 = <&pinctrl_pwm0_default &pinctrl_pwm1_default>;
+	fan at 0 {
+		reg = <0x00>;
+		aspeed,fan-tach-ch = /bits/ 8 <0x00>;
+	};
+	fan at 1 {
+		reg = <0x01>;
+		aspeed,fan-tach-ch = /bits/ 8 <0x02>;
+	};
+};
Signature · 1 line
2.7.4
Quoted 21 lines · Andrew Jeffery
Quoted 11 lines · Manikandan

Hi Andrew/Vijay,

Thanks for the review .

The following changes done in dts and tested in Facebook Yosemite V2 BMC platform,
1. LPC feature removed as not supported .
2. VUART feature removed as not supported.
3. Host UART feature removed
4. ADC pinctrl details added in dts.

Can you please re-send the patch as a v2 and inline to the mail rather than as an attachment?

Cheers,

Andrew