Hi Dirk, On Mon, May 9, 2016 at 9:52 AM, Dirk Behme <dirk.behme@xxxxxxxxxxxx> wrote: > On 09.05.2016 09:44, Geert Uytterhoeven wrote: >> >> On Mon, May 9, 2016 at 9:14 AM, Dirk Behme <dirk.behme@xxxxxxxxxxxx> >> wrote: >>> >>> --- a/arch/arm/boot/dts/r8a7790.dtsi >>> +++ b/arch/arm/boot/dts/r8a7790.dtsi >>> @@ -1102,6 +1102,11 @@ >>> #power-domain-cells = <0>; >>> }; >>> >>> + modemr: modemr@e6160060 { >>> + compatible = "renesas,modemr"; >>> + reg = <0 0xe6160060 0 0x4>; >>> + }; >>> + >> >> >> This should be a node for the complete RST module, not just for the RST >> module's >> MODEMR register. > > > > You mean > > reset: reset@e6160000 { > compatible = "renesas,reset"; > reg = <0 0xe6160000 0 0x200>; > }; > > and then use anything like > > #define MODEMR 0x60 > > in the code? Yes, more or less: reset-controller@e6160000 { compatible = "renesas,r8a7795-rst"; reg = <0 0xe6160000 0 0x0200>; }; Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds