Hi Uli, On Fri, Aug 17, 2018 at 3:19 PM Ulrich Hecht <uli+renesas@xxxxxxxx> wrote: > From: Khiem Nguyen <khiem.nguyen.xt@xxxxxxxxxxxxxxx> > > Enable cpuidle (core shutdown) support for R-Car H3 CA57 cores. > > Parameters were found after evaluation by gaku.inami.xw@xxxxxxxxxxxxxx; they > help to keep the performance and reduce the power consumption. > > Signed-off-by: Khiem Nguyen <khiem.nguyen.xt@xxxxxxxxxxxxxxx> > [dien.pham.ry: Apply new cpuidle parameters] > Signed-off-by: Dien Pham <dien.pham.ry@xxxxxxxxxxx> > Signed-off-by: Takeshi Kihara <takeshi.kihara.df@xxxxxxxxxxx> > Signed-off-by: Ulrich Hecht <uli+renesas@xxxxxxxx> Thanks for your patch! > --- a/arch/arm64/boot/dts/renesas/r8a7795.dtsi > +++ b/arch/arm64/boot/dts/renesas/r8a7795.dtsi > @@ -221,6 +225,20 @@ > cache-unified; > cache-level = <2>; > }; > + > + idle-states { > + entry-method = "psci"; > + > + CPU_SLEEP_0: cpu-sleep-0 { > + compatible = "arm,idle-state"; > + arm,psci-suspend-param = <0x0010000>; > + local-timer-stop; > + entry-latency-us = <400>; > + exit-latency-us = <500>; > + min-residency-us = <4000>; > + status = "okay"; Usually we just don't specify status for "virtual" nodes (applies to the other patches, too). > + }; > + }; > }; 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