[PATCH] disk: part: change from hextoul to dectoul
When 'ls' command is running with partition number, it's passed by a hex value. For example, if want to check a 15th partition, it has to input as 0xf. Before applied - ls mmc 0:f After applied - ls mmc 0:15
The using decimal number is more readable than passed by a hex value.
Signed-off-by: Jaehoon Chung jh80.chung@samsung.com --- disk/part.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/disk/part.c b/disk/part.c index 79955c7fb000..31a42d8a2312 100644 --- a/disk/part.c +++ b/disk/part.c @@ -540,7 +540,7 @@ int blk_get_device_part_str(const char *ifname, const char *dev_part_str, part = PART_AUTO; } else { /* Something specified -> use exactly that */ - part = (int)hextoul(part_str, &ep); + part = (int)dectoul(part_str, &ep); /* * Less than whole string converted, * or request for whole device, but caller requires partition.
On Fri, Jun 17, 2022 at 07:01:59PM +0900, Jaehoon Chung wrote:
When 'ls' command is running with partition number, it's passed by a hex value. For example, if want to check a 15th partition, it has to input as 0xf. Before applied
- ls mmc 0:f
After applied
- ls mmc 0:15
The using decimal number is more readable than passed by a hex value.
Signed-off-by: Jaehoon Chung jh80.chung@samsung.com
NAK. I agree it's a more obvious way of interacting. But the CLI is our API with our users and there's lots of stuff out there today that knows (and then deals with) needing to pass hex not decimal for partitions. And given SoCs that have large numbers of partitions, you can't assume that "12" isn't in intentional and referring to partition 18 in decimal.
We could maybe do this with a flag or environment variable to allow people to opt-in, since that would at least allow for multi-platform scripts to know what to expect and not have to guess / hard-code per platform.
Hi Tom,
On 6/17/22 21:27, Tom Rini wrote:
On Fri, Jun 17, 2022 at 07:01:59PM +0900, Jaehoon Chung wrote:
When 'ls' command is running with partition number, it's passed by a hex value. For example, if want to check a 15th partition, it has to input as 0xf. Before applied
- ls mmc 0:f
After applied
- ls mmc 0:15
The using decimal number is more readable than passed by a hex value.
Signed-off-by: Jaehoon Chung jh80.chung@samsung.com
NAK. I agree it's a more obvious way of interacting. But the CLI is our API with our users and there's lots of stuff out there today that knows (and then deals with) needing to pass hex not decimal for partitions. And given SoCs that have large numbers of partitions, you can't assume that "12" isn't in intentional and referring to partition 18 in decimal.
We could maybe do this with a flag or environment variable to allow people to opt-in, since that would at least allow for multi-platform scripts to know what to expect and not have to guess / hard-code per platform.
Okay. Frankly, I were not sure that it needs to change or not. In our boot script, I'm using the hex value and decimal value as other environment variable. I agreed yours.
Best Regards, Jaehoon Chung
participants (2)
-
Jaehoon Chung -
Tom Rini