On Mon, Oct 4, 2010 at 1:32 PM, John Rigby jcrigby@gmail.com wrote:
Alagu,
This never made it into Andy's tree nor upstream. Could you submit a new patch to current upstream with the config option removed as Andy said he did back in May? Maybe we can get it commited via some other maintainer. Steve Sakoman seems to have done some MMC work lately.
I've done some testing with this patch on OMAP3 and OMAP4, in the "always enabled" version.
I compared the old legacy mmc driver vs the new generic mmc driver vs new generic mmc driver with multiblock read support added.
The good news is that the generic mmc driver for OMAP is almost twice as fast as the old legacy driver. These patches should be in the next u-boot-ti pull request and so will be in mainline shortly.
The bad news is that multiblock reads seem to make no difference :-(
For reading a 45MB file from an ext3 partition using ext2load I get:
legacy: 62 seconds generic: 35 seconds generic w/ multi: 35 seconds
I'm happy to resubmit the multi-block read patch because it does seem to work properly and may even yield improvements on other architectures.
Would you like me to do that?
Steve
Yes please submit updated patches if you have them in your tree. Others might want to participate in a new round of review.
Sandeep