* [PATCH 0/2] Allow decimally formatted data for the commands "i2c_write" and "spi"
@ 2017-10-19 6:53 Ulrich Ölmann
2017-10-19 6:53 ` [PATCH 1/2] commands: i2c: do not restrict i2c_write's data to be hex-formatted Ulrich Ölmann
` (3 more replies)
0 siblings, 4 replies; 6+ messages in thread
From: Ulrich Ölmann @ 2017-10-19 6:53 UTC (permalink / raw)
To: Barebox List
Currently both commands set the default base for parsing user supplied data to
16 which makes it impossible at all to parse decimal data.
Presumably this originates from having the convenience to easily copy'n'paste
hex dumped data into these commands without further preprocessing it before.
Change the default base to 0 to particularly allow decimally formatted data.
This is in accordance to what the "mw" command does.
Changes since (implicit) v1 (20171017) (http://lists.infradead.org/pipermail/barebox/2017-October/031356.html):
- extended the commit messages to include the motivation for the change
- added Uwe's acked-by
Ulrich Ölmann (2):
commands: i2c: do not restrict i2c_write's data to be hex-formatted
commands: spi: do not restrict spi's data to be hex-formatted
commands/i2c.c | 2 +-
commands/spi.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
--
2.11.0
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
^ permalink raw reply [flat|nested] 6+ messages in thread* [PATCH 1/2] commands: i2c: do not restrict i2c_write's data to be hex-formatted
2017-10-19 6:53 [PATCH 0/2] Allow decimally formatted data for the commands "i2c_write" and "spi" Ulrich Ölmann
@ 2017-10-19 6:53 ` Ulrich Ölmann
2017-10-19 6:53 ` [PATCH 2/2] commands: spi: do not restrict spi's " Ulrich Ölmann
` (2 subsequent siblings)
3 siblings, 0 replies; 6+ messages in thread
From: Ulrich Ölmann @ 2017-10-19 6:53 UTC (permalink / raw)
To: Barebox List
Change the default base to 0 to be consistent with the "mw" command and
particularly allow decimally formatted data.
The price to pay is needing the usual prefix "0x" for each hex-formatted number.
Signed-off-by: Ulrich Ölmann <u.oelmann@pengutronix.de>
Acked-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
---
commands/i2c.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/commands/i2c.c b/commands/i2c.c
index 573032ab1588..b74c53509f38 100644
--- a/commands/i2c.c
+++ b/commands/i2c.c
@@ -129,7 +129,7 @@ static int do_i2c_write(int argc, char *argv[])
buf = xmalloc(count);
for (i = 0; i < count; i++)
- *(buf + i) = (char) simple_strtol(argv[optind+i], NULL, 16);
+ *(buf + i) = (char) simple_strtol(argv[optind+i], NULL, 0);
ret = i2c_write_reg(&client, reg | wide, buf, count);
if (ret != count) {
--
2.11.0
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
^ permalink raw reply [flat|nested] 6+ messages in thread* [PATCH 2/2] commands: spi: do not restrict spi's data to be hex-formatted
2017-10-19 6:53 [PATCH 0/2] Allow decimally formatted data for the commands "i2c_write" and "spi" Ulrich Ölmann
2017-10-19 6:53 ` [PATCH 1/2] commands: i2c: do not restrict i2c_write's data to be hex-formatted Ulrich Ölmann
@ 2017-10-19 6:53 ` Ulrich Ölmann
2017-10-19 7:44 ` [PATCH 0/2] Allow decimally formatted data for the commands "i2c_write" and "spi" Ulrich Ölmann
2017-10-23 7:27 ` Sascha Hauer
3 siblings, 0 replies; 6+ messages in thread
From: Ulrich Ölmann @ 2017-10-19 6:53 UTC (permalink / raw)
To: Barebox List
Change the default base to 0 to be consistent with the "mw" command and
particularly allow decimally formatted data.
The price to pay is needing the usual prefix "0x" for each hex-formatted number.
Signed-off-by: Ulrich Ölmann <u.oelmann@pengutronix.de>
Acked-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
---
commands/spi.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/commands/spi.c b/commands/spi.c
index 6603b34b673c..7bf193b0bbbb 100644
--- a/commands/spi.c
+++ b/commands/spi.c
@@ -84,7 +84,7 @@ static int do_spi(int argc, char *argv[])
rx_buf = xmalloc(read);
for (i = 0; i < count; i++)
- tx_buf[i] = (u8) simple_strtol(argv[optind + i], NULL, 16);
+ tx_buf[i] = (u8) simple_strtol(argv[optind + i], NULL, 0);
ret = spi_write_then_read(&spi, tx_buf, count, rx_buf, read);
if (ret)
--
2.11.0
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 0/2] Allow decimally formatted data for the commands "i2c_write" and "spi"
2017-10-19 6:53 [PATCH 0/2] Allow decimally formatted data for the commands "i2c_write" and "spi" Ulrich Ölmann
2017-10-19 6:53 ` [PATCH 1/2] commands: i2c: do not restrict i2c_write's data to be hex-formatted Ulrich Ölmann
2017-10-19 6:53 ` [PATCH 2/2] commands: spi: do not restrict spi's " Ulrich Ölmann
@ 2017-10-19 7:44 ` Ulrich Ölmann
2017-10-23 7:27 ` Sascha Hauer
3 siblings, 0 replies; 6+ messages in thread
From: Ulrich Ölmann @ 2017-10-19 7:44 UTC (permalink / raw)
To: Barebox List
Args, forgot the v2-infix in the subject...
--
Pengutronix e.K. | |
Industrial Linux Solutions | http://www.pengutronix.de/ |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 0/2] Allow decimally formatted data for the commands "i2c_write" and "spi"
2017-10-19 6:53 [PATCH 0/2] Allow decimally formatted data for the commands "i2c_write" and "spi" Ulrich Ölmann
` (2 preceding siblings ...)
2017-10-19 7:44 ` [PATCH 0/2] Allow decimally formatted data for the commands "i2c_write" and "spi" Ulrich Ölmann
@ 2017-10-23 7:27 ` Sascha Hauer
3 siblings, 0 replies; 6+ messages in thread
From: Sascha Hauer @ 2017-10-23 7:27 UTC (permalink / raw)
To: Ulrich Ölmann; +Cc: Barebox List
On Thu, Oct 19, 2017 at 08:53:06AM +0200, Ulrich Ölmann wrote:
> Currently both commands set the default base for parsing user supplied data to
> 16 which makes it impossible at all to parse decimal data.
>
> Presumably this originates from having the convenience to easily copy'n'paste
> hex dumped data into these commands without further preprocessing it before.
>
> Change the default base to 0 to particularly allow decimally formatted data.
> This is in accordance to what the "mw" command does.
>
> Changes since (implicit) v1 (20171017) (http://lists.infradead.org/pipermail/barebox/2017-October/031356.html):
> - extended the commit messages to include the motivation for the change
> - added Uwe's acked-by
>
> Ulrich Ölmann (2):
> commands: i2c: do not restrict i2c_write's data to be hex-formatted
> commands: spi: do not restrict spi's data to be hex-formatted
Applied, thanks
Sascha
--
Pengutronix e.K. | |
Industrial Linux Solutions | http://www.pengutronix.de/ |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 0/2] Allow decimally formatted data for the commands "i2c_write" and "spi"
@ 2017-10-17 13:16 Ulrich Ölmann
2017-10-17 13:16 ` [PATCH 2/2] commands: spi: do not restrict spi's data to be hex-formatted Ulrich Ölmann
0 siblings, 1 reply; 6+ messages in thread
From: Ulrich Ölmann @ 2017-10-17 13:16 UTC (permalink / raw)
To: Barebox List
Currently both commands set the default base for parsing user supplied data to
16 which makes it impossible at all to parse decimal data.
Presumably this originates from having the convenience to easily copy'n'paste
hex dumped data into these commands without further preprocessing it before.
Change the default base to 0 to particularly allow decimally formatted data.
This is in accordance to what the "mw" command does.
Ulrich Ölmann (2):
commands: i2c: do not restrict i2c_write's data to be hex-formatted
commands: spi: do not restrict spi's data to be hex-formatted
commands/i2c.c | 2 +-
commands/spi.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
--
2.11.0
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 2/2] commands: spi: do not restrict spi's data to be hex-formatted
2017-10-17 13:16 Ulrich Ölmann
@ 2017-10-17 13:16 ` Ulrich Ölmann
0 siblings, 0 replies; 6+ messages in thread
From: Ulrich Ölmann @ 2017-10-17 13:16 UTC (permalink / raw)
To: Barebox List
The price to pay is needing the usual prefix "0x" for each hex-formatted number.
Signed-off-by: Ulrich Ölmann <u.oelmann@pengutronix.de>
---
commands/spi.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/commands/spi.c b/commands/spi.c
index 6603b34b673c..7bf193b0bbbb 100644
--- a/commands/spi.c
+++ b/commands/spi.c
@@ -84,7 +84,7 @@ static int do_spi(int argc, char *argv[])
rx_buf = xmalloc(read);
for (i = 0; i < count; i++)
- tx_buf[i] = (u8) simple_strtol(argv[optind + i], NULL, 16);
+ tx_buf[i] = (u8) simple_strtol(argv[optind + i], NULL, 0);
ret = spi_write_then_read(&spi, tx_buf, count, rx_buf, read);
if (ret)
--
2.11.0
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2017-10-23 7:27 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-10-19 6:53 [PATCH 0/2] Allow decimally formatted data for the commands "i2c_write" and "spi" Ulrich Ölmann
2017-10-19 6:53 ` [PATCH 1/2] commands: i2c: do not restrict i2c_write's data to be hex-formatted Ulrich Ölmann
2017-10-19 6:53 ` [PATCH 2/2] commands: spi: do not restrict spi's " Ulrich Ölmann
2017-10-19 7:44 ` [PATCH 0/2] Allow decimally formatted data for the commands "i2c_write" and "spi" Ulrich Ölmann
2017-10-23 7:27 ` Sascha Hauer
-- strict thread matches above, loose matches on Subject: below --
2017-10-17 13:16 Ulrich Ölmann
2017-10-17 13:16 ` [PATCH 2/2] commands: spi: do not restrict spi's data to be hex-formatted Ulrich Ölmann
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox