[U-Boot] [PATCH v2] mmc: sdhci: Avoid commands errors by simple timeout adaptation.

Pantelis Antoniou panto at antoniou-consulting.com
Tue Oct 29 14:08:13 CET 2013


Dear Przemyslaw,

On Oct 29, 2013, at 3:02 PM, Przemyslaw Marczak wrote:

> Dear Pantelis,
> You don't reply for a long time. Could you look on it at this time?
> 

Is this a way to address people on a mailing list?

People, especially volunteers are busy with other things too.

Eventually things get merged, but not when people are being rude.

> On 10/15/2013 10:18 PM, Przemyslaw Marczak wrote:
>> Hello Pantelis,
>> 
>> Could you look at this patch, please?
>> 
>> On 10/08/2013 06:12 PM, Przemyslaw Marczak wrote:
>>> Old command timeout value was too small and it caused I/O errors which
>>> led to uncompleted read/write/erase operations and filesystem errors.
>>> Timeout adaptation fixes this issue.
>>> 
>>> Changes in sdhci_send_command() function:
>>> - change timeout variable to static
>>> - increase default command timeout to 100 ms
>>> - add definition of max command timeout value,
>>>   which can be redefined in each board config file
>>> - wait for card ready state for max defined time
>>>   if it doesn't exceed defined maximum or return COMM_ERR
>>> 
>>> Once successfully increased timeout value will be used in next function
>>> call. This fix was tested on Goni, Trats, Trats2 boards by testing UMS
>>> on MMC storage.
>>> 
>>> Changes v2:
>>> - move global variable cmd_timeout into function sdhci_send_command()
>>> - change condition "==" to ">=" when comparing time with timeout
>>> - print information about timeout increasing and card busy timeout
>>> 
>>> Signed-off-by: Przemyslaw Marczak <p.marczak at samsung.com>
>>> Cc: Pantelis Antoniou <panto at antoniou-consulting.com>
>> 
>> Thank you.
>> 
>> 
> 
> Regards
> -- 
> Przemyslaw Marczak
> Samsung R&D Institute Poland
> Samsung Electronics
> p.marczak at samsung.com

Regards

-- Pantelis


More information about the U-Boot mailing list