On Tue, Aug 11, 2015 at 08:55:35AM -0600, 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.
So, yes. I can see this being a useful way forward for dealing with FAT.
Note that this series is based on u-boot/master without the patch linked above applied. I wonder if it's a little risky to apply this series after -rc1; perhaps the patch above should be applied to v2015.10, and this series immediately after the release?
Yes, it's too late for v2015.10 but we can put this in for the next release assuming there's enough testing and agreement and so forth.