.../boot/dts/aspeed-bmc-ampere-mtmitchell.dts | 37 ++++++++++++++++++- 1 file changed, 36 insertions(+), 1 deletion(-)
Support the mac3 (RGMII4) as an NC-SI stack instead of an MDIO PHY.
The OCP slot #0 and OCP slot #1 use a common the BMC_NCSI signal,
so we use only one of them at the same time. The OCP slot #0 will
be enabled by PCA9539's setting by default.
Also, enable the OCP Auxiliary Power during booting.
Signed-off-by: Chanh Nguyen <chanh@os.amperecomputing.com>
---
Changes in v2:
- Change PCA9539APW node name. [Krzysztof]
---
.../boot/dts/aspeed-bmc-ampere-mtmitchell.dts | 37 ++++++++++++++++++-
1 file changed, 36 insertions(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/aspeed-bmc-ampere-mtmitchell.dts b/arch/arm/boot/dts/aspeed-bmc-ampere-mtmitchell.dts
index 4b91600eaf62..1e0e88465254 100644
--- a/arch/arm/boot/dts/aspeed-bmc-ampere-mtmitchell.dts
+++ b/arch/arm/boot/dts/aspeed-bmc-ampere-mtmitchell.dts
@@ -251,6 +251,14 @@
pinctrl-0 = <&pinctrl_rgmii1_default>;
};
+&mac3 {
+ status = "okay";
+ pinctrl-names = "default";
+ pinctrl-0 = <&pinctrl_rmii4_default>;
+ clock-names = "MACCLK", "RCLK";
+ use-ncsi;
+};
+
&fmc {
status = "okay";
flash@0 {
@@ -439,6 +447,26 @@
status = "okay";
};
+&i2c8 {
+ status = "okay";
+
+ gpio@77 {
+ compatible = "nxp,pca9539";
+ reg = <0x77>;
+ gpio-controller;
+ #address-cells = <1>;
+ #size-cells = <0>;
+ #gpio-cells = <2>;
+
+ bmc-ocp0-en-hog {
+ gpio-hog;
+ gpios = <7 GPIO_ACTIVE_LOW>;
+ output-high;
+ line-name = "bmc-ocp0-en-n";
+ };
+ };
+};
+
&i2c9 {
status = "okay";
};
@@ -530,13 +558,20 @@
/*V0-V7*/ "s0-hightemp-n","s0-fault-alert","s0-sys-auth-failure-n",
"host0-reboot-ack-n","host0-ready","host0-shd-req-n",
"host0-shd-ack-n","s0-overtemp-n",
- /*W0-W7*/ "ocp-aux-pwren","ocp-main-pwren","ocp-pgood","",
+ /*W0-W7*/ "","ocp-main-pwren","ocp-pgood","",
"bmc-ok","bmc-ready","spi0-program-sel","spi0-backup-sel",
/*X0-X7*/ "i2c-backup-sel","s1-fault-alert","s1-fw-boot-ok",
"s1-hightemp-n","s0-spi-auth-fail-n","s1-sys-auth-failure-n",
"s1-overtemp-n","s1-spi-auth-fail-n",
/*Y0-Y7*/ "","","","","","","","host0-special-boot",
/*Z0-Z7*/ "reset-button","ps0-pgood","ps1-pgood","","","","","";
+
+ ocp-aux-pwren-hog {
+ gpio-hog;
+ gpios = <ASPEED_GPIO(W, 0) GPIO_ACTIVE_HIGH>;
+ output-high;
+ line-name = "ocp-aux-pwren";
+ };
};
&gpio1 {
--
2.17.1
Dear Chanh, Thank you for the patch. Am 28.02.23 um 11:28 schrieb Chanh Nguyen: > Support the mac3 (RGMII4) as an NC-SI stack instead of an MDIO PHY. > > The OCP slot #0 and OCP slot #1 use a common the BMC_NCSI signal, *the* seems a leftover. > so we use only one of them at the same time. The OCP slot #0 will > be enabled by PCA9539's setting by default. > > Also, enable the OCP Auxiliary Power during booting. Is there a reason not to make this a separate commit? Kind regards, Paul > Signed-off-by: Chanh Nguyen <chanh@os.amperecomputing.com> > --- > Changes in v2: > - Change PCA9539APW node name. [Krzysztof] > --- > .../boot/dts/aspeed-bmc-ampere-mtmitchell.dts | 37 ++++++++++++++++++- > 1 file changed, 36 insertions(+), 1 deletion(-) > > diff --git a/arch/arm/boot/dts/aspeed-bmc-ampere-mtmitchell.dts b/arch/arm/boot/dts/aspeed-bmc-ampere-mtmitchell.dts > index 4b91600eaf62..1e0e88465254 100644 > --- a/arch/arm/boot/dts/aspeed-bmc-ampere-mtmitchell.dts > +++ b/arch/arm/boot/dts/aspeed-bmc-ampere-mtmitchell.dts > @@ -251,6 +251,14 @@ > pinctrl-0 = <&pinctrl_rgmii1_default>; > }; > > +&mac3 { > + status = "okay"; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_rmii4_default>; > + clock-names = "MACCLK", "RCLK"; > + use-ncsi; > +}; > + > &fmc { > status = "okay"; > flash@0 { > @@ -439,6 +447,26 @@ > status = "okay"; > }; > > +&i2c8 { > + status = "okay"; > + > + gpio@77 { > + compatible = "nxp,pca9539"; > + reg = <0x77>; > + gpio-controller; > + #address-cells = <1>; > + #size-cells = <0>; > + #gpio-cells = <2>; > + > + bmc-ocp0-en-hog { > + gpio-hog; > + gpios = <7 GPIO_ACTIVE_LOW>; > + output-high; > + line-name = "bmc-ocp0-en-n"; > + }; > + }; > +}; > + > &i2c9 { > status = "okay"; > }; > @@ -530,13 +558,20 @@ > /*V0-V7*/ "s0-hightemp-n","s0-fault-alert","s0-sys-auth-failure-n", > "host0-reboot-ack-n","host0-ready","host0-shd-req-n", > "host0-shd-ack-n","s0-overtemp-n", > - /*W0-W7*/ "ocp-aux-pwren","ocp-main-pwren","ocp-pgood","", > + /*W0-W7*/ "","ocp-main-pwren","ocp-pgood","", > "bmc-ok","bmc-ready","spi0-program-sel","spi0-backup-sel", > /*X0-X7*/ "i2c-backup-sel","s1-fault-alert","s1-fw-boot-ok", > "s1-hightemp-n","s0-spi-auth-fail-n","s1-sys-auth-failure-n", > "s1-overtemp-n","s1-spi-auth-fail-n", > /*Y0-Y7*/ "","","","","","","","host0-special-boot", > /*Z0-Z7*/ "reset-button","ps0-pgood","ps1-pgood","","","","",""; > + > + ocp-aux-pwren-hog { > + gpio-hog; > + gpios = <ASPEED_GPIO(W, 0) GPIO_ACTIVE_HIGH>; > + output-high; > + line-name = "ocp-aux-pwren"; > + }; > }; > > &gpio1 {
On 01/03/2023 14:51, Paul Menzel wrote: > Dear Chanh, > > > Thank you for the patch. > > Am 28.02.23 um 11:28 schrieb Chanh Nguyen: >> Support the mac3 (RGMII4) as an NC-SI stack instead of an MDIO PHY. >> >> The OCP slot #0 and OCP slot #1 use a common the BMC_NCSI signal, > > *the* seems a leftover. Thank Paul, I'll remove it. > >> so we use only one of them at the same time. The OCP slot #0 will >> be enabled by PCA9539's setting by default. >> >> Also, enable the OCP Auxiliary Power during booting. > > Is there a reason not to make this a separate commit? > > > Kind regards, > > Paul > I wouldn't like to separate it. This is "Enable NCSI" commit, so I would like to support all configs, that make NCSI feature works. Thanks, Chanh > >> Signed-off-by: Chanh Nguyen <chanh@os.amperecomputing.com> >> --- >> Changes in v2: >> - Change PCA9539APW node name. [Krzysztof] >> --- >> .../boot/dts/aspeed-bmc-ampere-mtmitchell.dts | 37 ++++++++++++++++++- >> 1 file changed, 36 insertions(+), 1 deletion(-) >> >> diff --git a/arch/arm/boot/dts/aspeed-bmc-ampere-mtmitchell.dts >> b/arch/arm/boot/dts/aspeed-bmc-ampere-mtmitchell.dts >> index 4b91600eaf62..1e0e88465254 100644 >> --- a/arch/arm/boot/dts/aspeed-bmc-ampere-mtmitchell.dts >> +++ b/arch/arm/boot/dts/aspeed-bmc-ampere-mtmitchell.dts >> @@ -251,6 +251,14 @@ >> pinctrl-0 = <&pinctrl_rgmii1_default>; >> }; >> +&mac3 { >> + status = "okay"; >> + pinctrl-names = "default"; >> + pinctrl-0 = <&pinctrl_rmii4_default>; >> + clock-names = "MACCLK", "RCLK"; >> + use-ncsi; >> +}; >> + >> &fmc { >> status = "okay"; >> flash@0 { >> @@ -439,6 +447,26 @@ >> status = "okay"; >> }; >> +&i2c8 { >> + status = "okay"; >> + >> + gpio@77 { >> + compatible = "nxp,pca9539"; >> + reg = <0x77>; >> + gpio-controller; >> + #address-cells = <1>; >> + #size-cells = <0>; >> + #gpio-cells = <2>; >> + >> + bmc-ocp0-en-hog { >> + gpio-hog; >> + gpios = <7 GPIO_ACTIVE_LOW>; >> + output-high; >> + line-name = "bmc-ocp0-en-n"; >> + }; >> + }; >> +}; >> + >> &i2c9 { >> status = "okay"; >> }; >> @@ -530,13 +558,20 @@ >> /*V0-V7*/ >> "s0-hightemp-n","s0-fault-alert","s0-sys-auth-failure-n", >> "host0-reboot-ack-n","host0-ready","host0-shd-req-n", >> "host0-shd-ack-n","s0-overtemp-n", >> - /*W0-W7*/ "ocp-aux-pwren","ocp-main-pwren","ocp-pgood","", >> + /*W0-W7*/ "","ocp-main-pwren","ocp-pgood","", >> "bmc-ok","bmc-ready","spi0-program-sel","spi0-backup-sel", >> /*X0-X7*/ "i2c-backup-sel","s1-fault-alert","s1-fw-boot-ok", >> >> "s1-hightemp-n","s0-spi-auth-fail-n","s1-sys-auth-failure-n", >> "s1-overtemp-n","s1-spi-auth-fail-n", >> /*Y0-Y7*/ "","","","","","","","host0-special-boot", >> /*Z0-Z7*/ "reset-button","ps0-pgood","ps1-pgood","","","","",""; >> + >> + ocp-aux-pwren-hog { >> + gpio-hog; >> + gpios = <ASPEED_GPIO(W, 0) GPIO_ACTIVE_HIGH>; >> + output-high; >> + line-name = "ocp-aux-pwren"; >> + }; >> }; >> &gpio1 {
On Sun, 5 Mar 2023 at 15:59, Chanh Nguyen <chanh@amperemail.onmicrosoft.com> wrote: > > > > On 01/03/2023 14:51, Paul Menzel wrote: > > Dear Chanh, > > > > > > Thank you for the patch. > > > > Am 28.02.23 um 11:28 schrieb Chanh Nguyen: > >> Support the mac3 (RGMII4) as an NC-SI stack instead of an MDIO PHY. > >> > >> The OCP slot #0 and OCP slot #1 use a common the BMC_NCSI signal, > > > > *the* seems a leftover. > > Thank Paul, > I'll remove it. > > > > >> so we use only one of them at the same time. The OCP slot #0 will > >> be enabled by PCA9539's setting by default. > >> > >> Also, enable the OCP Auxiliary Power during booting. > > > > Is there a reason not to make this a separate commit? > > > > > > Kind regards, > > > > Paul > > > > I wouldn't like to separate it. > > This is "Enable NCSI" commit, so I would like to support all configs, > that make NCSI feature works. That sounds reasonable. > > Thanks, > Chanh > > > >> Signed-off-by: Chanh Nguyen <chanh@os.amperecomputing.com> > >> --- > >> Changes in v2: > >> - Change PCA9539APW node name. [Krzysztof] > >> --- > >> .../boot/dts/aspeed-bmc-ampere-mtmitchell.dts | 37 ++++++++++++++++++- > >> 1 file changed, 36 insertions(+), 1 deletion(-) > >> > >> diff --git a/arch/arm/boot/dts/aspeed-bmc-ampere-mtmitchell.dts > >> b/arch/arm/boot/dts/aspeed-bmc-ampere-mtmitchell.dts > >> index 4b91600eaf62..1e0e88465254 100644 > >> --- a/arch/arm/boot/dts/aspeed-bmc-ampere-mtmitchell.dts > >> +++ b/arch/arm/boot/dts/aspeed-bmc-ampere-mtmitchell.dts > >> @@ -251,6 +251,14 @@ > >> pinctrl-0 = <&pinctrl_rgmii1_default>; > >> }; > >> +&mac3 { > >> + status = "okay"; > >> + pinctrl-names = "default"; > >> + pinctrl-0 = <&pinctrl_rmii4_default>; > >> + clock-names = "MACCLK", "RCLK"; > >> + use-ncsi; > >> +}; > >> + > >> &fmc { > >> status = "okay"; > >> flash@0 { > >> @@ -439,6 +447,26 @@ > >> status = "okay"; > >> }; > >> +&i2c8 { > >> + status = "okay"; > >> + > >> + gpio@77 { > >> + compatible = "nxp,pca9539"; > >> + reg = <0x77>; > >> + gpio-controller; > >> + #address-cells = <1>; > >> + #size-cells = <0>; > >> + #gpio-cells = <2>; > >> + > >> + bmc-ocp0-en-hog { > >> + gpio-hog; > >> + gpios = <7 GPIO_ACTIVE_LOW>; > >> + output-high; > >> + line-name = "bmc-ocp0-en-n"; > >> + }; > >> + }; > >> +}; > >> + > >> &i2c9 { > >> status = "okay"; > >> }; > >> @@ -530,13 +558,20 @@ > >> /*V0-V7*/ > >> "s0-hightemp-n","s0-fault-alert","s0-sys-auth-failure-n", > >> "host0-reboot-ack-n","host0-ready","host0-shd-req-n", > >> "host0-shd-ack-n","s0-overtemp-n", > >> - /*W0-W7*/ "ocp-aux-pwren","ocp-main-pwren","ocp-pgood","", > >> + /*W0-W7*/ "","ocp-main-pwren","ocp-pgood","", > >> "bmc-ok","bmc-ready","spi0-program-sel","spi0-backup-sel", > >> /*X0-X7*/ "i2c-backup-sel","s1-fault-alert","s1-fw-boot-ok", > >> > >> "s1-hightemp-n","s0-spi-auth-fail-n","s1-sys-auth-failure-n", > >> "s1-overtemp-n","s1-spi-auth-fail-n", > >> /*Y0-Y7*/ "","","","","","","","host0-special-boot", > >> /*Z0-Z7*/ "reset-button","ps0-pgood","ps1-pgood","","","","",""; > >> + > >> + ocp-aux-pwren-hog { > >> + gpio-hog; > >> + gpios = <ASPEED_GPIO(W, 0) GPIO_ACTIVE_HIGH>; > >> + output-high; > >> + line-name = "ocp-aux-pwren"; > >> + }; > >> }; > >> &gpio1 {
On Tue, 28 Feb 2023 at 10:28, Chanh Nguyen <chanh@os.amperecomputing.com> wrote: > > Support the mac3 (RGMII4) as an NC-SI stack instead of an MDIO PHY. > > The OCP slot #0 and OCP slot #1 use a common the BMC_NCSI signal, > so we use only one of them at the same time. The OCP slot #0 will > be enabled by PCA9539's setting by default. > > Also, enable the OCP Auxiliary Power during booting. > > Signed-off-by: Chanh Nguyen <chanh@os.amperecomputing.com> LGTM. I'll apply it once we have a -rc1. Reviewed-by: Joel Stanley <joel@jms.id.au> > --- > Changes in v2: > - Change PCA9539APW node name. [Krzysztof] > --- > .../boot/dts/aspeed-bmc-ampere-mtmitchell.dts | 37 ++++++++++++++++++- > 1 file changed, 36 insertions(+), 1 deletion(-) > > diff --git a/arch/arm/boot/dts/aspeed-bmc-ampere-mtmitchell.dts b/arch/arm/boot/dts/aspeed-bmc-ampere-mtmitchell.dts > index 4b91600eaf62..1e0e88465254 100644 > --- a/arch/arm/boot/dts/aspeed-bmc-ampere-mtmitchell.dts > +++ b/arch/arm/boot/dts/aspeed-bmc-ampere-mtmitchell.dts > @@ -251,6 +251,14 @@ > pinctrl-0 = <&pinctrl_rgmii1_default>; > }; > > +&mac3 { > + status = "okay"; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_rmii4_default>; > + clock-names = "MACCLK", "RCLK"; > + use-ncsi; > +}; > + > &fmc { > status = "okay"; > flash@0 { > @@ -439,6 +447,26 @@ > status = "okay"; > }; > > +&i2c8 { > + status = "okay"; > + > + gpio@77 { > + compatible = "nxp,pca9539"; > + reg = <0x77>; > + gpio-controller; > + #address-cells = <1>; > + #size-cells = <0>; > + #gpio-cells = <2>; > + > + bmc-ocp0-en-hog { > + gpio-hog; > + gpios = <7 GPIO_ACTIVE_LOW>; > + output-high; > + line-name = "bmc-ocp0-en-n"; > + }; > + }; > +}; > + > &i2c9 { > status = "okay"; > }; > @@ -530,13 +558,20 @@ > /*V0-V7*/ "s0-hightemp-n","s0-fault-alert","s0-sys-auth-failure-n", > "host0-reboot-ack-n","host0-ready","host0-shd-req-n", > "host0-shd-ack-n","s0-overtemp-n", > - /*W0-W7*/ "ocp-aux-pwren","ocp-main-pwren","ocp-pgood","", > + /*W0-W7*/ "","ocp-main-pwren","ocp-pgood","", > "bmc-ok","bmc-ready","spi0-program-sel","spi0-backup-sel", > /*X0-X7*/ "i2c-backup-sel","s1-fault-alert","s1-fw-boot-ok", > "s1-hightemp-n","s0-spi-auth-fail-n","s1-sys-auth-failure-n", > "s1-overtemp-n","s1-spi-auth-fail-n", > /*Y0-Y7*/ "","","","","","","","host0-special-boot", > /*Z0-Z7*/ "reset-button","ps0-pgood","ps1-pgood","","","","",""; > + > + ocp-aux-pwren-hog { > + gpio-hog; > + gpios = <ASPEED_GPIO(W, 0) GPIO_ACTIVE_HIGH>; > + output-high; > + line-name = "ocp-aux-pwren"; > + }; > }; > > &gpio1 { > -- > 2.17.1 >
On 01/03/2023 07:17, Joel Stanley wrote: > On Tue, 28 Feb 2023 at 10:28, Chanh Nguyen <chanh@os.amperecomputing.com> wrote: >> >> Support the mac3 (RGMII4) as an NC-SI stack instead of an MDIO PHY. >> >> The OCP slot #0 and OCP slot #1 use a common the BMC_NCSI signal, >> so we use only one of them at the same time. The OCP slot #0 will >> be enabled by PCA9539's setting by default. >> >> Also, enable the OCP Auxiliary Power during booting. >> >> Signed-off-by: Chanh Nguyen <chanh@os.amperecomputing.com> > > LGTM. I'll apply it once we have a -rc1. > > Reviewed-by: Joel Stanley <joel@jms.id.au> > Thank Joel very much! >> --- >> Changes in v2: >> - Change PCA9539APW node name. [Krzysztof] >> --- >> .../boot/dts/aspeed-bmc-ampere-mtmitchell.dts | 37 ++++++++++++++++++- >> 1 file changed, 36 insertions(+), 1 deletion(-) >> >> diff --git a/arch/arm/boot/dts/aspeed-bmc-ampere-mtmitchell.dts b/arch/arm/boot/dts/aspeed-bmc-ampere-mtmitchell.dts >> index 4b91600eaf62..1e0e88465254 100644 >> --- a/arch/arm/boot/dts/aspeed-bmc-ampere-mtmitchell.dts >> +++ b/arch/arm/boot/dts/aspeed-bmc-ampere-mtmitchell.dts >> @@ -251,6 +251,14 @@ >> pinctrl-0 = <&pinctrl_rgmii1_default>; >> }; >> >> +&mac3 { >> + status = "okay"; >> + pinctrl-names = "default"; >> + pinctrl-0 = <&pinctrl_rmii4_default>; >> + clock-names = "MACCLK", "RCLK"; >> + use-ncsi; >> +}; >> + >> &fmc { >> status = "okay"; >> flash@0 { >> @@ -439,6 +447,26 @@ >> status = "okay"; >> }; >> >> +&i2c8 { >> + status = "okay"; >> + >> + gpio@77 { >> + compatible = "nxp,pca9539"; >> + reg = <0x77>; >> + gpio-controller; >> + #address-cells = <1>; >> + #size-cells = <0>; >> + #gpio-cells = <2>; >> + >> + bmc-ocp0-en-hog { >> + gpio-hog; >> + gpios = <7 GPIO_ACTIVE_LOW>; >> + output-high; >> + line-name = "bmc-ocp0-en-n"; >> + }; >> + }; >> +}; >> + >> &i2c9 { >> status = "okay"; >> }; >> @@ -530,13 +558,20 @@ >> /*V0-V7*/ "s0-hightemp-n","s0-fault-alert","s0-sys-auth-failure-n", >> "host0-reboot-ack-n","host0-ready","host0-shd-req-n", >> "host0-shd-ack-n","s0-overtemp-n", >> - /*W0-W7*/ "ocp-aux-pwren","ocp-main-pwren","ocp-pgood","", >> + /*W0-W7*/ "","ocp-main-pwren","ocp-pgood","", >> "bmc-ok","bmc-ready","spi0-program-sel","spi0-backup-sel", >> /*X0-X7*/ "i2c-backup-sel","s1-fault-alert","s1-fw-boot-ok", >> "s1-hightemp-n","s0-spi-auth-fail-n","s1-sys-auth-failure-n", >> "s1-overtemp-n","s1-spi-auth-fail-n", >> /*Y0-Y7*/ "","","","","","","","host0-special-boot", >> /*Z0-Z7*/ "reset-button","ps0-pgood","ps1-pgood","","","","",""; >> + >> + ocp-aux-pwren-hog { >> + gpio-hog; >> + gpios = <ASPEED_GPIO(W, 0) GPIO_ACTIVE_HIGH>; >> + output-high; >> + line-name = "ocp-aux-pwren"; >> + }; >> }; >> >> &gpio1 { >> -- >> 2.17.1 >>
© 2016 - 2025 Red Hat, Inc.