Hello, miquel.raynal@xxxxxxxxxxx wrote on Tue, 15 Mar 2022 20:12:51 +0100: > The dmamux register is located within the system controller. > > Without syscon, we need an extra helper in order to give write access to > this register to a dmamux driver. > > Signed-off-by: Miquel Raynal <miquel.raynal@xxxxxxxxxxx> > --- > drivers/clk/renesas/r9a06g032-clocks.c | 35 ++++++++++++++++++- > include/linux/soc/renesas/r9a06g032-sysctrl.h | 11 ++++++ > 2 files changed, 45 insertions(+), 1 deletion(-) > create mode 100644 include/linux/soc/renesas/r9a06g032-sysctrl.h > [...] > /* register/bit pairs are encoded as an uint16_t */ > static void > clk_rdesc_set(struct r9a06g032_priv *clocks, > @@ -922,6 +948,7 @@ static int __init r9a06g032_clocks_probe(struct platform_device *pdev) > clocks->reg = of_iomap(np, 0); > if (WARN_ON(!clocks->reg)) > return -ENOMEM; > + As we are post -rc1 I will repost this series as a v6 after rebasing. While at it I'll get rid of this extra new line but that's basically all what I plan to change. As this series brings the basics for more RZN1 support (because of this specific sysctrl patch), it is kind of a base for more contributions which are in the pipe (UART, USB, RTC, Switch, etc) so hopefully it is now ready to be accepted. Thanks, Miquèl