Re: [PATCH 16/17] ASoC: Basic split of mpc5200 DMA code out from mpc5200_psc_i2s

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

 



At Tue, 28 Apr 2009 16:23:41 +0100,
Mark Brown wrote:
> 
> From: Jon Smirl <jonsmirl@xxxxxxxxx>
> 
> Basic split of mpc5200 DMA code out from i2s into a standalone file.
> 
> [Fixed trailing whitespace -- broonie.]
> 
> Signed-off-by: Jon Smirl <jonsmirl@xxxxxxxxx>
> Acked-by: Grant Likely <grant.likely@xxxxxxxxxxxx>
> Signed-off-by: Mark Brown <broonie@xxxxxxxxxxxxxxxxxxxxxxxxxxx>

This patch conflicts with the last patch I merged to add
SNDRV_PCM_INFO_BATCH to some drivers including mpc5200.
(It's in fix/pcm-info-batch branch and not merged to */asoc branch
 yet.  Feel free to pull to your tree by yourself.)
And, the jiffies check shall be anyway fixed in the upstream PCM core
side, thus there is no big merit to hurry to merge the unfinished
changes now.

So, could you back off mpc5200 changes and wait for a while until the
issue regarding PCM jiffies-check is resolved?

Note that the info flag should contain SNDRV_PCM_INFO_BATCH unless the
driver provides the precise position reporting in the pointer
callback.  The flag can be removed once after you implement that
feature, of course.


thanks,

Takashi
_______________________________________________
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