[U-Boot] [PATCH 0/9] Replace the FAT filesystem code

[Stephen Warren]

On 08/07/2015 09:09 PM, Stephen Warren wrote:
> The existing FAT filesystem implementation in U-Boot has some bugs that
> are tricky to fix cleanly without significant rework of the code. For
> example, see:
> 
> http://lists.denx.de/pipermail/u-boot/2015-July/221054.html
> [PATCH] fat: handle paths that include ../
> 
> This series replaces U-Boot's FAT filesystem implementation with the "ff"
> project; http://elm-chan.org/fsw/ff/00index_e.html. This project appears
> to be actively maintained, does not have at least the bug referenced in
> the patch link above, and is quite easy to integrate into U-Boot. An
> earlier version is also used in Barebox, so presumably it's had some
> level of testing/exposure there.

I've found a few issues with this series:

1) A warning when only building R/O FAT support, but not R/W FAT
support. I have fixed this locally.

2) Some functions only used by the SPL FAT support are missing, causing
build errors on some boards. I have fixed this locally. There are now no
new buildman failures on any architecture supported by buildman's
toolchains.

3) While the new FAT library (and the U-Boot integration code I wrote)
works on some test cases, others such as loading /zImage on an rpi, or
running ./test/fs/fs-test.sh fail. I'm in the process of investigating
these issues.

Still, I'd appreciate any thoughts on the general idea behind this series.