[PATCH] i2c_probe: fix wrong start and stop addresses

[PATCH] i2c_probe: fix wrong start and stop addresses

[Miquel Raynal]

This patch fixes the wrong range of i2c addresses used by the i2c_probe
command when a stop address is provided. In this case the start address
(startaddr) was taking the value of the stop address and the stop address
(stopaddr) was always set to its default value.

Signed-off-by: Miquel Raynal <raynal.miquel@gmail.com>
---
 commands/i2c.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/commands/i2c.c b/commands/i2c.c
index f4ffc99..e6750e3 100644
--- a/commands/i2c.c
+++ b/commands/i2c.c
@@ -55,7 +55,7 @@ static int do_i2c_probe(int argc, char *argv[])
 	if (argc > 2)
 		startaddr = simple_strtol(argv[2], NULL, 0);
 	if (argc > 3)
-		startaddr = simple_strtol(argv[3], NULL, 0);
+		stopaddr = simple_strtol(argv[3], NULL, 0);
 
 
 	if (startaddr > stopaddr)
-- 
1.7.10.4

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

Re: [PATCH] i2c_probe: fix wrong start and stop addresses

[Sascha Hauer]

On Wed, Apr 13, 2016 at 11:24:34PM +0200, Miquel Raynal wrote:
> This patch fixes the wrong range of i2c addresses used by the i2c_probe
> command when a stop address is provided. In this case the start address
> (startaddr) was taking the value of the stop address and the stop address
> (stopaddr) was always set to its default value.
> 
> Signed-off-by: Miquel Raynal <raynal.miquel@gmail.com>
> ---
>  commands/i2c.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

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