Hi Geert, thanks for the review! > > + mfis: system-controller@e6260000 { > > + compatible = "renesas,r8a779g0-mfis"; > > + reg = <0 0xe6260000 0 0xf000>; > > Why 0xf000 and not 0x10000? The sheet listing the memory map has a "MFI - Region 14" entry for 0xe626e000, but no region 15 for 0xe626f000. I thought I play safe. Can change if you want... > > + interrupt-names = "ch0e", "ch1e", "ch2e", "ch3e", "ch4e", "ch5e", "ch6e", "ch7e", "ch8e", "ch9e", > > This is way too longer for a single line, and the double spacing > doesn't really help (yeah, it does align ch1e and ch11e...). You are right, will fix this. Happy hacking, Wolfram