[U-Boot] [PATCH v3 0/6] ARM: kirkwood: mvebu_mmc: Speed up access time

[Pantelis Antoniou]

Hi Gerald,

Last pull-req for mmc was last Friday, I intend to issue another this Friday too.

Regards

— Pantelis

> On Dec 17, 2014, at 11:22 , drEagle <dreagle at doukki.net> wrote:
> 
> Hi,
> 
> Any chance to be included in the next release of u-boot ?
> 
> Regards,
> Gérald
> 
> Le 15/12/2014 12:14, Mario Schuknecht a écrit :
>> 
>> 2014-12-13 21:35 GMT+01:00 Gérald Kerma <drEagle at doukki.net <mailto:drEagle at doukki.net>>:
>> 
>> 
>>    This serie of patches speed up access time of MVEBUMMC driver
>> 
>>    This is allowed by a fix in MVEBUMMC init status check inspired from linux MVSDIO driver.
>>             * Hardware weirdness.  The FIFO_EMPTY bit of the HW_STATE
>>             * register is sometimes not set before a while when some
>>             * "unusual" data block sizes are used (such as with the SWITCH
>>             * command), even despite the fact that the XFER_DONE interrupt
>>             * was raised.  And if another data transfer starts before
>>             * this bit comes to good sense (which eventually happens by
>>             * itself) then the new transfer simply fails with a timeout.
>> 
>>    It allows about 10x to 40x faster access time transfer on SHEEVAPLUG MMC
>>    It may also fixes some SD types incompatibilities
>> 
>>    ### before patch
>> 
>>    Marvell>> ext2load mmc 0:1 0x800000 uImage
>>    1613392 bytes read in 977 ms (1.6 MiB/s)
>> 
>>    ### with fix
>> 
>>    Marvell>> ext2load mmc 0:1 0x800000 uImage
>>    1613392 bytes read in 83 ms (18.5 MiB/s)
>> 
>> 
>> 
>> Looks good to me.
>> ACK to this series of patches:
>> 
>> 
>>    Gérald Kerma (6):
>>      MVEBUMMC : Change copyright date
>>      MVEBUMMC : Speed up access time
>>      MVEBUMMC : FIX debug strings
>>      MVEBUMMC : REMOVE unnecessary delays
>>      MVEBUMMC : CLEAN code
>>      MVEBUMMC : REMOVE unnecessary delay from init
>> 
>> 
>>     drivers/mmc/mvebu_mmc.c | 103 ++++++++++++++++++++++++++++--------------------
>>     include/mvebu_mmc.h     |   1 +
>>     2 files changed, 61 insertions(+), 43 deletions(-)
>> 
>>    --
>>    2.1.3
>>    _______________________________________________
>>    U-Boot mailing list
>>    U-Boot at lists.denx.de <mailto:U-Boot at lists.denx.de>
>>    http://lists.denx.de/mailman/listinfo/u-boot
>> 
>>