Hi,
On Thu, 24 Jan 2019 at 03:24, tien.fong.chee@intel.com wrote:
From: Tien Fong Chee tien.fong.chee@intel.com
Firmware loader would encounter problem if the block device is accessed before initializing it. This patch would adding the support of probing block device and initializing block before the block device is accessed by firmware loader.
Signed-off-by: Tien Fong Chee tien.fong.chee@intel.com
Changes in v3:
- Initializing block device through probing the blk device
Changes in v2:
- Initializing MMC through probing the blk device
drivers/misc/fs_loader.c | 26 ++++++++++++++++++++++++++ 1 file changed, 26 insertions(+)
Looks good, a few nits below.
Also at some point (separate patch) we really shoud have a test for UCLASS_FS_FIRMWARE_LOADER,.
diff --git a/drivers/misc/fs_loader.c b/drivers/misc/fs_loader.c index 57a14a3..df35ec6 100644 --- a/drivers/misc/fs_loader.c +++ b/drivers/misc/fs_loader.c @@ -12,6 +12,7 @@ #include <linux/string.h> #include <mapmem.h> #include <malloc.h> +#include <mmc.h>
Can you drop this now?
#include <spl.h>
DECLARE_GLOBAL_DATA_PTR; @@ -252,6 +253,31 @@ static int fs_loader_ofdata_to_platdata(struct udevice *dev)
static int fs_loader_probe(struct udevice *dev) { +#if CONFIG_IS_ENABLED(DM) && CONFIG_IS_ENABLED(BLK)
int ret;struct device_platdata *plat = dev->platdata;if (plat->phandlepart.phandle) {ofnode node = ofnode_get_by_phandle(plat->phandlepart.phandle);
drop extra blank line
struct udevice *parent_dev = NULL;ret = device_get_global_by_ofnode(node, &parent_dev);
here too
if (!ret) {struct udevice *dev;ret = blk_get_from_parent(parent_dev, &dev);if (ret) {debug("fs_loader: No block device: %d\n",ret);return ret;}}}+#endif
return 0;};
-- 2.2.0