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

Reply via email to