Hi,
On 5 March 2018 at 02:43, tien.fong.chee@intel.com wrote:
From: Tien Fong Chee tien.fong.chee@intel.com
This is file system generic loader which can be used to load the file image from the storage into target such as memory. The consumer driver would then use this loader to program whatever, ie. the FPGA device.
Signed-off-by: Tien Fong Chee tien.fong.chee@intel.com Reviewed-by: Lothar Waßmann LW@KARO-electronics.de
common/Kconfig | 10 ++ common/Makefile | 1 + common/fs_loader.c | 353 +++++++++++++++++++++++++++++++++++++++++++++ doc/README.firmware_loader | 86 +++++++++++ include/fs_loader.h | 28 ++++ 5 files changed, 478 insertions(+) create mode 100644 common/fs_loader.c create mode 100644 doc/README.firmware_loader create mode 100644 include/fs_loader.h
This looks fine as a concept but I am not keen on the implementation.
1. It should use driver model (only) in U-Boot proper. If there is some SPL problem then add a specific function or feature for SPL. 2. It should not be necessary ti manually init subsystems - driver model does this for you 3. You can use the uclass name to find things
Please let me know if you need more info.
Regards, Simon