RE: [PATCHv2 0/7] TWL6030 audio codec initial support

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

 



Misa,

> -----Original Message-----
> From: linux-omap-owner@xxxxxxxxxxxxxxx [mailto:linux-omap-
> owner@xxxxxxxxxxxxxxx] On Behalf Of Lopez Cruz, Misael
> Sent: Saturday, September 26, 2009 7:32 AM
> To: alsa-devel@xxxxxxxxxxxxxxxx; linux-omap@xxxxxxxxxxxxxxx
> Cc: Mark Brown
> Subject: [PATCHv2 0/7] TWL6030 audio codec initial support
> 
> Following patch series adds initial support for TWL6030 codec
> driver.
> 
> TWL6030 codec depends on TWL6030_CORE, which is not accepted yet.
> The main intention then is to get comments, I'll repost the patches
> of this series when TWL6030_CORE support gets accepted and rework
> patches as needed.
Looking at your series, you have based your patches on Balaji's 6030 PMIC series as you mentioned " TWL6030 codec depends on TWL6030_CORE".

While doing that series, we realized that there is a lot of code common between TWL5030 and TWL6030 apart from mainly interrupt handling and hence a common file approach is taken to avoid code duplication. The patch " [PATCHv2 3/7] ASoC: TWL6030: Add twl6030 codec driver" gives me an impression that there is opportunity to have a single file supporting both IC's. IC specific code can be handled using the runtime check depending on the revision.
Can you explore this ? 
You can find more information on above discussion at below thread.

http://www.mail-archive.com/linux-omap@xxxxxxxxxxxxxxx/msg15756.html


>From 
> Changes from v1:
> - Renamed dai format for PDM as SND_SOC_DAIFMT_PDM
> - CODEC driver registered as a platform driver
> - AUDPWRON gpio line handled in the CODEC driver
> - Handle scenario when no AUDPWRON gpio line is provided
> - Corrected names of some widgets
> - Avoid multiple calls to power functions in set_bias_level
> 
> Thanks,
> -Misa
> 
> ---
> 
> Misael Lopez Cruz (7):
>       OMAP4: PMIC: Add support for twl6030 codec
>       ASoC: Add PDM DAI format definition
>       ASoC: TWL6030: Add twl6030 codec driver
>       ASoC: TWL6030: Manual power-up/down sequences
>       ASoC: TWL6030: Add support for low-power mode
>       ASoC: TWL6030: Enable audio interrupt
>       ASoC: TWL6030: Power-up seq completion through audio interrupt
> 
>  drivers/mfd/twl-core.c     |   15 +
>  include/linux/i2c/twl.h    |   11 +
>  include/sound/soc-dai.h    |    1 +
>  sound/soc/codecs/Kconfig   |    4 +
>  sound/soc/codecs/Makefile  |    2 +
>  sound/soc/codecs/twl6030.c | 1179
> ++++++++++++++++++++++++++++++++++++++++++++
>  sound/soc/codecs/twl6030.h |  137 +++++
>  7 files changed, 1349 insertions(+), 0 deletions(-)
>  create mode 100644 sound/soc/codecs/twl6030.c
>  create mode 100644 sound/soc/codecs/twl6030.h
> --
> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html


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

[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux