在 2010年6月7日 下午6:41，Nicolas Ferre <nicolas.ferre@xxxxxxxxx> 写道：
> Le 28/05/2010 04:27, Wan ZongShun :
>> Dear Nicolas,
>>
>> I am seeing the atmel codes as reference of my alsa driver programming.
>> Finding a few code issues, I make a patch to fix, thanks!
>
> After a brief review, I come back to this
>
>> Signed-off-by: Wan ZongShun<mcuos.com@xxxxxxxxx>
>>
>> ---
>> sound/soc/atmel/atmel-pcm.c | 2 +-
>> sound/soc/atmel/atmel_ssc_dai.c | 3 +--
>> 2 files changed, 2 insertions(+), 3 deletions(-)
>>
>> diff --git a/sound/soc/atmel/atmel-pcm.c b/sound/soc/atmel/atmel-pcm.c
>> index f6b3cc0..c69a8fb 100644
>> --- a/sound/soc/atmel/atmel-pcm.c
>> +++ b/sound/soc/atmel/atmel-pcm.c
>> @@ -478,7 +478,7 @@ struct snd_soc_platform atmel_soc_platform = {
>> .pcm_free = atmel_pcm_free_dma_buffers,
>> .suspend = atmel_pcm_suspend,
>> .resume = atmel_pcm_resume,
>> -};
>> +}
>
> Not ok for this. It is a structure declaration: we should keep the ";".
>
Okay, thanks!
>> EXPORT_SYMBOL_GPL(atmel_soc_platform);
>>
>> static int __init atmel_pcm_modinit(void)
>> diff --git a/sound/soc/atmel/atmel_ssc_dai.c b/sound/soc/atmel/atmel_ssc_dai.c
>> index 0b59806..4c44d13 100644
>> --- a/sound/soc/atmel/atmel_ssc_dai.c
>> +++ b/sound/soc/atmel/atmel_ssc_dai.c
>> @@ -549,7 +549,6 @@ static int atmel_ssc_hw_params(struct snd_pcm_substream
>> *substream,
>> printk(KERN_WARNING "atmel_ssc_dai: unsupported DAI format 0x%x\n",
>> ssc_p->daifmt);
>> return -EINVAL;
>> - break;
>
> Ok.
>
>> }
>> pr_debug("atmel_ssc_hw_params: "
>> "RCMR=%08x RFMR=%08x TCMR=%08x TFMR=%08x\n",
>> @@ -760,7 +759,7 @@ struct snd_soc_dai atmel_ssc_dai[NUM_SSC_DEVICES] = {
>> .private_data = &ssc_info[2],
>> },
>> #endif
>> -};
>> +}
>
> Not ok for this either. ditto.
>
>> EXPORT_SYMBOL_GPL(atmel_ssc_dai);
>>
>> static int __init atmel_ssc_modinit(void)
>
>
> --
> Nicolas Ferre
>
>
--
*linux-arm-kernel mailing list
mail addr:linux-arm-kernel@xxxxxxxxxxxxxxxxxxx
you can subscribe by:
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
* linux-arm-NUC900 mailing list
mail addr:NUC900@xxxxxxxxxxxxxxxx
main web: https://groups.google.com/group/NUC900
you can subscribe it by sending me mail:
mcuos.com@xxxxxxxxx
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
