All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 0/2] ASoC: multi-comp: twl4030 updates
@ 2010-08-06  5:49 Peter Ujfalusi
  2010-08-06  5:49 ` [PATCH 1/2] ASoC: multi-comp: twl4030: TI CODECs Peter Ujfalusi
                   ` (3 more replies)
  0 siblings, 4 replies; 6+ messages in thread
From: Peter Ujfalusi @ 2010-08-06  5:49 UTC (permalink / raw)
  To: alsa-devel; +Cc: broonie, lrg

Hello Liam, Mark!

The following two patch is for the current multi-comp
branch with twl4030 related fixes.

I have separated them, so they can be squased to the
corresponding commit in multi-comp branch for now.

The changes in short:
- We do not need the header sound/soc/codecs/twl4030.h
  anymore, so remove it
- Fix machine drivers with theis header removal
- The codec no longer get's the former setup data, all
  configuration is coming from board files via the
  platform_data. So change the code accordingly
- Move the twl4030_priv allocation to a proper place
- Codec can shall go OFF instead of STANDBY, so add back
  the idle_bias_off flag
- The MFD driver is twl4030-audio

TODO after the multi-comp is merged:
- Rename the mfd driver to twl4030-audio.
  Clean up the internal naming within the mfd
  This going to touch many files...

One more notice:
The multi-comp branch does not have the
"ASoC: TWL4030: Capture route runtime DAPM ordering fix" commit
yet, and I have tested my changes on top of that, but it should
not be a big problem, when you rebase this driver.

What do you think?

Peter

---
Peter Ujfalusi (2):
  ASoC: multi-comp: twl4030: TI CODECs
  ASoC: multi-comp: twl4030: TI OMAP Platform

 drivers/mfd/twl-core.c        |    2 +-
 drivers/mfd/twl4030-codec.c   |    4 +-
 include/linux/i2c/twl.h       |    6 +++-
 sound/soc/codecs/twl4030.c    |   73 ++++++++++++++++++++---------------------
 sound/soc/codecs/twl4030.h    |   52 -----------------------------
 sound/soc/omap/igep0020.c     |    1 -
 sound/soc/omap/omap2evm.c     |    1 -
 sound/soc/omap/omap3beagle.c  |    1 -
 sound/soc/omap/omap3evm.c     |    1 -
 sound/soc/omap/omap3pandora.c |    1 -
 sound/soc/omap/overo.c        |    1 -
 sound/soc/omap/sdp3430.c      |    4 ++-
 sound/soc/omap/zoom2.c        |    4 ++-
 13 files changed, 50 insertions(+), 101 deletions(-)
 delete mode 100644 sound/soc/codecs/twl4030.h

--
1.7.2

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH 1/2] ASoC: multi-comp: twl4030: TI CODECs
  2010-08-06  5:49 [PATCH 0/2] ASoC: multi-comp: twl4030 updates Peter Ujfalusi
@ 2010-08-06  5:49 ` Peter Ujfalusi
  2010-08-06  5:49 ` [PATCH 2/2] ASoC: multi-comp: twl4030: TI OMAP Platform Peter Ujfalusi
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 6+ messages in thread
From: Peter Ujfalusi @ 2010-08-06  5:49 UTC (permalink / raw)
  To: alsa-devel; +Cc: broonie, lrg

Corrections for the twl4030 codec driver on top of the
current multi-comp branch.
Short log:
- twl4030.h header removed, since it is not needed anymore
- all codec parameters are coming from platform_data
- put back the idle_bias_off=1

This can be squashed to 'ASoC: multi-component - TI CODECs'
commit for now.

Signed-off-by: Peter Ujfalusi <peter.ujfalusi@nokia.com>
---
 sound/soc/codecs/twl4030.c |   73 +++++++++++++++++++++----------------------
 sound/soc/codecs/twl4030.h |   52 -------------------------------
 2 files changed, 36 insertions(+), 89 deletions(-)
 delete mode 100644 sound/soc/codecs/twl4030.h

diff --git a/sound/soc/codecs/twl4030.c b/sound/soc/codecs/twl4030.c
index cab013e..7bde165 100644
--- a/sound/soc/codecs/twl4030.c
+++ b/sound/soc/codecs/twl4030.c
@@ -36,7 +36,16 @@
 #include <sound/initval.h>
 #include <sound/tlv.h>
 
-#include "twl4030.h"
+/* Register descriptions are here */
+#include <linux/mfd/twl4030-codec.h>
+
+/* Shadow register used by the audio driver */
+#define TWL4030_REG_SW_SHADOW		0x4A
+#define TWL4030_CACHEREGNUM	(TWL4030_REG_SW_SHADOW + 1)
+
+/* TWL4030_REG_SW_SHADOW (0x4A) Fields */
+#define TWL4030_HFL_EN			0x01
+#define TWL4030_HFR_EN			0x02
 
 /*
  * twl4030 register cache & default register settings
@@ -279,17 +288,17 @@ static inline void twl4030_reset_registers(struct snd_soc_codec *codec)
 
 static void twl4030_init_chip(struct snd_soc_codec *codec)
 {
-	struct twl4030_setup_data *setup = dev_get_platdata(codec->dev);
+	struct twl4030_codec_audio_data *pdata = dev_get_platdata(codec->dev);
 	struct twl4030_priv *twl4030 = snd_soc_codec_get_drvdata(codec);
 	u8 reg, byte;
 	int i = 0;
 
 	/* Check defaults, if instructed before anything else */
-	if (setup && setup->check_defaults)
+	if (pdata && pdata->check_defaults)
 		twl4030_check_defaults(codec);
 
 	/* Reset registers, if no setup data or if instructed to do so */
-	if (!setup || (setup && setup->reset_registers))
+	if (!pdata || (pdata && pdata->reset_registers))
 		twl4030_reset_registers(codec);
 
 	/* Refresh APLL_CTL register from HW */
@@ -310,20 +319,14 @@ static void twl4030_init_chip(struct snd_soc_codec *codec)
 	twl4030_write(codec, TWL4030_REG_ARXR2_APGA_CTL, 0x32);
 
 	/* Machine dependent setup */
-	if (!setup)
+	if (!pdata)
 		return;
 
-	twl4030->digimic_delay = setup->digimic_delay;
-
-	/* Configuration for headset ramp delay from setup data */
-	if (setup->sysclk != twl4030->sysclk)
-		dev_warn(codec->dev,
-				"Mismatch in APLL mclk: %u (configured: %u)\n",
-				setup->sysclk, twl4030->sysclk);
+	twl4030->digimic_delay = pdata->digimic_delay;
 
 	reg = twl4030_read_reg_cache(codec, TWL4030_REG_HS_POPN_SET);
 	reg &= ~TWL4030_RAMP_DELAY;
-	reg |= (setup->ramp_delay_value << 2);
+	reg |= (pdata->ramp_delay_value << 2);
 	twl4030_write_reg_cache(codec, TWL4030_REG_HS_POPN_SET, reg);
 
 	/* initiate offset cancellation */
@@ -331,7 +334,7 @@ static void twl4030_init_chip(struct snd_soc_codec *codec)
 
 	reg = twl4030_read_reg_cache(codec, TWL4030_REG_ANAMICL);
 	reg &= ~TWL4030_OFFSET_CNCL_SEL;
-	reg |= setup->offset_cncl_path;
+	reg |= pdata->offset_cncl_path;
 	twl4030_write(codec, TWL4030_REG_ANAMICL,
 		reg | TWL4030_CNCL_OFFSET_START);
 
@@ -617,7 +620,7 @@ static int micpath_event(struct snd_soc_dapm_widget *w,
 static int pin_name##pga_event(struct snd_soc_dapm_widget *w,		\
 		struct snd_kcontrol *kcontrol, int event)		\
 {									\
-	struct twl4030_priv *twl4030 = snd_soc_codec_get_drvdata(w->codec);	\
+	struct twl4030_priv *twl4030 = snd_soc_codec_get_drvdata(w->codec); \
 									\
 	switch (event) {						\
 	case SND_SOC_DAPM_POST_PMU:					\
@@ -746,7 +749,7 @@ static int aif_event(struct snd_soc_dapm_widget *w,
 
 static void headset_ramp(struct snd_soc_codec *codec, int ramp)
 {
-	struct twl4030_codec_audio_data *setup = codec->dev->platform_data;
+	struct twl4030_codec_audio_data *pdata = codec->dev->platform_data;
 	unsigned char hs_gain, hs_pop;
 	struct twl4030_priv *twl4030 = snd_soc_codec_get_drvdata(codec);
 	/* Base values for ramp delay calculation: 2^19 - 2^26 */
@@ -758,9 +761,9 @@ static void headset_ramp(struct snd_soc_codec *codec, int ramp)
 
 	/* Enable external mute control, this dramatically reduces
 	 * the pop-noise */
-	if (setup && setup->hs_extmute) {
-		if (setup->set_hs_extmute) {
-			setup->set_hs_extmute(1);
+	if (pdata && pdata->hs_extmute) {
+		if (pdata->set_hs_extmute) {
+			pdata->set_hs_extmute(1);
 		} else {
 			hs_pop |= TWL4030_EXTMUTE;
 			twl4030_write(codec, TWL4030_REG_HS_POPN_SET, hs_pop);
@@ -798,9 +801,9 @@ static void headset_ramp(struct snd_soc_codec *codec, int ramp)
 	}
 
 	/* Disable external mute */
-	if (setup && setup->hs_extmute) {
-		if (setup->set_hs_extmute) {
-			setup->set_hs_extmute(0);
+	if (pdata && pdata->hs_extmute) {
+		if (pdata->set_hs_extmute) {
+			pdata->set_hs_extmute(0);
 		} else {
 			hs_pop &= ~TWL4030_EXTMUTE;
 			twl4030_write(codec, TWL4030_REG_HS_POPN_SET, hs_pop);
@@ -2189,7 +2192,7 @@ static int twl4030_voice_set_tristate(struct snd_soc_dai *dai, int tristate)
 #define TWL4030_RATES	 (SNDRV_PCM_RATE_8000_48000)
 #define TWL4030_FORMATS	 (SNDRV_PCM_FMTBIT_S16_LE | SNDRV_PCM_FORMAT_S24_LE)
 
-struct snd_soc_dai_ops twl4030_dai_ops = {
+static struct snd_soc_dai_ops twl4030_dai_hifi_ops = {
 	.startup	= twl4030_startup,
 	.shutdown	= twl4030_shutdown,
 	.hw_params	= twl4030_hw_params,
@@ -2222,7 +2225,7 @@ static struct snd_soc_dai_driver twl4030_dai[] = {
 		.channels_max = 4,
 		.rates = TWL4030_RATES,
 		.formats = TWL4030_FORMATS,},
-	.ops = &twl4030_dai_ops,
+	.ops = &twl4030_dai_hifi_ops,
 },
 {
 	.name = "twl4030-voice",
@@ -2242,9 +2245,7 @@ static struct snd_soc_dai_driver twl4030_dai[] = {
 },
 };
 
-
-static int twl4030_soc_suspend(struct snd_soc_codec *codec,
-		pm_message_t state)
+static int twl4030_soc_suspend(struct snd_soc_codec *codec, pm_message_t state)
 {
 	twl4030_set_bias_level(codec, SND_SOC_BIAS_OFF);
 	return 0;
@@ -2258,11 +2259,18 @@ static int twl4030_soc_resume(struct snd_soc_codec *codec)
 
 static int twl4030_soc_probe(struct snd_soc_codec *codec)
 {
-	struct twl4030_priv *twl4030 = snd_soc_codec_get_drvdata(codec);
+	struct twl4030_priv *twl4030;
 
+	twl4030 = kzalloc(sizeof(struct twl4030_priv), GFP_KERNEL);
+	if (twl4030 == NULL) {
+		printk("Can not allocate memroy\n");
+		return -ENOMEM;
+	}
+	snd_soc_codec_set_drvdata(codec, twl4030);
 	/* Set the defaults, and power up the codec */
 	twl4030->sysclk = twl4030_codec_get_mclk() / 1000;
 	codec->bias_level = SND_SOC_BIAS_OFF;
+	codec->idle_bias_off = 1;
 
 	twl4030_init_chip(codec);
 
@@ -2294,21 +2302,12 @@ static struct snd_soc_codec_driver soc_codec_dev_twl4030 = {
 static int __devinit twl4030_codec_probe(struct platform_device *pdev)
 {
 	struct twl4030_codec_audio_data *pdata = pdev->dev.platform_data;
-	struct twl4030_priv *twl4030;
 
 	if (!pdata) {
 		dev_err(&pdev->dev, "platform_data is missing\n");
 		return -EINVAL;
 	}
 
-	twl4030 = kzalloc(sizeof(struct twl4030_priv), GFP_KERNEL);
-	if (twl4030 == NULL) {
-		dev_err(&pdev->dev, "Can not allocate memroy\n");
-		return -ENOMEM;
-	}
-
-	dev_set_drvdata(&pdev->dev, twl4030);
-
 	return snd_soc_register_codec(&pdev->dev, &soc_codec_dev_twl4030,
 			twl4030_dai, ARRAY_SIZE(twl4030_dai));
 }
diff --git a/sound/soc/codecs/twl4030.h b/sound/soc/codecs/twl4030.h
deleted file mode 100644
index 12b5e4e..0000000
--- a/sound/soc/codecs/twl4030.h
+++ /dev/null
@@ -1,52 +0,0 @@
-/*
- * ALSA SoC TWL4030 codec driver
- *
- * Author: Steve Sakoman <steve@sakoman.com>
- *
- * This program is free software; you can redistribute it and/or
- * modify it under the terms of the GNU General Public License
- * version 2 as published by the Free Software Foundation.
- *
- * This program is distributed in the hope that it will be useful, but
- * WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
- * General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA
- * 02110-1301 USA
- *
- */
-
-#ifndef __TWL4030_AUDIO_H__
-#define __TWL4030_AUDIO_H__
-
-/* Register descriptions are here */
-#include <linux/mfd/twl4030-codec.h>
-
-/* Shadow register used by the audio driver */
-#define TWL4030_REG_SW_SHADOW		0x4A
-#define TWL4030_CACHEREGNUM	(TWL4030_REG_SW_SHADOW + 1)
-
-/* TWL4030_REG_SW_SHADOW (0x4A) Fields */
-#define TWL4030_HFL_EN			0x01
-#define TWL4030_HFR_EN			0x02
-
-#define TWL4030_DAI_HIFI		0
-#define TWL4030_DAI_VOICE		1
-
-struct twl4030_setup_data {
-	unsigned int ramp_delay_value;
-	unsigned int digimic_delay; /* in ms */
-	unsigned int sysclk;
-	unsigned int offset_cncl_path;
-	unsigned int check_defaults:1;
-	unsigned int reset_registers:1;
-	unsigned int hs_extmute:1;
-	void (*set_hs_extmute)(int mute);
-};
-
-#endif	/* End of __TWL4030_AUDIO_H__ */
-
-
-- 
1.7.2

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [PATCH 2/2] ASoC: multi-comp: twl4030: TI OMAP Platform
  2010-08-06  5:49 [PATCH 0/2] ASoC: multi-comp: twl4030 updates Peter Ujfalusi
  2010-08-06  5:49 ` [PATCH 1/2] ASoC: multi-comp: twl4030: TI CODECs Peter Ujfalusi
@ 2010-08-06  5:49 ` Peter Ujfalusi
  2010-08-06 12:14 ` [PATCH 0/2] ASoC: multi-comp: twl4030 updates Mark Brown
  2010-08-07 12:03 ` Liam Girdwood
  3 siblings, 0 replies; 6+ messages in thread
From: Peter Ujfalusi @ 2010-08-06  5:49 UTC (permalink / raw)
  To: alsa-devel; +Cc: broonie, lrg

Changes for twl4030 codec:
- in mfd level
- in machine level

This can be squashed to 'ASoC: multi-component - TI OMAP Platform'
commit for now.

Signed-off-by: Peter Ujfalusi <peter.ujfalusi@nokia.com>
---
 drivers/mfd/twl-core.c        |    2 +-
 drivers/mfd/twl4030-codec.c   |    4 ++--
 include/linux/i2c/twl.h       |    6 +++++-
 sound/soc/omap/igep0020.c     |    1 -
 sound/soc/omap/omap2evm.c     |    1 -
 sound/soc/omap/omap3beagle.c  |    1 -
 sound/soc/omap/omap3evm.c     |    1 -
 sound/soc/omap/omap3pandora.c |    1 -
 sound/soc/omap/overo.c        |    1 -
 sound/soc/omap/sdp3430.c      |    4 +++-
 sound/soc/omap/zoom2.c        |    4 +++-
 11 files changed, 14 insertions(+), 12 deletions(-)

diff --git a/drivers/mfd/twl-core.c b/drivers/mfd/twl-core.c
index be15203..5d0fb60 100644
--- a/drivers/mfd/twl-core.c
+++ b/drivers/mfd/twl-core.c
@@ -698,7 +698,7 @@ add_children(struct twl4030_platform_data *pdata, unsigned long features)
 
 	if (twl_has_codec() && pdata->codec && twl_class_is_4030()) {
 		sub_chip_id = twl_map[TWL_MODULE_AUDIO_VOICE].sid;
-		child = add_child(sub_chip_id, "twl4030_audio",
+		child = add_child(sub_chip_id, "twl4030-audio",
 				pdata->codec, sizeof(*pdata->codec),
 				false, 0, 0);
 		if (IS_ERR(child))
diff --git a/drivers/mfd/twl4030-codec.c b/drivers/mfd/twl4030-codec.c
index e9e64e1..9a4b196 100644
--- a/drivers/mfd/twl4030-codec.c
+++ b/drivers/mfd/twl4030-codec.c
@@ -249,14 +249,14 @@ static int __devexit twl4030_codec_remove(struct platform_device *pdev)
 	return 0;
 }
 
-MODULE_ALIAS("platform:twl4030_audio");
+MODULE_ALIAS("platform:twl4030-audio");
 
 static struct platform_driver twl4030_codec_driver = {
 	.probe		= twl4030_codec_probe,
 	.remove		= __devexit_p(twl4030_codec_remove),
 	.driver		= {
 		.owner	= THIS_MODULE,
-		.name	= "twl4030_audio",
+		.name	= "twl4030-audio",
 	},
 };
 
diff --git a/include/linux/i2c/twl.h b/include/linux/i2c/twl.h
index 6de90bf..4793d8a 100644
--- a/include/linux/i2c/twl.h
+++ b/include/linux/i2c/twl.h
@@ -553,8 +553,12 @@ extern void twl4030_power_init(struct twl4030_power_data *triton2_scripts);
 extern int twl4030_remove_script(u8 flags);
 
 struct twl4030_codec_audio_data {
-	unsigned int	audio_mclk;
+	unsigned int audio_mclk; /* not used, will be removed */
+	unsigned int digimic_delay; /* in ms */
 	unsigned int ramp_delay_value;
+	unsigned int offset_cncl_path;
+	unsigned int check_defaults:1;
+	unsigned int reset_registers:1;
 	unsigned int hs_extmute:1;
 	void (*set_hs_extmute)(int mute);
 };
diff --git a/sound/soc/omap/igep0020.c b/sound/soc/omap/igep0020.c
index 050d81f..d296cfc 100644
--- a/sound/soc/omap/igep0020.c
+++ b/sound/soc/omap/igep0020.c
@@ -33,7 +33,6 @@
 
 #include "omap-mcbsp.h"
 #include "omap-pcm.h"
-#include "../codecs/twl4030.h"
 
 static int igep2_hw_params(struct snd_pcm_substream *substream,
 	struct snd_pcm_hw_params *params)
diff --git a/sound/soc/omap/omap2evm.c b/sound/soc/omap/omap2evm.c
index 3af2c17..38cd189 100644
--- a/sound/soc/omap/omap2evm.c
+++ b/sound/soc/omap/omap2evm.c
@@ -35,7 +35,6 @@
 
 #include "omap-mcbsp.h"
 #include "omap-pcm.h"
-#include "../codecs/twl4030.h"
 
 static int omap2evm_hw_params(struct snd_pcm_substream *substream,
 	struct snd_pcm_hw_params *params)
diff --git a/sound/soc/omap/omap3beagle.c b/sound/soc/omap/omap3beagle.c
index ebad1de..7c11e1a 100644
--- a/sound/soc/omap/omap3beagle.c
+++ b/sound/soc/omap/omap3beagle.c
@@ -33,7 +33,6 @@
 
 #include "omap-mcbsp.h"
 #include "omap-pcm.h"
-#include "../codecs/twl4030.h"
 
 static int omap3beagle_hw_params(struct snd_pcm_substream *substream,
 	struct snd_pcm_hw_params *params)
diff --git a/sound/soc/omap/omap3evm.c b/sound/soc/omap/omap3evm.c
index bd45f62..1ac5bab 100644
--- a/sound/soc/omap/omap3evm.c
+++ b/sound/soc/omap/omap3evm.c
@@ -31,7 +31,6 @@
 
 #include "omap-mcbsp.h"
 #include "omap-pcm.h"
-#include "../codecs/twl4030.h"
 
 static int omap3evm_hw_params(struct snd_pcm_substream *substream,
 	struct snd_pcm_hw_params *params)
diff --git a/sound/soc/omap/omap3pandora.c b/sound/soc/omap/omap3pandora.c
index f306f8f..dbd9d96 100644
--- a/sound/soc/omap/omap3pandora.c
+++ b/sound/soc/omap/omap3pandora.c
@@ -35,7 +35,6 @@
 
 #include "omap-mcbsp.h"
 #include "omap-pcm.h"
-#include "../codecs/twl4030.h"
 
 #define OMAP3_PANDORA_DAC_POWER_GPIO	118
 #define OMAP3_PANDORA_AMP_POWER_GPIO	14
diff --git a/sound/soc/omap/overo.c b/sound/soc/omap/overo.c
index 81e284e..e95a607 100644
--- a/sound/soc/omap/overo.c
+++ b/sound/soc/omap/overo.c
@@ -33,7 +33,6 @@
 
 #include "omap-mcbsp.h"
 #include "omap-pcm.h"
-#include "../codecs/twl4030.h"
 
 static int overo_hw_params(struct snd_pcm_substream *substream,
 	struct snd_pcm_hw_params *params)
diff --git a/sound/soc/omap/sdp3430.c b/sound/soc/omap/sdp3430.c
index 953abfb..76ce77b9 100644
--- a/sound/soc/omap/sdp3430.c
+++ b/sound/soc/omap/sdp3430.c
@@ -36,9 +36,11 @@
 #include <mach/gpio.h>
 #include <plat/mcbsp.h>
 
+/* Register descriptions for twl4030 codec part */
+#include <linux/mfd/twl4030-codec.h>
+
 #include "omap-mcbsp.h"
 #include "omap-pcm.h"
-#include "../codecs/twl4030.h"
 
 /* TWL4030 PMBR1 Register */
 #define TWL4030_INTBR_PMBR1		0x0D
diff --git a/sound/soc/omap/zoom2.c b/sound/soc/omap/zoom2.c
index 0810b99..338dc95 100644
--- a/sound/soc/omap/zoom2.c
+++ b/sound/soc/omap/zoom2.c
@@ -32,9 +32,11 @@
 #include <mach/board-zoom.h>
 #include <plat/mcbsp.h>
 
+/* Register descriptions for twl4030 codec part */
+#include <linux/mfd/twl4030-codec.h>
+
 #include "omap-mcbsp.h"
 #include "omap-pcm.h"
-#include "../codecs/twl4030.h"
 
 #define ZOOM2_HEADSET_MUX_GPIO		(OMAP_MAX_GPIO_LINES + 15)
 
-- 
1.7.2

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH 0/2] ASoC: multi-comp: twl4030 updates
  2010-08-06  5:49 [PATCH 0/2] ASoC: multi-comp: twl4030 updates Peter Ujfalusi
  2010-08-06  5:49 ` [PATCH 1/2] ASoC: multi-comp: twl4030: TI CODECs Peter Ujfalusi
  2010-08-06  5:49 ` [PATCH 2/2] ASoC: multi-comp: twl4030: TI OMAP Platform Peter Ujfalusi
@ 2010-08-06 12:14 ` Mark Brown
  2010-08-06 12:43   ` Peter Ujfalusi
  2010-08-07 12:03 ` Liam Girdwood
  3 siblings, 1 reply; 6+ messages in thread
From: Mark Brown @ 2010-08-06 12:14 UTC (permalink / raw)
  To: Peter Ujfalusi; +Cc: alsa-devel, lrg

On Fri, Aug 06, 2010 at 08:49:53AM +0300, Peter Ujfalusi wrote:

> The following two patch is for the current multi-comp
> branch with twl4030 related fixes.

Acked-by: Mark Brown <broonie@opensource.wolfsonmicro.com>

though patch 2 could've used a better changelog!

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 0/2] ASoC: multi-comp: twl4030 updates
  2010-08-06 12:14 ` [PATCH 0/2] ASoC: multi-comp: twl4030 updates Mark Brown
@ 2010-08-06 12:43   ` Peter Ujfalusi
  0 siblings, 0 replies; 6+ messages in thread
From: Peter Ujfalusi @ 2010-08-06 12:43 UTC (permalink / raw)
  To: ext Mark Brown; +Cc: alsa-devel, lrg

On Friday 06 August 2010 15:14:24 ext Mark Brown wrote:
> On Fri, Aug 06, 2010 at 08:49:53AM +0300, Peter Ujfalusi wrote:
> > The following two patch is for the current multi-comp
> > branch with twl4030 related fixes.
> 
> Acked-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
> 
> though patch 2 could've used a better changelog!

Oops. The commit message in  patch 2 not even close to informative :(
I did however detailed the changes in the intro mail. These patches will be 
not visible at the end, since Liam will squash the changes to the multi-comp 
patch-set, and I did not wanted to put more effort on the commit message for 
the individual patches (I'm lazy, yes).
Just wanted to give Liam a hand with the twl4030 codec, and make sure that it 
is going to work flawlessly with multi-comp.

Never the less, I'll try to put more effort next time, since the commit message 
on top of the patch is easier to read, than a separate mail.

Thanks,
Péter

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 0/2] ASoC: multi-comp: twl4030 updates
  2010-08-06  5:49 [PATCH 0/2] ASoC: multi-comp: twl4030 updates Peter Ujfalusi
                   ` (2 preceding siblings ...)
  2010-08-06 12:14 ` [PATCH 0/2] ASoC: multi-comp: twl4030 updates Mark Brown
@ 2010-08-07 12:03 ` Liam Girdwood
  3 siblings, 0 replies; 6+ messages in thread
From: Liam Girdwood @ 2010-08-07 12:03 UTC (permalink / raw)
  To: Peter Ujfalusi; +Cc: alsa-devel, broonie

On Fri, 2010-08-06 at 08:49 +0300, Peter Ujfalusi wrote:
> Hello Liam, Mark!
> 
> The following two patch is for the current multi-comp
> branch with twl4030 related fixes.
> 
> I have separated them, so they can be squased to the
> corresponding commit in multi-comp branch for now.
> 
> The changes in short:
> - We do not need the header sound/soc/codecs/twl4030.h
>   anymore, so remove it
> - Fix machine drivers with theis header removal
> - The codec no longer get's the former setup data, all
>   configuration is coming from board files via the
>   platform_data. So change the code accordingly
> - Move the twl4030_priv allocation to a proper place
> - Codec can shall go OFF instead of STANDBY, so add back
>   the idle_bias_off flag
> - The MFD driver is twl4030-audio
> 
> TODO after the multi-comp is merged:
> - Rename the mfd driver to twl4030-audio.
>   Clean up the internal naming within the mfd
>   This going to touch many files...
> 
> One more notice:
> The multi-comp branch does not have the
> "ASoC: TWL4030: Capture route runtime DAPM ordering fix" commit
> yet, and I have tested my changes on top of that, but it should
> not be a big problem, when you rebase this driver.
> 

Applied.

Thanks

Liam
-- 
Freelance Developer, SlimLogic Ltd
ASoC and Voltage Regulator Maintainer.
http://www.slimlogic.co.uk

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2010-08-07 12:03 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-08-06  5:49 [PATCH 0/2] ASoC: multi-comp: twl4030 updates Peter Ujfalusi
2010-08-06  5:49 ` [PATCH 1/2] ASoC: multi-comp: twl4030: TI CODECs Peter Ujfalusi
2010-08-06  5:49 ` [PATCH 2/2] ASoC: multi-comp: twl4030: TI OMAP Platform Peter Ujfalusi
2010-08-06 12:14 ` [PATCH 0/2] ASoC: multi-comp: twl4030 updates Mark Brown
2010-08-06 12:43   ` Peter Ujfalusi
2010-08-07 12:03 ` Liam Girdwood

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.