* [PATCH] davinci: da8xx/omap-l1xx: match codec_name with i2c ids
@ 2011-01-21 14:24 Rajashekhara, Sudhakar
2011-01-21 14:38 ` Liam Girdwood
0 siblings, 1 reply; 7+ messages in thread
From: Rajashekhara, Sudhakar @ 2011-01-21 14:24 UTC (permalink / raw)
To: linux-arm-kernel
The codec_name entry for da8xx evm in sound/soc/davinci/davinci-evm.c
is not matching with the i2c ids in the board file. Without this fix the
soundcard does not get detected on da850/omap-l138/am18x evm.
Signed-off-by: Rajashekhara, Sudhakar <sudhakar.raj@ti.com>
Tested-by: Dan Sharon <dansharon@nanometrics.ca>
---
This patch applies to Linus's kernel tree.
sound/soc/davinci/davinci-evm.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/sound/soc/davinci/davinci-evm.c b/sound/soc/davinci/davinci-evm.c
index 0c2d6ba..b36f0b3 100644
--- a/sound/soc/davinci/davinci-evm.c
+++ b/sound/soc/davinci/davinci-evm.c
@@ -223,7 +223,7 @@ static struct snd_soc_dai_link da8xx_evm_dai = {
.stream_name = "AIC3X",
.cpu_dai_name= "davinci-mcasp.0",
.codec_dai_name = "tlv320aic3x-hifi",
- .codec_name = "tlv320aic3x-codec.0-001a",
+ .codec_name = "tlv320aic3x-codec.1-0018",
.platform_name = "davinci-pcm-audio",
.init = evm_aic3x_init,
.ops = &evm_ops,
--
1.7.1
^ permalink raw reply related [flat|nested] 7+ messages in thread* [PATCH] davinci: da8xx/omap-l1xx: match codec_name with i2c ids
2011-01-21 14:24 [PATCH] davinci: da8xx/omap-l1xx: match codec_name with i2c ids Rajashekhara, Sudhakar
@ 2011-01-21 14:38 ` Liam Girdwood
2011-01-24 23:00 ` Kevin Hilman
0 siblings, 1 reply; 7+ messages in thread
From: Liam Girdwood @ 2011-01-21 14:38 UTC (permalink / raw)
To: linux-arm-kernel
On Fri, 2011-01-21 at 19:54 +0530, Rajashekhara, Sudhakar wrote:
> The codec_name entry for da8xx evm in sound/soc/davinci/davinci-evm.c
> is not matching with the i2c ids in the board file. Without this fix the
> soundcard does not get detected on da850/omap-l138/am18x evm.
>
> Signed-off-by: Rajashekhara, Sudhakar <sudhakar.raj@ti.com>
> Tested-by: Dan Sharon <dansharon@nanometrics.ca>
> ---
> This patch applies to Linus's kernel tree.
>
> sound/soc/davinci/davinci-evm.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/sound/soc/davinci/davinci-evm.c b/sound/soc/davinci/davinci-evm.c
> index 0c2d6ba..b36f0b3 100644
> --- a/sound/soc/davinci/davinci-evm.c
> +++ b/sound/soc/davinci/davinci-evm.c
> @@ -223,7 +223,7 @@ static struct snd_soc_dai_link da8xx_evm_dai = {
> .stream_name = "AIC3X",
> .cpu_dai_name= "davinci-mcasp.0",
> .codec_dai_name = "tlv320aic3x-hifi",
> - .codec_name = "tlv320aic3x-codec.0-001a",
> + .codec_name = "tlv320aic3x-codec.1-0018",
> .platform_name = "davinci-pcm-audio",
> .init = evm_aic3x_init,
> .ops = &evm_ops,
Acked-by: Liam Girdwood <lrg@slimlogic.co.uk>
--
Freelance Developer, SlimLogic Ltd
ASoC and Voltage Regulator Maintainer.
http://www.slimlogic.co.uk
^ permalink raw reply [flat|nested] 7+ messages in thread* [PATCH] davinci: da8xx/omap-l1xx: match codec_name with i2c ids
2011-01-21 14:38 ` Liam Girdwood
@ 2011-01-24 23:00 ` Kevin Hilman
2011-01-24 23:09 ` Kevin Hilman
0 siblings, 1 reply; 7+ messages in thread
From: Kevin Hilman @ 2011-01-24 23:00 UTC (permalink / raw)
To: linux-arm-kernel
Liam Girdwood <lrg@slimlogic.co.uk> writes:
> On Fri, 2011-01-21 at 19:54 +0530, Rajashekhara, Sudhakar wrote:
>> The codec_name entry for da8xx evm in sound/soc/davinci/davinci-evm.c
>> is not matching with the i2c ids in the board file. Without this fix the
>> soundcard does not get detected on da850/omap-l138/am18x evm.
>>
>> Signed-off-by: Rajashekhara, Sudhakar <sudhakar.raj@ti.com>
>> Tested-by: Dan Sharon <dansharon@nanometrics.ca>
>> ---
>> This patch applies to Linus's kernel tree.
>>
>> sound/soc/davinci/davinci-evm.c | 2 +-
>> 1 files changed, 1 insertions(+), 1 deletions(-)
>>
>> diff --git a/sound/soc/davinci/davinci-evm.c b/sound/soc/davinci/davinci-evm.c
>> index 0c2d6ba..b36f0b3 100644
>> --- a/sound/soc/davinci/davinci-evm.c
>> +++ b/sound/soc/davinci/davinci-evm.c
>> @@ -223,7 +223,7 @@ static struct snd_soc_dai_link da8xx_evm_dai = {
>> .stream_name = "AIC3X",
>> .cpu_dai_name= "davinci-mcasp.0",
>> .codec_dai_name = "tlv320aic3x-hifi",
>> - .codec_name = "tlv320aic3x-codec.0-001a",
>> + .codec_name = "tlv320aic3x-codec.1-0018",
>> .platform_name = "davinci-pcm-audio",
>> .init = evm_aic3x_init,
>> .ops = &evm_ops,
>
> Acked-by: Liam Girdwood <lrg@slimlogic.co.uk>
Liam,
I'm assuming you'll merge this one via the ASoC tree?
Kevin
^ permalink raw reply [flat|nested] 7+ messages in thread* [PATCH] davinci: da8xx/omap-l1xx: match codec_name with i2c ids
2011-01-24 23:00 ` Kevin Hilman
@ 2011-01-24 23:09 ` Kevin Hilman
2011-01-25 10:59 ` [alsa-devel] " Liam Girdwood
0 siblings, 1 reply; 7+ messages in thread
From: Kevin Hilman @ 2011-01-24 23:09 UTC (permalink / raw)
To: linux-arm-kernel
Kevin Hilman <khilman@ti.com> writes:
> Liam Girdwood <lrg@slimlogic.co.uk> writes:
>
>> On Fri, 2011-01-21 at 19:54 +0530, Rajashekhara, Sudhakar wrote:
>>> The codec_name entry for da8xx evm in sound/soc/davinci/davinci-evm.c
>>> is not matching with the i2c ids in the board file. Without this fix the
>>> soundcard does not get detected on da850/omap-l138/am18x evm.
>>>
>>> Signed-off-by: Rajashekhara, Sudhakar <sudhakar.raj@ti.com>
>>> Tested-by: Dan Sharon <dansharon@nanometrics.ca>
>>> ---
>>> This patch applies to Linus's kernel tree.
>>>
>>> sound/soc/davinci/davinci-evm.c | 2 +-
>>> 1 files changed, 1 insertions(+), 1 deletions(-)
>>>
>>> diff --git a/sound/soc/davinci/davinci-evm.c b/sound/soc/davinci/davinci-evm.c
>>> index 0c2d6ba..b36f0b3 100644
>>> --- a/sound/soc/davinci/davinci-evm.c
>>> +++ b/sound/soc/davinci/davinci-evm.c
>>> @@ -223,7 +223,7 @@ static struct snd_soc_dai_link da8xx_evm_dai = {
>>> .stream_name = "AIC3X",
>>> .cpu_dai_name= "davinci-mcasp.0",
>>> .codec_dai_name = "tlv320aic3x-hifi",
>>> - .codec_name = "tlv320aic3x-codec.0-001a",
>>> + .codec_name = "tlv320aic3x-codec.1-0018",
>>> .platform_name = "davinci-pcm-audio",
>>> .init = evm_aic3x_init,
>>> .ops = &evm_ops,
>>
>> Acked-by: Liam Girdwood <lrg@slimlogic.co.uk>
>
> Liam,
>
> I'm assuming you'll merge this one via the ASoC tree?
>
On second thought, these should probably merge for .38-rc3.
If you're OK with it, I can merge this and the platform fix together for
.38-rc3.
Kevin
^ permalink raw reply [flat|nested] 7+ messages in thread* [alsa-devel] [PATCH] davinci: da8xx/omap-l1xx: match codec_name with i2c ids
2011-01-24 23:09 ` Kevin Hilman
@ 2011-01-25 10:59 ` Liam Girdwood
2011-01-25 12:35 ` Mark Brown
0 siblings, 1 reply; 7+ messages in thread
From: Liam Girdwood @ 2011-01-25 10:59 UTC (permalink / raw)
To: linux-arm-kernel
On Mon, 2011-01-24 at 15:09 -0800, Kevin Hilman wrote:
> Kevin Hilman <khilman@ti.com> writes:
>
> > Liam Girdwood <lrg@slimlogic.co.uk> writes:
> >
> >> On Fri, 2011-01-21 at 19:54 +0530, Rajashekhara, Sudhakar wrote:
> >>> The codec_name entry for da8xx evm in sound/soc/davinci/davinci-evm.c
> >>> is not matching with the i2c ids in the board file. Without this fix the
> >>> soundcard does not get detected on da850/omap-l138/am18x evm.
> >>>
> >>> Signed-off-by: Rajashekhara, Sudhakar <sudhakar.raj@ti.com>
> >>> Tested-by: Dan Sharon <dansharon@nanometrics.ca>
> >>> ---
> >>> This patch applies to Linus's kernel tree.
> >>>
> >>> sound/soc/davinci/davinci-evm.c | 2 +-
> >>> 1 files changed, 1 insertions(+), 1 deletions(-)
> >>>
> >>> diff --git a/sound/soc/davinci/davinci-evm.c b/sound/soc/davinci/davinci-evm.c
> >>> index 0c2d6ba..b36f0b3 100644
> >>> --- a/sound/soc/davinci/davinci-evm.c
> >>> +++ b/sound/soc/davinci/davinci-evm.c
> >>> @@ -223,7 +223,7 @@ static struct snd_soc_dai_link da8xx_evm_dai = {
> >>> .stream_name = "AIC3X",
> >>> .cpu_dai_name= "davinci-mcasp.0",
> >>> .codec_dai_name = "tlv320aic3x-hifi",
> >>> - .codec_name = "tlv320aic3x-codec.0-001a",
> >>> + .codec_name = "tlv320aic3x-codec.1-0018",
> >>> .platform_name = "davinci-pcm-audio",
> >>> .init = evm_aic3x_init,
> >>> .ops = &evm_ops,
> >>
> >> Acked-by: Liam Girdwood <lrg@slimlogic.co.uk>
> >
> > Liam,
> >
> > I'm assuming you'll merge this one via the ASoC tree?
> >
>
> On second thought, these should probably merge for .38-rc3.
>
> If you're OK with it, I can merge this and the platform fix together for
> .38-rc3.
Yes please :)
Thanks !
Liam
--
Freelance Developer, SlimLogic Ltd
ASoC and Voltage Regulator Maintainer.
http://www.slimlogic.co.uk
^ permalink raw reply [flat|nested] 7+ messages in thread* [alsa-devel] [PATCH] davinci: da8xx/omap-l1xx: match codec_name with i2c ids
2011-01-25 10:59 ` [alsa-devel] " Liam Girdwood
@ 2011-01-25 12:35 ` Mark Brown
2011-01-25 18:36 ` Kevin Hilman
0 siblings, 1 reply; 7+ messages in thread
From: Mark Brown @ 2011-01-25 12:35 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Jan 25, 2011 at 10:59:05AM +0000, Liam Girdwood wrote:
> On Mon, 2011-01-24 at 15:09 -0800, Kevin Hilman wrote:
> > On second thought, these should probably merge for .38-rc3.
> > If you're OK with it, I can merge this and the platform fix together for
> > .38-rc3.
> Yes please :)
I applied it and sent it off towards Linux a few days ago.
^ permalink raw reply [flat|nested] 7+ messages in thread
* [alsa-devel] [PATCH] davinci: da8xx/omap-l1xx: match codec_name with i2c ids
2011-01-25 12:35 ` Mark Brown
@ 2011-01-25 18:36 ` Kevin Hilman
0 siblings, 0 replies; 7+ messages in thread
From: Kevin Hilman @ 2011-01-25 18:36 UTC (permalink / raw)
To: linux-arm-kernel
Mark Brown <broonie@opensource.wolfsonmicro.com> writes:
> On Tue, Jan 25, 2011 at 10:59:05AM +0000, Liam Girdwood wrote:
>> On Mon, 2011-01-24 at 15:09 -0800, Kevin Hilman wrote:
>
>> > On second thought, these should probably merge for .38-rc3.
>
>> > If you're OK with it, I can merge this and the platform fix together for
>> > .38-rc3.
>
>> Yes please :)
>
> I applied it and sent it off towards Linux a few days ago.
OK, missed that.
I'll queue up the platform one then.
Thanks,
Kevin
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2011-01-25 18:36 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-01-21 14:24 [PATCH] davinci: da8xx/omap-l1xx: match codec_name with i2c ids Rajashekhara, Sudhakar
2011-01-21 14:38 ` Liam Girdwood
2011-01-24 23:00 ` Kevin Hilman
2011-01-24 23:09 ` Kevin Hilman
2011-01-25 10:59 ` [alsa-devel] " Liam Girdwood
2011-01-25 12:35 ` Mark Brown
2011-01-25 18:36 ` Kevin Hilman
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).