Re: [PATCH] ARM: dt: tegra: harmony: add regulators

2012-08-24 Thread Stephen Warren
On 08/22/2012 01:07 PM, Mark Brown wrote: > On Mon, Aug 20, 2012 at 12:37:44PM -0600, Stephen Warren wrote: >> On 08/20/2012 12:14 PM, Mark Brown wrote: > >>> Why not just pull the patch in via the regulator tree? The >>> idea of merging the entire regulator drivers branch into the >>> Tegra tree

Re: [PATCH] ARM: dt: tegra: harmony: add regulators

2012-08-22 Thread Mark Brown
On Mon, Aug 20, 2012 at 12:37:44PM -0600, Stephen Warren wrote: > On 08/20/2012 12:14 PM, Mark Brown wrote: > > Why not just pull the patch in via the regulator tree? The idea > > of merging the entire regulator drivers branch into the Tegra tree > > doesn't seem awesome... > I think that'd end

Re: [PATCH] ARM: dt: tegra: harmony: add regulators

2012-08-20 Thread Stephen Warren
On 08/20/2012 12:14 PM, Mark Brown wrote: > On Mon, Aug 20, 2012 at 12:08:45PM -0600, Stephen Warren wrote: > >> The patch in this thread adds all the required regulators to the >> DT file. However, it should also remove the special case so the >> regulators don't get registered twice, once from

Re: [PATCH] ARM: dt: tegra: harmony: add regulators

2012-08-20 Thread Mark Brown
On Mon, Aug 20, 2012 at 12:08:45PM -0600, Stephen Warren wrote: > The patch in this thread adds all the required regulators to the DT > file. However, it should also remove the special case so the > regulators don't get registered twice, once from the board file and > once from DT. However, doing

Re: [PATCH] ARM: dt: tegra: harmony: add regulators

2012-08-20 Thread Stephen Warren
On 08/20/2012 11:54 AM, Mark Brown wrote: > On Mon, Aug 20, 2012 at 11:49:11AM -0600, Stephen Warren wrote: > >> The problem there is that will prevent any regulators from being >> registered at all, until the patch is merged with the regulator >> tree. That will also prevent PCIe working on Harm

Re: [PATCH] ARM: dt: tegra: harmony: add regulators

2012-08-20 Thread Mark Brown
On Mon, Aug 20, 2012 at 11:49:11AM -0600, Stephen Warren wrote: > The problem there is that will prevent any regulators from being > registered at all, until the patch is merged with the regulator tree. > That will also prevent PCIe working on Harmony. Those both work right > now, so that's a bise

Re: [PATCH] ARM: dt: tegra: harmony: add regulators

2012-08-20 Thread Stephen Warren
On 08/20/2012 11:22 AM, Laxman Dewangan wrote: > On Saturday 18 August 2012 01:25 AM, Stephen Warren wrote: >> On 08/17/2012 01:27 PM, Stephen Warren wrote: >>> On 08/17/2012 12:59 AM, Laxman Dewangan wrote: Harmony uses a TPS6586x regulator. Instantiate this, and hook up a couple of fixe

Re: [PATCH] ARM: dt: tegra: harmony: add regulators

2012-08-20 Thread Laxman Dewangan
On Saturday 18 August 2012 01:25 AM, Stephen Warren wrote: On 08/17/2012 01:27 PM, Stephen Warren wrote: On 08/17/2012 12:59 AM, Laxman Dewangan wrote: Harmony uses a TPS6586x regulator. Instantiate this, and hook up a couple of fixed GPIO-controlled regulators too. Based on Ventana regulator

Re: [PATCH] ARM: dt: tegra: harmony: add regulators

2012-08-17 Thread Stephen Warren
On 08/17/2012 01:27 PM, Stephen Warren wrote: > On 08/17/2012 12:59 AM, Laxman Dewangan wrote: >> Harmony uses a TPS6586x regulator. Instantiate this, and hook up a >> couple of fixed GPIO-controlled regulators too. >> >> Based on Ventana regulator patch by Stephen Warren >> and converted to Harmo

Re: [PATCH] ARM: dt: tegra: harmony: add regulators

2012-08-17 Thread Stephen Warren
On 08/17/2012 12:59 AM, Laxman Dewangan wrote: > Harmony uses a TPS6586x regulator. Instantiate this, and hook up a > couple of fixed GPIO-controlled regulators too. > > Based on Ventana regulator patch by Stephen Warren > and converted to Harmony. Thanks, applied to Tegra's for-3.7/dt branch.

[PATCH] ARM: dt: tegra: harmony: add regulators

2012-08-17 Thread Laxman Dewangan
Harmony uses a TPS6586x regulator. Instantiate this, and hook up a couple of fixed GPIO-controlled regulators too. Based on Ventana regulator patch by Stephen Warren and converted to Harmony. Signed-off-by: Laxman Dewangan --- arch/arm/boot/dts/tegra20-harmony.dts | 210 ++