Re: [PATCH] ARM: dt: tegra: harmony: add regulators
From: Stephen Warren <hidden>
Date: 2012-08-20 18:37:49
Also in:
lkml
On 08/20/2012 12:14 PM, Mark Brown wrote:
On Mon, Aug 20, 2012 at 12:08:45PM -0600, Stephen Warren wrote:quoted
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 that would prevent regulators getting registered at all, since v3.6-rc* don't have all the required (TPS6586x) regulator (driver) patches to get the regulators instantiated from DT. Everything will only work in linux-next or sometime during the 3.7 merge window. Hence, git bisect would be broken.quoted
One solution to this is to bring the regulator tree into the Tegra tree as a dependency, and then apply a patch which adds the regulators to DT, and removes the special case from the board file. I'm asking for your OK to do that.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 up causing some annoying conflicts with other changes that also touch arch/arm/match-tegra/board-dt-tegra20.c. Perhaps they're manageable; they probably aren't that bad, but it seems better to avoid them.