Re: [alsa-devel] [PATCH v2 0/4] Tegra: Harmony: Add internal speaker support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, 2011-01-19 at 13:50 -0700, Stephen Warren wrote:
> Harmony has headers on the board for an internal speaker and internal mic.
> This patch series adds various infra-structure, and enables usage of the
> internal speaker.
> 
> The patch series includes both changes to code within the ASoC
> subsystem and Tegra subsystem, the latter provided only for context when
> reading the ASoC changes. Would you prefer I separated these so that the
> ASoC maintainers can grab a whole series and apply it, rather than just
> a subset of the patches?
> 
> While patch 2 is physically located in the Tegra subsystem, I'd
> consider it part of the ASoC driver. Can such a change be checked into
> the ASoC tree? This new header file will be required to build to ASoC driver
> once the following patches are in. I was considering placing this in
> include/sound, following the example of various codecs, but it doesn't seem
> a good idea to pollute that directory with platform-specific files. The
> current location for the file follows the example of s3c24xx/simtec. Perhaps
> it should be in sound/soc/tegra?
> 
> Stephen Warren (4):
>   ASoC: WM8903: Expose GPIOs through gpiolib
>   ARM: tegra: Add Harmony sound platform data type
>   ARM: tegra: Platform data fixes for ASoC driver updates
>   ASoC: tegra: Harmony: Support the internal speaker
> 
>  arch/arm/mach-tegra/board-harmony.c              |   31 ++++++
>  arch/arm/mach-tegra/gpio-names.h                 |    2 +
>  arch/arm/mach-tegra/include/mach/harmony_audio.h |   19 ++++
>  include/sound/wm8903.h                           |    7 +-
>  sound/soc/codecs/wm8903.c                        |  117 +++++++++++++++++++++-
>  sound/soc/tegra/harmony.c                        |   89 ++++++++++++++--
>  6 files changed, 251 insertions(+), 14 deletions(-)
>  create mode 100644 arch/arm/mach-tegra/include/mach/harmony_audio.h
> 

All

Acked-by: Liam Girdwood <lrg@xxxxxxxxxxxxxxx>
-- 
Freelance Developer, SlimLogic Ltd
ASoC and Voltage Regulator Maintainer.
http://www.slimlogic.co.uk

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


[Index of Archives]     [ARM Kernel]     [Linux ARM]     [Linux ARM MSM]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux