Hello.
David Brownell wrote:
+ cppi_ch->channel.actual_len = 0;
Wait, Shouldn't that be set by the musb_{host|gadget}.c? It indeed is. But
musb_gadget doesn't do it...
To be consistent, we should now remove the duplicate setting in
mush_host.c as tusb6010_omap.c does set this field itself too.
Actually I'd be all for initializing it in only one place ...
... but that implies a substantial cleanup of the DMA paths,
Some cleanup is coming soon, at least for the host Tx path -- as part
of ISO Tx DMA fix.
and I'll be content to defer such only-one-place cleanup until
that happens.
To summarize, musb_hdrc has four basic DMA paths today, the
cross product of {RX, TX} and {host, gadget}, each of which
looks more or less like
if (cppi/davinci)
do this
else if (omap native/TUSB)
do that
These two seem to be the same in musb_host..Not in musb_gadget though...
else if (mentor's DMA)
do something else
Or at least, that's how it worked before the Blackfin updates.
It shouldn't be that messy.
Well, the mess at least allows to earn many patch points. :-)
- Dave
WBR, Sergei
_______________________________________________
Davinci-linux-open-source mailing list
[email protected]
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source
