Chris,
On Wed, Oct 29, 2014 at 4:52 AM, Chris Zhong <zyw@xxxxxxxxxxxxxx> wrote:
> global_pwroff would be pull to high when RK3288 entering suspend,
> this pin is a sleep signal for RK808, so RK808 could goto sleep
> mode, and some regulators would be disable.
>
> Signed-off-by: Chris Zhong <zyw@xxxxxxxxxxxxxx>
>
> ---
>
> Changes in v6: None
> Changes in v5: None
> Changes in v4: None
> Changes in v3: None
> Changes in v2: None
>
> arch/arm/boot/dts/rk3288-evb-rk808.dts | 16 +++++++++++++++-
> 1 file changed, 15 insertions(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/rk3288-evb-rk808.dts b/arch/arm/boot/dts/rk3288-evb-rk808.dts
> index ff522f8..319fcb6 100644
> --- a/arch/arm/boot/dts/rk3288-evb-rk808.dts
> +++ b/arch/arm/boot/dts/rk3288-evb-rk808.dts
> @@ -27,7 +27,7 @@
> interrupt-parent = <&gpio0>;
> interrupts = <4 IRQ_TYPE_LEVEL_LOW>;
> pinctrl-names = "default";
> - pinctrl-0 = <&pmic_int>;
> + pinctrl-0 = <&pmic_int &global_pwroff>;
> rockchip,system-power-controller;
> wakeup-source;
> #clock-cells = <1>;
> @@ -46,6 +46,7 @@
> regulator-min-microvolt = <750000>;
> regulator-max-microvolt = <1300000>;
> regulator-name = "vdd_arm";
> + regulator-suspend-mem-disabled;
NAK. "regulator-suspend-mem-disabled" is a local property added by
local Chrome OS patches and doesn't belong in an upstream submission.
You should using the new patches from Chanwoo.
40e20d6 regulator: of: Add support for parsing regulator_state for suspend state
291d761 regulator: Document binding for regulator suspend state for PM state
In your case, it would look like:
regulator-state-mem {
regulator-off-in-suspend;
};
> @@ -76,6 +80,7 @@
> regulator-min-microvolt = <3300000>;
> regulator-max-microvolt = <3300000>;
> regulator-name = "vccio_pmu";
> + regulator-suspend-mem-microvolt = <3300000>;
Similarly this property isn't upstream. You can see Javier's work on
this in <https://patchwork.kernel.org/patch/5106351/> and I think
you'll need an rk808-specific patch just like he needs an max77802
patch. You probably want to wait for him to spin it first, though,
since Mark had feedback on his last patch.
-Doug
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
