2014-09-24 16:26 GMT+08:00 Mark Brown broonie@kernel.org:
On Wed, Sep 24, 2014 at 05:55:20PM +0800, Scott Jiang wrote:
Bf5xx and bf6xx reused bf5xx-i2s-pcm.c before. Recently tdm driver has been merged into bf5xx-i2s-pcm.c and it's not compatible with bf6xx audio. So create this dma driver and add tdm support later.
Can you go into a bit more detail on the differences - clearly there's some similarlities since the drivers got merged, can we really not share any code?
First, sport2 only supports i2s and left justified mode, while sport3 adds i2s packed and right justified mode. Second, in tdm mode for sport2, tx and rx are dependent and have a window granularity of 8 channel. While sport3 doesn't have these limits. What's more, sport3 is not only used on blackfin. That means many patches can only apply to this driver.
Thanks, Scott