Re: [PATCH] power: reset: Add MAX77620 support

[Thierry Reding]

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

Attachment: signature.asc
Description: PGP signature