On 12/09/2023 04:49, k.son@xxxxxxxxxxx wrote: > From: Kwanghoon Son <k.son@xxxxxxxxxxx> > > Add reset device tree for th1520 SoC > > Signed-off-by: Kwanghoon Son <k.son@xxxxxxxxxxx> > --- > arch/riscv/boot/dts/thead/th1520.dtsi | 8 ++++++++ > 1 file changed, 8 insertions(+) > > diff --git a/arch/riscv/boot/dts/thead/th1520.dtsi b/arch/riscv/boot/dts/thead/th1520.dtsi > index ce708183b6f6..7e592449b5e4 100644 > --- a/arch/riscv/boot/dts/thead/th1520.dtsi > +++ b/arch/riscv/boot/dts/thead/th1520.dtsi > @@ -5,6 +5,7 @@ > */ > > #include <dt-bindings/interrupt-controller/irq.h> > +#include <dt-bindings/reset/th1520-reset.h> > > / { > compatible = "thead,th1520"; > @@ -418,5 +419,12 @@ portf: gpio-controller@0 { > interrupts = <55 IRQ_TYPE_LEVEL_HIGH>; > }; > }; > + > + rst: reset-controller@ffef014000 { > + compatible = "thead,th1520-reset", "syscon"; > + reg = <0xff 0xef014000 0x0 0x1000>; > + #reset-cells = <1>; > + status = "okay"; Drop, it's by default. Look at the other nodes - they do not have it, so neither this node should. Best regards, Krzysztof