From mboxrd@z Thu Jan  1 00:00:00 1970
From: Marek Vasut <marex@denx.de>
Date: Sun, 29 Jul 2012 20:08:50 +0200
Subject: [U-Boot] [PATCH 2/5] ehci-hcd: Boost transfer speed
In-Reply-To: <624121478.771929.1343571293331.JavaMail.root@advansee.com>
References: <624121478.771929.1343571293331.JavaMail.root@advansee.com>
Message-ID: <201207292008.50837.marex@denx.de>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Dear Beno?t Th?baudeau,

> 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.

I'm still trying to make some sense of it, gimme one more day please.

> Best regards,
> Beno?t

Best regards,
Marek Vasut