Re: [rfc/rft/patch v2 09/19] usb: musb: gadget: dma enabling for musb_gadget rx

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

 



Hello.

Felipe Balbi wrote:

we would have a bit of code duplication, no ??

  Actually, we shouldn't. The code is just much tangled now but could
be cleanly separated I think.

if it's possible... why not ? Then musb_dma_completion() would call the other versions of musb_g_[rt]x().

   Maybe I'll try to come up with something...

no... if this was caused by a dma irq, request->actuall should be
exactly the same as request->length.


  Then why call channel_abort()?

to update request->actual.

Hum... does Mentor DMA indeed not interrupt on a receiving a short packet? Ah, that must be a consequence of using DMA mode 1... hm, need to study the current code more WRT CPPI...

We only get dma irq on rx when we program dma for e.g. 64k and we
receive exactly 64k. If we get a short in the middle, we won't see a
dma irq.

  You're naive -- we'll get an IRQ of course with e.g. CPPI. :-)

heh, I don't know about all dma engines used with musb that's why the patches were sent out as rfc.

Ah... the current CPPI code only uses Rx DMA mode 0, and so but you're changing that to always use mode 1... nevermind then, I just didn't know well how that mode works.

WBR, Sergei
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux