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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxx>
> ---
> 
> 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: signature.asc
Description: This is a digitally signed message part.


[Index of Archives]     [ARM Kernel]     [Linux ARM]     [Linux ARM MSM]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux