linux-kernel - Re: [PATCH] power: reset: Add MAX77620 support

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20170119122744.GG30182@ulmo.ba.sec>
Date:   Thu, 19 Jan 2017 13:27:44 +0100
From:   Thierry Reding <thierry.reding@...il.com>
To:     Laxman Dewangan <ldewangan@...dia.com>
Cc:     Sebastian Reichel <sre@...nel.org>,
        Martin Michlmayr <tbm@...ius.com>, linux-pm@...r.kernel.org,
        linux-kernel@...r.kernel.org, Chaitanya Bandi <bandik@...dia.com>
Subject: Re: [PATCH] power: reset: Add MAX77620 support

On Thu, Jan 12, 2017 at 11:05:05PM +0530, Laxman Dewangan wrote:
> 
> On Thursday 12 January 2017 11:05 PM, Thierry Reding wrote:
> > * PGP Signed by an unknown key
> > 
> > On Thu, Jan 12, 2017 at 10:06:24PM +0530, Laxman Dewangan wrote:
> > > On Thursday 12 January 2017 09:45 PM, Thierry Reding wrote:
> > > > 
> > > > +	dev_dbg(&pdev->dev, "event recorder: %#x\n", value);
> > > > +
> > > > +	system_power_controller = power;
> > > > +	pm_power_off = max77620_pm_power_off;
> > > > +#if defined(CONFIG_ARM) || defined(CONFIG_ARM64)
> > > > +	arm_pm_restart = max77620_pm_restart;
> > > What if we want to reset via the Tegra and not through PMIC?
> > In that case I assume we either don't have a PMIC, or we should not add
> > the system-power-controller device tree property to the PMIC node. In
> > the latter case this driver won't install any power off or restart
> > callbacks.
> 
> 
> We need reset from Tegra and power of from PMIC. Tegra does not support
> power OFF.

Can you explain in more details what exactly the combination is that we
need to support? Why can't the PMIC both be the reset and power off
controller? Are there any advantages to resetting with the SoC rather
than the PMIC?

Thierry

Download attachment "signature.asc" of type "application/pgp-signature" (834 bytes)