[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