From: "Eric Bénard" <eric@eukrea.com>
To: s.hauer@pengutronix.de
Cc: barebox@lists.infradead.org
Subject: [PATCH v5] commands: add i2c commands
Date: Tue, 25 May 2010 11:25:32 +0200 [thread overview]
Message-ID: <1274779532-8110-1-git-send-email-eric@eukrea.com> (raw)
In-Reply-To: <4BFB9224.9030507@pengutronix.de>
theses commands allow low level access to i2c bus and can be useful
to setup an i2c device without having to add code, compile and flash
barebox.
Signed-off-by: Eric Bénard <eric@eukrea.com>
---
v5 :
updated as per Marc's comments
better error handling in write
better default bus handling in read & write
v4 :
updated as per Sascha and Marc's comments
use bus 0 as a default for i2c_read & write
replace malloc with xmalloc
fix simple_strtol configuration for parameters
v3 :
enable brain and fix v2
tested on iMX27
v2 :
updated as per Sascha's comments
commands/Kconfig | 8 ++
commands/Makefile | 1 +
commands/i2c.c | 211 +++++++++++++++++++++++++++++++++++++++++++++++++++++
3 files changed, 220 insertions(+), 0 deletions(-)
create mode 100644 commands/i2c.c
diff --git a/commands/Kconfig b/commands/Kconfig
index 0c09f91..0ea32d9 100644
--- a/commands/Kconfig
+++ b/commands/Kconfig
@@ -314,4 +314,12 @@ config CMD_UNLZO
Say yes here to get the unlzo command. lzo is a fast compression
algorithm by Markus Franz Xaver Johannes Oberhumer.
+config CMD_I2C
+ bool
+ depends on I2C
+ prompt "i2c commands"
+ help
+ include i2c_probe, i2c_read and i2c_write commands to communicate
+ on i2c bus.
+
endmenu
diff --git a/commands/Makefile b/commands/Makefile
index 74b0994..3eef5de 100644
--- a/commands/Makefile
+++ b/commands/Makefile
@@ -48,3 +48,4 @@ obj-$(CONFIG_CMD_BMP) += bmp.o
obj-$(CONFIG_USB_GADGET_DFU) += dfu.o
obj-$(CONFIG_CMD_GPIO) += gpio.o
obj-$(CONFIG_CMD_UNLZO) += unlzo.o
+obj-$(CONFIG_CMD_I2C) += i2c.o
diff --git a/commands/i2c.c b/commands/i2c.c
new file mode 100644
index 0000000..b8f2ee6
--- /dev/null
+++ b/commands/i2c.c
@@ -0,0 +1,211 @@
+/*
+ * i2c.c - i2c commands
+ *
+ * Copyright (c) 2010 Eric Bénard <eric@eukrea.Com>, Eukréa Electromatique
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation; either version 2 of
+ * the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
+ * MA 02111-1307 USA
+ */
+
+#include <common.h>
+#include <command.h>
+#include <errno.h>
+#include <malloc.h>
+#include <getopt.h>
+#include <i2c/i2c.h>
+
+static int do_i2c_probe(struct command *cmdtp, int argc, char *argv[])
+{
+ struct i2c_adapter *adapter;
+ struct i2c_client client;
+ int startaddr = -1, stopaddr = -1, addr, ret;
+ u8 reg;
+
+ if (argc < 4)
+ return COMMAND_ERROR_USAGE;
+
+ adapter = i2c_get_adapter(simple_strtoul(argv[1], NULL, 0));
+ if (!adapter)
+ return -ENODEV;
+ client.adapter = adapter;
+
+ startaddr = simple_strtol(argv[2], NULL, 0);
+ stopaddr = simple_strtol(argv[3], NULL, 0);
+ if ((startaddr == -1) || (stopaddr == -1) || (startaddr > stopaddr))
+ return COMMAND_ERROR_USAGE;
+
+ if (stopaddr > 0x7F)
+ stopaddr = 0x7F;
+
+ printf("probing i2c range 0X%02x - 0x%02x :\n", startaddr, stopaddr);
+ for (addr = startaddr; addr <= stopaddr; addr++) {
+ client.addr = addr;
+ ret = i2c_write_reg(&client, 0x00, ®, 0);
+ if (ret == 0)
+ printf("0x%02x ", addr);
+ }
+ printf("\n");
+ return 0;
+}
+
+static const __maybe_unused char cmd_i2c_probe_help[] =
+"Usage: i2c_probe bus 0xstartaddr 0xstopaddr\n"
+"probe a range of i2c addresses.\n";
+
+BAREBOX_CMD_START(i2c_probe)
+ .cmd = do_i2c_probe,
+ .usage = "probe for an i2c device",
+ BAREBOX_CMD_HELP(cmd_i2c_probe_help)
+BAREBOX_CMD_END
+
+static int do_i2c_write(struct command *cmdtp, int argc, char *argv[])
+{
+ struct i2c_adapter *adapter = NULL;
+ struct i2c_client client;
+ int addr = -1, reg = -1, count = -1, verbose = 0, ret, opt, i, bus = 0;
+ u8 *buf;
+ u8 tmp;
+
+ while ((opt = getopt(argc, argv, "a:b:r:v")) > 0) {
+ switch (opt) {
+ case 'a':
+ addr = simple_strtol(optarg, NULL, 0);
+ break;
+ case 'r':
+ reg = simple_strtol(optarg, NULL, 0);
+ break;
+ case 'b':
+ bus = simple_strtoul(optarg, NULL, 0);
+ break;
+ case 'v':
+ verbose = 1;
+ break;
+ }
+ }
+
+ count = argc - optind;
+
+ if ((addr < 0) || (reg < 0) || (count == 0) || (addr > 0x7F))
+ return COMMAND_ERROR_USAGE;
+
+ adapter = i2c_get_adapter(bus);
+ if (!adapter)
+ return -ENODEV;
+
+ client.adapter = adapter;
+ client.addr = addr;
+
+ buf = xmalloc(count);
+ for (i = 0; i < count; i++)
+ *(buf + i) = (char) simple_strtol(argv[optind+i], NULL, 16);
+
+ ret = i2c_write_reg(&client, reg, buf, count);
+ if (ret != count)
+ goto out;
+ ret = 0;
+
+ if (verbose) {
+ printf("wrote %i bytes starting at reg 0x%02x to i2cdev 0x%02x on bus %i\n",
+ count, reg, addr, adapter->nr);
+ for (i = 0; i < count; i++)
+ printf("0x%02x ", *(buf + i));
+ printf("\n");
+ }
+
+out:
+ free(buf);
+ return ret;
+}
+
+static const __maybe_unused char cmd_i2c_write_help[] =
+"Usage: i2c_write [OPTION] ... hexdatas\n"
+"write to i2c device.\n"
+" -a 0x<addr> i2c device address\n"
+" -b <bus_num> i2c bus number (default = 0)\n"
+" -r 0x<reg> start register\n";
+
+BAREBOX_CMD_START(i2c_write)
+ .cmd = do_i2c_write,
+ .usage = "write to an i2c device",
+ BAREBOX_CMD_HELP(cmd_i2c_write_help)
+BAREBOX_CMD_END
+
+static int do_i2c_read(struct command *cmdtp, int argc, char *argv[])
+{
+ struct i2c_adapter *adapter = NULL;
+ struct i2c_client client;
+ u8 *buf;
+ int count = -1, addr = -1, reg = -1, verbose = 0, ret, opt, bus = 0;
+
+ while ((opt = getopt(argc, argv, "a:b:c:r:v")) > 0) {
+ switch (opt) {
+ case 'a':
+ addr = simple_strtol(optarg, NULL, 0);
+ break;
+ case 'c':
+ count = simple_strtoul(optarg, NULL, 0);
+ break;
+ case 'b':
+ bus = simple_strtoul(optarg, NULL, 0);
+ break;
+ case 'r':
+ reg = simple_strtol(optarg, NULL, 0);
+ break;
+ case 'v':
+ verbose = 1;
+ break;
+ }
+ }
+
+ if ((addr < 0) || (reg < 0) || (count == 0) || (addr > 0x7F))
+ return COMMAND_ERROR_USAGE;
+
+ adapter = i2c_get_adapter(bus);
+ if (!adapter)
+ return -ENODEV;
+
+ client.adapter = adapter;
+ client.addr = addr;
+
+ buf = xmalloc(count);
+ ret = i2c_read_reg(&client, reg, buf, count);
+ if (ret == count) {
+ int i;
+ if (verbose)
+ printf("read %i bytes starting at reg 0x%02x from i2cdev 0x%02x on bus %i\n",
+ count, reg, addr, adapter->nr);
+ for (i = 0; i < count; i++)
+ printf("0x%02x ", *(buf + i));
+ printf("\n");
+ ret = 0;
+ }
+
+ free(buf);
+ return ret;
+}
+
+static const __maybe_unused char cmd_i2c_read_help[] =
+"Usage: i2c_read [OPTION]\n"
+"read i2c device.\n"
+" -a 0x<addr> i2c device address\n"
+" -b <bus_num> i2c bus number (default = 0)\n"
+" -r 0x<reg> start register\n"
+" -c <count> byte count\n";
+
+BAREBOX_CMD_START(i2c_read)
+ .cmd = do_i2c_read,
+ .usage = "read from an i2c device",
+ BAREBOX_CMD_HELP(cmd_i2c_read_help)
+BAREBOX_CMD_END
--
1.6.3.3
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
prev parent reply other threads:[~2010-05-25 9:25 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-05-25 8:37 [PATCH 1/6] i2c-imx: change log level for No ACK Eric Bénard
2010-05-25 8:37 ` [PATCH v2 2/6] i2c: implement i2c_get_adapter() Eric Bénard
2010-05-25 8:37 ` [PATCH v4 3/6] commands: add i2c commands Eric Bénard
2010-05-25 8:37 ` [PATCH 4/6] eukrea_cpuimx27: add fb support Eric Bénard
2010-05-25 8:37 ` [PATCH 5/6] eukrea_cpuimx27: update env Eric Bénard
2010-05-25 8:37 ` [PATCH 6/6] eukrea_cpuimx27: update defconfig Eric Bénard
2010-05-25 9:02 ` [PATCH v4 3/6] commands: add i2c commands Marc Kleine-Budde
2010-05-25 9:17 ` Eric Bénard
2010-05-25 9:32 ` Marc Kleine-Budde
2010-05-25 9:43 ` Eric Bénard
2010-05-25 9:47 ` Marc Kleine-Budde
2010-05-25 9:57 ` Eric Bénard
2010-05-25 9:25 ` Eric Bénard [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1274779532-8110-1-git-send-email-eric@eukrea.com \
--to=eric@eukrea.com \
--cc=barebox@lists.infradead.org \
--cc=s.hauer@pengutronix.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox