Dear Marek Vasut, On Sun, Jul 29, 2012 at 03:40:32 AM, Marek Vasut wrote: > > 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.
Great. There are also questions for you in my answer to Stefan. Please take a look at these. I know, it's long to read, sorry. Best regards, Benoît _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot