Hello.

Gupta, Ajay Kumar wrote:

-----Original Message-----
From: [email protected] [mailto:[email protected]] 
On Behalf Of Sergei
Shtylyov
Sent: Wednesday, January 28, 2009 2:59 AM
To: [email protected]; [email protected]
Cc: [email protected]; [email protected]; 
[email protected]
Subject: [PATCH 2/8] musb_host: fix endpoint allocation

The driver prevents using the same numbered Rx/Tx endpoints simultaneously for
the periodic transfers -- which would actually be correct unless they share the
same FIFO. Use 'in_qh' and 'out_qh' fields of the 'struct musb_hw_ep' to check
the endpoint's business and get rid of now completely useless 'periodic' array
in the 'struct musb'.  While at it, optimize the loop induction variable in the
endpoint lookup code and remove duplicate/unneeded code elsewhere...

Signed-off-by: Sergei Shtylyov <[email protected]>

---
This is hopefully a better replacement for the patch originally posted by Ajay
Kumar Gupta (http://marc.info/?l=linux-usb&m=122337221417651).
The patch is against the recent Linus' kernel...

Why didn't you remove periodic[32] field from "struct musb"?
        struct musb_qh          *periodic[32];  /* tree of interrupt+iso */

Oops, probably a reject was missed (though I seem to remember fixing it)... will repost.

WBR, Sergei



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

Reply via email to