Hi Claudiu, On Thu, Feb 8, 2024 at 1:44 PM Claudiu <claudiu.beznea@xxxxxxxxx> wrote: > From: Claudiu Beznea <claudiu.beznea.uj@xxxxxxxxxxxxxx> > > Update CPG #power-domain-cells = <1> and move all the IPs to be part of the > always on power domain as the driver has been modified to support multiple > power domains. > > Signed-off-by: Claudiu Beznea <claudiu.beznea.uj@xxxxxxxxxxxxxx> Thanks for your patch! > --- a/arch/arm64/boot/dts/renesas/r9a07g044.dtsi > +++ b/arch/arm64/boot/dts/renesas/r9a07g044.dtsi > @@ -238,7 +238,7 @@ mtu3: timer@10001200 { > "tgia8", "tgib8", "tgic8", "tgid8", > "tciv8", "tciu8"; > clocks = <&cpg CPG_MOD R9A07G044_MTU_X_MCK_MTU3>; > - power-domains = <&cpg>; > + power-domains = <&cpg R9A07G044_PD_ALWAYS_ON>; This should be R9A07G044_PD_MTU, but that domain is not yet supported by the driver. Instead of adding incorrect hardware descriptions, I'd rather defer this until all domains are handled by the driver. > resets = <&cpg R9A07G044_MTU_X_PRESET_MTU3>; > #pwm-cells = <2>; > status = "disabled"; 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