ARM64: dts: rockchip: Change PWM regulators to 300kHz for gru base
authorDouglas Anderson <dianders@chromium.org>
Tue, 12 Apr 2016 00:51:56 +0000 (17:51 -0700)
committerHuang, Tao <huangtao@rock-chips.com>
Wed, 13 Apr 2016 02:28:55 +0000 (10:28 +0800)
Apparently the time constant for for the PWM regulator circuit on
gru-based devices is different than EVB.  Let's run at 300kHz which
should make us work well.

BUG=None
TEST=None

Change-Id: I0973f416d026de27908c3ef527c1e9274b967fc8
Signed-off-by: Douglas Anderson <dianders@chromium.org>
Reviewed-on: https://chrome-internal-review.googlesource.com/254648
Reviewed-by: Stephen Barber <smbarber@google.com>
Tested-by: Doug Anderson <dianders@google.com>
arch/arm64/boot/dts/rockchip/rk3399-gru.dtsi

index d53e77b5774b9836049a6959a262b9f8ca579603..b9442612819920c53474e4823bb52401cd29b385 100644 (file)
                regulator-name = "ppvar_bigcpu";
                status = "okay";
 
-               // TODO: schematics say 600kHz so period of 1667 ns
-               // Rockchip has 25000 (40kHz) for these though.
-               pwms = <&pwm1 0 1667 0>;
+               pwms = <&pwm1 0 3333 0>;
 
                // TODO: where's the mapping of duty cycle to voltage???
                // TODO: we probably need in-flight PWM regulator series?
                regulator-name = "ppvar_litcpu";
                status = "okay";
 
-               // TODO: schematics say 600kHz so period of 1667 ns
-               // Rockchip has 25000 (40kHz) for these though.
-               pwms = <&pwm2 0 1667 0>;
+               pwms = <&pwm2 0 3333 0>;
 
                /* EC turns on w/ ap_core_en; always on for AP */
                regulator-always-on;
                regulator-name = "ppvar_gpu";
                status = "okay";
 
-               // TODO: schematics say 600kHz so period of 1667 ns
-               // Rockchip has 25000 (40kHz) for these though.
-               pwms = <&pwm0 0 1667 0>;
+               pwms = <&pwm0 0 3333 0>;
 
                /* EC turns on w/ ap_core_en; always on for AP */
                regulator-always-on;
                regulator-name = "ppvar_centerlogic";
                status = "okay";
 
-               // TODO: schematics say 600kHz so period of 1667 ns
-               // Rockchip has 25000 (40kHz) for these though.
-               pwms = <&pwm3 0 1667 0>;
+               pwms = <&pwm3 0 3333 0>;
 
                /* EC turns on w/ ppvar_centerlogic_en; always on for AP */
                regulator-always-on;