Re: [PATCH v2 1/3]AsoC/nuc900: patch for SUBSTREAM_TYPE', 'PCM_TX' and 'PCM_RX' removal

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Mark,

2010/6/2 Mark Brown <broonie@xxxxxxxxxxxxxxxxxxxxxxxxxxx>:
> On Wed, Jun 02, 2010 at 10:43:07AM +0100, Liam Girdwood wrote:
>> On Wed, 2010-06-02 at 13:54 +0800, Wan ZongShun wrote:
>> > This patch is to remove the 'SUBSTREAM_TYPE','PCM_TX' and 'PCM_RX' definition.
>> >
>> > There is no need to redefine SNDRV_PCM_STREAM_PLAYBACK as PCM_TX,
>> > the SUBSTREAM_TYPE(substream) can be deleted too, the playback or record can be
>> > judged by 'if (substream->stream == SNDRV_PCM_STREAM_PLAYBACK)' directly rather
>> > than 'if (PCM_TX == stype)', which makes the codes easy to read.
>> >
>> > Signed-off-by: Wan ZongShun<mcuos.com@xxxxxxxxx>
>
> Note here that you're still missing a space betwen your name and e-mail
> address.
>

Do I need re-sent this patch with the missing space fix?

>> All
>
>> Acked-by: Liam Girdwood <lrg@xxxxxxxxxxxxxxx>
>
> I've applied all the patches, since I had to fix up the changelogs
> anyway I updated the ms to us in the changelog for patch 2.  Thanks.
> _______________________________________________
> Alsa-devel mailing list
> Alsa-devel@xxxxxxxxxxxxxxxx
> http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
>



-- 
*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



[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux