Generic firmware loader

From: Tien Fong Chee <[email protected]>

This patch removes the static declation on spl_mmc_find_device_function so this function is accessible by the caller from other file. This patch is required for later patch.

Signed-off-by Tien Fong Chee <[email protected]>

Reviewed-by Tom Rini <[email protected]>

---

From: Tien Fong Chee <[email protected]>

cmd_ubifs_umount() function would be called directly instead of involving whole command machinery in generic firmware loader, so checking on ubifs_initialized status need to be done in cmd_ubifs_umount() without breaking original functionality design.

Signed-off-by Tien Fong Chee <[email protected]>64db5518baa2ea1a8a0e81cc485d760b850c7052

From: Tien Fong Chee <[email protected]>

cmd_ubifs_mount() function would be called directly instead of involving whole command machinery for mounting ubifs in generic firmware loader, so some checking codes need to be factored out into cmd_ubifs_mount() without breaking original functionality design.

Signed-off-by Tien Fong Chee <[email protected]>

Quoted 11 lines · tien.fong.chee at intel.com

From: Tien Fong Chee <[email protected]>

cmd_ubifs_mount() function would be called directly instead of involving whole command machinery for mounting ubifs in generic firmware loader, so some checking codes need to be factored out
into cmd_ubifs_mount() without breaking original functionality design.

Signed-off-by Tien Fong Chee <[email protected]>

[...]

Quoted 1 line

Quoted 10 lines

include/ubi_uboot.h +0 -0

index 827dbfc..fb300e8 100644
@@ -76,5 +76,6 @@ extern int ubi_volume_read(char *volume, char
 *buf, size_t size);

 extern struct ubi_device *ubi_devices[];
 int cmd_ubifs_umount(void);
+int cmd_ubifs_mount(char *vol_name);

Swap these two -- mount first and umount seconds.

With that

Reviewed-by Marek Vasut <[email protected]>

From: Tien Fong Chee <[email protected]>

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 <[email protected]>

Reviewed-by Lothar Waßmann <[email protected]>

common/fs_loader.c new file +17 -0

+#include <spl.h>
+#endif
+#include <linux/string.h>
+#ifdef CONFIG_CMD_UBIFS
+#include <ubi_uboot.h>
+#include <ubifs_uboot.h>
+#endif
+#include <usb.h>
+
+struct firmware_priv {
+	const char *name;	/* Filename */
+	u32 offset;		/* Offset of reading a file */
+};
+
+static struct device_location default_locations[] = {
+	{
+		.name = "mmc",

common/fs_loader.c new file +5 -0

+};
+
+/* USB build is not supported yet in SPL */
+#ifndef CONFIG_SPL_BUILD
+#ifdef CONFIG_USB_STORAGE

common/fs_loader.c new file +9 -0

+	if (err)
+		return err;
+#endif
+
+	return err;
+}
+#else
+static int init_usb(void)
+{

common/fs_loader.c new file +37 -0

+}
+#else
+static int init_storage_sata(void)
+{
+	debug("Error: Cannot load image: no SATA support\n");
+	return -ENOSYS;
+}
+#endif
+
+#ifdef CONFIG_CMD_UBIFS
+static int mount_ubifs(struct device_location *location)
+{
+	int ret;
+	char cmd[32];
+
+	if (ret = ubi_part(location->mtdpart, NULL)) {
+		debug("Cannot find mtd partition %s\n", location->mtdpart);
+		return ret;
+	}
+
+	return cmd_ubifs_mount(location->ubivol);
+}
+
+static int umount_ubifs(void)
+{
+	return cmd_ubifs_umount();
+}
+#else
+static int mount_ubifs(struct device_location *location)
+{
+	debug("Error: Cannot load image: no UBIFS support\n");
+	return -ENOSYS;
+}
+#endif
+
+#if defined(CONFIG_SPL_MMC_SUPPORT) && defined(CONFIG_SPL_BUILD)
+__weak int init_mmc(void)

common/fs_loader.c new file +18 -0

+	return -ENOENT;
+#endif
+
+	err = mmc_init(mmc);
+	if (err) {
+		debug("spl: mmc init failed with error: %d\n", err);
+		return err;
+	}
+
+	return err;
+}
+#else
+__weak int init_mmc(void)
+{
+	/* Expect somewhere already initialize MMC */
+	return 0;
+}
+#endif

Quoted 10 lines · tien.fong.chee at intel.com

From: Tien Fong Chee <[email protected]>

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 <[email protected]>

Reviewed-by Lothar Waßmann <[email protected]>

[...]

Quoted 1 line

common/fs_loader.c line 15

Quoted 9 lines

common/fs_loader.c new file +9 -0

+#include <common.h>
+#include <errno.h>
+#include <fs.h>
+#include <fs_loader.h>
+#include <nand.h>
+#include <sata.h>
+#include <spi.h>
+#include <spi_flash.h>
+#ifdef CONFIG_SPL

Are the ifdefs needed ?

Quoted 18 lines

common/fs_loader.c new file +17 -0

+#include <spl.h>
+#endif
+#include <linux/string.h>
+#ifdef CONFIG_CMD_UBIFS
+#include <ubi_uboot.h>
+#include <ubifs_uboot.h>
+#endif
+#include <usb.h>
+
+struct firmware_priv {
+	const char *name;	/* Filename */
+	u32 offset;		/* Offset of reading a file */
+};
+
+static struct device_location default_locations[] = {
+	{
+		.name = "mmc",

common/fs_loader.c line 42

Quoted 10 lines

common/fs_loader.c new file +10 -0

+		.devpart = "0:1",
+	},
+	{
+		.name = "usb",
+		.devpart = "0:1",
+	},
+	{
+		.name = "sata",
+		.devpart = "0:1",
+	},

How can this load from UBI if it's not listed here ?

Quoted 1 line

Quoted 5 lines

common/fs_loader.c new file +5 -0

+};
+
+/* USB build is not supported yet in SPL */
+#ifndef CONFIG_SPL_BUILD
+#ifdef CONFIG_USB_STORAGE

common/fs_loader.c line 57

Quoted 10 lines

common/fs_loader.c new file +10 -0

+static int init_usb(void)
+{
+	int err;
+
+	err = usb_init();
+	if (err)
+		return err;
+
+#ifndef CONFIG_DM_USB
+	err = usb_stor_scan(1);

mode = 0 I think, we don't need this verbose output.

Quoted 14 lines

common/fs_loader.c new file +13 -0

+	if (err)
+		return err;
+#endif
+
+	return err;
+}
+#else
+static int init_usb(void)
+{
+	debug("Error: Cannot load flash image: no USB support\n");
+	return -ENOSYS;
+}
+#endif

common/fs_loader.c line 76

Quoted 6 lines

common/fs_loader.c new file +6 -0

+#endif
+
+#ifdef CONFIG_SATA
+static int init_storage_sata(void)
+{
+	return sata_probe(0);

Shouldn't the sata device number be pulled from devpart in default_locations table ?

Quoted 43 lines

common/fs_loader.c new file +17 -0

+}
+#else
+static int init_storage_sata(void)
+{
+	debug("Error: Cannot load image: no SATA support\n");
+	return -ENOSYS;
+}
+#endif
+
+#ifdef CONFIG_CMD_UBIFS
+static int mount_ubifs(struct device_location *location)
+{
+	int ret;
+	char cmd[32];
+
+	if (ret = ubi_part(location->mtdpart, NULL)) {
+		debug("Cannot find mtd partition %s\n", location-

Quoted 1 line

mtdpart);

common/fs_loader.c new file +24 -0

+		return ret;
+	}
+
+	return cmd_ubifs_mount(location->ubivol);
+}
+
+static int umount_ubifs(void)
+{
+	return cmd_ubifs_umount();
+}
+#else
+static int mount_ubifs(struct device_location *location)
+{
+	debug("Error: Cannot load image: no UBIFS support\n");
+	return -ENOSYS;
+}
+#endif
+
+#if defined(CONFIG_SPL_MMC_SUPPORT) && defined(CONFIG_SPL_BUILD)
+__weak int init_mmc(void)
+{
+	/* Just for the case MMC is not yet initialized */
+	struct mmc *mmc = NULL;
+	int err;

common/fs_loader.c line 123

Quoted 7 lines

common/fs_loader.c new file +5 -0

+
+#ifdef CONFIG_SPL
+	spl_mmc_find_device(&mmc, spl_boot_device());
+#else
+	debug("#define CONFIG_SPL is required or overrriding

%s\n",

common/fs_loader.c new file +1 -0

+	      __func__);

This can be a compile-time error, maybe ?

Quoted 20 lines

common/fs_loader.c new file +6 -0

+	return -ENOENT;
+#endif
+
+	err = mmc_init(mmc);
+	if (err) {
+		debug("spl: mmc init failed with error: %d\n",

err);

common/fs_loader.c new file +12 -0

+		return err;
+	}
+
+	return err;
+}
+#else
+__weak int init_mmc(void)
+{
+	/* Expect somewhere already initialize MMC */
+	return 0;
+}
+#endif

[...]

Quoted 14 lines · Marek Vasut

Quoted 10 lines · tien.fong.chee at intel.com

From: Tien Fong Chee <[email protected]>

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 <[email protected]>

Reviewed-by Lothar Waßmann <[email protected]>

[...]

Quoted 1 line

Quoted 11 lines

common/fs_loader.c line 15

Quoted 9 lines

common/fs_loader.c new file +9 -0

+#include <common.h>
+#include <errno.h>
+#include <fs.h>
+#include <fs_loader.h>
+#include <nand.h>
+#include <sata.h>
+#include <spi.h>
+#include <spi_flash.h>
+#ifdef CONFIG_SPL

Are the ifdefs needed ?

Because spl.h contains some codes have its dependency with SPL. So, Tom adviced to make this part of code depend on CONFIG_SPL. However, only __weak int init_mmc() depend on the codes from spl.h, so user can override their own init_mmc() if SPL is not used.

Quoted 20 lines

Quoted 20 lines

common/fs_loader.c new file +20 -0

+#include <spl.h>
+#endif
+#include <linux/string.h>
+#ifdef CONFIG_CMD_UBIFS
+#include <ubi_uboot.h>
+#include <ubifs_uboot.h>
+#endif
+#include <usb.h>
+
+struct firmware_priv {
+	const char *name;	/* Filename */
+	u32 offset;		/* Offset of reading a file */
+};
+
+static struct device_location default_locations[] = {
+	{
+		.name = "mmc",
+		.devpart = "0:1",
+	},
+	{

Quoted 8 lines

common/fs_loader.c line 42

Quoted 7 lines

common/fs_loader.c new file +7 -0

+		.name = "usb",
+		.devpart = "0:1",
+	},
+	{
+		.name = "sata",
+		.devpart = "0:1",
+	},

How can this load from UBI if it's not listed here ?

Quoted 7 lines

common/fs_loader.c line 76

Quoted 4 lines

common/fs_loader.c new file +4 -0

+#ifdef CONFIG_SATA
+static int init_storage_sata(void)
+{
+	return sata_probe(0);

Shouldn't the sata device number be pulled from devpart in default_locations table ?

This may need to add the logic for splitting the device number and partition into integer from the string. Let me think how to do it, may be i can leverage existing code.

Quoted 8 lines

common/fs_loader.c line 123

Quoted 6 lines

common/fs_loader.c new file +4 -0

+#ifdef CONFIG_SPL
+	spl_mmc_find_device(&mmc, spl_boot_device());
+#else
+	debug("#define CONFIG_SPL is required or overrriding

%s\n",

common/fs_loader.c new file +1 -0

+	      __func__);

This can be a compile-time error, maybe ?

No compile error. When you open the file, "%s\n" is actually same line with debug("..... .

Quoted 21 lines

Quoted 20 lines

common/fs_loader.c new file +6 -0

+	return -ENOENT;
+#endif
+
+	err = mmc_init(mmc);
+	if (err) {
+		debug("spl: mmc init failed with error: %d\n",

err);

common/fs_loader.c new file +12 -0

+		return err;
+	}
+
+	return err;
+}
+#else
+__weak int init_mmc(void)
+{
+	/* Expect somewhere already initialize MMC */
+	return 0;
+}
+#endif

[...]

Quoted 26 lines · Tien Fong

Quoted 25 lines · Marek Vasut

Quoted 10 lines · tien.fong.chee at intel.com

From: Tien Fong Chee <[email protected]>

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 <[email protected]>

Reviewed-by Lothar Waßmann <[email protected]>

[...]

common/fs_loader.c line 15

Quoted 10 lines

common/fs_loader.c new file +9 -0

+#include <common.h>
+#include <errno.h>
+#include <fs.h>
+#include <fs_loader.h>
+#include <nand.h>
+#include <sata.h>
+#include <spi.h>
+#include <spi_flash.h>
+#ifdef CONFIG_SPL

Are the ifdefs needed ?

Quoted 4 lines

Because spl.h contains some codes have its dependency with SPL. So, Tom adviced to make this part of code depend on CONFIG_SPL. However, only __weak int init_mmc() depend on the codes from spl.h, so user can override their own init_mmc() if SPL is not used.

You probably dont need those ifdefs around headers.

Quoted 33 lines · Marek Vasut

Quoted 31 lines · Tien Fong

Quoted 29 lines · Marek Vasut

Quoted 12 lines · tien.fong.chee at intel.com

From: Tien Fong Chee <[email protected]>

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 <[email protected]>

Reviewed-by Lothar Waßmann <[email protected]>

[...]

common/fs_loader.c line 15

Quoted 11 lines

common/fs_loader.c new file +9 -0

+#include <common.h>
+#include <errno.h>
+#include <fs.h>
+#include <fs_loader.h>
+#include <nand.h>
+#include <sata.h>
+#include <spi.h>
+#include <spi_flash.h>
+#ifdef CONFIG_SPL

Are the ifdefs needed ?

Quoted 7 lines

Quoted 6 lines

Because spl.h contains some codes have its dependency with SPL. So, Tom
adviced to make this part of code depend on CONFIG_SPL. However, only __weak int init_mmc() depend on the codes from spl.h, so
user can override their own init_mmc() if SPL is not used.

You probably dont need those ifdefs around headers.

In this case, we do.  You can only include <spl.h> on architectures which have SPL support.

I wouldn't object to a separate patch series that adds a dummy asm-generic/spl.h and we go that route, if it also cleans up more of the
code in general.  But I think that's separate from this series.  Thanks!

Quoted 31 lines · Tien Fong

Quoted 18 lines · Marek Vasut

Quoted 12 lines · tien.fong.chee at intel.com

From: Tien Fong Chee <[email protected]>

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 <[email protected]>

Reviewed-by Lothar Waßmann <[email protected]>

[...]

Quoted 2 lines

Quoted 11 lines

common/fs_loader.c line 15

Quoted 9 lines

common/fs_loader.c new file +9 -0

+#include <common.h>
+#include <errno.h>
+#include <fs.h>
+#include <fs_loader.h>
+#include <nand.h>
+#include <sata.h>
+#include <spi.h>
+#include <spi_flash.h>
+#ifdef CONFIG_SPL

Are the ifdefs needed ?

Quoted 6 lines

Because spl.h contains some codes have its dependency with SPL. So, Tom
adviced to make this part of code depend on CONFIG_SPL. However, only __weak int init_mmc() depend on the codes from spl.h, so
user can override their own init_mmc() if SPL is not used.

You probably dont need those ifdefs around headers.

Quoted 24 lines

Quoted 24 lines

Quoted 24 lines

common/fs_loader.c new file +12 -0

+#include <spl.h>
+#endif
+#include <linux/string.h>
+#ifdef CONFIG_CMD_UBIFS
+#include <ubi_uboot.h>
+#include <ubifs_uboot.h>
+#endif
+#include <usb.h>
+
+struct firmware_priv {
+	const char *name;	/* Filename */
+	u32 offset;		/* Offset of reading a file

*/

common/fs_loader.c new file +8 -0

+};
+
+static struct device_location default_locations[] = {
+	{
+		.name = "mmc",
+		.devpart = "0:1",
+	},
+	{

Quoted 8 lines

Quoted 8 lines

common/fs_loader.c line 42

Quoted 7 lines

common/fs_loader.c new file +7 -0

+		.name = "usb",
+		.devpart = "0:1",
+	},
+	{
+		.name = "sata",
+		.devpart = "0:1",
+	},

How can this load from UBI if it's not listed here ?

Well ?

[...]

Quoted 10 lines

Quoted 10 lines

common/fs_loader.c line 76

Quoted 7 lines

common/fs_loader.c new file +4 -0

+#ifdef CONFIG_SATA
+static int init_storage_sata(void)
+{
+	return sata_probe(0);

Shouldn't the sata device number be pulled from devpart in default_locations table ?

Quoted 4 lines

This may need to add the logic for splitting the device number and partition into integer from the string. Let me think how to do it, may
be i can leverage existing code.

strtok(), strtol() or something ?

[...]

Quoted 11 lines

Quoted 11 lines

common/fs_loader.c line 123

Quoted 9 lines

common/fs_loader.c new file +4 -0

+#ifdef CONFIG_SPL
+	spl_mmc_find_device(&mmc, spl_boot_device());
+#else
+	debug("#define CONFIG_SPL is required or overrriding

%s\n",

common/fs_loader.c new file +1 -0

+	      __func__);

This can be a compile-time error, maybe ?

Quoted 3 lines

No compile error. When you open the file, "%s\n" is actually same line
with debug("..... .

So what ? This missing macro can be detected at runtime, heck this code
can even depend on CONFIG_SPL in Kconfig.

Quoted 24 lines

Quoted 24 lines

Quoted 23 lines

common/fs_loader.c new file +6 -0

+	return -ENOENT;
+#endif
+
+	err = mmc_init(mmc);
+	if (err) {
+		debug("spl: mmc init failed with error: %d\n",

err);

common/fs_loader.c new file +12 -0

+		return err;
+	}
+
+	return err;
+}
+#else
+__weak int init_mmc(void)
+{
+	/* Expect somewhere already initialize MMC */
+	return 0;
+}
+#endif

[...]

From: Tien Fong Chee <[email protected]>

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 <[email protected]>

Reviewed-by Lothar Waßmann <[email protected]>

---
 common/Kconfig             |   9 ++
 common/Makefile            |   1 +
 common/fs_loader.c         | 320 +++++++++++++++++++++++++++++++++++++++++++++
 doc/README.firmware_loader |  76 +++++++++++
 include/fs_loader.h        |  28 ++++
 5 files changed, 434 insertions(+)
 create mode 100644 common/fs_loader.c
 create mode 100644 doc/README.firmware_loader
 create mode 100644 include/fs_loader.h

Hi,

Quoted 22 lines · On Mon,  5 Feb 2018 15:06:49 +0800 tien.fong.chee at intel.com wrote:

From: Tien Fong Chee <[email protected]>

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 <[email protected]>

Reviewed-by Lothar Waßmann <[email protected]>

---
 common/Kconfig             |   9 ++
 common/Makefile            |   1 +
 common/fs_loader.c         | 320
+++++++++++++++++++++++++++++++++++++++++++++
 doc/README.firmware_loader |  76 +++++++++++
 include/fs_loader.h        |  28 ++++

 5 files changed, 434 insertions(+)

 create mode 100644 common/fs_loader.c
 create mode 100644 doc/README.firmware_loader
 create mode 100644 include/fs_loader.h

[...]

Quoted 1 line

Quoted 11 lines

common/fs_loader.c new file +11 -0

+#if defined(CONFIG_SPL_MMC_SUPPORT) && defined(CONFIG_SPL_BUILD)
+__weak int init_mmc(void)
+{
+	/* Just for the case MMC is not yet initialized */
+	struct mmc *mmc = NULL;
+	int err;
+
+#ifdef CONFIG_SPL
+	spl_mmc_find_device(&mmc, spl_boot_device());
+#else
+	debug("#define CONFIG_SPL is required or overrriding

Quoted 2 lines

%s\n",

s/overrr/overr/

Lothar Waßmann

From: Tien Fong Chee <[email protected]>

This patchset contains generic firmware loader which is very close to Linux
firmware loader but for U-Boot framework. Generic firmware loader can be used
load whatever into target location, and then consumer driver would use it to
program whatever, ie. the FPGA. This version mainly resolved comments from
Marek Vasut in [v7].

This series is working on top of u-boot.git -
 http://git.denx.de/u-boot.git .

[v7]: https://www.mail-archive.com/u-boot at lists.denx.de/msg276775.htm l

v7 -> v8 changes:
-----------------
- Return cmd_ubifs_umount() directly.
- Call the UBI functions directly instead of invoking the U-Boot command parser.

Patchset history
----------------
[v1]: https://www.mail-archive.com/u-boot at lists.denx.de/msg271905.htm l
[v2]: https://www.mail-archive.com/u-boot at lists.denx.de/msg271979.htm l
[v3]: https://www.mail-archive.com/u-boot at lists.denx.de/msg272039.htm l
[v4]: https://www.mail-archive.com/u-boot at lists.denx.de/msg272432.htm l
[v5]: https://www.mail-archive.com/u-boot at lists.denx.de/msg272771.htm l
[v6]: https://www.mail-archive.com/u-boot at lists.denx.de/msg273961.htm l

Tien Fong Chee (4):
  spl: Remove static declaration on spl_mmc_find_device function
  cmd: ubifs: Move ubifs_initialized checking into cmd_ubifs_umount()
  cmd: ubifs: Factor out some checking codes into cmd_ubifs_mount()
  common: Generic firmware loader for file system

 cmd/ubifs.c                |  40 +++---
 common/Kconfig             |   9 ++
 common/Makefile            |   1 +
 common/fs_loader.c         | 320
+++++++++++++++++++++++++++++++++++++++++++++
 common/spl/spl_mmc.c       |   2 +-
 doc/README.firmware_loader |  76 +++++++++++
 include/fs_loader.h        |  28 ++++
 include/spl.h              |   2 +
 include/ubi_uboot.h        |   2 +

 9 files changed, 462 insertions(+), 18 deletions(-)

 create mode 100644 common/fs_loader.c
 create mode 100644 doc/README.firmware_loader
 create mode 100644 include/fs_loader.h

From: Tien Fong Chee <[email protected]>

This patchset contains generic firmware loader which is very close to Linux
firmware loader but for U-Boot framework. Generic firmware loader can be used
load whatever into target location, and then consumer driver would use it to
program whatever, ie. the FPGA. This version mainly resolved comments from
Marek Vasut in [v7].

This series is working on top of u-boot.git -
 http://git.denx.de/u-boot.git .

[v7]: https://www.mail-archive.com/u-boot at lists.denx.de/msg276775.htm l

v7 -> v8 changes:
-----------------
- Return cmd_ubifs_umount() directly.
- Call the UBI functions directly instead of invoking the U-Boot command parser.

Patchset history
----------------
[v1]: https://www.mail-archive.com/u-boot at lists.denx.de/msg271905.htm l
[v2]: https://www.mail-archive.com/u-boot at lists.denx.de/msg271979.htm l
[v3]: https://www.mail-archive.com/u-boot at lists.denx.de/msg272039.htm l
[v4]: https://www.mail-archive.com/u-boot at lists.denx.de/msg272432.htm l
[v5]: https://www.mail-archive.com/u-boot at lists.denx.de/msg272771.htm l
[v6]: https://www.mail-archive.com/u-boot at lists.denx.de/msg273961.htm l

Tien Fong Chee (4):
  spl: Remove static declaration on spl_mmc_find_device function
  cmd: ubifs: Move ubifs_initialized checking into cmd_ubifs_umount()
  cmd: ubifs: Factor out some checking codes into cmd_ubifs_mount()
  common: Generic firmware loader for file system

 cmd/ubifs.c                |  40 +++---
 common/Kconfig             |   9 ++
 common/Makefile            |   1 +
 common/fs_loader.c         | 320
+++++++++++++++++++++++++++++++++++++++++++++
 common/spl/spl_mmc.c       |   2 +-
 doc/README.firmware_loader |  76 +++++++++++
 include/fs_loader.h        |  28 ++++
 include/spl.h              |   2 +
 include/ubi_uboot.h        |   2 +

 9 files changed, 462 insertions(+), 18 deletions(-)

 create mode 100644 common/fs_loader.c
 create mode 100644 doc/README.firmware_loader
 create mode 100644 include/fs_loader.h