On 10/20/2015 01:10 PM, Tom Rini wrote:
On Tue, Oct 20, 2015 at 12:57:32PM -0600, Stephen Warren wrote:
On 10/02/2015 12:06 AM, Stephen Warren wrote:
Enhance f_read() to find the maximum contiguous set of clusters to read,
and read it all at once (which is fast) rather one by one (which is
slow).
Hmm. I had hoped that the author of ff.c would accept this patch
upstream, so we could pick up a later upstream version that included
this patch. However, it seems the author of ff.c has a policy of not
accepting outside contributions:
http://elm-chan.org/fsw/ff/bd/?show=2472
(That's a link to the author's reply to my patch, on the forum
system associated with his/her SW)
The bit about the license is at
http://elm-chan.org/fsw/ff/en/appnote.html#license
I wonder how much of a liability incorporating ff.c into U-Boot will
be, if we can't ever get any fixes merged upstream. Perhaps we just
fork it, although I had hoped we'd be able to keep picking up new
versions.
Arg, that really does take away one of the potential nice features. I
guess, sadly, at this point I'd rather stick with the version we have
unless you want to deal with re-syncing their releases but still
effectively doing a fork (so that we can also make use of caches which I
think you said before you thought might be part of the performance
problem.
FWIW, the caching (of FAT bitmap, not CPU memory) turned out not to be
an issue at all; it was just my wild conjecture before I'd investigated
the read performance issues.
_______________________________________________
U-Boot mailing list
[email protected]
http://lists.denx.de/mailman/listinfo/u-boot
