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 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot