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. _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot