[U-Boot] [PATCH] i2c: add i2c reset command.

Signed-off-by: Heiko Schocher hs@denx.de --- common/cmd_i2c.c | 9 +++++++++ 1 files changed, 9 insertions(+), 0 deletions(-)
diff --git a/common/cmd_i2c.c b/common/cmd_i2c.c index ef3928e..414c888 100644 --- a/common/cmd_i2c.c +++ b/common/cmd_i2c.c @@ -1182,6 +1182,12 @@ int do_sdram (cmd_tbl_t * cmdtp, int flag, int argc, char *argv[]) #endif
#if defined(CONFIG_I2C_CMD_TREE) +int do_i2c_res(cmd_tbl_t * cmdtp, int flag, int argc, char *argv[]) +{ + i2c_init (CFG_I2C_SPEED, CFG_I2C_SLAVE); + return 0; +} + #if defined(CONFIG_I2C_MULTI_BUS) int do_i2c_bus_num(cmd_tbl_t * cmdtp, int flag, int argc, char *argv[]) { @@ -1240,6 +1246,8 @@ int do_i2c(cmd_tbl_t * cmdtp, int flag, int argc, char *argv[]) return do_i2c_probe(cmdtp, flag, --argc, ++argv); if (!strncmp(argv[1], "lo", 2)) return do_i2c_loop(cmdtp, flag, --argc, ++argv); + if (!strncmp(argv[1], "re", 2)) + return do_i2c_res(cmdtp, flag, --argc, ++argv); #if defined(CONFIG_CMD_SDRAM) if (!strncmp(argv[1], "sd", 2)) return do_sdram(cmdtp, flag, --argc, ++argv); @@ -1267,6 +1275,7 @@ U_BOOT_CMD( "i2c crc32 chip address[.0, .1, .2] count - compute CRC32 checksum\n" "i2c probe - show devices on the I2C bus\n" "i2c loop chip address[.0, .1, .2] [# of objects] - looping read of device\n" + "i2c res - resets the I2C Controller\n" #if defined(CONFIG_CMD_SDRAM) "i2c sdram chip - print SDRAM configuration information\n" #endif

Dear Heiko Schocher,
In message 48E0894D.2010101@denx.de you wrote:
Signed-off-by: Heiko Schocher hs@denx.de
common/cmd_i2c.c | 9 +++++++++ 1 files changed, 9 insertions(+), 0 deletions(-)
diff --git a/common/cmd_i2c.c b/common/cmd_i2c.c index ef3928e..414c888 100644 --- a/common/cmd_i2c.c +++ b/common/cmd_i2c.c @@ -1182,6 +1182,12 @@ int do_sdram (cmd_tbl_t * cmdtp, int flag, int argc, char *argv[]) #endif
#if defined(CONFIG_I2C_CMD_TREE) +int do_i2c_res(cmd_tbl_t * cmdtp, int flag, int argc, char *argv[]) +{
- i2c_init (CFG_I2C_SPEED, CFG_I2C_SLAVE);
- return 0;
+}
What does "res" mean? "Resource"? "resync"? "rescale"? "rescan"?
If you mean reset, then please write reset.
Also, in which way is this supposed to reset the I2C controller?
In short: which problem is this supposed to fix?
Best regards,
Wolfgang Denk

Dear Heiko Schocher,
In message 48E0894D.2010101@denx.de you wrote:
Signed-off-by: Heiko Schocher hs@denx.de
common/cmd_i2c.c | 9 +++++++++ 1 files changed, 9 insertions(+), 0 deletions(-)
Added to "next" branch, thanks.
Best regards,
Wolfgang Denk
participants (2)
-
Heiko Schocher
-
Wolfgang Denk