Hi Morimoto-san, On Wed, Jan 25, 2023 at 11:39 PM Kuninori Morimoto <kuninori.morimoto.gx@xxxxxxxxxxx> wrote: > > > + DEF_MOD("sys-dmac0", 709, R8A779A0_CLK_S1D2), > > > + DEF_MOD("sys-dmac1", 710, R8A779A0_CLK_S1D2), > > > > Actually Rev 0.5 of the documentation does call the channels "SYS-DMAC1" > > and "SYS-DMAC2". > > Shall we use that numbering? > > We also have labels "dmac1" and "dmac2" in DTS. > > OK, I see. > > I'm happy if you can modify it. > Of course I'm happy to post v2 patch. Thanks, no need for that, I will fix while applying. Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx> i.e. will queue in renesas-clk for v6.3. 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