[PATCH] disk: part: change from hextoul to dectoul

[PATCH] disk: part: change from hextoul to dectoul

[Jaehoon Chung]

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.
-- 
2.25.1

Re: [PATCH] disk: part: change from hextoul to dectoul

[Tom Rini]

[-- Attachment #1: Type: text/plain, Size: 1036 bytes --]

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.

-- 
Tom

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]

Re: [PATCH] disk: part: change from hextoul to dectoul

[Jaehoon Chung]

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


>