Re: [U-Boot] [PATCH 1/2] mmc: dw_mmc: Increase timeout to 20 seconds

2015-09-21 Thread Pantelis Antoniou
Hi Tom, > On Sep 18, 2015, at 22:27 , Tom Rini wrote: > > On Fri, Sep 18, 2015 at 09:32:47AM +0200, Lukasz Majewski wrote: >> Hi Tom, >> >>> On Thu, Sep 17, 2015 at 04:43:33PM +0200, Lukasz Majewski wrote: >>> Hi Tom, > On Monday, September 14, 2015 at 01:22:20 PM, Lukasz Majews

Re: [U-Boot] [PATCH 1/2] mmc: dw_mmc: Increase timeout to 20 seconds

2015-09-18 Thread Tom Rini
On Fri, Sep 18, 2015 at 09:32:47AM +0200, Lukasz Majewski wrote: > Hi Tom, > > > On Thu, Sep 17, 2015 at 04:43:33PM +0200, Lukasz Majewski wrote: > > > > > Hi Tom, > > > > > > > On Monday, September 14, 2015 at 01:22:20 PM, Lukasz Majewski > > > > wrote: > > > > > Hi Alexey, > > > > > > > > > >

Re: [U-Boot] [PATCH 1/2] mmc: dw_mmc: Increase timeout to 20 seconds

2015-09-18 Thread Przemyslaw Marczak
Hello, On 09/18/2015 09:32 AM, Lukasz Majewski wrote: Hi Tom, On Thu, Sep 17, 2015 at 04:43:33PM +0200, Lukasz Majewski wrote: Hi Tom, On Monday, September 14, 2015 at 01:22:20 PM, Lukasz Majewski wrote: Hi Alexey, Hi Marek, Lukasz, On Sun, 2015-09-13 at 16:00 +0200, Marek Vasut wrote:

Re: [U-Boot] [PATCH 1/2] mmc: dw_mmc: Increase timeout to 20 seconds

2015-09-18 Thread Lukasz Majewski
Hi Tom, > On Thu, Sep 17, 2015 at 04:43:33PM +0200, Lukasz Majewski wrote: > > > Hi Tom, > > > > > On Monday, September 14, 2015 at 01:22:20 PM, Lukasz Majewski > > > wrote: > > > > Hi Alexey, > > > > > > > > > Hi Marek, Lukasz, > > > > > > > > > > On Sun, 2015-09-13 at 16:00 +0200, Marek Vasu

Re: [U-Boot] [PATCH 1/2] mmc: dw_mmc: Increase timeout to 20 seconds

2015-09-17 Thread Tom Rini
On Thu, Sep 17, 2015 at 04:43:33PM +0200, Lukasz Majewski wrote: > Hi Tom, > > > On Monday, September 14, 2015 at 01:22:20 PM, Lukasz Majewski wrote: > > > Hi Alexey, > > > > > > > Hi Marek, Lukasz, > > > > > > > > On Sun, 2015-09-13 at 16:00 +0200, Marek Vasut wrote: > > > > > On Sunday, Septe

Re: [U-Boot] [PATCH 1/2] mmc: dw_mmc: Increase timeout to 20 seconds

2015-09-17 Thread Lukasz Majewski
Hi Tom, > On Monday, September 14, 2015 at 01:22:20 PM, Lukasz Majewski wrote: > > Hi Alexey, > > > > > Hi Marek, Lukasz, > > > > > > On Sun, 2015-09-13 at 16:00 +0200, Marek Vasut wrote: > > > > On Sunday, September 13, 2015 at 12:03:18 PM, Lukasz Majewski > > > > wrote: > > > > > Hi Marek, > >

Re: [U-Boot] [PATCH 1/2] mmc: dw_mmc: Increase timeout to 20 seconds

2015-09-14 Thread Marek Vasut
On Monday, September 14, 2015 at 01:22:20 PM, Lukasz Majewski wrote: > Hi Alexey, > > > Hi Marek, Lukasz, > > > > On Sun, 2015-09-13 at 16:00 +0200, Marek Vasut wrote: > > > On Sunday, September 13, 2015 at 12:03:18 PM, Lukasz Majewski wrote: > > > > Hi Marek, > > > > > > Hi, > > > > > > [...]

Re: [U-Boot] [PATCH 1/2] mmc: dw_mmc: Increase timeout to 20 seconds

2015-09-14 Thread Lukasz Majewski
Hi Alexey, > Hi Marek, Lukasz, > > On Sun, 2015-09-13 at 16:00 +0200, Marek Vasut wrote: > > On Sunday, September 13, 2015 at 12:03:18 PM, Lukasz Majewski wrote: > > > Hi Marek, > > > > Hi, > > > > [...] > > > > > > > > > Still we need to fix regression first with virtually > > > > > > infini

Re: [U-Boot] [PATCH 1/2] mmc: dw_mmc: Increase timeout to 20 seconds

2015-09-14 Thread Przemyslaw Marczak
Hi Alexey, On 09/14/2015 12:30 PM, Alexey Brodkin wrote: On Fri, 2015-09-11 at 23:45 +0200, Lukasz Majewski wrote: Hi Alexey, FWIW I faced similar problem even reading data. At least on one of my boards reading of ~8Mb file took ~1.7 seconds and so 1 second timeout was interrupting data exch

Re: [U-Boot] [PATCH 1/2] mmc: dw_mmc: Increase timeout to 20 seconds

2015-09-14 Thread Przemyslaw Marczak
Hi all, On 09/09/2015 09:01 AM, Lukasz Majewski wrote: Hi, The commit: d9dbb97be0e4a550457aec5f11afefb446169c90 "mmc: dw_mmc: Zap endless timeout" removed endless loop waiting for end of dw mmc transfer. For some workloads - dfu test @ Odroid XU3 (sending 8MiB file) - and SD cards (e.g. Micro

Re: [U-Boot] [PATCH 1/2] mmc: dw_mmc: Increase timeout to 20 seconds

2015-09-14 Thread Alexey Brodkin
On Fri, 2015-09-11 at 23:45 +0200, Lukasz Majewski wrote: > Hi Alexey, > > FWIW I faced similar problem even reading data. > > At least on one of my boards reading of ~8Mb file > > took ~1.7 seconds and so 1 second timeout was interrupting data > > exchange. > > Was it SD card or eMMC device? It

Re: [U-Boot] [PATCH 1/2] mmc: dw_mmc: Increase timeout to 20 seconds

2015-09-14 Thread Alexey Brodkin
Hi Marek, Lukasz, On Sun, 2015-09-13 at 16:00 +0200, Marek Vasut wrote: > On Sunday, September 13, 2015 at 12:03:18 PM, Lukasz Majewski wrote: > > Hi Marek, > > Hi, > > [...] > > > > > > Still we need to fix regression first with virtually infinite > > > > > timeout :) I would even thing that

Re: [U-Boot] [PATCH 1/2] mmc: dw_mmc: Increase timeout to 20 seconds

2015-09-13 Thread Marek Vasut
On Sunday, September 13, 2015 at 12:03:18 PM, Lukasz Majewski wrote: > Hi Marek, Hi, [...] > > > > Now from this thread I see that there're other reasons that might > > > > affect length of at least write operation. In other words it > > > > could be complicated unfortunately. > > > > > > My gu

Re: [U-Boot] [PATCH 1/2] mmc: dw_mmc: Increase timeout to 20 seconds

2015-09-13 Thread Lukasz Majewski
Hi Marek, > On Friday, September 11, 2015 at 11:45:28 PM, Lukasz Majewski wrote: > > Hi Alexey, > > Hi, > > > > Hi Marek, Lukasz, > > > > > > > On Wednesday, September 09, 2015 at 09:01:30 AM, Lukasz Majewski > > > > > > > > wrote: > > > > > Hi, > > > > > > > > > > > The commit: d9dbb97be0e4a

Re: [U-Boot] [PATCH 1/2] mmc: dw_mmc: Increase timeout to 20 seconds

2015-09-12 Thread Marek Vasut
On Friday, September 11, 2015 at 11:45:28 PM, Lukasz Majewski wrote: > Hi Alexey, Hi, > > Hi Marek, Lukasz, > > > > > On Wednesday, September 09, 2015 at 09:01:30 AM, Lukasz Majewski > > > > > > wrote: > > > > Hi, > > > > > > > > > The commit: d9dbb97be0e4a550457aec5f11afefb446169c90 > > > > >

Re: [U-Boot] [PATCH 1/2] mmc: dw_mmc: Increase timeout to 20 seconds

2015-09-11 Thread Lukasz Majewski
Hi Alexey, > Hi Marek, Lukasz, > > > On Wednesday, September 09, 2015 at 09:01:30 AM, Lukasz Majewski > > wrote: > > > Hi, > > > > > > > The commit: d9dbb97be0e4a550457aec5f11afefb446169c90 > > > > "mmc: dw_mmc: Zap endless timeout" removed endless loop waiting > > > > for end of dw mmc transfer

Re: [U-Boot] [PATCH 1/2] mmc: dw_mmc: Increase timeout to 20 seconds

2015-09-11 Thread Alexey Brodkin
Hi Marek, Lukasz, > On Wednesday, September 09, 2015 at 09:01:30 AM, Lukasz Majewski wrote: > > Hi, > > > > > The commit: d9dbb97be0e4a550457aec5f11afefb446169c90 > > > "mmc: dw_mmc: Zap endless timeout" removed endless loop waiting for > > > end of dw mmc transfer. > > > > > > For some workload

Re: [U-Boot] [PATCH 1/2] mmc: dw_mmc: Increase timeout to 20 seconds

2015-09-09 Thread Marek Vasut
On Wednesday, September 09, 2015 at 09:01:30 AM, Lukasz Majewski wrote: > Hi, > > > The commit: d9dbb97be0e4a550457aec5f11afefb446169c90 > > "mmc: dw_mmc: Zap endless timeout" removed endless loop waiting for > > end of dw mmc transfer. > > > > For some workloads - dfu test @ Odroid XU3 (sending

Re: [U-Boot] [PATCH 1/2] mmc: dw_mmc: Increase timeout to 20 seconds

2015-09-09 Thread Lukasz Majewski
Hi, > The commit: d9dbb97be0e4a550457aec5f11afefb446169c90 > "mmc: dw_mmc: Zap endless timeout" removed endless loop waiting for > end of dw mmc transfer. > > For some workloads - dfu test @ Odroid XU3 (sending 8MiB file) - > and SD cards (e.g. MicroSD Kingston 4GiB, Adata 4GiB) > the default tim

Re: [U-Boot] [PATCH 1/2] mmc: dw_mmc: Increase timeout to 20 seconds

2015-09-02 Thread Marek Vasut
On Tuesday, September 01, 2015 at 06:22:08 PM, Pantelis Antoniou wrote: > Hi Marek, Hi! [...] > >> So you suggest adding board_mmc_getcd() in several places in the mmc > >> subsystem driver to detect removal of the SD card? > > > > H, I'm not sure about this one. Panto ? > > I’m fine with t

Re: [U-Boot] [PATCH 1/2] mmc: dw_mmc: Increase timeout to 20 seconds

2015-09-01 Thread Pantelis Antoniou
Hi Marek, > On Aug 29, 2015, at 22:19 , Marek Vasut wrote: > > On Saturday, August 29, 2015 at 06:38:48 PM, Lukasz Majewski wrote: >> Hi Marek, > > Hi Lukasz, > >>> On Saturday, August 29, 2015 at 01:55:36 PM, Lukasz Majewski wrote: On Fri, 28 Aug 2015 23:55:17 +0200 >>> >>> Hi! >>> >>>

Re: [U-Boot] [PATCH 1/2] mmc: dw_mmc: Increase timeout to 20 seconds

2015-09-01 Thread Marek Vasut
On Tuesday, September 01, 2015 at 05:25:00 PM, Lukasz Majewski wrote: > Hi Marek, > > > On Tuesday, September 01, 2015 at 01:19:09 PM, Lukasz Majewski wrote: > > > Hi Marek, > > > > Hi! > > > > > > On Saturday, August 29, 2015 at 06:38:48 PM, Lukasz Majewski > > > > > > > > wrote: > > > > > Hi

Re: [U-Boot] [PATCH 1/2] mmc: dw_mmc: Increase timeout to 20 seconds

2015-09-01 Thread Lukasz Majewski
Hi Marek, > On Tuesday, September 01, 2015 at 01:19:09 PM, Lukasz Majewski wrote: > > Hi Marek, > > Hi! > > > > On Saturday, August 29, 2015 at 06:38:48 PM, Lukasz Majewski > > > wrote: > > > > Hi Marek, > > > > > > Hi Lukasz, > > > > > > > > On Saturday, August 29, 2015 at 01:55:36 PM, Lukasz

Re: [U-Boot] [PATCH 1/2] mmc: dw_mmc: Increase timeout to 20 seconds

2015-09-01 Thread Marek Vasut
On Tuesday, September 01, 2015 at 01:19:09 PM, Lukasz Majewski wrote: > Hi Marek, Hi! > > On Saturday, August 29, 2015 at 06:38:48 PM, Lukasz Majewski wrote: > > > Hi Marek, > > > > Hi Lukasz, > > > > > > On Saturday, August 29, 2015 at 01:55:36 PM, Lukasz Majewski > > > > > > > > wrote: > > >

Re: [U-Boot] [PATCH 1/2] mmc: dw_mmc: Increase timeout to 20 seconds

2015-09-01 Thread Lukasz Majewski
Hi Marek, > On Saturday, August 29, 2015 at 06:38:48 PM, Lukasz Majewski wrote: > > Hi Marek, > > Hi Lukasz, > > > > On Saturday, August 29, 2015 at 01:55:36 PM, Lukasz Majewski > > > wrote: > > > > On Fri, 28 Aug 2015 23:55:17 +0200 > > > > > > Hi! > > > > > > > Marek Vasut wrote: > > > > >

Re: [U-Boot] [PATCH 1/2] mmc: dw_mmc: Increase timeout to 20 seconds

2015-08-29 Thread Marek Vasut
On Saturday, August 29, 2015 at 06:38:48 PM, Lukasz Majewski wrote: > Hi Marek, Hi Lukasz, > > On Saturday, August 29, 2015 at 01:55:36 PM, Lukasz Majewski wrote: > > > On Fri, 28 Aug 2015 23:55:17 +0200 > > > > Hi! > > > > > Marek Vasut wrote: > > > > On Friday, August 28, 2015 at 03:50:20 PM

Re: [U-Boot] [PATCH 1/2] mmc: dw_mmc: Increase timeout to 20 seconds

2015-08-29 Thread Lukasz Majewski
Hi Marek, > On Saturday, August 29, 2015 at 01:55:36 PM, Lukasz Majewski wrote: > > On Fri, 28 Aug 2015 23:55:17 +0200 > > Hi! > > > Marek Vasut wrote: > > > On Friday, August 28, 2015 at 03:50:20 PM, Lukasz Majewski wrote: > > > > The commit: d9dbb97be0e4a550457aec5f11afefb446169c90 > > > > "m

Re: [U-Boot] [PATCH 1/2] mmc: dw_mmc: Increase timeout to 20 seconds

2015-08-29 Thread Marek Vasut
On Saturday, August 29, 2015 at 01:55:36 PM, Lukasz Majewski wrote: > On Fri, 28 Aug 2015 23:55:17 +0200 Hi! > Marek Vasut wrote: > > On Friday, August 28, 2015 at 03:50:20 PM, Lukasz Majewski wrote: > > > The commit: d9dbb97be0e4a550457aec5f11afefb446169c90 > > > "mmc: dw_mmc: Zap endless timeo

Re: [U-Boot] [PATCH 1/2] mmc: dw_mmc: Increase timeout to 20 seconds

2015-08-29 Thread Lukasz Majewski
On Fri, 28 Aug 2015 23:55:17 +0200 Marek Vasut wrote: > On Friday, August 28, 2015 at 03:50:20 PM, Lukasz Majewski wrote: > > The commit: d9dbb97be0e4a550457aec5f11afefb446169c90 > > "mmc: dw_mmc: Zap endless timeout" removed endless loop waiting for > > end of dw mmc transfer. > > > > For some

Re: [U-Boot] [PATCH 1/2] mmc: dw_mmc: Increase timeout to 20 seconds

2015-08-28 Thread Marek Vasut
On Friday, August 28, 2015 at 03:50:20 PM, Lukasz Majewski wrote: > The commit: d9dbb97be0e4a550457aec5f11afefb446169c90 > "mmc: dw_mmc: Zap endless timeout" removed endless loop waiting for end > of dw mmc transfer. > > For some workloads - dfu test @ Odroid XU3 (sending 8MiB file) - > and SD car

[U-Boot] [PATCH 1/2] mmc: dw_mmc: Increase timeout to 20 seconds

2015-08-28 Thread Lukasz Majewski
The commit: d9dbb97be0e4a550457aec5f11afefb446169c90 "mmc: dw_mmc: Zap endless timeout" removed endless loop waiting for end of dw mmc transfer. For some workloads - dfu test @ Odroid XU3 (sending 8MiB file) - and SD cards (e.g. MicroSD Kingston 4GiB, Adata 4GiB) the default timeout is to short.