diff options
author | Heinrich Schuchardt <heinrich.schuchardt@canonical.com> | 2023-08-15 18:07:36 +0200 |
---|---|---|
committer | Heinrich Schuchardt <heinrich.schuchardt@canonical.com> | 2023-08-19 04:12:52 +0200 |
commit | 4ad85a9fea1ad5c401272102e68398f3bbdd20dd (patch) | |
tree | 9844e35ca53e8d199e6d81be4800a8df8803312c /common/spl | |
parent | 97f4bc372f750fe4fba8b302f277e191a7157df0 (diff) | |
download | u-boot-4ad85a9fea1ad5c401272102e68398f3bbdd20dd.tar.gz u-boot-4ad85a9fea1ad5c401272102e68398f3bbdd20dd.tar.bz2 u-boot-4ad85a9fea1ad5c401272102e68398f3bbdd20dd.zip |
spl: don't assume NVMe partition 1 exists
There is no requirement that a partition 1 exists in a partition table.
We should not try to retrieve information about it.
We should not even try reading with partition number
CONFIG_SYS_NVME_BOOT_PARTITION here as this is done in the fs_set_blk_dev()
call anyway.
Fixes: 8ce6a2e17577 ("spl: blk: Support loading images from fs")
Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>
Reviewed-by: Tom Rini <trini@konsulko.com>
Diffstat (limited to 'common/spl')
-rw-r--r-- | common/spl/spl_blk_fs.c | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/common/spl/spl_blk_fs.c b/common/spl/spl_blk_fs.c index 16ecece702..eb6f526689 100644 --- a/common/spl/spl_blk_fs.c +++ b/common/spl/spl_blk_fs.c @@ -44,7 +44,6 @@ int spl_blk_load_image(struct spl_image_info *spl_image, enum uclass_id uclass_id, int devnum, int partnum) { const char *filename = CONFIG_SPL_FS_LOAD_PAYLOAD_NAME; - struct disk_partition part_info = {}; struct legacy_img_hdr *header; struct blk_desc *blk_desc; loff_t actlen, filesize; @@ -59,11 +58,6 @@ int spl_blk_load_image(struct spl_image_info *spl_image, blk_show_device(uclass_id, devnum); header = spl_get_load_buffer(-sizeof(*header), sizeof(*header)); - ret = part_get_info(blk_desc, 1, &part_info); - if (ret) { - printf("spl: no partition table found. Err - %d\n", ret); - goto out; - } dev.ifname = blk_get_uclass_name(uclass_id); snprintf(dev.dev_part_str, sizeof(dev.dev_part_str) - 1, "%x:%x", |