Re: [U-Boot] [PATCH] usb: Add new command to regress USB devices

2016-03-13 Thread Wolfgang Denk
Dear Simon, In message you wrote: > > We do have an 'sf test' command. I think it is useful, particularly if > it measures speed as well. I tend to agree with Hans - this should normally be implemented in a script, or, if reallyneeed (but I can't ssee why this would be necessary) in board spec

Re: [U-Boot] [PATCH] usb: Add new command to regress USB devices

2016-03-12 Thread Simon Glass
Hi, On 9 March 2016 at 21:52, Rajesh Bhagat wrote: > > >> -Original Message- >> From: s...@google.com [mailto:s...@google.com] On Behalf Of Simon Glass >> Sent: Thursday, March 10, 2016 3:09 AM >> To: Rajat Srivastava >> Cc: U-Boot Mailing List ; Marek Vašut ; >> Rajesh Bhagat >> Subjec

Re: [U-Boot] [PATCH] usb: Add new command to regress USB devices

2016-03-10 Thread Rajesh Bhagat
> -Original Message- > From: s...@google.com [mailto:s...@google.com] On Behalf Of Simon Glass > Sent: Thursday, March 10, 2016 3:09 AM > To: Rajat Srivastava > Cc: U-Boot Mailing List ; Marek Vašut ; > Rajesh Bhagat > Subject: Re: [PATCH] usb: Add new command to regress USB devices >

Re: [U-Boot] [PATCH] usb: Add new command to regress USB devices

2016-03-09 Thread Simon Glass
Hi Rajat, On 9 March 2016 at 04:22, Rajat Srivastava wrote: > This patch adds a new 'usb regress' command, that can be used to > regress test a USB device. It performs the following operations: > > 1. starts the USB device > 2. performs read/write operations > 3. stops the USB device > 4. verifie

Re: [U-Boot] [PATCH] usb: Add new command to regress USB devices

2016-03-09 Thread Simon Glass
Hi, On 9 March 2016 at 05:55, Marek Vasut wrote: > > On 03/09/2016 01:21 PM, Hans de Goede wrote: > > Hi, > > > > On 09-03-16 12:28, Marek Vasut wrote: > >> On 03/09/2016 12:22 PM, Rajat Srivastava wrote: > >>> This patch adds a new 'usb regress' command, that can be used to > >>> regress test a

Re: [U-Boot] [PATCH] usb: Add new command to regress USB devices

2016-03-09 Thread Rajesh Bhagat
> -Original Message- > From: Marek Vasut [mailto:ma...@denx.de] > Sent: Wednesday, March 09, 2016 4:59 PM > To: Rajat Srivastava ; u-boot@lists.denx.de > Cc: s...@chromium.org; Rajesh Bhagat > Subject: Re: [PATCH] usb: Add new command to regress USB devices > > On 03/09/2016 12:22 PM, R

[U-Boot] [PATCH] usb: Add new command to regress USB devices

2016-03-09 Thread Rajat Srivastava
This patch adds a new 'usb regress' command, that can be used to regress test a USB device. It performs the following operations: 1. starts the USB device 2. performs read/write operations 3. stops the USB device 4. verifies the contents of read/write operations Sample Output: => usb regress 8100

Re: [U-Boot] [PATCH] usb: Add new command to regress USB devices

2016-03-09 Thread Marek Vasut
On 03/09/2016 01:21 PM, Hans de Goede wrote: > Hi, > > On 09-03-16 12:28, Marek Vasut wrote: >> On 03/09/2016 12:22 PM, Rajat Srivastava wrote: >>> This patch adds a new 'usb regress' command, that can be used to >>> regress test a USB device. It performs the following operations: >>> >>> 1. start

Re: [U-Boot] [PATCH] usb: Add new command to regress USB devices

2016-03-09 Thread Marek Vasut
On 03/09/2016 01:33 PM, Rajesh Bhagat wrote: > > >> -Original Message- >> From: Marek Vasut [mailto:ma...@denx.de] >> Sent: Wednesday, March 09, 2016 4:59 PM >> To: Rajat Srivastava ; u-boot@lists.denx.de >> Cc: s...@chromium.org; Rajesh Bhagat >> Subject: Re: [PATCH] usb: Add new comman

Re: [U-Boot] [PATCH] usb: Add new command to regress USB devices

2016-03-09 Thread Hans de Goede
Hi, On 09-03-16 12:28, Marek Vasut wrote: On 03/09/2016 12:22 PM, Rajat Srivastava wrote: This patch adds a new 'usb regress' command, that can be used to regress test a USB device. It performs the following operations: 1. starts the USB device 2. performs read/write operations 3. stops the US

Re: [U-Boot] [PATCH] usb: Add new command to regress USB devices

2016-03-09 Thread Marek Vasut
On 03/09/2016 12:22 PM, Rajat Srivastava wrote: > This patch adds a new 'usb regress' command, that can be used to > regress test a USB device. It performs the following operations: > > 1. starts the USB device > 2. performs read/write operations > 3. stops the USB device > 4. verifies the content