* [PATCH 0/4 v2] ASoC/MFD/OMAP: TWL4030: APLL_CTL handling change @ 2009-11-04 7:58 ` Peter Ujfalusi 0 siblings, 0 replies; 21+ messages in thread From: Peter Ujfalusi @ 2009-11-04 7:58 UTC (permalink / raw) To: alsa-devel, linux-omap, sameo; +Cc: linux-kernel, tony, broonie Hello, Changes from the first series: - Typos fixed and warning/error messages for APLL frequency mismatch now consistent - twl4030-codec MFD driver audio_mclk validity check simplified - Commit message for the twl4030-codec MFD driver has been changed also - Patch: "ASoC: TWL4030: Make sure, that the codec is powered on startup" has been removed from the series, since it is already taken. Intro text from the first series: The following series changes the way how the APLL_CTL register is handled by the twl4030 codec related drivers. Moving the configuration of APLL_CTL register from child drivers (specifically from soc codec driver) to the twl4030-codec MFD driver. Because the audio_mclk is static, board specific and can not be changed in runtime, the codec MFD is the right place to configure it early enough, so child can use the hardware correctly. Before this patch for example the APLL_INFREQ was configured first, when the user played or recorded audio via the twl4030. The digital bypass needs correct APLL_INFREQ to be configured in order to work properly. If the digital bypass enabled before any audio activity, in some cases the bypassed audio was not correct. The series also addresses the issue, that after boot-up, neither of the loopbacks was working, because the codec was actually off. Setting the codec->bias_level to _OFF, before setting the codec bias level to STANDBY fixes this issue. This series is on top of Takashi's sound-2.6 topic/asoc branch, which has the twl403-codec MFD and related patches for 2.6.33. It would be really nice, if this series would make it also to the same branch, since this fixes some real anomalies brought in by the cleanup of the twl4030 codec related drivers. These problems were hidden before because of the inconsistent handling of bits and states in the codec driver in the past. --- Peter Ujfalusi (4): MFD: TWL4030: Add audio_mclk to the codec platform data OMAP: Configure audio_mclk for twl4030-codec MFD MFD: twl4030-codec: APLL_INFREQ handling in the MFD driver ASoC: TWL4030: Do not modify the APLL_CTL register arch/arm/mach-omap2/board-3430sdp.c | 1 + arch/arm/mach-omap2/board-omap3beagle.c | 1 + arch/arm/mach-omap2/board-omap3evm.c | 1 + arch/arm/mach-omap2/board-omap3pandora.c | 1 + arch/arm/mach-omap2/board-overo.c | 1 + arch/arm/mach-omap2/board-zoom2.c | 1 + drivers/mfd/twl4030-codec.c | 35 ++++++++++++++ include/linux/i2c/twl4030.h | 1 + include/linux/mfd/twl4030-codec.h | 1 + sound/soc/codecs/twl4030.c | 76 ++++++++++++------------------ 10 files changed, 74 insertions(+), 45 deletions(-) ^ permalink raw reply [flat|nested] 21+ messages in thread
* [PATCH 0/4 v2] ASoC/MFD/OMAP: TWL4030: APLL_CTL handling change @ 2009-11-04 7:58 ` Peter Ujfalusi 0 siblings, 0 replies; 21+ messages in thread From: Peter Ujfalusi @ 2009-11-04 7:58 UTC (permalink / raw) To: alsa-devel, linux-omap, sameo; +Cc: tony, broonie, linux-kernel Hello, Changes from the first series: - Typos fixed and warning/error messages for APLL frequency mismatch now consistent - twl4030-codec MFD driver audio_mclk validity check simplified - Commit message for the twl4030-codec MFD driver has been changed also - Patch: "ASoC: TWL4030: Make sure, that the codec is powered on startup" has been removed from the series, since it is already taken. Intro text from the first series: The following series changes the way how the APLL_CTL register is handled by the twl4030 codec related drivers. Moving the configuration of APLL_CTL register from child drivers (specifically from soc codec driver) to the twl4030-codec MFD driver. Because the audio_mclk is static, board specific and can not be changed in runtime, the codec MFD is the right place to configure it early enough, so child can use the hardware correctly. Before this patch for example the APLL_INFREQ was configured first, when the user played or recorded audio via the twl4030. The digital bypass needs correct APLL_INFREQ to be configured in order to work properly. If the digital bypass enabled before any audio activity, in some cases the bypassed audio was not correct. The series also addresses the issue, that after boot-up, neither of the loopbacks was working, because the codec was actually off. Setting the codec->bias_level to _OFF, before setting the codec bias level to STANDBY fixes this issue. This series is on top of Takashi's sound-2.6 topic/asoc branch, which has the twl403-codec MFD and related patches for 2.6.33. It would be really nice, if this series would make it also to the same branch, since this fixes some real anomalies brought in by the cleanup of the twl4030 codec related drivers. These problems were hidden before because of the inconsistent handling of bits and states in the codec driver in the past. --- Peter Ujfalusi (4): MFD: TWL4030: Add audio_mclk to the codec platform data OMAP: Configure audio_mclk for twl4030-codec MFD MFD: twl4030-codec: APLL_INFREQ handling in the MFD driver ASoC: TWL4030: Do not modify the APLL_CTL register arch/arm/mach-omap2/board-3430sdp.c | 1 + arch/arm/mach-omap2/board-omap3beagle.c | 1 + arch/arm/mach-omap2/board-omap3evm.c | 1 + arch/arm/mach-omap2/board-omap3pandora.c | 1 + arch/arm/mach-omap2/board-overo.c | 1 + arch/arm/mach-omap2/board-zoom2.c | 1 + drivers/mfd/twl4030-codec.c | 35 ++++++++++++++ include/linux/i2c/twl4030.h | 1 + include/linux/mfd/twl4030-codec.h | 1 + sound/soc/codecs/twl4030.c | 76 ++++++++++++------------------ 10 files changed, 74 insertions(+), 45 deletions(-) ^ permalink raw reply [flat|nested] 21+ messages in thread
* [PATCH 1/4 v2] MFD: TWL4030: Add audio_mclk to the codec platform data 2009-11-04 7:58 ` Peter Ujfalusi @ 2009-11-04 7:58 ` Peter Ujfalusi -1 siblings, 0 replies; 21+ messages in thread From: Peter Ujfalusi @ 2009-11-04 7:58 UTC (permalink / raw) To: alsa-devel, linux-omap, sameo; +Cc: linux-kernel, tony, broonie Add audio_mclk to the platform data struct for the twl4030-codec MFD driver. Signed-off-by: Peter Ujfalusi <peter.ujfalusi@nokia.com> --- include/linux/i2c/twl4030.h | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/include/linux/i2c/twl4030.h b/include/linux/i2c/twl4030.h index 42d6c72..c188961 100644 --- a/include/linux/i2c/twl4030.h +++ b/include/linux/i2c/twl4030.h @@ -414,6 +414,7 @@ struct twl4030_codec_vibra_data { }; struct twl4030_codec_data { + unsigned int audio_mclk; struct twl4030_codec_audio_data *audio; struct twl4030_codec_vibra_data *vibra; }; -- 1.6.5.2 ^ permalink raw reply related [flat|nested] 21+ messages in thread
* [PATCH 1/4 v2] MFD: TWL4030: Add audio_mclk to the codec platform data @ 2009-11-04 7:58 ` Peter Ujfalusi 0 siblings, 0 replies; 21+ messages in thread From: Peter Ujfalusi @ 2009-11-04 7:58 UTC (permalink / raw) To: alsa-devel, linux-omap, sameo; +Cc: tony, broonie, linux-kernel Add audio_mclk to the platform data struct for the twl4030-codec MFD driver. Signed-off-by: Peter Ujfalusi <peter.ujfalusi@nokia.com> --- include/linux/i2c/twl4030.h | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/include/linux/i2c/twl4030.h b/include/linux/i2c/twl4030.h index 42d6c72..c188961 100644 --- a/include/linux/i2c/twl4030.h +++ b/include/linux/i2c/twl4030.h @@ -414,6 +414,7 @@ struct twl4030_codec_vibra_data { }; struct twl4030_codec_data { + unsigned int audio_mclk; struct twl4030_codec_audio_data *audio; struct twl4030_codec_vibra_data *vibra; }; -- 1.6.5.2 ^ permalink raw reply related [flat|nested] 21+ messages in thread
* [PATCH 2/4 v2] OMAP: Configure audio_mclk for twl4030-codec MFD 2009-11-04 7:58 ` Peter Ujfalusi @ 2009-11-04 7:58 ` Peter Ujfalusi -1 siblings, 0 replies; 21+ messages in thread From: Peter Ujfalusi @ 2009-11-04 7:58 UTC (permalink / raw) To: alsa-devel, linux-omap, sameo; +Cc: linux-kernel, tony, broonie audio_mclk value is going to be handled by the twl4030-codec MFD driver, configure the correct value for boards, which is using the twl4030 audio. Signed-off-by: Peter Ujfalusi <peter.ujfalusi@nokia.com> --- arch/arm/mach-omap2/board-3430sdp.c | 1 + arch/arm/mach-omap2/board-omap3beagle.c | 1 + arch/arm/mach-omap2/board-omap3evm.c | 1 + arch/arm/mach-omap2/board-omap3pandora.c | 1 + arch/arm/mach-omap2/board-overo.c | 1 + arch/arm/mach-omap2/board-zoom2.c | 1 + 6 files changed, 6 insertions(+), 0 deletions(-) diff --git a/arch/arm/mach-omap2/board-3430sdp.c b/arch/arm/mach-omap2/board-3430sdp.c index 4f91f7a..9afd957 100644 --- a/arch/arm/mach-omap2/board-3430sdp.c +++ b/arch/arm/mach-omap2/board-3430sdp.c @@ -415,6 +415,7 @@ static struct twl4030_codec_audio_data sdp3430_audio = { }; static struct twl4030_codec_data sdp3430_codec = { + .audio_mclk = 26000000, .audio = &sdp3430_audio, }; diff --git a/arch/arm/mach-omap2/board-omap3beagle.c b/arch/arm/mach-omap2/board-omap3beagle.c index 2161d85..8f0c106 100644 --- a/arch/arm/mach-omap2/board-omap3beagle.c +++ b/arch/arm/mach-omap2/board-omap3beagle.c @@ -259,6 +259,7 @@ static struct twl4030_codec_audio_data beagle_audio_data = { }; static struct twl4030_codec_data beagle_codec_data = { + .audio_mclk = 26000000, .audio = &beagle_audio_data, }; diff --git a/arch/arm/mach-omap2/board-omap3evm.c b/arch/arm/mach-omap2/board-omap3evm.c index d9a6103..5bb30cb 100644 --- a/arch/arm/mach-omap2/board-omap3evm.c +++ b/arch/arm/mach-omap2/board-omap3evm.c @@ -199,6 +199,7 @@ static struct twl4030_codec_audio_data omap3evm_audio_data = { }; static struct twl4030_codec_data omap3evm_codec_data = { + .audio_mclk = 26000000, .audio = &omap3evm_audio_data, }; diff --git a/arch/arm/mach-omap2/board-omap3pandora.c b/arch/arm/mach-omap2/board-omap3pandora.c index 5036b56..77790ee 100644 --- a/arch/arm/mach-omap2/board-omap3pandora.c +++ b/arch/arm/mach-omap2/board-omap3pandora.c @@ -286,6 +286,7 @@ static struct twl4030_codec_audio_data omap3pandora_audio_data = { }; static struct twl4030_codec_data omap3pandora_codec_data = { + .audio_mclk = 26000000, .audio = &omap3pandora_audio_data, }; diff --git a/arch/arm/mach-omap2/board-overo.c b/arch/arm/mach-omap2/board-overo.c index dc55008..e1fb504 100644 --- a/arch/arm/mach-omap2/board-overo.c +++ b/arch/arm/mach-omap2/board-overo.c @@ -334,6 +334,7 @@ static struct twl4030_codec_audio_data overo_audio_data = { }; static struct twl4030_codec_data overo_codec_data = { + .audio_mclk = 26000000, .audio = &overo_audio_data, }; diff --git a/arch/arm/mach-omap2/board-zoom2.c b/arch/arm/mach-omap2/board-zoom2.c index f1b4e7c..de3a38d 100644 --- a/arch/arm/mach-omap2/board-zoom2.c +++ b/arch/arm/mach-omap2/board-zoom2.c @@ -234,6 +234,7 @@ static struct twl4030_codec_audio_data zoom2_audio_data = { }; static struct twl4030_codec_data zoom2_codec_data = { + .audio_mclk = 26000000, .audio = &zoom2_audio_data, }; -- 1.6.5.2 ^ permalink raw reply related [flat|nested] 21+ messages in thread
* [PATCH 2/4 v2] OMAP: Configure audio_mclk for twl4030-codec MFD @ 2009-11-04 7:58 ` Peter Ujfalusi 0 siblings, 0 replies; 21+ messages in thread From: Peter Ujfalusi @ 2009-11-04 7:58 UTC (permalink / raw) To: alsa-devel, linux-omap, sameo; +Cc: tony, broonie, linux-kernel audio_mclk value is going to be handled by the twl4030-codec MFD driver, configure the correct value for boards, which is using the twl4030 audio. Signed-off-by: Peter Ujfalusi <peter.ujfalusi@nokia.com> --- arch/arm/mach-omap2/board-3430sdp.c | 1 + arch/arm/mach-omap2/board-omap3beagle.c | 1 + arch/arm/mach-omap2/board-omap3evm.c | 1 + arch/arm/mach-omap2/board-omap3pandora.c | 1 + arch/arm/mach-omap2/board-overo.c | 1 + arch/arm/mach-omap2/board-zoom2.c | 1 + 6 files changed, 6 insertions(+), 0 deletions(-) diff --git a/arch/arm/mach-omap2/board-3430sdp.c b/arch/arm/mach-omap2/board-3430sdp.c index 4f91f7a..9afd957 100644 --- a/arch/arm/mach-omap2/board-3430sdp.c +++ b/arch/arm/mach-omap2/board-3430sdp.c @@ -415,6 +415,7 @@ static struct twl4030_codec_audio_data sdp3430_audio = { }; static struct twl4030_codec_data sdp3430_codec = { + .audio_mclk = 26000000, .audio = &sdp3430_audio, }; diff --git a/arch/arm/mach-omap2/board-omap3beagle.c b/arch/arm/mach-omap2/board-omap3beagle.c index 2161d85..8f0c106 100644 --- a/arch/arm/mach-omap2/board-omap3beagle.c +++ b/arch/arm/mach-omap2/board-omap3beagle.c @@ -259,6 +259,7 @@ static struct twl4030_codec_audio_data beagle_audio_data = { }; static struct twl4030_codec_data beagle_codec_data = { + .audio_mclk = 26000000, .audio = &beagle_audio_data, }; diff --git a/arch/arm/mach-omap2/board-omap3evm.c b/arch/arm/mach-omap2/board-omap3evm.c index d9a6103..5bb30cb 100644 --- a/arch/arm/mach-omap2/board-omap3evm.c +++ b/arch/arm/mach-omap2/board-omap3evm.c @@ -199,6 +199,7 @@ static struct twl4030_codec_audio_data omap3evm_audio_data = { }; static struct twl4030_codec_data omap3evm_codec_data = { + .audio_mclk = 26000000, .audio = &omap3evm_audio_data, }; diff --git a/arch/arm/mach-omap2/board-omap3pandora.c b/arch/arm/mach-omap2/board-omap3pandora.c index 5036b56..77790ee 100644 --- a/arch/arm/mach-omap2/board-omap3pandora.c +++ b/arch/arm/mach-omap2/board-omap3pandora.c @@ -286,6 +286,7 @@ static struct twl4030_codec_audio_data omap3pandora_audio_data = { }; static struct twl4030_codec_data omap3pandora_codec_data = { + .audio_mclk = 26000000, .audio = &omap3pandora_audio_data, }; diff --git a/arch/arm/mach-omap2/board-overo.c b/arch/arm/mach-omap2/board-overo.c index dc55008..e1fb504 100644 --- a/arch/arm/mach-omap2/board-overo.c +++ b/arch/arm/mach-omap2/board-overo.c @@ -334,6 +334,7 @@ static struct twl4030_codec_audio_data overo_audio_data = { }; static struct twl4030_codec_data overo_codec_data = { + .audio_mclk = 26000000, .audio = &overo_audio_data, }; diff --git a/arch/arm/mach-omap2/board-zoom2.c b/arch/arm/mach-omap2/board-zoom2.c index f1b4e7c..de3a38d 100644 --- a/arch/arm/mach-omap2/board-zoom2.c +++ b/arch/arm/mach-omap2/board-zoom2.c @@ -234,6 +234,7 @@ static struct twl4030_codec_audio_data zoom2_audio_data = { }; static struct twl4030_codec_data zoom2_codec_data = { + .audio_mclk = 26000000, .audio = &zoom2_audio_data, }; -- 1.6.5.2 ^ permalink raw reply related [flat|nested] 21+ messages in thread
* [PATCH 3/4 v2] MFD: twl4030-codec: APLL_INFREQ handling in the MFD driver 2009-11-04 7:58 ` Peter Ujfalusi @ 2009-11-04 7:58 ` Peter Ujfalusi -1 siblings, 0 replies; 21+ messages in thread From: Peter Ujfalusi @ 2009-11-04 7:58 UTC (permalink / raw) To: alsa-devel, linux-omap, sameo; +Cc: linux-kernel, tony, broonie Configure the APLL_INFREQ field in the APLL_CTL register based on the platform data. Provide also a function for childs to query the audio_mclk frequency. Signed-off-by: Peter Ujfalusi <peter.ujfalusi@nokia.com> --- drivers/mfd/twl4030-codec.c | 35 +++++++++++++++++++++++++++++++++++ include/linux/mfd/twl4030-codec.h | 1 + 2 files changed, 36 insertions(+), 0 deletions(-) diff --git a/drivers/mfd/twl4030-codec.c b/drivers/mfd/twl4030-codec.c index 9710307..77b9149 100644 --- a/drivers/mfd/twl4030-codec.c +++ b/drivers/mfd/twl4030-codec.c @@ -41,6 +41,7 @@ struct twl4030_codec_resource { }; struct twl4030_codec { + unsigned int audio_mclk; struct mutex mutex; struct twl4030_codec_resource resource[TWL4030_CODEC_RES_MAX]; struct mfd_cell cells[TWL4030_CODEC_CELLS]; @@ -145,12 +146,45 @@ int twl4030_codec_disable_resource(unsigned id) } EXPORT_SYMBOL_GPL(twl4030_codec_disable_resource); +unsigned int twl4030_codec_get_mclk(void) +{ + struct twl4030_codec *codec = platform_get_drvdata(twl4030_codec_dev); + + return codec->audio_mclk; +} +EXPORT_SYMBOL_GPL(twl4030_codec_get_mclk); + static int __devinit twl4030_codec_probe(struct platform_device *pdev) { struct twl4030_codec *codec; struct twl4030_codec_data *pdata = pdev->dev.platform_data; struct mfd_cell *cell = NULL; int ret, childs = 0; + u8 val; + + if (!pdata) { + dev_err(&pdev->dev, "Platform data is missing\n"); + return -EINVAL; + } + + /* Configure APLL_INFREQ and disable APLL if enabled */ + val = 0; + switch (pdata->audio_mclk) { + case 19200000: + val |= TWL4030_APLL_INFREQ_19200KHZ; + break; + case 26000000: + val |= TWL4030_APLL_INFREQ_26000KHZ; + break; + case 38400000: + val |= TWL4030_APLL_INFREQ_38400KHZ; + break; + default: + dev_err(&pdev->dev, "Invalid audio_mclk\n"); + return -EINVAL; + } + twl4030_i2c_write_u8(TWL4030_MODULE_AUDIO_VOICE, + val, TWL4030_REG_APLL_CTL); codec = kzalloc(sizeof(struct twl4030_codec), GFP_KERNEL); if (!codec) @@ -160,6 +194,7 @@ static int __devinit twl4030_codec_probe(struct platform_device *pdev) twl4030_codec_dev = pdev; mutex_init(&codec->mutex); + codec->audio_mclk = pdata->audio_mclk; /* Codec power */ codec->resource[TWL4030_CODEC_RES_POWER].reg = TWL4030_REG_CODEC_MODE; diff --git a/include/linux/mfd/twl4030-codec.h b/include/linux/mfd/twl4030-codec.h index ef0a304..2ec317c 100644 --- a/include/linux/mfd/twl4030-codec.h +++ b/include/linux/mfd/twl4030-codec.h @@ -267,5 +267,6 @@ enum twl4030_codec_res { int twl4030_codec_disable_resource(enum twl4030_codec_res id); int twl4030_codec_enable_resource(enum twl4030_codec_res id); +unsigned int twl4030_codec_get_mclk(void); #endif /* End of __TWL4030_CODEC_H__ */ -- 1.6.5.2 ^ permalink raw reply related [flat|nested] 21+ messages in thread
* [PATCH 3/4 v2] MFD: twl4030-codec: APLL_INFREQ handling in the MFD driver @ 2009-11-04 7:58 ` Peter Ujfalusi 0 siblings, 0 replies; 21+ messages in thread From: Peter Ujfalusi @ 2009-11-04 7:58 UTC (permalink / raw) To: alsa-devel, linux-omap, sameo; +Cc: tony, broonie, linux-kernel Configure the APLL_INFREQ field in the APLL_CTL register based on the platform data. Provide also a function for childs to query the audio_mclk frequency. Signed-off-by: Peter Ujfalusi <peter.ujfalusi@nokia.com> --- drivers/mfd/twl4030-codec.c | 35 +++++++++++++++++++++++++++++++++++ include/linux/mfd/twl4030-codec.h | 1 + 2 files changed, 36 insertions(+), 0 deletions(-) diff --git a/drivers/mfd/twl4030-codec.c b/drivers/mfd/twl4030-codec.c index 9710307..77b9149 100644 --- a/drivers/mfd/twl4030-codec.c +++ b/drivers/mfd/twl4030-codec.c @@ -41,6 +41,7 @@ struct twl4030_codec_resource { }; struct twl4030_codec { + unsigned int audio_mclk; struct mutex mutex; struct twl4030_codec_resource resource[TWL4030_CODEC_RES_MAX]; struct mfd_cell cells[TWL4030_CODEC_CELLS]; @@ -145,12 +146,45 @@ int twl4030_codec_disable_resource(unsigned id) } EXPORT_SYMBOL_GPL(twl4030_codec_disable_resource); +unsigned int twl4030_codec_get_mclk(void) +{ + struct twl4030_codec *codec = platform_get_drvdata(twl4030_codec_dev); + + return codec->audio_mclk; +} +EXPORT_SYMBOL_GPL(twl4030_codec_get_mclk); + static int __devinit twl4030_codec_probe(struct platform_device *pdev) { struct twl4030_codec *codec; struct twl4030_codec_data *pdata = pdev->dev.platform_data; struct mfd_cell *cell = NULL; int ret, childs = 0; + u8 val; + + if (!pdata) { + dev_err(&pdev->dev, "Platform data is missing\n"); + return -EINVAL; + } + + /* Configure APLL_INFREQ and disable APLL if enabled */ + val = 0; + switch (pdata->audio_mclk) { + case 19200000: + val |= TWL4030_APLL_INFREQ_19200KHZ; + break; + case 26000000: + val |= TWL4030_APLL_INFREQ_26000KHZ; + break; + case 38400000: + val |= TWL4030_APLL_INFREQ_38400KHZ; + break; + default: + dev_err(&pdev->dev, "Invalid audio_mclk\n"); + return -EINVAL; + } + twl4030_i2c_write_u8(TWL4030_MODULE_AUDIO_VOICE, + val, TWL4030_REG_APLL_CTL); codec = kzalloc(sizeof(struct twl4030_codec), GFP_KERNEL); if (!codec) @@ -160,6 +194,7 @@ static int __devinit twl4030_codec_probe(struct platform_device *pdev) twl4030_codec_dev = pdev; mutex_init(&codec->mutex); + codec->audio_mclk = pdata->audio_mclk; /* Codec power */ codec->resource[TWL4030_CODEC_RES_POWER].reg = TWL4030_REG_CODEC_MODE; diff --git a/include/linux/mfd/twl4030-codec.h b/include/linux/mfd/twl4030-codec.h index ef0a304..2ec317c 100644 --- a/include/linux/mfd/twl4030-codec.h +++ b/include/linux/mfd/twl4030-codec.h @@ -267,5 +267,6 @@ enum twl4030_codec_res { int twl4030_codec_disable_resource(enum twl4030_codec_res id); int twl4030_codec_enable_resource(enum twl4030_codec_res id); +unsigned int twl4030_codec_get_mclk(void); #endif /* End of __TWL4030_CODEC_H__ */ -- 1.6.5.2 ^ permalink raw reply related [flat|nested] 21+ messages in thread
* [PATCH 4/4 v2] ASoC: TWL4030: Do not modify the APLL_CTL register 2009-11-04 7:58 ` Peter Ujfalusi @ 2009-11-04 7:58 ` Peter Ujfalusi -1 siblings, 0 replies; 21+ messages in thread From: Peter Ujfalusi @ 2009-11-04 7:58 UTC (permalink / raw) To: alsa-devel, linux-omap, sameo; +Cc: linux-kernel, tony, broonie APLL_CTL register is configured by the twl4030-codec MFD driver. Remove code, which makes changes in the APLL_CTL register, and replace those with checks against the configured audio_mclk configuration done in the MFD driver. Signed-off-by: Peter Ujfalusi <peter.ujfalusi@nokia.com> --- sound/soc/codecs/twl4030.c | 76 ++++++++++++++++++-------------------------- 1 files changed, 31 insertions(+), 45 deletions(-) diff --git a/sound/soc/codecs/twl4030.c b/sound/soc/codecs/twl4030.c index c0b47df..bd8ea35 100644 --- a/sound/soc/codecs/twl4030.c +++ b/sound/soc/codecs/twl4030.c @@ -214,7 +214,8 @@ static void twl4030_init_chip(struct snd_soc_codec *codec) /* set all audio section registers to reasonable defaults */ for (i = TWL4030_REG_OPTION; i <= TWL4030_REG_MISC_SET_2; i++) - twl4030_write(codec, i, cache[i]); + if (i != TWL4030_REG_APLL_CTL) + twl4030_write(codec, i, cache[i]); } @@ -1753,30 +1754,23 @@ static int twl4030_set_dai_sysclk(struct snd_soc_dai *codec_dai, { struct snd_soc_codec *codec = codec_dai->codec; struct twl4030_priv *twl4030 = codec->private_data; - u8 apll_ctrl; - apll_ctrl = twl4030_read_reg_cache(codec, TWL4030_REG_APLL_CTL); - apll_ctrl &= ~TWL4030_APLL_INFREQ; switch (freq) { case 19200000: - apll_ctrl |= TWL4030_APLL_INFREQ_19200KHZ; - twl4030->sysclk = 19200; - break; case 26000000: - apll_ctrl |= TWL4030_APLL_INFREQ_26000KHZ; - twl4030->sysclk = 26000; - break; case 38400000: - apll_ctrl |= TWL4030_APLL_INFREQ_38400KHZ; - twl4030->sysclk = 38400; break; default: - printk(KERN_ERR "TWL4030 set sysclk: unknown rate %d\n", - freq); + dev_err(codec->dev, "Unsupported APLL mclk: %u\n", freq); return -EINVAL; } - twl4030_write(codec, TWL4030_REG_APLL_CTL, apll_ctrl); + if ((freq / 1000) != twl4030->sysclk) { + dev_err(codec->dev, + "Mismatch in APLL mclk: %u (configured: %u)\n", + freq, twl4030->sysclk * 1000); + return -EINVAL; + } return 0; } @@ -1874,18 +1868,16 @@ static int twl4030_voice_startup(struct snd_pcm_substream *substream, struct snd_soc_pcm_runtime *rtd = substream->private_data; struct snd_soc_device *socdev = rtd->socdev; struct snd_soc_codec *codec = socdev->card->codec; - u8 infreq; + struct twl4030_priv *twl4030 = codec->private_data; u8 mode; /* If the system master clock is not 26MHz, the voice PCM interface is * not avilable. */ - infreq = twl4030_read_reg_cache(codec, TWL4030_REG_APLL_CTL) - & TWL4030_APLL_INFREQ; - - if (infreq != TWL4030_APLL_INFREQ_26000KHZ) { - printk(KERN_ERR "TWL4030 voice startup: " - "MCLK is not 26MHz, call set_sysclk() on init\n"); + if (twl4030->sysclk != 26000) { + dev_err(codec->dev, "The board is configured for %u Hz, while" + "the Voice interface needs 26MHz APLL mclk\n", + twl4030->sysclk * 1000); return -EINVAL; } @@ -1958,22 +1950,19 @@ static int twl4030_voice_set_dai_sysclk(struct snd_soc_dai *codec_dai, int clk_id, unsigned int freq, int dir) { struct snd_soc_codec *codec = codec_dai->codec; - u8 apll_ctrl; + struct twl4030_priv *twl4030 = codec->private_data; - apll_ctrl = twl4030_read_reg_cache(codec, TWL4030_REG_APLL_CTL); - apll_ctrl &= ~TWL4030_APLL_INFREQ; - switch (freq) { - case 26000000: - apll_ctrl |= TWL4030_APLL_INFREQ_26000KHZ; - break; - default: - printk(KERN_ERR "TWL4030 voice set sysclk: unknown rate %d\n", - freq); + if (freq != 26000000) { + dev_err(codec->dev, "Unsupported APLL mclk: %u, the Voice" + "interface needs 26MHz APLL mclk\n", freq); + return -EINVAL; + } + if ((freq / 1000) != twl4030->sysclk) { + dev_err(codec->dev, + "Mismatch in APLL mclk: %u (configured: %u)\n", + freq, twl4030->sysclk * 1000); return -EINVAL; } - - twl4030_write(codec, TWL4030_REG_APLL_CTL, apll_ctrl); - return 0; } @@ -2131,17 +2120,15 @@ static int twl4030_soc_probe(struct platform_device *pdev) if (setup) { unsigned char hs_pop; - if (setup->sysclk) - twl4030->sysclk = setup->sysclk; - else - twl4030->sysclk = 26000; + if (setup->sysclk != twl4030->sysclk) + dev_warn(&pdev->dev, + "Mismatch in APLL mclk: %u (configured: %u)\n", + setup->sysclk, twl4030->sysclk); hs_pop = twl4030_read_reg_cache(codec, TWL4030_REG_HS_POPN_SET); hs_pop &= ~TWL4030_RAMP_DELAY; hs_pop |= (setup->ramp_delay_value << 2); twl4030_write_reg_cache(codec, TWL4030_REG_HS_POPN_SET, hs_pop); - } else { - twl4030->sysclk = 26000; } /* register pcms */ @@ -2191,10 +2178,8 @@ static int __devinit twl4030_codec_probe(struct platform_device *pdev) struct twl4030_priv *twl4030; int ret; - if (!pdata || !(pdata->audio_mclk == 19200000 || - pdata->audio_mclk == 26000000 || - pdata->audio_mclk == 38400000)) { - dev_err(&pdev->dev, "Invalid platform_data\n"); + if (!pdata) { + dev_err(&pdev->dev, "platform_data is missing\n"); return -EINVAL; } @@ -2233,6 +2218,7 @@ static int __devinit twl4030_codec_probe(struct platform_device *pdev) twl4030_codec = codec; /* Set the defaults, and power up the codec */ + twl4030->sysclk = twl4030_codec_get_mclk() / 1000; twl4030_init_chip(codec); codec->bias_level = SND_SOC_BIAS_OFF; twl4030_set_bias_level(codec, SND_SOC_BIAS_STANDBY); -- 1.6.5.2 ^ permalink raw reply related [flat|nested] 21+ messages in thread
* [PATCH 4/4 v2] ASoC: TWL4030: Do not modify the APLL_CTL register @ 2009-11-04 7:58 ` Peter Ujfalusi 0 siblings, 0 replies; 21+ messages in thread From: Peter Ujfalusi @ 2009-11-04 7:58 UTC (permalink / raw) To: alsa-devel, linux-omap, sameo; +Cc: tony, broonie, linux-kernel APLL_CTL register is configured by the twl4030-codec MFD driver. Remove code, which makes changes in the APLL_CTL register, and replace those with checks against the configured audio_mclk configuration done in the MFD driver. Signed-off-by: Peter Ujfalusi <peter.ujfalusi@nokia.com> --- sound/soc/codecs/twl4030.c | 76 ++++++++++++++++++-------------------------- 1 files changed, 31 insertions(+), 45 deletions(-) diff --git a/sound/soc/codecs/twl4030.c b/sound/soc/codecs/twl4030.c index c0b47df..bd8ea35 100644 --- a/sound/soc/codecs/twl4030.c +++ b/sound/soc/codecs/twl4030.c @@ -214,7 +214,8 @@ static void twl4030_init_chip(struct snd_soc_codec *codec) /* set all audio section registers to reasonable defaults */ for (i = TWL4030_REG_OPTION; i <= TWL4030_REG_MISC_SET_2; i++) - twl4030_write(codec, i, cache[i]); + if (i != TWL4030_REG_APLL_CTL) + twl4030_write(codec, i, cache[i]); } @@ -1753,30 +1754,23 @@ static int twl4030_set_dai_sysclk(struct snd_soc_dai *codec_dai, { struct snd_soc_codec *codec = codec_dai->codec; struct twl4030_priv *twl4030 = codec->private_data; - u8 apll_ctrl; - apll_ctrl = twl4030_read_reg_cache(codec, TWL4030_REG_APLL_CTL); - apll_ctrl &= ~TWL4030_APLL_INFREQ; switch (freq) { case 19200000: - apll_ctrl |= TWL4030_APLL_INFREQ_19200KHZ; - twl4030->sysclk = 19200; - break; case 26000000: - apll_ctrl |= TWL4030_APLL_INFREQ_26000KHZ; - twl4030->sysclk = 26000; - break; case 38400000: - apll_ctrl |= TWL4030_APLL_INFREQ_38400KHZ; - twl4030->sysclk = 38400; break; default: - printk(KERN_ERR "TWL4030 set sysclk: unknown rate %d\n", - freq); + dev_err(codec->dev, "Unsupported APLL mclk: %u\n", freq); return -EINVAL; } - twl4030_write(codec, TWL4030_REG_APLL_CTL, apll_ctrl); + if ((freq / 1000) != twl4030->sysclk) { + dev_err(codec->dev, + "Mismatch in APLL mclk: %u (configured: %u)\n", + freq, twl4030->sysclk * 1000); + return -EINVAL; + } return 0; } @@ -1874,18 +1868,16 @@ static int twl4030_voice_startup(struct snd_pcm_substream *substream, struct snd_soc_pcm_runtime *rtd = substream->private_data; struct snd_soc_device *socdev = rtd->socdev; struct snd_soc_codec *codec = socdev->card->codec; - u8 infreq; + struct twl4030_priv *twl4030 = codec->private_data; u8 mode; /* If the system master clock is not 26MHz, the voice PCM interface is * not avilable. */ - infreq = twl4030_read_reg_cache(codec, TWL4030_REG_APLL_CTL) - & TWL4030_APLL_INFREQ; - - if (infreq != TWL4030_APLL_INFREQ_26000KHZ) { - printk(KERN_ERR "TWL4030 voice startup: " - "MCLK is not 26MHz, call set_sysclk() on init\n"); + if (twl4030->sysclk != 26000) { + dev_err(codec->dev, "The board is configured for %u Hz, while" + "the Voice interface needs 26MHz APLL mclk\n", + twl4030->sysclk * 1000); return -EINVAL; } @@ -1958,22 +1950,19 @@ static int twl4030_voice_set_dai_sysclk(struct snd_soc_dai *codec_dai, int clk_id, unsigned int freq, int dir) { struct snd_soc_codec *codec = codec_dai->codec; - u8 apll_ctrl; + struct twl4030_priv *twl4030 = codec->private_data; - apll_ctrl = twl4030_read_reg_cache(codec, TWL4030_REG_APLL_CTL); - apll_ctrl &= ~TWL4030_APLL_INFREQ; - switch (freq) { - case 26000000: - apll_ctrl |= TWL4030_APLL_INFREQ_26000KHZ; - break; - default: - printk(KERN_ERR "TWL4030 voice set sysclk: unknown rate %d\n", - freq); + if (freq != 26000000) { + dev_err(codec->dev, "Unsupported APLL mclk: %u, the Voice" + "interface needs 26MHz APLL mclk\n", freq); + return -EINVAL; + } + if ((freq / 1000) != twl4030->sysclk) { + dev_err(codec->dev, + "Mismatch in APLL mclk: %u (configured: %u)\n", + freq, twl4030->sysclk * 1000); return -EINVAL; } - - twl4030_write(codec, TWL4030_REG_APLL_CTL, apll_ctrl); - return 0; } @@ -2131,17 +2120,15 @@ static int twl4030_soc_probe(struct platform_device *pdev) if (setup) { unsigned char hs_pop; - if (setup->sysclk) - twl4030->sysclk = setup->sysclk; - else - twl4030->sysclk = 26000; + if (setup->sysclk != twl4030->sysclk) + dev_warn(&pdev->dev, + "Mismatch in APLL mclk: %u (configured: %u)\n", + setup->sysclk, twl4030->sysclk); hs_pop = twl4030_read_reg_cache(codec, TWL4030_REG_HS_POPN_SET); hs_pop &= ~TWL4030_RAMP_DELAY; hs_pop |= (setup->ramp_delay_value << 2); twl4030_write_reg_cache(codec, TWL4030_REG_HS_POPN_SET, hs_pop); - } else { - twl4030->sysclk = 26000; } /* register pcms */ @@ -2191,10 +2178,8 @@ static int __devinit twl4030_codec_probe(struct platform_device *pdev) struct twl4030_priv *twl4030; int ret; - if (!pdata || !(pdata->audio_mclk == 19200000 || - pdata->audio_mclk == 26000000 || - pdata->audio_mclk == 38400000)) { - dev_err(&pdev->dev, "Invalid platform_data\n"); + if (!pdata) { + dev_err(&pdev->dev, "platform_data is missing\n"); return -EINVAL; } @@ -2233,6 +2218,7 @@ static int __devinit twl4030_codec_probe(struct platform_device *pdev) twl4030_codec = codec; /* Set the defaults, and power up the codec */ + twl4030->sysclk = twl4030_codec_get_mclk() / 1000; twl4030_init_chip(codec); codec->bias_level = SND_SOC_BIAS_OFF; twl4030_set_bias_level(codec, SND_SOC_BIAS_STANDBY); -- 1.6.5.2 ^ permalink raw reply related [flat|nested] 21+ messages in thread
* Re: [alsa-devel] [PATCH 2/4 v2] OMAP: Configure audio_mclk for twl4030-codec MFD 2009-11-04 7:58 ` Peter Ujfalusi @ 2009-11-09 7:46 ` Peter Ujfalusi -1 siblings, 0 replies; 21+ messages in thread From: Peter Ujfalusi @ 2009-11-09 7:46 UTC (permalink / raw) To: alsa-devel, tony; +Cc: linux-omap, sameo, broonie, linux-kernel Hello Tony, On Wednesday 04 November 2009 09:58:18 Ujfalusi Peter (Nokia-D/Tampere) wrote: > audio_mclk value is going to be handled by the > twl4030-codec MFD driver, configure the correct > value for boards, which is using the twl4030 audio. > > Signed-off-by: Peter Ujfalusi <peter.ujfalusi@nokia.com> > --- > arch/arm/mach-omap2/board-3430sdp.c | 1 + > arch/arm/mach-omap2/board-omap3beagle.c | 1 + > arch/arm/mach-omap2/board-omap3evm.c | 1 + > arch/arm/mach-omap2/board-omap3pandora.c | 1 + > arch/arm/mach-omap2/board-overo.c | 1 + > arch/arm/mach-omap2/board-zoom2.c | 1 + > 6 files changed, 6 insertions(+), 0 deletions(-) > > diff --git a/arch/arm/mach-omap2/board-3430sdp.c > b/arch/arm/mach-omap2/board-3430sdp.c index 4f91f7a..9afd957 100644 > --- a/arch/arm/mach-omap2/board-3430sdp.c > +++ b/arch/arm/mach-omap2/board-3430sdp.c > @@ -415,6 +415,7 @@ static struct twl4030_codec_audio_data sdp3430_audio = > { }; > > static struct twl4030_codec_data sdp3430_codec = { > + .audio_mclk = 26000000, > .audio = &sdp3430_audio, > }; > > diff --git a/arch/arm/mach-omap2/board-omap3beagle.c > b/arch/arm/mach-omap2/board-omap3beagle.c index 2161d85..8f0c106 100644 > --- a/arch/arm/mach-omap2/board-omap3beagle.c > +++ b/arch/arm/mach-omap2/board-omap3beagle.c > @@ -259,6 +259,7 @@ static struct twl4030_codec_audio_data > beagle_audio_data = { }; > > static struct twl4030_codec_data beagle_codec_data = { > + .audio_mclk = 26000000, > .audio = &beagle_audio_data, > }; > > diff --git a/arch/arm/mach-omap2/board-omap3evm.c > b/arch/arm/mach-omap2/board-omap3evm.c index d9a6103..5bb30cb 100644 > --- a/arch/arm/mach-omap2/board-omap3evm.c > +++ b/arch/arm/mach-omap2/board-omap3evm.c > @@ -199,6 +199,7 @@ static struct twl4030_codec_audio_data > omap3evm_audio_data = { }; > > static struct twl4030_codec_data omap3evm_codec_data = { > + .audio_mclk = 26000000, > .audio = &omap3evm_audio_data, > }; > > diff --git a/arch/arm/mach-omap2/board-omap3pandora.c > b/arch/arm/mach-omap2/board-omap3pandora.c index 5036b56..77790ee 100644 > --- a/arch/arm/mach-omap2/board-omap3pandora.c > +++ b/arch/arm/mach-omap2/board-omap3pandora.c > @@ -286,6 +286,7 @@ static struct twl4030_codec_audio_data > omap3pandora_audio_data = { }; > > static struct twl4030_codec_data omap3pandora_codec_data = { > + .audio_mclk = 26000000, > .audio = &omap3pandora_audio_data, > }; > > diff --git a/arch/arm/mach-omap2/board-overo.c > b/arch/arm/mach-omap2/board-overo.c index dc55008..e1fb504 100644 > --- a/arch/arm/mach-omap2/board-overo.c > +++ b/arch/arm/mach-omap2/board-overo.c > @@ -334,6 +334,7 @@ static struct twl4030_codec_audio_data overo_audio_data > = { }; > > static struct twl4030_codec_data overo_codec_data = { > + .audio_mclk = 26000000, > .audio = &overo_audio_data, > }; > > diff --git a/arch/arm/mach-omap2/board-zoom2.c > b/arch/arm/mach-omap2/board-zoom2.c index f1b4e7c..de3a38d 100644 > --- a/arch/arm/mach-omap2/board-zoom2.c > +++ b/arch/arm/mach-omap2/board-zoom2.c > @@ -234,6 +234,7 @@ static struct twl4030_codec_audio_data zoom2_audio_data > = { }; > > static struct twl4030_codec_data zoom2_codec_data = { > + .audio_mclk = 26000000, > .audio = &zoom2_audio_data, > }; > > -- > 1.6.5.2 Can you take a look at this patch? The rest of the series has already got Ack from Samuel and Mark, and only your verdict for the OMAP part is missing. Thank you, Péter ^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH 2/4 v2] OMAP: Configure audio_mclk for twl4030-codec MFD @ 2009-11-09 7:46 ` Peter Ujfalusi 0 siblings, 0 replies; 21+ messages in thread From: Peter Ujfalusi @ 2009-11-09 7:46 UTC (permalink / raw) To: alsa-devel, tony; +Cc: linux-kernel, linux-omap, sameo, broonie Hello Tony, On Wednesday 04 November 2009 09:58:18 Ujfalusi Peter (Nokia-D/Tampere) wrote: > audio_mclk value is going to be handled by the > twl4030-codec MFD driver, configure the correct > value for boards, which is using the twl4030 audio. > > Signed-off-by: Peter Ujfalusi <peter.ujfalusi@nokia.com> > --- > arch/arm/mach-omap2/board-3430sdp.c | 1 + > arch/arm/mach-omap2/board-omap3beagle.c | 1 + > arch/arm/mach-omap2/board-omap3evm.c | 1 + > arch/arm/mach-omap2/board-omap3pandora.c | 1 + > arch/arm/mach-omap2/board-overo.c | 1 + > arch/arm/mach-omap2/board-zoom2.c | 1 + > 6 files changed, 6 insertions(+), 0 deletions(-) > > diff --git a/arch/arm/mach-omap2/board-3430sdp.c > b/arch/arm/mach-omap2/board-3430sdp.c index 4f91f7a..9afd957 100644 > --- a/arch/arm/mach-omap2/board-3430sdp.c > +++ b/arch/arm/mach-omap2/board-3430sdp.c > @@ -415,6 +415,7 @@ static struct twl4030_codec_audio_data sdp3430_audio = > { }; > > static struct twl4030_codec_data sdp3430_codec = { > + .audio_mclk = 26000000, > .audio = &sdp3430_audio, > }; > > diff --git a/arch/arm/mach-omap2/board-omap3beagle.c > b/arch/arm/mach-omap2/board-omap3beagle.c index 2161d85..8f0c106 100644 > --- a/arch/arm/mach-omap2/board-omap3beagle.c > +++ b/arch/arm/mach-omap2/board-omap3beagle.c > @@ -259,6 +259,7 @@ static struct twl4030_codec_audio_data > beagle_audio_data = { }; > > static struct twl4030_codec_data beagle_codec_data = { > + .audio_mclk = 26000000, > .audio = &beagle_audio_data, > }; > > diff --git a/arch/arm/mach-omap2/board-omap3evm.c > b/arch/arm/mach-omap2/board-omap3evm.c index d9a6103..5bb30cb 100644 > --- a/arch/arm/mach-omap2/board-omap3evm.c > +++ b/arch/arm/mach-omap2/board-omap3evm.c > @@ -199,6 +199,7 @@ static struct twl4030_codec_audio_data > omap3evm_audio_data = { }; > > static struct twl4030_codec_data omap3evm_codec_data = { > + .audio_mclk = 26000000, > .audio = &omap3evm_audio_data, > }; > > diff --git a/arch/arm/mach-omap2/board-omap3pandora.c > b/arch/arm/mach-omap2/board-omap3pandora.c index 5036b56..77790ee 100644 > --- a/arch/arm/mach-omap2/board-omap3pandora.c > +++ b/arch/arm/mach-omap2/board-omap3pandora.c > @@ -286,6 +286,7 @@ static struct twl4030_codec_audio_data > omap3pandora_audio_data = { }; > > static struct twl4030_codec_data omap3pandora_codec_data = { > + .audio_mclk = 26000000, > .audio = &omap3pandora_audio_data, > }; > > diff --git a/arch/arm/mach-omap2/board-overo.c > b/arch/arm/mach-omap2/board-overo.c index dc55008..e1fb504 100644 > --- a/arch/arm/mach-omap2/board-overo.c > +++ b/arch/arm/mach-omap2/board-overo.c > @@ -334,6 +334,7 @@ static struct twl4030_codec_audio_data overo_audio_data > = { }; > > static struct twl4030_codec_data overo_codec_data = { > + .audio_mclk = 26000000, > .audio = &overo_audio_data, > }; > > diff --git a/arch/arm/mach-omap2/board-zoom2.c > b/arch/arm/mach-omap2/board-zoom2.c index f1b4e7c..de3a38d 100644 > --- a/arch/arm/mach-omap2/board-zoom2.c > +++ b/arch/arm/mach-omap2/board-zoom2.c > @@ -234,6 +234,7 @@ static struct twl4030_codec_audio_data zoom2_audio_data > = { }; > > static struct twl4030_codec_data zoom2_codec_data = { > + .audio_mclk = 26000000, > .audio = &zoom2_audio_data, > }; > > -- > 1.6.5.2 Can you take a look at this patch? The rest of the series has already got Ack from Samuel and Mark, and only your verdict for the OMAP part is missing. Thank you, Péter ^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [alsa-devel] [PATCH 2/4 v2] OMAP: Configure audio_mclk for twl4030-codec MFD 2009-11-09 7:46 ` Peter Ujfalusi @ 2009-11-09 22:01 ` Tony Lindgren -1 siblings, 0 replies; 21+ messages in thread From: Tony Lindgren @ 2009-11-09 22:01 UTC (permalink / raw) To: Peter Ujfalusi; +Cc: alsa-devel, linux-omap, sameo, broonie, linux-kernel Hi, * Peter Ujfalusi <peter.ujfalusi@nokia.com> [091108 23:46]: > Hello Tony, > > On Wednesday 04 November 2009 09:58:18 Ujfalusi Peter (Nokia-D/Tampere) wrote: > > audio_mclk value is going to be handled by the > > twl4030-codec MFD driver, configure the correct > > value for boards, which is using the twl4030 audio. > > > > Signed-off-by: Peter Ujfalusi <peter.ujfalusi@nokia.com> > > --- > > arch/arm/mach-omap2/board-3430sdp.c | 1 + > > arch/arm/mach-omap2/board-omap3beagle.c | 1 + > > arch/arm/mach-omap2/board-omap3evm.c | 1 + > > arch/arm/mach-omap2/board-omap3pandora.c | 1 + > > arch/arm/mach-omap2/board-overo.c | 1 + > > arch/arm/mach-omap2/board-zoom2.c | 1 + > > 6 files changed, 6 insertions(+), 0 deletions(-) > > > > diff --git a/arch/arm/mach-omap2/board-3430sdp.c > > b/arch/arm/mach-omap2/board-3430sdp.c index 4f91f7a..9afd957 100644 > > --- a/arch/arm/mach-omap2/board-3430sdp.c > > +++ b/arch/arm/mach-omap2/board-3430sdp.c > > @@ -415,6 +415,7 @@ static struct twl4030_codec_audio_data sdp3430_audio = > > { }; > > > > static struct twl4030_codec_data sdp3430_codec = { > > + .audio_mclk = 26000000, > > .audio = &sdp3430_audio, > > }; > > <snip> > Can you take a look at this patch? > The rest of the series has already got Ack from Samuel and Mark, and only your > verdict for the OMAP part is missing. Looks good to me. It seems like it's best to queue this along with the related ASoC patches. Acked-by: Tony Lindgren <tony@atomide.com> ^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH 2/4 v2] OMAP: Configure audio_mclk for twl4030-codec MFD @ 2009-11-09 22:01 ` Tony Lindgren 0 siblings, 0 replies; 21+ messages in thread From: Tony Lindgren @ 2009-11-09 22:01 UTC (permalink / raw) To: Peter Ujfalusi; +Cc: linux-kernel, alsa-devel, linux-omap, sameo, broonie Hi, * Peter Ujfalusi <peter.ujfalusi@nokia.com> [091108 23:46]: > Hello Tony, > > On Wednesday 04 November 2009 09:58:18 Ujfalusi Peter (Nokia-D/Tampere) wrote: > > audio_mclk value is going to be handled by the > > twl4030-codec MFD driver, configure the correct > > value for boards, which is using the twl4030 audio. > > > > Signed-off-by: Peter Ujfalusi <peter.ujfalusi@nokia.com> > > --- > > arch/arm/mach-omap2/board-3430sdp.c | 1 + > > arch/arm/mach-omap2/board-omap3beagle.c | 1 + > > arch/arm/mach-omap2/board-omap3evm.c | 1 + > > arch/arm/mach-omap2/board-omap3pandora.c | 1 + > > arch/arm/mach-omap2/board-overo.c | 1 + > > arch/arm/mach-omap2/board-zoom2.c | 1 + > > 6 files changed, 6 insertions(+), 0 deletions(-) > > > > diff --git a/arch/arm/mach-omap2/board-3430sdp.c > > b/arch/arm/mach-omap2/board-3430sdp.c index 4f91f7a..9afd957 100644 > > --- a/arch/arm/mach-omap2/board-3430sdp.c > > +++ b/arch/arm/mach-omap2/board-3430sdp.c > > @@ -415,6 +415,7 @@ static struct twl4030_codec_audio_data sdp3430_audio = > > { }; > > > > static struct twl4030_codec_data sdp3430_codec = { > > + .audio_mclk = 26000000, > > .audio = &sdp3430_audio, > > }; > > <snip> > Can you take a look at this patch? > The rest of the series has already got Ack from Samuel and Mark, and only your > verdict for the OMAP part is missing. Looks good to me. It seems like it's best to queue this along with the related ASoC patches. Acked-by: Tony Lindgren <tony@atomide.com> ^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH 1/4 v2] MFD: TWL4030: Add audio_mclk to the codec platform data 2009-11-04 7:58 ` Peter Ujfalusi @ 2009-11-10 12:08 ` Mark Brown -1 siblings, 0 replies; 21+ messages in thread From: Mark Brown @ 2009-11-10 12:08 UTC (permalink / raw) To: Peter Ujfalusi; +Cc: alsa-devel, linux-omap, sameo, linux-kernel, tony On Wed, Nov 04, 2009 at 09:58:17AM +0200, Peter Ujfalusi wrote: > Add audio_mclk to the platform data struct for the > twl4030-codec MFD driver. > > Signed-off-by: Peter Ujfalusi <peter.ujfalusi@nokia.com> > Acked-by: Samuel Ortiz <sameo@linux.intel.com> Applied all, thanks. ^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH 1/4 v2] MFD: TWL4030: Add audio_mclk to the codec platform data @ 2009-11-10 12:08 ` Mark Brown 0 siblings, 0 replies; 21+ messages in thread From: Mark Brown @ 2009-11-10 12:08 UTC (permalink / raw) To: Peter Ujfalusi; +Cc: tony, alsa-devel, linux-omap, sameo, linux-kernel On Wed, Nov 04, 2009 at 09:58:17AM +0200, Peter Ujfalusi wrote: > Add audio_mclk to the platform data struct for the > twl4030-codec MFD driver. > > Signed-off-by: Peter Ujfalusi <peter.ujfalusi@nokia.com> > Acked-by: Samuel Ortiz <sameo@linux.intel.com> Applied all, thanks. ^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH 0/4 v2] ASoC/MFD/OMAP: TWL4030: APLL_CTL handling change 2009-11-04 7:58 ` Peter Ujfalusi @ 2009-11-04 10:20 ` Mark Brown -1 siblings, 0 replies; 21+ messages in thread From: Mark Brown @ 2009-11-04 10:20 UTC (permalink / raw) To: Peter Ujfalusi; +Cc: alsa-devel, linux-omap, sameo, linux-kernel, tony On Wed, Nov 04, 2009 at 09:58:16AM +0200, Peter Ujfalusi wrote: > The following series changes the way how the APLL_CTL register is handled by > the twl4030 codec related drivers. I'm OK with this but I'll wait for acks from Tony and Samuel before applying. ^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH 0/4 v2] ASoC/MFD/OMAP: TWL4030: APLL_CTL handling change @ 2009-11-04 10:20 ` Mark Brown 0 siblings, 0 replies; 21+ messages in thread From: Mark Brown @ 2009-11-04 10:20 UTC (permalink / raw) To: Peter Ujfalusi; +Cc: tony, alsa-devel, linux-omap, sameo, linux-kernel On Wed, Nov 04, 2009 at 09:58:16AM +0200, Peter Ujfalusi wrote: > The following series changes the way how the APLL_CTL register is handled by > the twl4030 codec related drivers. I'm OK with this but I'll wait for acks from Tony and Samuel before applying. ^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH 0/4 v2] ASoC/MFD/OMAP: TWL4030: APLL_CTL handling change 2009-11-04 10:20 ` Mark Brown (?) @ 2009-11-04 16:36 ` Samuel Ortiz -1 siblings, 0 replies; 21+ messages in thread From: Samuel Ortiz @ 2009-11-04 16:36 UTC (permalink / raw) To: Mark Brown; +Cc: Peter Ujfalusi, alsa-devel, linux-omap, linux-kernel, tony Hi Mark, On Wed, Nov 04, 2009 at 10:20:41AM +0000, Mark Brown wrote: > On Wed, Nov 04, 2009 at 09:58:16AM +0200, Peter Ujfalusi wrote: > > > The following series changes the way how the APLL_CTL register is handled by > > the twl4030 codec related drivers. > > I'm OK with this but I'll wait for acks from Tony and Samuel before > applying. You can add my Acked-by: Samuel Ortiz <sameo@linux.intel.com> to the 2 MFD patches. Peter: thanks for the patches, and Mark: thanks for taking them :) Cheers, Samuel. -- Intel Open Source Technology Centre http://oss.intel.com/ ^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH 0/4 v2] ASoC/MFD/OMAP: TWL4030: APLL_CTL handling change 2009-11-04 10:20 ` Mark Brown @ 2009-11-09 22:02 ` Tony Lindgren -1 siblings, 0 replies; 21+ messages in thread From: Tony Lindgren @ 2009-11-09 22:02 UTC (permalink / raw) To: Mark Brown; +Cc: Peter Ujfalusi, alsa-devel, linux-omap, sameo, linux-kernel * Mark Brown <broonie@opensource.wolfsonmicro.com> [091104 02:20]: > On Wed, Nov 04, 2009 at 09:58:16AM +0200, Peter Ujfalusi wrote: > > > The following series changes the way how the APLL_CTL register is handled by > > the twl4030 codec related drivers. > > I'm OK with this but I'll wait for acks from Tony and Samuel before > applying. Looks good to me, replied with my ack in the board-*.c files patch in this series. Regards, Tony ^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH 0/4 v2] ASoC/MFD/OMAP: TWL4030: APLL_CTL handling change @ 2009-11-09 22:02 ` Tony Lindgren 0 siblings, 0 replies; 21+ messages in thread From: Tony Lindgren @ 2009-11-09 22:02 UTC (permalink / raw) To: Mark Brown; +Cc: alsa-devel, linux-omap, Peter Ujfalusi, sameo, linux-kernel * Mark Brown <broonie@opensource.wolfsonmicro.com> [091104 02:20]: > On Wed, Nov 04, 2009 at 09:58:16AM +0200, Peter Ujfalusi wrote: > > > The following series changes the way how the APLL_CTL register is handled by > > the twl4030 codec related drivers. > > I'm OK with this but I'll wait for acks from Tony and Samuel before > applying. Looks good to me, replied with my ack in the board-*.c files patch in this series. Regards, Tony ^ permalink raw reply [flat|nested] 21+ messages in thread
end of thread, other threads:[~2009-11-10 12:08 UTC | newest] Thread overview: 21+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2009-11-04 7:58 [PATCH 0/4 v2] ASoC/MFD/OMAP: TWL4030: APLL_CTL handling change Peter Ujfalusi 2009-11-04 7:58 ` Peter Ujfalusi 2009-11-04 7:58 ` [PATCH 1/4 v2] MFD: TWL4030: Add audio_mclk to the codec platform data Peter Ujfalusi 2009-11-04 7:58 ` Peter Ujfalusi 2009-11-04 7:58 ` [PATCH 2/4 v2] OMAP: Configure audio_mclk for twl4030-codec MFD Peter Ujfalusi 2009-11-04 7:58 ` Peter Ujfalusi 2009-11-04 7:58 ` [PATCH 3/4 v2] MFD: twl4030-codec: APLL_INFREQ handling in the MFD driver Peter Ujfalusi 2009-11-04 7:58 ` Peter Ujfalusi 2009-11-04 7:58 ` [PATCH 4/4 v2] ASoC: TWL4030: Do not modify the APLL_CTL register Peter Ujfalusi 2009-11-04 7:58 ` Peter Ujfalusi 2009-11-09 7:46 ` [alsa-devel] [PATCH 2/4 v2] OMAP: Configure audio_mclk for twl4030-codec MFD Peter Ujfalusi 2009-11-09 7:46 ` Peter Ujfalusi 2009-11-09 22:01 ` [alsa-devel] " Tony Lindgren 2009-11-09 22:01 ` Tony Lindgren 2009-11-10 12:08 ` [PATCH 1/4 v2] MFD: TWL4030: Add audio_mclk to the codec platform data Mark Brown 2009-11-10 12:08 ` Mark Brown 2009-11-04 10:20 ` [PATCH 0/4 v2] ASoC/MFD/OMAP: TWL4030: APLL_CTL handling change Mark Brown 2009-11-04 10:20 ` Mark Brown 2009-11-04 16:36 ` Samuel Ortiz 2009-11-09 22:02 ` Tony Lindgren 2009-11-09 22:02 ` Tony Lindgren
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.