Hi Iwona, I have done r-b, Just could you do a small modification On Thu, 27 Jul 2023 at 23:23, Iwona Winiarska <iwona.winiarska@xxxxxxxxx> wrote: > > Add PECI controller node with all required information. > > Signed-off-by: Iwona Winiarska <iwona.winiarska@xxxxxxxxx> > --- > arch/arm/boot/dts/nuvoton/nuvoton-common-npcm7xx.dtsi | 9 +++++++++ > 1 file changed, 9 insertions(+) > > diff --git a/arch/arm/boot/dts/nuvoton/nuvoton-common-npcm7xx.dtsi b/arch/arm/boot/dts/nuvoton/nuvoton-common-npcm7xx.dtsi > index c7b5ef15b716..cccc33441050 100644 > --- a/arch/arm/boot/dts/nuvoton/nuvoton-common-npcm7xx.dtsi > +++ b/arch/arm/boot/dts/nuvoton/nuvoton-common-npcm7xx.dtsi > @@ -220,6 +220,15 @@ kcs3: kcs3@0 { > }; > }; > > + peci0: peci-controller@f0100000 { Please modify the peci0 to peci we have only one PECI controller. > + compatible = "nuvoton,npcm750-peci"; > + reg = <0xf0100000 0x200>; > + interrupts = <GIC_SPI 6 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&clk NPCM7XX_CLK_APB3>; > + cmd-timeout-ms = <1000>; > + status = "disabled"; > + }; > + > spi0: spi@200000 { > compatible = "nuvoton,npcm750-pspi"; > reg = <0x200000 0x1000>; > -- > 2.40.1 > Reviewed-by: Tomer Maimon <tmaimon77@xxxxxxxxx> Thanks, Tomer