[PATCH] ASoC: simple-card: Fix property name for simple-audio-card, format

[PATCH] ASoC: simple-card: Fix property name for simple-audio-card, format

[Peter Ujfalusi]

The "format" part of the property name is missing from the code.

Signed-off-by: Peter Ujfalusi <peter.ujfalusi@ti.com>
---
 sound/soc/generic/simple-card.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sound/soc/generic/simple-card.c b/sound/soc/generic/simple-card.c
index 7cabcc5c8703..064b404ed252 100644
--- a/sound/soc/generic/simple-card.c
+++ b/sound/soc/generic/simple-card.c
@@ -165,7 +165,7 @@ static int asoc_simple_card_parse_of(struct device_node *node,
 	snd_soc_of_parse_card_name(&priv->snd_card, "simple-audio-card,name");
 
 	/* get CPU/CODEC common format via simple-audio-card,format */
-	priv->daifmt = snd_soc_of_parse_daifmt(node, "simple-audio-card,") &
+	priv->daifmt = snd_soc_of_parse_daifmt(node, "simple-audio-card,format") &
 		(SND_SOC_DAIFMT_FORMAT_MASK | SND_SOC_DAIFMT_INV_MASK);
 
 	/* off-codec widgets */
-- 
1.9.0

Re: [PATCH] ASoC: simple-card: Fix property name for simple-audio-card, format

[Mark Brown]

[-- Attachment #1.1: Type: text/plain, Size: 149 bytes --]

On Fri, Mar 07, 2014 at 02:18:31PM +0200, Peter Ujfalusi wrote:
> The "format" part of the property name is missing from the code.

Applied, thanks.

[-- Attachment #1.2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

[-- Attachment #2: Type: text/plain, Size: 0 bytes --]

Re: [PATCH] ASoC: simple-card: Fix property name for simple-audio-card, format

[Li.Xiubo]

> --- a/sound/soc/generic/simple-card.c
> +++ b/sound/soc/generic/simple-card.c
> @@ -165,7 +165,7 @@ static int asoc_simple_card_parse_of(struct device_node
> *node,
>  	snd_soc_of_parse_card_name(&priv->snd_card, "simple-audio-card,name");
> 
>  	/* get CPU/CODEC common format via simple-audio-card,format */
> -	priv->daifmt = snd_soc_of_parse_daifmt(node, "simple-audio-card,") &
> +	priv->daifmt = snd_soc_of_parse_daifmt(node, "simple-audio-card,format")


@Peter

Sorry, the interface of snd_soc_of_parse_daifmt(struct device_node *np,
const char *prefix), it's intended not only to parse the 'XXX,format', but also
'XXX,continuous-clock', 'XXX, bitclock-inversion', 'XXX, frame-inversion',
'XXX,bitclock-master' and 'XXX,frame-master' at the same time.

Maybe just the comment here is not very correct.

Thanks,

--
Best Regards,
Xiubo


> &
>  		(SND_SOC_DAIFMT_FORMAT_MASK | SND_SOC_DAIFMT_INV_MASK);
> 
>  	/* off-codec widgets */
> --
> 1.9.0
> 
> 

Re: [PATCH] ASoC: simple-card: Fix property name for simple-audio-card, format

[Mark Brown]

[-- Attachment #1.1: Type: text/plain, Size: 480 bytes --]

On Mon, Mar 10, 2014 at 02:25:02AM +0000, Li.Xiubo@freescale.com wrote:

> Sorry, the interface of snd_soc_of_parse_daifmt(struct device_node *np,
> const char *prefix), it's intended not only to parse the 'XXX,format', but also
> 'XXX,continuous-clock', 'XXX, bitclock-inversion', 'XXX, frame-inversion',
> 'XXX,bitclock-master' and 'XXX,frame-master' at the same time.

> Maybe just the comment here is not very correct.

Yes, the comment needs fixing.  I've dropped the patch.

[-- Attachment #1.2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

[-- Attachment #2: Type: text/plain, Size: 0 bytes --]

Re: [PATCH] ASoC: simple-card: Fix property name for simple-audio-card, format

[Peter Ujfalusi]

On 03/10/2014 04:25 AM, Li.Xiubo@freescale.com wrote:
>> --- a/sound/soc/generic/simple-card.c
>> +++ b/sound/soc/generic/simple-card.c
>> @@ -165,7 +165,7 @@ static int asoc_simple_card_parse_of(struct device_node
>> *node,
>>  	snd_soc_of_parse_card_name(&priv->snd_card, "simple-audio-card,name");
>>
>>  	/* get CPU/CODEC common format via simple-audio-card,format */
>> -	priv->daifmt = snd_soc_of_parse_daifmt(node, "simple-audio-card,") &
>> +	priv->daifmt = snd_soc_of_parse_daifmt(node, "simple-audio-card,format")
> 
> 
> @Peter
> 
> Sorry, the interface of snd_soc_of_parse_daifmt(struct device_node *np,
> const char *prefix), it's intended not only to parse the 'XXX,format', but also
> 'XXX,continuous-clock', 'XXX, bitclock-inversion', 'XXX, frame-inversion',
> 'XXX,bitclock-master' and 'XXX,frame-master' at the same time.

Yes, I overlooked that. I have not checked the snd_soc_of_parse_daifmt() before.

> Maybe just the comment here is not very correct.

Probably it would be better to re-word it since the code at this point only
interested on the i2s coding and the bit/frame inversion.

> Thanks,
> 
> --
> Best Regards,
> Xiubo
> 
> 
>> &
>>  		(SND_SOC_DAIFMT_FORMAT_MASK | SND_SOC_DAIFMT_INV_MASK);
>>
>>  	/* off-codec widgets */
>> --
>> 1.9.0
>>
>>
> 


-- 
Péter