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

[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 patch by Stephen Warren<swarren@xxxxxxxxxx>
> > > and converted to Harmony.
> > Thanks, applied to Tegra's for-3.7/dt branch.
> Oh hang on, this adds regulators to DT, but doesn't do anything to
> remove the board-file registration of Harmony's regulators from
> board-dt-tegra20.c, which end up conflicting, and preventing the PCIe
> driver from being registered.
>
> So, this patch really wants to remove the legacy code to, but doing so
> will be problematic; to avoid runtime git bisect failures, we'd need to
> merge the regulator tree in first, or wait until next kernel release or
> something... Thoughts?

Why not just remove the regulator registration from board files along 
with the dt entry (in one patch). That will resolve the issue of bisect.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/