
29 Jul
2012
29 Jul
'12
3:40 a.m.
Dear Benoît Thébaudeau,
Dear Marek,
On Fri, Jul 27, 2012 at 04:13:45 PM, Benoît Thébaudeau wrote:
On Fri, Jul 27, 2012 at 04:01:11 PM, Marek Vasut wrote:
[...]
token = (toggle << 31) |
(xfr_bytes << 16) |
((req == NULL ? 1 : 0) << 15) |
(0 << 12) |
(3 << 10) |
((usb_pipein(pipe) ? 1 : 0) << 8) | (0x80 <<
0);
If you could fix all this magic afterwards (not in these patches), that'd be great.
Do you only mean #defining all those values?
Yes, but let's do this in a subsequent patch. It can wait for later.
OK.
What would you think about merging that together with the definition of 4096 into the current patch 1/5? In the next version, this patch would thus become a general cosmetic patch for EHCI to define all used constants.
That's all right with me.
Best regards, Benoît
Best regards, Marek Vasut