Re: [PATCH] ARM: dt: tegra: paz00: configure power off

[Marc Dietrich <marvin24-Mmb7MZpHnFY@public.gmane.org>]

[-- Attachment #1: Type: text/plain, Size: 1470 bytes --]

Hi Micha,

Am Samstag, 12. März 2016, 00:00:00 CET schrieb Misha Komarovskiy:
> Add DT property to tell the regulator that it should provide
> the pm_power_off() implementation. This fix "shutdown" issue
> on this device.
> 
> Signed-off-by: Misha Komarovskiy <zombah-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
> ---
> 
> Tested on current mainline version, ac100 models 116 and 117,
> successful rate of "shutdown" increased from 10% to 90%.
> 
>  arch/arm/boot/dts/tegra20-paz00.dts | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/tegra20-paz00.dts
> b/arch/arm/boot/dts/tegra20-paz00.dts index ed7e100..c82e3ae 100644
> --- a/arch/arm/boot/dts/tegra20-paz00.dts
> +++ b/arch/arm/boot/dts/tegra20-paz00.dts
> @@ -316,6 +316,8 @@
>  			#gpio-cells = <2>;
>  			gpio-controller;
> 
> +			ti,system-power-controller;
> +
>  			sys-supply = <&p5valw_reg>;
>  			vin-sm0-supply = <&sys_reg>;
>  			vin-sm1-supply = <&sys_reg>;

I think this is the wrong fix. The power_off hook is already occupied by nvec. 
So either the tps or nvec is responsible for shutting down the system. AFAIR, 
nvec has some hard wired connection to the tps for doing this, so in the end, 
it will shut down (but I can't find it now in the schematics). See paz00-
diag.c in the original sources on how the system is beeing shut down.

In any case, we can only have one power_off handler. Either via tps or nvec, 
not both.

Marc

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 473 bytes --]