[U-Boot] [PATCH 5/6] usb: add Marvell Kirkwood ehci host controller driver

Remy Bohmer linux at bohmer.net
Thu Jul 9 09:29:55 CEST 2009


Hello Prafulla,

> As stated in other email thread too.
> Now all dependencies for Sheevaplug board support are in accepted state except this patch http://lists.denx.de/pipermail/u-boot/2009-June/055112.html

I know that but I cannot test it because I do not have the hardware,
so I depend on others here. (it might be even you). I already
mentioned that I would prefer the cleanup patch from Michael
Trimarchi, if it is correct.

At least there is 1 significant difference between your patch and the
patch of Michael

Your patch contains this:
--------------------------------------------------------
-	typeReq = req->request << 8 | req->requesttype;
+	typeReq = req->request | req->requesttype << 8;

 	switch (le16_to_cpu(typeReq)) {
--------------------------------------------------------

and Michael patch contains this:
--------------------------------------------------------
-	typeReq = req->request << 8 | req->requesttype;
+	typeReq = req->request | req->requesttype << 8;

-	switch (le16_to_cpu(typeReq)) {
+	switch (typeReq) {
--------------------------------------------------------

So, at least one of them is wrong.
This difference is reason enough for me to not pull either patch in
until this is clear.

Kind Regards,

Remy


More information about the U-Boot mailing list