Re: [U-Boot] [PATCH v3 0/8] usb: ss: Some fixes and cleanup for USB super-speed support

2013-04-18 Thread Vivek Gautam
Hi Marek,


On Sun, Apr 14, 2013 at 11:43 PM, Marek Vasut ma...@denx.de wrote:
 Dear Vivek Gautam,

 Based on 'u-boot-usb' master branch.

 This patch-series includes majorly some clean-up, few fixes and
 then some basic super-speed usb infrastructure addition, to help
 put support for XHCI in near future.

 btw can you test your patches with MAKEALL -a arm? I get this:

 - SUMMARY 
 Boards compiled: 306
 Boards with errors: 65 ( qong mx35pdk gplugd at91sam9m10g45ek_nandflash 
 pogo_e02
 dns325 iconnect lschlv2 lsxhl d2net_v2 inetspace_v2 net2big_v2 
 netspace_lite_v2
 netspace_max_v2 netspace_v2 wireless_space dreamplug guruplug mv88f6281gtw_ge
 openrd_base openrd_client openrd_ultimate rd6281a sheevaplug ib62x0 dockstar
 tk71 zmx25 mx23_olinuxino apx4devkit mx23evk m28evk mx28evk sc_sps_1 edminiv2
 mx51_efikamx mx51_efikasb mx51evk mx53loco mx6qsabreauto mx6qsabrelite
 nitrogen6dl nitrogen6dl2g nitrogen6q nitrogen6q2g nitrogen6s nitrogen6s1g 
 cm_t35
 mt_ventoux omap3_beagle mcx twister omap4_panda snow smdk5250 harmony seaboard
 ventana whistler colibri_t20_iris plutux medcom-wide tec paz00 trimslice )
 --

Tried with MAKEALL
got following result

- SUMMARY 
Boards compiled: 306
Boards with errors: 1 ( omap3_evm )
Boards with warnings but no errors: 17 ( VCMA9 smdk2410 kzm9g balloon3
h2200 lubbock palmld palmtc polaris pxa255_idp trizepsiv
vpac270_nor_128 vpac270_nor_256 vpac270_ond_256 xaeniax zipitz2
colibri_pxa270 )
--

There's something to do with Cross Compiler version ??
btw what environment are you compiling the source with.


-- 
Thanks  Regards
Vivek
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v3 0/8] usb: ss: Some fixes and cleanup for USB super-speed support

2013-04-18 Thread Vivek Gautam
Hi,


On Sun, Apr 14, 2013 at 10:44 PM, Marek Vasut ma...@denx.de wrote:
 Dear Vivek Gautam,

 Based on 'u-boot-usb' master branch.

 This patch-series includes majorly some clean-up, few fixes and
 then some basic super-speed usb infrastructure addition, to help
 put support for XHCI in near future.

 Changes from v2:
  - Added a patch usb: common: Weed out USB_**_PRINTFs from usb framework
to replace USB_***_PRINTF with debug() so as to get rid of unnecessary
DEBUG macros in usb common framework and thereby rebasing other patches
on top of this so that no USB_PRINTF appears further.
  - Added a patch to reset only USB 2.0 ports, since USB 3.0 protocol ports
don't require a reset to happen to move to 'enabled' state.
  - Added a patch to move definition of 'min3()' out of ehci-hcd and putting
the same as macro definition in common header.
  - Using a 'switch-case' in portspeed() in cmd_usb.c

 Changes from v1:
  - Fixing the issue with 'ifno' as well as added 'if_desc'.
  - Instead of turning-on only powered-off hub ports, power-cycle all
the hub ports (aka: turning off and then turning on again power to
all the ports.
  - Fixing commenting style problem in
'usb: Update device class in usb device's descriptor'
  - Fixing typo in commit message for
'usb: hub: Fix enumration timeout'
  - Removing separate definition for 'struct usb_ep_desc'; thereby adding
'struct usb_ss_ep_comp_descriptor' to 'struct usb_interface' only.
As a result modifying the patch accordingly also dropping the patch
'usb: eth: Fix for updated usb interface descriptor structure'
  - Dropping the patch 'usb: hub: Increase device enumeration timeout for
 broken drives' for now (will come back with a solution at later point of
 time).

 Applied all but the last one, let's now base all subsequent patches on u-boot-
 usb/next please.

And i can't see these patches on u-boot-usb/next. Possibly you haven't
pushed out yet ??



-- 
Thanks  Regards
Vivek
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v3 0/8] usb: ss: Some fixes and cleanup for USB super-speed support

2013-04-18 Thread Vivek Gautam
On Thu, Apr 18, 2013 at 11:54 AM, Vivek Gautam
gautamvivek1...@gmail.com wrote:
 Hi Marek,


 On Sun, Apr 14, 2013 at 11:43 PM, Marek Vasut ma...@denx.de wrote:
 Dear Vivek Gautam,

 Based on 'u-boot-usb' master branch.

 This patch-series includes majorly some clean-up, few fixes and
 then some basic super-speed usb infrastructure addition, to help
 put support for XHCI in near future.

 btw can you test your patches with MAKEALL -a arm? I get this:

 - SUMMARY 
 Boards compiled: 306
 Boards with errors: 65 ( qong mx35pdk gplugd at91sam9m10g45ek_nandflash 
 pogo_e02
 dns325 iconnect lschlv2 lsxhl d2net_v2 inetspace_v2 net2big_v2 
 netspace_lite_v2
 netspace_max_v2 netspace_v2 wireless_space dreamplug guruplug mv88f6281gtw_ge
 openrd_base openrd_client openrd_ultimate rd6281a sheevaplug ib62x0 dockstar
 tk71 zmx25 mx23_olinuxino apx4devkit mx23evk m28evk mx28evk sc_sps_1 edminiv2
 mx51_efikamx mx51_efikasb mx51evk mx53loco mx6qsabreauto mx6qsabrelite
 nitrogen6dl nitrogen6dl2g nitrogen6q nitrogen6q2g nitrogen6s nitrogen6s1g 
 cm_t35
 mt_ventoux omap3_beagle mcx twister omap4_panda snow smdk5250 harmony 
 seaboard
 ventana whistler colibri_t20_iris plutux medcom-wide tec paz00 trimslice )
 --

 Tried with MAKEALL
 got following result

 - SUMMARY 
 Boards compiled: 306
 Boards with errors: 1 ( omap3_evm )
 Boards with warnings but no errors: 17 ( VCMA9 smdk2410 kzm9g balloon3
 h2200 lubbock palmld palmtc polaris pxa255_idp trizepsiv
 vpac270_nor_128 vpac270_nor_256 vpac270_ond_256 xaeniax zipitz2
 colibri_pxa270 )
 --


**Without my patches i get following result

- SUMMARY 
Boards compiled: 306
Boards with warnings but no errors: 17 ( VCMA9 smdk2410 kzm9g balloon3
h2200 lubbock palmld palmtc polaris pxa255_idp trizepsiv
vpac270_nor_128 vpac270_nor_256 vpac270_ond_256 xaeniax zipitz2
colibri_pxa270 )
--


 There's something to do with Cross Compiler version ??
 btw what environment are you compiling the source with.


-- 
Thanks  Regards
Vivek
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v3 0/8] usb: ss: Some fixes and cleanup for USB super-speed support

2013-04-18 Thread Vivek Gautam
HI Marek,


On Thu, Apr 18, 2013 at 4:29 PM, Vivek Gautam gautamvivek1...@gmail.com wrote:
 On Thu, Apr 18, 2013 at 11:54 AM, Vivek Gautam
 gautamvivek1...@gmail.com wrote:
 Hi Marek,


 On Sun, Apr 14, 2013 at 11:43 PM, Marek Vasut ma...@denx.de wrote:
 Dear Vivek Gautam,

 Based on 'u-boot-usb' master branch.

 This patch-series includes majorly some clean-up, few fixes and
 then some basic super-speed usb infrastructure addition, to help
 put support for XHCI in near future.

 btw can you test your patches with MAKEALL -a arm? I get this:

 - SUMMARY 
 Boards compiled: 306
 Boards with errors: 65 ( qong mx35pdk gplugd at91sam9m10g45ek_nandflash 
 pogo_e02
 dns325 iconnect lschlv2 lsxhl d2net_v2 inetspace_v2 net2big_v2 
 netspace_lite_v2
 netspace_max_v2 netspace_v2 wireless_space dreamplug guruplug 
 mv88f6281gtw_ge
 openrd_base openrd_client openrd_ultimate rd6281a sheevaplug ib62x0 dockstar
 tk71 zmx25 mx23_olinuxino apx4devkit mx23evk m28evk mx28evk sc_sps_1 
 edminiv2
 mx51_efikamx mx51_efikasb mx51evk mx53loco mx6qsabreauto mx6qsabrelite
 nitrogen6dl nitrogen6dl2g nitrogen6q nitrogen6q2g nitrogen6s nitrogen6s1g 
 cm_t35
 mt_ventoux omap3_beagle mcx twister omap4_panda snow smdk5250 harmony 
 seaboard
 ventana whistler colibri_t20_iris plutux medcom-wide tec paz00 trimslice )
 --

 Tried with MAKEALL
 got following result

 - SUMMARY 
 Boards compiled: 306
 Boards with errors: 1 ( omap3_evm )
 Boards with warnings but no errors: 17 ( VCMA9 smdk2410 kzm9g balloon3
 h2200 lubbock palmld palmtc polaris pxa255_idp trizepsiv
 vpac270_nor_128 vpac270_nor_256 vpac270_ond_256 xaeniax zipitz2
 colibri_pxa270 )
 --

I actually checked now just for omap3_evm configuration by trying out:
  make distclean
  make omap3_evm_config
  make

But strangely i didn't get any build errros for omap3_evm board on
explicitly compiling for it.
Any clue ?



 **Without my patches i get following result

 - SUMMARY 
 Boards compiled: 306
 Boards with warnings but no errors: 17 ( VCMA9 smdk2410 kzm9g balloon3
 h2200 lubbock palmld palmtc polaris pxa255_idp trizepsiv
 vpac270_nor_128 vpac270_nor_256 vpac270_ond_256 xaeniax zipitz2
 colibri_pxa270 )
 --


 There's something to do with Cross Compiler version ??
 btw what environment are you compiling the source with.

I am using arm-2011.09 cross toolchain.



-- 
Thanks  Regards
Vivek
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v3 0/8] usb: ss: Some fixes and cleanup for USB super-speed support

2013-04-18 Thread Marek Vasut
Dear Vivek Gautam,

 Hi,
 
 On Sun, Apr 14, 2013 at 10:44 PM, Marek Vasut ma...@denx.de wrote:
  Dear Vivek Gautam,
  
  Based on 'u-boot-usb' master branch.
  
  This patch-series includes majorly some clean-up, few fixes and
  then some basic super-speed usb infrastructure addition, to help
  put support for XHCI in near future.
  
  Changes from v2:
   - Added a patch usb: common: Weed out USB_**_PRINTFs from usb
   framework
   
 to replace USB_***_PRINTF with debug() so as to get rid of
 unnecessary DEBUG macros in usb common framework and thereby
 rebasing other patches on top of this so that no USB_PRINTF appears
 further.
   
   - Added a patch to reset only USB 2.0 ports, since USB 3.0 protocol
   ports
   
 don't require a reset to happen to move to 'enabled' state.
   
   - Added a patch to move definition of 'min3()' out of ehci-hcd and
   putting
   
 the same as macro definition in common header.
   
   - Using a 'switch-case' in portspeed() in cmd_usb.c
  
  Changes from v1:
   - Fixing the issue with 'ifno' as well as added 'if_desc'.
   - Instead of turning-on only powered-off hub ports, power-cycle all
   
 the hub ports (aka: turning off and then turning on again power to
 all the ports.
   
   - Fixing commenting style problem in
   
 'usb: Update device class in usb device's descriptor'
   
   - Fixing typo in commit message for
   
 'usb: hub: Fix enumration timeout'
   
   - Removing separate definition for 'struct usb_ep_desc'; thereby adding
   
 'struct usb_ss_ep_comp_descriptor' to 'struct usb_interface' only.
 As a result modifying the patch accordingly also dropping the patch
 'usb: eth: Fix for updated usb interface descriptor structure'
   
   - Dropping the patch 'usb: hub: Increase device enumeration timeout for
  
  broken drives' for now (will come back with a solution at later point of
  time).
  
  Applied all but the last one, let's now base all subsequent patches on
  u-boot- usb/next please.
 
 And i can't see these patches on u-boot-usb/next. Possibly you haven't
 pushed out yet ??

You're right, sorry! Fixed now

Best regards,
Marek Vasut
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v3 0/8] usb: ss: Some fixes and cleanup for USB super-speed support

2013-04-18 Thread Marek Vasut
Dear Vivek Gautam,

 HI Marek,
 
 On Thu, Apr 18, 2013 at 4:29 PM, Vivek Gautam gautamvivek1...@gmail.com 
wrote:
  On Thu, Apr 18, 2013 at 11:54 AM, Vivek Gautam
  
  gautamvivek1...@gmail.com wrote:
  Hi Marek,
  
  On Sun, Apr 14, 2013 at 11:43 PM, Marek Vasut ma...@denx.de wrote:
  Dear Vivek Gautam,
  
  Based on 'u-boot-usb' master branch.
  
  This patch-series includes majorly some clean-up, few fixes and
  then some basic super-speed usb infrastructure addition, to help
  put support for XHCI in near future.
  
  btw can you test your patches with MAKEALL -a arm? I get this:
  
  - SUMMARY 
  Boards compiled: 306
  Boards with errors: 65 ( qong mx35pdk gplugd at91sam9m10g45ek_nandflash
  pogo_e02 dns325 iconnect lschlv2 lsxhl d2net_v2 inetspace_v2
  net2big_v2 netspace_lite_v2 netspace_max_v2 netspace_v2 wireless_space
  dreamplug guruplug mv88f6281gtw_ge openrd_base openrd_client
  openrd_ultimate rd6281a sheevaplug ib62x0 dockstar tk71 zmx25
  mx23_olinuxino apx4devkit mx23evk m28evk mx28evk sc_sps_1 edminiv2
  mx51_efikamx mx51_efikasb mx51evk mx53loco mx6qsabreauto mx6qsabrelite
  nitrogen6dl nitrogen6dl2g nitrogen6q nitrogen6q2g nitrogen6s
  nitrogen6s1g cm_t35 mt_ventoux omap3_beagle mcx twister omap4_panda
  snow smdk5250 harmony seaboard ventana whistler colibri_t20_iris
  plutux medcom-wide tec paz00 trimslice )
  --
  
  Tried with MAKEALL
  got following result
  
  - SUMMARY 
  Boards compiled: 306
  Boards with errors: 1 ( omap3_evm )
  Boards with warnings but no errors: 17 ( VCMA9 smdk2410 kzm9g balloon3
  h2200 lubbock palmld palmtc polaris pxa255_idp trizepsiv
  vpac270_nor_128 vpac270_nor_256 vpac270_ond_256 xaeniax zipitz2
  colibri_pxa270 )
  --
 
 I actually checked now just for omap3_evm configuration by trying out:
   make distclean
   make omap3_evm_config
   make
 
 But strangely i didn't get any build errros for omap3_evm board on
 explicitly compiling for it.
 Any clue ?
 
  **Without my patches i get following result
  
  - SUMMARY 
  Boards compiled: 306
  Boards with warnings but no errors: 17 ( VCMA9 smdk2410 kzm9g balloon3
  h2200 lubbock palmld palmtc polaris pxa255_idp trizepsiv
  vpac270_nor_128 vpac270_nor_256 vpac270_ond_256 xaeniax zipitz2
  colibri_pxa270 )
  --
  
  There's something to do with Cross Compiler version ??
  btw what environment are you compiling the source with.
 
 I am using arm-2011.09 cross toolchain.

I use ELDK 5.3 and Debian 4.7.2-5 to do by builds. But now that I'm looking at 
it, it's this patch that caused it, sorry.

commit 28b31a5937b89528c40df24dd6c9122578880605
Author: Julius Werner jwer...@chromium.org
Date:   Thu Feb 28 18:08:40 2013 +

usb: Add new command to set USB 2.0 port test modes

diff --git a/drivers/usb/host/ehci-hcd.c b/drivers/usb/host/ehci-hcd.c
index 8464850..19d4352 100644
--- a/drivers/usb/host/ehci-hcd.c
+++ b/drivers/usb/host/ehci-hcd.c
@@ -630,7 +630,7 @@ ehci_submit_root(struct usb_device *dev, unsigned long 
pipe, 
void *buffer,
printf(The request port(%d) is not configured\n, port - 1);
return -1;
}
-   status_reg = (uint32_t *)hcor-or_portsc[port - 1];
+   status_reg = (uint32_t *)ctrl-hcor-or_portsc[port - 1];
srclen = 0;
 
debug(req=%u (%#x), type=%u (%#x), value=%u, index=%u\n,

This change fixes is, right Julius ?

Best regards,
Marek Vasut
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v3 0/8] usb: ss: Some fixes and cleanup for USB super-speed support

2013-04-18 Thread Vivek Gautam
On Thu, Apr 18, 2013 at 6:08 PM, Marek Vasut ma...@denx.de wrote:
 Dear Vivek Gautam,

 Hi,

 On Sun, Apr 14, 2013 at 10:44 PM, Marek Vasut ma...@denx.de wrote:
  Dear Vivek Gautam,
 
  Based on 'u-boot-usb' master branch.
 
  This patch-series includes majorly some clean-up, few fixes and
  then some basic super-speed usb infrastructure addition, to help
  put support for XHCI in near future.
 
  Changes from v2:
   - Added a patch usb: common: Weed out USB_**_PRINTFs from usb
   framework
 
 to replace USB_***_PRINTF with debug() so as to get rid of
 unnecessary DEBUG macros in usb common framework and thereby
 rebasing other patches on top of this so that no USB_PRINTF appears
 further.
 
   - Added a patch to reset only USB 2.0 ports, since USB 3.0 protocol
   ports
 
 don't require a reset to happen to move to 'enabled' state.
 
   - Added a patch to move definition of 'min3()' out of ehci-hcd and
   putting
 
 the same as macro definition in common header.
 
   - Using a 'switch-case' in portspeed() in cmd_usb.c
 
  Changes from v1:
   - Fixing the issue with 'ifno' as well as added 'if_desc'.
   - Instead of turning-on only powered-off hub ports, power-cycle all
 
 the hub ports (aka: turning off and then turning on again power to
 all the ports.
 
   - Fixing commenting style problem in
 
 'usb: Update device class in usb device's descriptor'
 
   - Fixing typo in commit message for
 
 'usb: hub: Fix enumration timeout'
 
   - Removing separate definition for 'struct usb_ep_desc'; thereby adding
 
 'struct usb_ss_ep_comp_descriptor' to 'struct usb_interface' only.
 As a result modifying the patch accordingly also dropping the patch
 'usb: eth: Fix for updated usb interface descriptor structure'
 
   - Dropping the patch 'usb: hub: Increase device enumeration timeout for
 
  broken drives' for now (will come back with a solution at later point of
  time).
 
  Applied all but the last one, let's now base all subsequent patches on
  u-boot- usb/next please.

 And i can't see these patches on u-boot-usb/next. Possibly you haven't
 pushed out yet ??

 You're right, sorry! Fixed now

Thanks you so much !!

-- 
Thanks  Regards
Vivek
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v3 0/8] usb: ss: Some fixes and cleanup for USB super-speed support

2013-04-18 Thread Julius Werner
Hi Marek,

I'm sorry, that must have slipped by when I ported the change from my
local fork. Your patch is correct, you just need to add the ctrl-
there.

On Thu, Apr 18, 2013 at 5:43 AM, Marek Vasut ma...@denx.de wrote:
 Dear Vivek Gautam,

 HI Marek,

 On Thu, Apr 18, 2013 at 4:29 PM, Vivek Gautam gautamvivek1...@gmail.com
 wrote:
  On Thu, Apr 18, 2013 at 11:54 AM, Vivek Gautam
 
  gautamvivek1...@gmail.com wrote:
  Hi Marek,
 
  On Sun, Apr 14, 2013 at 11:43 PM, Marek Vasut ma...@denx.de wrote:
  Dear Vivek Gautam,
 
  Based on 'u-boot-usb' master branch.
 
  This patch-series includes majorly some clean-up, few fixes and
  then some basic super-speed usb infrastructure addition, to help
  put support for XHCI in near future.
 
  btw can you test your patches with MAKEALL -a arm? I get this:
 
  - SUMMARY 
  Boards compiled: 306
  Boards with errors: 65 ( qong mx35pdk gplugd at91sam9m10g45ek_nandflash
  pogo_e02 dns325 iconnect lschlv2 lsxhl d2net_v2 inetspace_v2
  net2big_v2 netspace_lite_v2 netspace_max_v2 netspace_v2 wireless_space
  dreamplug guruplug mv88f6281gtw_ge openrd_base openrd_client
  openrd_ultimate rd6281a sheevaplug ib62x0 dockstar tk71 zmx25
  mx23_olinuxino apx4devkit mx23evk m28evk mx28evk sc_sps_1 edminiv2
  mx51_efikamx mx51_efikasb mx51evk mx53loco mx6qsabreauto mx6qsabrelite
  nitrogen6dl nitrogen6dl2g nitrogen6q nitrogen6q2g nitrogen6s
  nitrogen6s1g cm_t35 mt_ventoux omap3_beagle mcx twister omap4_panda
  snow smdk5250 harmony seaboard ventana whistler colibri_t20_iris
  plutux medcom-wide tec paz00 trimslice )
  --
 
  Tried with MAKEALL
  got following result
 
  - SUMMARY 
  Boards compiled: 306
  Boards with errors: 1 ( omap3_evm )
  Boards with warnings but no errors: 17 ( VCMA9 smdk2410 kzm9g balloon3
  h2200 lubbock palmld palmtc polaris pxa255_idp trizepsiv
  vpac270_nor_128 vpac270_nor_256 vpac270_ond_256 xaeniax zipitz2
  colibri_pxa270 )
  --

 I actually checked now just for omap3_evm configuration by trying out:
   make distclean
   make omap3_evm_config
   make

 But strangely i didn't get any build errros for omap3_evm board on
 explicitly compiling for it.
 Any clue ?

  **Without my patches i get following result
 
  - SUMMARY 
  Boards compiled: 306
  Boards with warnings but no errors: 17 ( VCMA9 smdk2410 kzm9g balloon3
  h2200 lubbock palmld palmtc polaris pxa255_idp trizepsiv
  vpac270_nor_128 vpac270_nor_256 vpac270_ond_256 xaeniax zipitz2
  colibri_pxa270 )
  --
 
  There's something to do with Cross Compiler version ??
  btw what environment are you compiling the source with.

 I am using arm-2011.09 cross toolchain.

 I use ELDK 5.3 and Debian 4.7.2-5 to do by builds. But now that I'm looking at
 it, it's this patch that caused it, sorry.

 commit 28b31a5937b89528c40df24dd6c9122578880605
 Author: Julius Werner jwer...@chromium.org
 Date:   Thu Feb 28 18:08:40 2013 +

 usb: Add new command to set USB 2.0 port test modes

 diff --git a/drivers/usb/host/ehci-hcd.c b/drivers/usb/host/ehci-hcd.c
 index 8464850..19d4352 100644
 --- a/drivers/usb/host/ehci-hcd.c
 +++ b/drivers/usb/host/ehci-hcd.c
 @@ -630,7 +630,7 @@ ehci_submit_root(struct usb_device *dev, unsigned long 
 pipe,
 void *buffer,
 printf(The request port(%d) is not configured\n, port - 1);
 return -1;
 }
 -   status_reg = (uint32_t *)hcor-or_portsc[port - 1];
 +   status_reg = (uint32_t *)ctrl-hcor-or_portsc[port - 1];
 srclen = 0;

 debug(req=%u (%#x), type=%u (%#x), value=%u, index=%u\n,

 This change fixes is, right Julius ?

 Best regards,
 Marek Vasut
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v3 0/8] usb: ss: Some fixes and cleanup for USB super-speed support

2013-04-18 Thread Marek Vasut
Hi Julius,

 Hi Marek,
 
 I'm sorry, that must have slipped by when I ported the change from my
 local fork. Your patch is correct, you just need to add the ctrl-
 there.

Well, next time please make sure to compile-test your patches at least before 
you send them.

The stuff is now pushed, Vivek.

Best regards,
Marek Vasut
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v3 0/8] usb: ss: Some fixes and cleanup for USB super-speed support

2013-04-18 Thread Vivek Gautam
On Fri, Apr 19, 2013 at 12:45 AM, Marek Vasut ma...@denx.de wrote:
 Hi Julius,

 Hi Marek,

 I'm sorry, that must have slipped by when I ported the change from my
 local fork. Your patch is correct, you just need to add the ctrl-
 there.

 Well, next time please make sure to compile-test your patches at least before
 you send them.

 The stuff is now pushed, Vivek.

Thanks Marek.



-- 
Best Regards
Vivek
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v3 0/8] usb: ss: Some fixes and cleanup for USB super-speed support

2013-04-14 Thread Marek Vasut
Dear Vivek Gautam,

 Based on 'u-boot-usb' master branch.
 
 This patch-series includes majorly some clean-up, few fixes and
 then some basic super-speed usb infrastructure addition, to help
 put support for XHCI in near future.
 
 Changes from v2:
  - Added a patch usb: common: Weed out USB_**_PRINTFs from usb framework
to replace USB_***_PRINTF with debug() so as to get rid of unnecessary
DEBUG macros in usb common framework and thereby rebasing other patches
on top of this so that no USB_PRINTF appears further.
  - Added a patch to reset only USB 2.0 ports, since USB 3.0 protocol ports
don't require a reset to happen to move to 'enabled' state.
  - Added a patch to move definition of 'min3()' out of ehci-hcd and putting
the same as macro definition in common header.
  - Using a 'switch-case' in portspeed() in cmd_usb.c
 
 Changes from v1:
  - Fixing the issue with 'ifno' as well as added 'if_desc'.
  - Instead of turning-on only powered-off hub ports, power-cycle all
the hub ports (aka: turning off and then turning on again power to
all the ports.
  - Fixing commenting style problem in
'usb: Update device class in usb device's descriptor'
  - Fixing typo in commit message for
'usb: hub: Fix enumration timeout'
  - Removing separate definition for 'struct usb_ep_desc'; thereby adding
'struct usb_ss_ep_comp_descriptor' to 'struct usb_interface' only.
As a result modifying the patch accordingly also dropping the patch
'usb: eth: Fix for updated usb interface descriptor structure'
  - Dropping the patch 'usb: hub: Increase device enumeration timeout for
 broken drives' for now (will come back with a solution at later point of
 time).

Applied all but the last one, let's now base all subsequent patches on u-boot-
usb/next please.

Best regards,
Marek Vasut
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v3 0/8] usb: ss: Some fixes and cleanup for USB super-speed support

2013-04-14 Thread Marek Vasut
Dear Vivek Gautam,

 Based on 'u-boot-usb' master branch.
 
 This patch-series includes majorly some clean-up, few fixes and
 then some basic super-speed usb infrastructure addition, to help
 put support for XHCI in near future.

btw can you test your patches with MAKEALL -a arm? I get this:

- SUMMARY 
Boards compiled: 306
Boards with errors: 65 ( qong mx35pdk gplugd at91sam9m10g45ek_nandflash 
pogo_e02 
dns325 iconnect lschlv2 lsxhl d2net_v2 inetspace_v2 net2big_v2 netspace_lite_v2 
netspace_max_v2 netspace_v2 wireless_space dreamplug guruplug mv88f6281gtw_ge 
openrd_base openrd_client openrd_ultimate rd6281a sheevaplug ib62x0 dockstar 
tk71 zmx25 mx23_olinuxino apx4devkit mx23evk m28evk mx28evk sc_sps_1 edminiv2 
mx51_efikamx mx51_efikasb mx51evk mx53loco mx6qsabreauto mx6qsabrelite 
nitrogen6dl nitrogen6dl2g nitrogen6q nitrogen6q2g nitrogen6s nitrogen6s1g 
cm_t35 
mt_ventoux omap3_beagle mcx twister omap4_panda snow smdk5250 harmony seaboard 
ventana whistler colibri_t20_iris plutux medcom-wide tec paz00 trimslice )
--

Best regards,
Marek Vasut
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH v3 0/8] usb: ss: Some fixes and cleanup for USB super-speed support

2013-04-12 Thread Vivek Gautam
Based on 'u-boot-usb' master branch.

This patch-series includes majorly some clean-up, few fixes and
then some basic super-speed usb infrastructure addition, to help
put support for XHCI in near future.

Changes from v2:
 - Added a patch usb: common: Weed out USB_**_PRINTFs from usb framework
   to replace USB_***_PRINTF with debug() so as to get rid of unnecessary
   DEBUG macros in usb common framework and thereby rebasing other patches
   on top of this so that no USB_PRINTF appears further.
 - Added a patch to reset only USB 2.0 ports, since USB 3.0 protocol ports
   don't require a reset to happen to move to 'enabled' state.
 - Added a patch to move definition of 'min3()' out of ehci-hcd and putting
   the same as macro definition in common header.
 - Using a 'switch-case' in portspeed() in cmd_usb.c

Changes from v1:
 - Fixing the issue with 'ifno' as well as added 'if_desc'.
 - Instead of turning-on only powered-off hub ports, power-cycle all
   the hub ports (aka: turning off and then turning on again power to
   all the ports.
 - Fixing commenting style problem in
   'usb: Update device class in usb device's descriptor'
 - Fixing typo in commit message for
   'usb: hub: Fix enumration timeout'
 - Removing separate definition for 'struct usb_ep_desc'; thereby adding
   'struct usb_ss_ep_comp_descriptor' to 'struct usb_interface' only.
   As a result modifying the patch accordingly also dropping the patch
   'usb: eth: Fix for updated usb interface descriptor structure'
 - Dropping the patch 'usb: hub: Increase device enumeration timeout for broken
   drives' for now (will come back with a solution at later point of time).

Vivek Gautam (8):
  usb: common: Weed out USB_**_PRINTFs from usb framework
  USB: Some cleanup prior to USB 3.0 interface addition
  usb: hub: Power-cycle on root-hub ports
  usb: Update device class in usb device's descriptor
  usb: hub: Fix enumration timeout
  USB: SS: Add support for Super Speed USB interface
  usb: hub: Reset only usb 2.0 ports
  usb: common: Use a global macro for 'min3'

 common/cmd_usb.c|   24 +++-
 common/usb.c|  116 +-
 common/usb_hub.c|  228 ---
 common/usb_kbd.c|   24 ++---
 common/usb_storage.c|  283 +--
 drivers/usb/host/ehci-hcd.c |   10 --
 include/common.h|2 +
 include/usb.h   |6 +
 include/usb_defs.h  |   26 -
 9 files changed, 389 insertions(+), 330 deletions(-)

-- 
1.7.6.5

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot