Subject: [RFC 3/5] PCI: tegra: Implement accurate power supply scheme

From: [email-masked] (Thierry Reding)
Date: Tue, 8 Apr 2014 21:55:37 +0200

On Tue, Apr 08, 2014 at 09:20:57PM +0200, Stephen Warren wrote:
> On 04/04/2014 08:49 AM, Thierry Reding wrote:
> > The current description of power supplies doesn't match the hardware.
> > Instead it's designed to support the needs of current designs, which
> > will break as soon as a new design appears that cannot be described
> > using the current assumptions.
> > 
> > In order to fully support all possible future designs, all power supply
> > inputs to the PCIe block need to be accurately described and separately
> > configurable.
> 
> > diff --git a/drivers/pci/host/pci-tegra.c b/drivers/pci/host/pci-tegra.c
> 
> > @@ -930,23 +927,9 @@ static void tegra_pcie_power_off(struct tegra_pcie *pcie)
> 
> > -	err = regulator_disable(pcie->pex_clk_supply);
> > +	err = regulator_bulk_disable(pcie->num_supplies, pcie->supplies);
> >  	if (err < 0)
> > -		dev_warn(pcie->dev, "failed to disable pex-clk regulator: %d\n",
> > -			 err);
> > -
> > -	err = regulator_disable(pcie->vdd_supply);
> > -	if (err < 0)
> > -		dev_warn(pcie->dev, "failed to disable VDD regulator: %d\n",
> > -			 err);
> > +		dev_warn(pcie->dev, "failed to enable regulators: %d\n", err);
> 
> s/enable/disable/

Good catch. Fixed, thanks,

Thierry
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20140408/23ac3062/attachment.sig>