[PATCH] cmd: fs: Use part_get_info_by_dev_and_name_or_num to parse partitions

Heinrich Schuchardt xypron.glpk at gmx.de
Tue Mar 9 22:21:22 CET 2021


Am 9. März 2021 21:56:24 MEZ schrieb Sean Anderson <sean.anderson at seco.com>:
>This allows using dev#partlabel syntax.

Allowing more widespread use of that syntax makes sense to me.

Unfortunately you do not mention which commands and devices are affected.

>
>Signed-off-by: Sean Anderson <sean.anderson at seco.com>
>---
>
> fs/fs.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
>diff --git a/fs/fs.c b/fs/fs.c
>index 900928c394..b7936fd4cf 100644
>--- a/fs/fs.c
>+++ b/fs/fs.c
>@@ -385,8 +385,8 @@ int fs_set_blk_dev(const char *ifname, const char
>*dev_part_str, int fstype)
> 	}
> #endif
> 
>-	part = blk_get_device_part_str(ifname, dev_part_str, &fs_dev_desc,
>-					&fs_partition, 1);
>+	part = part_get_info_by_dev_and_name_or_num(ifname, dev_part_str,
>&fs_dev_desc,
>+						    &fs_partition, 1);
> 	if (part < 0)
> 		return -1;
> 

It would be good if we could test the new capabilities. Can we make use of sandbox_mmc.c?

Best regards

Heinrich






More information about the U-Boot mailing list