| Message ID | 20210819055227.140980-1-liuxiwei@inspur.com |
|---|---|
| State | New |
| Headers | show |
| Series | ARM: dts: fp5280g2: Enable KCS 3 for MCTP binding | expand |
Hey Joel: do you have any comments at this patch? Thx -- George Liu On Thu, Aug 19, 2021 at 1:52 PM George Liu <liuxiwei1013@gmail.com> wrote: > > Signed-off-by: George Liu <liuxiwei@inspur.com> > --- > arch/arm/boot/dts/aspeed-bmc-inspur-fp5280g2.dts | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/arch/arm/boot/dts/aspeed-bmc-inspur-fp5280g2.dts b/arch/arm/boot/dts/aspeed-bmc-inspur-fp5280g2.dts > index 1752f3250e44..d0c3acbf6c8c 100644 > --- a/arch/arm/boot/dts/aspeed-bmc-inspur-fp5280g2.dts > +++ b/arch/arm/boot/dts/aspeed-bmc-inspur-fp5280g2.dts > @@ -3,6 +3,7 @@ > #include "aspeed-g5.dtsi" > #include <dt-bindings/gpio/aspeed-gpio.h> > #include <dt-bindings/leds/leds-pca955x.h> > +#include <dt-bindings/interrupt-controller/irq.h> > > / { > model = "FP5280G2 BMC"; > @@ -902,4 +903,10 @@ fan@7 { > > }; > > +&kcs3 { > + status = "okay"; > + aspeed,lpc-io-reg = <0xca2>; > + aspeed,lpc-interrupts = <11 IRQ_TYPE_LEVEL_LOW>; > +}; > + > #include "ibm-power9-dual.dtsi" > -- > 2.30.2 >
On Tue, 14 Sept 2021 at 08:16, George Liu <liuxiwei1013@gmail.com> wrote: > > Hey Joel: > do you have any comments at this patch? > > Thx -- George Liu > > On Thu, Aug 19, 2021 at 1:52 PM George Liu <liuxiwei1013@gmail.com> wrote: > > > > Signed-off-by: George Liu <liuxiwei@inspur.com> It looks okay to me. Reviewed-by: Joel Stanley <joel@jms.id.au> I have applied it for 5.16. > > --- > > arch/arm/boot/dts/aspeed-bmc-inspur-fp5280g2.dts | 7 +++++++ > > 1 file changed, 7 insertions(+) > > > > diff --git a/arch/arm/boot/dts/aspeed-bmc-inspur-fp5280g2.dts b/arch/arm/boot/dts/aspeed-bmc-inspur-fp5280g2.dts > > index 1752f3250e44..d0c3acbf6c8c 100644 > > --- a/arch/arm/boot/dts/aspeed-bmc-inspur-fp5280g2.dts > > +++ b/arch/arm/boot/dts/aspeed-bmc-inspur-fp5280g2.dts > > @@ -3,6 +3,7 @@ > > #include "aspeed-g5.dtsi" > > #include <dt-bindings/gpio/aspeed-gpio.h> > > #include <dt-bindings/leds/leds-pca955x.h> > > +#include <dt-bindings/interrupt-controller/irq.h> > > > > / { > > model = "FP5280G2 BMC"; > > @@ -902,4 +903,10 @@ fan@7 { > > > > }; > > > > +&kcs3 { > > + status = "okay"; > > + aspeed,lpc-io-reg = <0xca2>; > > + aspeed,lpc-interrupts = <11 IRQ_TYPE_LEVEL_LOW>; > > +}; > > + > > #include "ibm-power9-dual.dtsi" > > -- > > 2.30.2 > >
Thanks! On Tue, Sep 14, 2021 at 4:45 PM Joel Stanley <joel@jms.id.au> wrote: > > On Tue, 14 Sept 2021 at 08:16, George Liu <liuxiwei1013@gmail.com> wrote: > > > > Hey Joel: > > do you have any comments at this patch? > > > > Thx -- George Liu > > > > On Thu, Aug 19, 2021 at 1:52 PM George Liu <liuxiwei1013@gmail.com> wrote: > > > > > > Signed-off-by: George Liu <liuxiwei@inspur.com> > > It looks okay to me. > > Reviewed-by: Joel Stanley <joel@jms.id.au> > > I have applied it for 5.16. > > > > --- > > > arch/arm/boot/dts/aspeed-bmc-inspur-fp5280g2.dts | 7 +++++++ > > > 1 file changed, 7 insertions(+) > > > > > > diff --git a/arch/arm/boot/dts/aspeed-bmc-inspur-fp5280g2.dts b/arch/arm/boot/dts/aspeed-bmc-inspur-fp5280g2.dts > > > index 1752f3250e44..d0c3acbf6c8c 100644 > > > --- a/arch/arm/boot/dts/aspeed-bmc-inspur-fp5280g2.dts > > > +++ b/arch/arm/boot/dts/aspeed-bmc-inspur-fp5280g2.dts > > > @@ -3,6 +3,7 @@ > > > #include "aspeed-g5.dtsi" > > > #include <dt-bindings/gpio/aspeed-gpio.h> > > > #include <dt-bindings/leds/leds-pca955x.h> > > > +#include <dt-bindings/interrupt-controller/irq.h> > > > > > > / { > > > model = "FP5280G2 BMC"; > > > @@ -902,4 +903,10 @@ fan@7 { > > > > > > }; > > > > > > +&kcs3 { > > > + status = "okay"; > > > + aspeed,lpc-io-reg = <0xca2>; > > > + aspeed,lpc-interrupts = <11 IRQ_TYPE_LEVEL_LOW>; > > > +}; > > > + > > > #include "ibm-power9-dual.dtsi" > > > -- > > > 2.30.2 > > >
diff --git a/arch/arm/boot/dts/aspeed-bmc-inspur-fp5280g2.dts b/arch/arm/boot/dts/aspeed-bmc-inspur-fp5280g2.dts index 1752f3250e44..d0c3acbf6c8c 100644 --- a/arch/arm/boot/dts/aspeed-bmc-inspur-fp5280g2.dts +++ b/arch/arm/boot/dts/aspeed-bmc-inspur-fp5280g2.dts @@ -3,6 +3,7 @@ #include "aspeed-g5.dtsi" #include <dt-bindings/gpio/aspeed-gpio.h> #include <dt-bindings/leds/leds-pca955x.h> +#include <dt-bindings/interrupt-controller/irq.h> / { model = "FP5280G2 BMC"; @@ -902,4 +903,10 @@ fan@7 { }; +&kcs3 { + status = "okay"; + aspeed,lpc-io-reg = <0xca2>; + aspeed,lpc-interrupts = <11 IRQ_TYPE_LEVEL_LOW>; +}; + #include "ibm-power9-dual.dtsi"
Signed-off-by: George Liu <liuxiwei@inspur.com> --- arch/arm/boot/dts/aspeed-bmc-inspur-fp5280g2.dts | 7 +++++++ 1 file changed, 7 insertions(+)