Re: Driver code with mpc5200 pointer problem.

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

 



On Mon, Apr 27, 2009 at 10:05 AM, Mark Brown <broonie@xxxxxxxxxxxxx> wrote:
> On Mon, Apr 27, 2009 at 10:02:47AM -0400, Jon Smirl wrote:
>
>> Is there really a problem with sending the reorg in now? There are
>> only two users of the code and we have both ack'd it.
>
> It's a fairly hefty change for -rc4.  From the sounds of it you just
> need to add the constraint?

it is also broken because of the additional requirement of needing to
estimate the current position of the DMA transfer. The changes in
pcm_lib.c have made it non-functional.  When I went into fix those
problems I found a couple more issues.  It's ok for i2s to be broken
in this release, Grant and I both know it is broken and won't ship
anything based on it.

Here's the reorg series on linux-ppc
http://ozlabs.org/pipermail/linuxppc-dev/2009-April/071206.html
I want to do the rename of the globals that Grant suggested.

The pcm_lib.c changes may have broken other embedded drivers too.  And
yes, these other drivers are probably implemented wrong and the
changes in pcm_lib.c exposed their errors. But they will be
non-functional until someone notices and fixes them.

-- 
Jon Smirl
jonsmirl@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