Thread (11 messages) 11 messages, 3 authors, 2012-08-24

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.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help