> >> >> This patch causes that the sound can not work normally 
> on OMAP_L138.
> >> <snip>
> >> After revert it, the audio works fine.
> >> 
> >
> > This patch works fine on DM644x which has McBSP but breaks 
> audio on OMAP L138
> > (as you had mentioned) which has McASP. Ideally McBSP/McASP 
> should start after
> > EDMA is started. If not then this patch clears the EDMA 
> event which is actually
> > set by McBSP/McASP. As this patch is working fine on 
> DM644x, I think there is
> > some issue in the audio driver which needs to be debugged.
> 
> In the mean time, I think it makes sense to revert $SUBJECT patch in
> davinci git until the audio driver is debugged.

Kevin - 

What is the process or timeline for getting this patch reapplied?  I am
working right now on resubmitting my patch for the Davinci SPI driver,
however that driver won't work as is without this patch.  I could
probably hack something into the SPI driver to compensate, however if
all are agreed that drivers shouldn't rely on DMA events that occur
before the DMA is started, then it seems better to handle the issue in
the DMA driver as I did in the reverted patch.

Thanks!

Brian

_______________________________________________
Davinci-linux-open-source mailing list
[email protected]
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source

Reply via email to