[PATCH 3/3] ARM: sun7i: cubieboard2: Enable the AXP209

[Chen-Yu Tsai]

Hi,

On Sun, Sep 21, 2014 at 4:06 AM, Maxime Ripard
<maxime.ripard at free-electrons.com> wrote:
> Add the AXP209 PMIC with the regulators in use on the cubie2.
>
> The RTC regulator can be disabled, but since the RTC driver doesn't grab the
> regulator for now, the driver wouldn't work anymore.
>
> Signed-off-by: Maxime Ripard <maxime.ripard at free-electrons.com>
> ---
>  arch/arm/boot/dts/sun7i-a20-cubieboard2.dts | 31 +++++++++++++++++++++++++++++
>  1 file changed, 31 insertions(+)
>
> diff --git a/arch/arm/boot/dts/sun7i-a20-cubieboard2.dts b/arch/arm/boot/dts/sun7i-a20-cubieboard2.dts
> index 53680983461a..f0b74a38a800 100644
> --- a/arch/arm/boot/dts/sun7i-a20-cubieboard2.dts
> +++ b/arch/arm/boot/dts/sun7i-a20-cubieboard2.dts
> @@ -91,6 +91,29 @@
>
>                                 interrupt-controller;
>                                 #interrupt-cells = <1>;
> +
> +                               regulators {
> +                                       x-powers,dcdc-freq = <1500>;
> +
> +                                       vdd_cpu: dcdc2 {
> +                                               vin-supply = <&reg_axp_ipsout>;

The driver ignores the vin-supply properties.

I think the original intended design was to have

    acin-supply = <&reg_axp_ipsout>;
    vin2-supply = <&reg_axp_ipsout>;
    vin3-supply = <&reg_axp_ipsout>;
    ldo24in-supply = <&reg_axp_ipsout>;
    ldo3in-supply = <&reg_axp_ipsout>;
    ldo5in-supply = <&reg_axp_ipsout>;

under the axp209 node.

With this change, the kernel does find the supplies:

LDO1: 1300 mV
LDO1: supplied by axp-ipsout
LDO2: at 3000 mV
LDO2: supplied by axp-ipsout
LDO3: at 2275 mV
LDO3: supplied by axp-ipsout
LDO4: at 2800 mV
LDO4: supplied by axp-ipsout
LDO5: at 2800 mV
LDO5: supplied by axp-ipsout
DCDC2: at 1400 mV
DCDC2: supplied by axp-ipsout
DCDC3: at 1250 mV
DCDC3: supplied by axp-ipsout

ChenYu

P.S. my /sys/kernel/debug/regulator/supply_map is always empty.
Is that normal?

> +                                               regulator-always-on;
> +                                       };
> +
> +                                       vdd_dll: dcdc3 {
> +                                               vin-supply = <&reg_axp_ipsout>;
> +                                               regulator-always-on;
> +                                       };
> +
> +                                       vdd_rtc: ldo1 {
> +                                               vin-supply = <&reg_axp_ipsout>;
> +                                       };
> +
> +                                       avcc: ldo2 {
> +                                               vin-supply = <&reg_axp_ipsout>;
> +                                               regulator-always-on;
> +                                       };
> +                               };
>                         };
>                 };
>
> @@ -140,4 +163,12 @@
>         reg_usb2_vbus: usb2-vbus {
>                 status = "okay";
>         };
> +
> +       reg_axp_ipsout: axp_ipsout {
> +               compatible = "regulator-fixed";
> +               regulator-name = "axp-ipsout";
> +               regulator-min-microvolt = <5000000>;
> +               regulator-max-microvolt = <5000000>;
> +               regulator-always-on;
> +       };
>  };
> --
> 2.1.0
>
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel