Re: [PATCH v3 3/4] ARM: tegra: Platform data fixes for ASoC driver updates

[Stephen Warren <swarren@nvidia.com>]

Mark Brown wrote on Friday, January 21, 2011 10:50:
> On Fri, Jan 21, 2011 at 09:45:56AM -0800, Stephen Warren wrote:
> 
> > > I'm happy with this version of the series, I guess Liam will be too.
> > > Ideally someone from the Tegra side should sign off on the arch/arm
> > > side - that'll need to go through the same tree as the ASoC side for
> > > build dependencies.
> 
> > I assume you're only talking about:
> 
> > a882c75eadfcdc56f47962dd3f7870d6def62847
> > ARM: tegra: Add Harmony sound platform data type
> 
> That and the addition of the defines for function codes IIRC.

The wm8903 GPIO function defines? They're in wm8903.h and don't rely
on anything else. The arch/arm patches don't rely on these defines.

> > The other arch/arm patches won't apply anywhere yet since they build
> > on top of various other patches that haven't been submitted for or
> > applied to the linux-tegra tree yet.
> 
> Oh, I see.  I hadn't realised there wer eother dependencies, please
> ensure you mention any such dependencies in the changlogs so we avoid
> introducing breakage.

That was the intent of what I wrote in the "cover letter" "patch".

To be explicit, the only dependency is that the patch quoted above, which
adds some platform data types, needs to be checked in to te ASoC tree
rather than the usual ARM tree, before the following ASoC patch is
applied:

ASoC: tegra: Harmony: Support th internal speaker

Every other ASoC patch I've submitted doesn't rely on anything outside
the ASoC tree to compile, although actually running it usefully obviously
requires the arch/arm changes.

Hope this clears things up!

--
nvpublic