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. Or we take a look at borrowing the kernel's code, similar to how we leverage UBIFS today. Regardless, thanks for the time you've already put in on this! -- Tom
signature.asc
Description: Digital signature
_______________________________________________ U-Boot mailing list [email protected] http://lists.denx.de/mailman/listinfo/u-boot
