Hi Prabhakar, reduced CC list added CPUidle people
On Fri, May 15, 2020 at 5:10 PM Lad Prabhakar <[email protected]> wrote: > Add DT nodes for the Advanced Power Management Units (APMU), and use the > enable-method to point out that the APMU should be used for SMP support. > > Signed-off-by: Lad Prabhakar <[email protected]> > Reviewed-by: Marian-Cristian Rotariu > <[email protected]> Thanks for your patch! > --- a/arch/arm/boot/dts/r8a7742.dtsi > +++ b/arch/arm/boot/dts/r8a7742.dtsi > @@ -18,6 +18,7 @@ > cpus { > #address-cells = <1>; > #size-cells = <0>; > + enable-method = "renesas,apmu"; According to Documentation/devicetree/bindings/arm/cpus.yaml, "enable-method" should be a property of the individual CPU nodes, and not of the parent "cpus" container node. However, so far we always put it in the parents "cpus" node, which works from secondary CPU bringup, but may cause issues with CPUidle? See also "[PATCH/RFC v2] ARM: dts: r8a7791: Move enable-method to CPU nodes" https://lore.kernel.org/linux-arm-kernel/[email protected]/ which so far has received no feedback from the DT or CPUidle people. Thanks! > cpu0: cpu@0 { > device_type = "cpu"; > @@ -305,6 +306,18 @@ > #reset-cells = <1>; > }; > > + apmu@e6151000 { > + compatible = "renesas,r8a7742-apmu", "renesas,apmu"; > + reg = <0 0xe6151000 0 0x188>; > + cpus = <&cpu4 &cpu5 &cpu6 &cpu7>; > + }; > + > + apmu@e6152000 { > + compatible = "renesas,r8a7742-apmu", "renesas,apmu"; > + reg = <0 0xe6152000 0 0x188>; > + cpus = <&cpu0 &cpu1 &cpu2 &cpu3>; > + }; > + > rst: reset-controller@e6160000 { > compatible = "renesas,r8a7742-rst"; > reg = <0 0xe6160000 0 0x0100>; Regardless: Reviewed-by: Geert Uytterhoeven <[email protected]> and I'll see what I will queue in renesas-devel for v5.9 ;-) Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected] 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
