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