On Thursday 11 December 2014 21:50:56 walter harms wrote:
>
> Am 11.12.2014 16:36, schrieb Bartosz Golaszewski:
> > This version includes fixes suggested for v1 and implements i2cset. Any
> > testing and suggestions for size reduction are welcome.
> >
> > Next: i2cdump and i2cdetect.
> >
> > v1:
> > http://lists.busybox.net/pipermail/busybox/2014-December/082071.html
> >
> > Signed-off-by: Bartosz Golaszewski <[email protected]>
> > ---
> > miscutils/i2c_tools.c | 727
> > ++++++++++++++++++++++++++++++++++++++++++++++++++
> > 1 file changed, 727 insertions(+)
> > create mode 100644 miscutils/i2c_tools.c
> >
> > diff --git a/miscutils/i2c_tools.c b/miscutils/i2c_tools.c
> > new file mode 100644
> > index 0000000..ad04e0d
> > --- /dev/null
> > +++ b/miscutils/i2c_tools.c
> > @@ -0,0 +1,727 @@
> > +/* vi: set sw=4 ts=4: */
> > +/*
> > + * Minimal i2c-tools implementation for busybox.
> > + * Parts of code ported from i2c-tools:
> > + * http://www.lm-sensors.org/wiki/I2CTools.
> > + *
> > + * Copyright (C) 2014 by Bartosz Golaszewski <[email protected]>
> > + *
> > + * Licensed under GPLv2 or later, see file LICENSE in this source tree.
> > + */
> > +
> > +//config:config I2CGET
> > +//config: bool "i2cget"
> > +//config: default y
> > +//config: select PLATFORM_LINUX
> > +//config: help
> > +//config: Read from I2C/SMBus chip registers.
> > +//config:
> > +//config:config I2CSET
> > +//config: bool "i2cset"
> > +//config: default y
> > +//config: select PLATFORM_LINUX
> > +//config: help
> > +//config: Set I2C registers.
> > +//config:
> > +
> > +//applet:IF_I2CGET(APPLET(i2cget, BB_DIR_USR_SBIN, BB_SUID_DROP))
> > +//applet:IF_I2CSET(APPLET(i2cset, BB_DIR_USR_SBIN, BB_SUID_DROP))
> > +
> > +//kbuild:lib-$(CONFIG_I2CGET) += i2c_tools.o
> > +//kbuild:lib-$(CONFIG_I2CSET) += i2c_tools.o
> > +
> > +//usage:#define i2cget_trivial_usage
> > +//usage: "[-f] [-y] BUS CHIP-ADDRESS [DATA-ADDRESS [MODE]]"
> > +//usage:#define i2cget_full_usage "\n\n"
> > +//usage: "Read from I2C/SMBus chip registers\n"
> > +//usage: "\n I2CBUS is an i2c bus number"
> > +//usage: "\n ADDRESS is an integer (0x03 - 0x77)"
> > +//usage: "\n MODE is one of:"
> > +//usage: "\n b (read byte data, default)"
> > +//usage: "\n w (read word data)"
> > +//usage: "\n c (write byte/read byte)"
> > +//usage: "\n\n Append p for SMBus PEC"
> > +//usage: "\n\n -f - force access to device"
> > +//usage: "\n -y - disable interactive mode"
> > +//usage:
> > +//usage:#define i2cset_trivial_usage
> > +//usage: "[-f] [-y] [-m MASK] BUS CHIP-ADDR DATA-ADDR [VALUE] ...
> > [MODE]"
> > +//usage:#define i2cset_full_usage "\n\n"
> > +//usage: "Set I2C registers\n"
> > +//usage: "\n I2CBUS is an i2c bus number"
> > +//usage: "\n ADDRESS is an integer (0x03 - 0x77)"
> > +//usage: "\n MODE is one of:"
> > +//usage: "\n c (byte, no value)"
> > +//usage: "\n b (byte data, default)"
> > +//usage: "\n w (word data)"
> > +//usage: "\n i (I2C block data)"
> > +//usage: "\n s (SMBus block data)"
> > +//usage: "\n\n Append p for SMBus PEC"
> > +//usage: "\n\n -f - force access to device"
> > +//usage: "\n -y - disable interactive mode"
> > +//usage: "\n -r - read back and compare the result"
> > +//usage: "\n -m - mask specifying which bits to write"
> > +//usage:
> > +
> > +#include "libbb.h"
> > +
> > +/*
> > + * /dev/i2c-X ioctl commands. The ioctl's parameter is always an unsigned
> > long,
> > + * except for:
> > + * - I2C_FUNCS, takes pointer to an unsigned long
> > + * - I2C_RDWR, takes pointer to struct i2c_rdwr_ioctl_data
> > + * - I2C_SMBUS, takes pointer to struct i2c_smbus_ioctl_data
> > + */
> > +
> > +/* Number of times a device address should be polled when not
> > acknowledging */
> > +#define I2C_RETRIES 0x0701
> > +/* Set timeout in units of 10 ms */
> > +#define I2C_TIMEOUT 0x0702
> > +
> > +/*
> > + * NOTE: Slave address is 7 or 10 bits, but 10-bit addresses
> > + * are NOT supported! (due to code brokenness)
> > + */
> > +
> > +/* Use this slave address */
> > +#define I2C_SLAVE 0x0703
> > +/* Use this slave address, even if it is already in use by a driver! */
> > +#define I2C_SLAVE_FORCE 0x0706
> > +/* 0 for 7 bit addrs, != 0 for 10 bit */
> > +#define I2C_TENBIT 0x0704
> > +/* Get the adapter functionality mask */
> > +#define I2C_FUNCS 0x0705
> > +/* Combined R/W transfer (one STOP only) */
> > +#define I2C_RDWR 0x0707
> > +/* != 0 to use PEC with SMBus */
> > +#define I2C_PEC 0x0708
> > +/* SMBus transfer */
> > +#define I2C_SMBUS 0x0720
> > +
> > +/* This is the structure as used in the I2C_SMBUS ioctl call */
> > +struct i2c_smbus_ioctl_data {
> > + uint8_t read_write;
> > + uint8_t command;
> > + uint32_t size;
> > + union i2c_smbus_data *data;
> > +};
> > +
> > +/* This is the structure as used in the I2C_RDWR ioctl call */
> > +struct i2c_rdwr_ioctl_data {
> > + struct i2c_msg *msgs; /* pointers to i2c_msgs */
> > + uint32_t nmsgs; /* number of i2c_msgs */
> > +};
> > +
> > +/* As specified in SMBus standard */
> > +#define I2C_SMBUS_BLOCK_MAX 32
> > +/* Not specified but we use same structure */
> > +#define I2C_SMBUS_I2C_BLOCK_MAX 32
> > +
> > +/*
> > + * Data for SMBus Messages
> > + */
> > +union i2c_smbus_data {
> > + uint8_t byte;
> > + uint16_t word;
> > + /* block[0] is used for length and one more for PEC */
> > + uint8_t block[I2C_SMBUS_BLOCK_MAX + 2];
> > +};
> > +
> > +#define I2C_RDRW_IOCTL_MAX_MSGS 42
> > +
> > +/* smbus_access read or write markers */
> > +#define I2C_SMBUS_READ 1
> > +#define I2C_SMBUS_WRITE 0
> > +
> > +/* SMBus transaction types (size parameter in the below functions). */
> > +#define I2C_SMBUS_QUICK 0
> > +#define I2C_SMBUS_BYTE 1
> > +#define I2C_SMBUS_BYTE_DATA 2
> > +#define I2C_SMBUS_WORD_DATA 3
> > +#define I2C_SMBUS_PROC_CALL 4
> > +#define I2C_SMBUS_BLOCK_DATA 5
> > +#define I2C_SMBUS_I2C_BLOCK_BROKEN 6
> > +#define I2C_SMBUS_BLOCK_PROC_CALL 7
> > +#define I2C_SMBUS_I2C_BLOCK_DATA 8
> > +
> > +/* Defines to determine what functionality is present */
> > +#define I2C_FUNC_I2C 0x00000001
> > +#define I2C_FUNC_10BIT_ADDR 0x00000002
> > +#define I2C_FUNC_PROTOCOL_MANGLING 0x00000004
> > +#define I2C_FUNC_SMBUS_PEC 0x00000008
> > +#define I2C_FUNC_SMBUS_BLOCK_PROC_CALL 0x00008000
> > +#define I2C_FUNC_SMBUS_QUICK 0x00010000
> > +#define I2C_FUNC_SMBUS_READ_BYTE 0x00020000
> > +#define I2C_FUNC_SMBUS_WRITE_BYTE 0x00040000
> > +#define I2C_FUNC_SMBUS_READ_BYTE_DATA 0x00080000
> > +#define I2C_FUNC_SMBUS_WRITE_BYTE_DATA 0x00100000
> > +#define I2C_FUNC_SMBUS_READ_WORD_DATA 0x00200000
> > +#define I2C_FUNC_SMBUS_WRITE_WORD_DATA 0x00400000
> > +#define I2C_FUNC_SMBUS_PROC_CALL 0x00800000
> > +#define I2C_FUNC_SMBUS_READ_BLOCK_DATA 0x01000000
> > +#define I2C_FUNC_SMBUS_WRITE_BLOCK_DATA 0x02000000
> > +#define I2C_FUNC_SMBUS_READ_I2C_BLOCK 0x04000000
> > +#define I2C_FUNC_SMBUS_WRITE_I2C_BLOCK 0x08000000
> > +
> > +/*
> > + * This is needed for ioctl_or_perror_and_die() since it only accepts
> > pointer.
> > + */
> > +#define INT_TO_PTR(i) ((void*)((long)i))
> > +
> > +static int32_t i2c_smbus_access(int fd, char read_write, uint8_t cmd,
> > + int size, union i2c_smbus_data *data)
> > +{
> > + struct i2c_smbus_ioctl_data args;
> > +
> > + args.read_write = read_write;
> > + args.command = cmd;
> > + args.size = size;
> > + args.data = data;
> > +
> > + return ioctl(fd, I2C_SMBUS, &args);
> > +}
> > +
> > +static int32_t i2c_smbus_read_byte(int fd)
> > +{
> > + union i2c_smbus_data data;
> > + int err;
> > +
> > + err = i2c_smbus_access(fd, I2C_SMBUS_READ, 0, I2C_SMBUS_BYTE, &data);
> > + if (err < 0)
> > + return err;
> > +
> > + return 0xff & data.byte;
> > +}
> > +
> > +static int32_t i2c_smbus_write_byte(int fd, uint8_t val)
> > +{
> > + return i2c_smbus_access(fd, I2C_SMBUS_WRITE,
> > + val, I2C_SMBUS_BYTE, NULL);
> > +}
> > +
> > +static int32_t i2c_smbus_read_byte_data(int fd, uint8_t cmd)
> > +{
> > + union i2c_smbus_data data;
> > + int err;
> > +
> > + err = i2c_smbus_access(fd, I2C_SMBUS_READ, cmd,
> > + I2C_SMBUS_BYTE_DATA, &data);
> > + if (err < 0)
> > + return err;
> > +
> > + return 0xff & data.byte;
> > +}
> > +
> > +static int32_t i2c_smbus_read_word_data(int fd, uint8_t cmd)
> > +{
> > + union i2c_smbus_data data;
> > + int err;
> > +
> > + err = i2c_smbus_access(fd, I2C_SMBUS_READ, cmd,
> > + I2C_SMBUS_WORD_DATA, &data);
> > + if (err < 0)
> > + return err;
> > +
> > + return 0xffff & data.word;
> > +}
> > +
> > +#if ENABLE_I2CSET
> > +static int32_t i2c_smbus_write_byte_data(int file,
> > + uint8_t cmd, uint8_t value)
> > +{
> > + union i2c_smbus_data data;
> > +
> > + data.byte = value;
> > +
> > + return i2c_smbus_access(file, I2C_SMBUS_WRITE, cmd,
> > + I2C_SMBUS_BYTE_DATA, &data);
> > +}
> > +
> > +static int32_t i2c_smbus_write_word_data(int file, uint8_t cmd, uint16_t
> > value)
> > +{
> > + union i2c_smbus_data data;
> > +
> > + data.word = value;
> > +
> > + return i2c_smbus_access(file, I2C_SMBUS_WRITE, cmd,
> > + I2C_SMBUS_WORD_DATA, &data);
> > +}
> > +
> > +static int32_t i2c_smbus_write_block_data(int file, uint8_t cmd,
> > + uint8_t length, const uint8_t *values)
> > +{
> > + union i2c_smbus_data data;
> > + int i;
> > +
> > + if (length > I2C_SMBUS_BLOCK_MAX)
> > + length = I2C_SMBUS_BLOCK_MAX;
> > + for (i = 1; i <= length; i++)
> > + data.block[i] = values[i-1];
> > + data.block[0] = length;
> > +
> > + return i2c_smbus_access(file, I2C_SMBUS_WRITE, cmd,
> > + I2C_SMBUS_BLOCK_DATA, &data);
> > +}
> > +
> > +static int32_t i2c_smbus_write_i2c_block_data(int file, uint8_t cmd,
> > + uint8_t length, const uint8_t *values)
> > +{
> > + union i2c_smbus_data data;
> > + int i;
> > +
> > + if (length > I2C_SMBUS_BLOCK_MAX)
> > + length = I2C_SMBUS_BLOCK_MAX;
> > + for (i = 1; i <= length; i++)
> > + data.block[i] = values[i-1];
>
> I have no idea how often this is uses but perhaps it is useful
> to copy the whole block in once instead of byte wise.
> memcpy(data.block+1,values,length); /* untested */
> (Same above)
>
>
> > + data.block[0] = length;
>
> > +
> > + return i2c_smbus_access(file, I2C_SMBUS_WRITE, cmd,
> > + I2C_SMBUS_I2C_BLOCK_BROKEN, &data);
> > +}
> > +#endif /* ENABLE_I2CSET */
> > +
> > +static int i2c_bus_lookup(const char *bus_str)
> > +{
> > + return xstrtoi_range(bus_str, 10, 0, 0xfffff);
> > +}
> > +
> > +static int i2c_parse_bus_addr(const char *addr_str)
> > +{
> > + return xstrtoi_range(addr_str, 16, 0x03, 0x77);
> > +}
> > +
> > +static int i2c_parse_data_addr(const char *data_addr)
> > +{
> > + return xstrtoi_range(data_addr, 16, 0, 0xff);
> > +}
> i am not an i2c expert, can you drop a comment here about the magic numbers ?
>
>
>
> > +/* Opens the device file associated with given i2c bus. */
> > +static int i2c_dev_open(int i2cbus)
> > +{
> > + char filename[32];
> > + int fd;
> > +
> > + snprintf(filename, sizeof(filename), "/dev/i2c-%d", i2cbus);
> > + fd = open_or_warn(filename, O_RDWR);
>
> now user of "/dev/i2c/%d" get warning here even when the next open works.
> I do not think that this was you intention.
>
> /* having one open is more easy :) */
>
> just my 2 cents,
> wh
Hi,
I think that this is better than having an error message like this:
f (fd < 0)
bb_perror_msg_and_die("Could not open file: /dev/i2c-%d or
/dev/i2c/%d", i2cbus, i2cbus);
and not knowing exactly which device you was not able to open.
Ciao,
Tito
> > + if (fd < 0 && errno == ENOENT) {
> > + snprintf(filename, sizeof(filename), "/dev/i2c/%d", i2cbus);
> > + fd = xopen(filename, O_RDWR);
> > + }
> > +
> > + return fd;
> > +}
> > +
> > +static void i2c_set_slave_addr(int fd, int addr, int force)
> > +{
> > + ioctl_or_perror_and_die(fd, force ? I2C_SLAVE_FORCE : I2C_SLAVE,
> > + INT_TO_PTR(addr),
> > + "Could not set address to 0x%02x", addr);
> > +}
> > +
> > +static void i2c_set_pec(int fd, int pec)
> > +{
> > + ioctl_or_perror_and_die(fd, I2C_PEC,
> > + INT_TO_PTR(pec ? 1 : 0),
> > + "Could not set PEC");
> > +}
> > +
> > +/* Size reducing helpers for xxx_check_funcs(). */
> > +static void get_funcs_matrix(int fd, unsigned long *funcs)
> > +{
> > + ioctl_or_perror_and_die(fd, I2C_FUNCS, funcs,
> > + "Could not get the adapter functionality matrix");
> > +}
> > +
> > +static void check_funcs_test_end(int funcs, int pec, const char *err)
> > +{
> > + if (pec && !(funcs & (I2C_FUNC_SMBUS_PEC | I2C_FUNC_I2C)))
> > + fprintf(stderr,
> > + "Warning: Adapter does not seem to support PEC\n");
> > +
> > + if (err)
> > + bb_error_msg_and_die("Adapter does not have %s capability",
> > + err);
> > +}
> > +
> > +/*
> > + * The below functions emit an error message and exit if the adapter
> > doesn't
> > + * support desired functionalities.
> > + */
> > +#if ENABLE_I2CGET
> > +static void i2cget_check_funcs(int fd, int mode, int data_addr, int pec)
> > +{
> > + unsigned long funcs;
> > + const char *err = NULL;
> > +
> > + get_funcs_matrix(fd, &funcs);
> > + switch (mode) {
> > + case I2C_SMBUS_BYTE:
> > + if (!(funcs & I2C_FUNC_SMBUS_READ_BYTE)) {
> > + err = "SMBus receive byte";
> > + break;
> > + }
> > + if (data_addr >= 0 && !(funcs & I2C_FUNC_SMBUS_WRITE_BYTE))
> > + err = "SMBus send byte";
> > + break;
> > + case I2C_SMBUS_BYTE_DATA:
> > + if (!(funcs & I2C_FUNC_SMBUS_READ_BYTE_DATA))
> > + err = "SMBus read byte";
> > + break;
> > + case I2C_SMBUS_WORD_DATA:
> > + if (!(funcs & I2C_FUNC_SMBUS_READ_WORD_DATA))
> > + err = "SMBus read word";
> > + break;
> > + }
> > + check_funcs_test_end(funcs, pec, err);
> > +}
> > +#endif /* ENABLE_I2CGET */
> > +
> > +#if ENABLE_I2CSET
> > +static void i2cset_check_funcs(int fd, int mode, int pec)
> > +{
> > + unsigned long funcs;
> > + const char *err = NULL;
> > +
> > + get_funcs_matrix(fd, &funcs);
> > + switch (mode) {
> > + case I2C_SMBUS_BYTE:
> > + if (!(funcs & I2C_FUNC_SMBUS_WRITE_BYTE))
> > + err = "SMBus send byte";
> > + break;
> > +
> > + case I2C_SMBUS_BYTE_DATA:
> > + if (!(funcs & I2C_FUNC_SMBUS_WRITE_BYTE_DATA))
> > + err = "SMBus write byte";
> > + break;
> > +
> > + case I2C_SMBUS_WORD_DATA:
> > + if (!(funcs & I2C_FUNC_SMBUS_WRITE_WORD_DATA))
> > + err = "SMBus write word";
> > + break;
> > +
> > + case I2C_SMBUS_BLOCK_DATA:
> > + if (!(funcs & I2C_FUNC_SMBUS_WRITE_BLOCK_DATA))
> > + err = "SMBus block write";
> > + break;
> > + case I2C_SMBUS_I2C_BLOCK_DATA:
> > + if (!(funcs & I2C_FUNC_SMBUS_WRITE_I2C_BLOCK))
> > + err = "I2C block write";
> > + break;
> > + }
> > + check_funcs_test_end(funcs, pec, err);
> > +}
> > +#endif /* ENABLE_I2CSET */
> > +
> > +/*********************************** XXXX
> > ***********************************/
> > +/*
> > + * XXX read_line() exists already in util-linux/fdisk.c - move it to
> > + * libbb and reuse here in the final version of the patch. This will save
> > + * additional 116 bytes.
> > + */
> > +static char *line_ptr;
> > +static char line_buffer[128];
> > +
> > +static int read_line(const char *prompt)
> > +{
> > + int sz;
> > +
> > + sz = read_line_input(NULL, prompt, line_buffer,
> > + sizeof(line_buffer), /*timeout*/ -1);
> > + if (sz <= 0)
> > + exit(EXIT_SUCCESS); /* Ctrl-D or Ctrl-C */
> > +
> > + if (line_buffer[sz-1] == '\n')
> > + line_buffer[--sz] = '\0';
> > +
> > + line_ptr = line_buffer;
> > + while (*line_ptr != '\0' && (unsigned char)*line_ptr <= ' ')
> > + line_ptr++;
> > + return *line_ptr;
> > +}
> > +
> > +static void user_confirm(void)
> > +{
> > + if (tolower(read_line("Continue? [y/N] ")) != 'y') {
> > + fprintf(stderr, "Aborting.\n");
> > + xfunc_die();
> > + }
> > +}
> > +/*********************************** XXXX
> > ***********************************/
> > +
> > +/*
> > + * Return only if user confirms the action, abort otherwise.
> > + *
> > + * The messages displayed here are much less elaborate than their i2c-tools
> > + * counterparts - this is done for size reduction.
> > + */
> > +static void confirm_action(int bus_addr, int mode, int data_addr, int pec)
> > +{
> > + fprintf(stderr, "WARNING! This program can confuse your I2C bus\n");
> > +
> > + /* Don't let the user break his/her EEPROMs */
> > + if (bus_addr >= 0x50 && bus_addr <= 0x57 && pec) {
> > + fprintf(stderr, "This is I2C not smbus - using PEC on I2C "
> > + "devices may result in data loss. Aborting.\n");
> > + xfunc_die();
> > + }
> > +
> > + if (mode == I2C_SMBUS_BYTE && data_addr >= 0 && pec)
> > + fprintf(stderr, "WARNING! May interpret a write byte command "
> > + "with PEC as a write byte data command!\n");
> > +
> > + if (pec)
> > + fprintf(stderr, "PEC checking enabled.\n");
> > +
> > + user_confirm();
> > +}
> > +
> > +#if ENABLE_I2CGET
> > +int i2cget_main(int argc, char **argv) MAIN_EXTERNALLY_VISIBLE;
> > +int i2cget_main(int argc, char **argv)
> > +{
> > + static const unsigned opt_f = (1 << 0), opt_y = (1 << 1);
> > + static const char *const optstr = "fy";
> > +
> > + int bus_num, bus_addr, data_addr = -1, status;
> > + int mode = I2C_SMBUS_BYTE, pec = 0, fd;
> > + unsigned opts;
> > +
> > + opts = getopt32(argv, optstr);
> > + argv += optind;
> > + argc -= optind;
> > +
> > + if (argc < 2)
> > + bb_show_usage();
> > +
> > + bus_num = i2c_bus_lookup(argv[0]);
> > + bus_addr = i2c_parse_bus_addr(argv[1]);
> > +
> > + if (argc > 2) {
> > + data_addr = i2c_parse_data_addr(argv[2]);
> > + mode = I2C_SMBUS_BYTE_DATA;
> > + }
> > +
> > + if (argc > 3) {
> > + switch (argv[3][0]) {
> > + case 'b': /* Already set */ break;
> > + case 'w': mode = I2C_SMBUS_WORD_DATA; break;
> > + case 'c': mode = I2C_SMBUS_BYTE; break;
> > + default:
> > + bb_error_msg("Invalid mode");
> > + bb_show_usage();
> > + }
> > + pec = argv[3][1] == 'p';
> > + }
> > +
> > + fd = i2c_dev_open(bus_num);
> > + i2cget_check_funcs(fd, mode, data_addr, pec);
> > + i2c_set_slave_addr(fd, bus_addr, opts & opt_f);
> > +
> > + if (!(opts & opt_y))
> > + confirm_action(bus_addr, mode, data_addr, pec);
> > +
> > + if (pec)
> > + i2c_set_pec(fd, 1);
> > +
> > + switch (mode) {
> > + case I2C_SMBUS_BYTE:
> > + if (data_addr >= 0) {
> > + status = i2c_smbus_write_byte(fd, data_addr);
> > + if (status < 0)
> > + fprintf(stderr, "Warning - write failed\n");
> > + }
> > + status = i2c_smbus_read_byte(fd);
> > + break;
> > + case I2C_SMBUS_WORD_DATA:
> > + status = i2c_smbus_read_word_data(fd, data_addr);
> > + break;
> > + default: /* I2C_SMBUS_BYTE_DATA */
> > + status = i2c_smbus_read_byte_data(fd, data_addr);
> > + }
> > + close(fd);
> > +
> > + if (status < 0)
> > + bb_perror_msg_and_die("Read failed");
> > +
> > + printf("0x%0*x\n", mode == I2C_SMBUS_WORD_DATA ? 4 : 2, status);
> > +
> > + return 0;
> > +}
> > +#endif /* ENABLE_I2CGET */
> > +
> > +#if ENABLE_I2CSET
> > +int i2cset_main(int argc, char **argv) MAIN_EXTERNALLY_VISIBLE;
> > +int i2cset_main(int argc, char **argv)
> > +{
> > + static const unsigned opt_f = (1 << 0), opt_y = (1 << 1),
> > + opt_m = (1 << 2), opt_r = (1 << 3);
> > + static const char *const optstr = "fym:r";
> > +
> > + int bus_num, bus_addr, data_addr, mode = I2C_SMBUS_BYTE, pec = 0;
> > + int val, blen = 0, mask = 0, fd, status;
> > + unsigned char block[I2C_SMBUS_BLOCK_MAX];
> > + char *opt_m_arg = NULL;
> > + unsigned opts;
> > +
> > + opts = getopt32(argv, optstr, &opt_m_arg);
> > + argv += optind;
> > + argc -= optind;
> > +
> > + if (argc < 3)
> > + bb_show_usage();
> > +
> > + bus_num = i2c_bus_lookup(argv[0]);
> > + bus_addr = i2c_parse_bus_addr(argv[1]);
> > + data_addr = i2c_parse_data_addr(argv[2]);
> > +
> > + if (argc > 3) {
> > + switch (argv[argc-1][0]) {
> > + case 'c': mode = I2C_SMBUS_BYTE; break;
> > + case 'b': mode = I2C_SMBUS_BYTE_DATA; break;
> > + case 'w': mode = I2C_SMBUS_WORD_DATA; break;
> > + case 's': mode = I2C_SMBUS_BLOCK_DATA; break;
> > + case 'i': mode = I2C_SMBUS_I2C_BLOCK_DATA; break;
> > + default:
> > + bb_error_msg("Invalid mode");
> > + bb_show_usage();
> > + }
> > + pec = argv[argc-1][1] == 'p';
> > + if (mode == I2C_SMBUS_BLOCK_DATA ||
> > + mode == I2C_SMBUS_I2C_BLOCK_DATA) {
> > + if (pec && mode == I2C_SMBUS_I2C_BLOCK_DATA)
> > + bb_error_msg_and_die("PEC not supported for "
> > + "I2C block writes!");
> > + if (opts & opt_m)
> > + bb_error_msg_and_die("Mask not supported "
> > + "for block writes!\n");
> > + }
> > + }
> > +
> > + /* Prepare the value(s) to be written according to current mode. */
> > + switch (mode) {
> > + case I2C_SMBUS_BYTE_DATA:
> > + val = xstrtoi_range(argv[3], 0, 0, 0xff);
> > + break;
> > + case I2C_SMBUS_WORD_DATA:
> > + val = xstrtoi_range(argv[3], 0, 0, 0xffff);
> > + break;
> > + case I2C_SMBUS_BLOCK_DATA:
> > + case I2C_SMBUS_I2C_BLOCK_DATA:
> > + for (blen = 3; blen < (argc - 1); blen++)
> > + block[blen] = xstrtoi_range(argv[blen], 0, 0, 0xff);
> > + val = -1;
> > + break;
> > + default:
> > + val = -1;
> > + break;
> > + }
> > +
> > + if (opts & opt_m) {
> > + mask = xstrtoi_range(opt_m_arg, 0, 0,
> > + (mode == I2C_SMBUS_BYTE ||
> > + mode == I2C_SMBUS_BYTE_DATA) ? 0xff : 0xffff);
> > + }
> > +
> > + fd = i2c_dev_open(bus_num);
> > + i2cset_check_funcs(fd, mode, pec);
> > + i2c_set_slave_addr(fd, bus_addr, opts & opt_f);
> > +
> > + if (!(opts & opt_y))
> > + confirm_action(bus_addr, mode, data_addr, pec);
> > +
> > + /*
> > + * If we're using mask - read the current value here and adjust the
> > + * value to be written.
> > + */
> > + if (opts & opt_m) {
> > + int tmpval;
> > +
> > + switch (mode) {
> > + case I2C_SMBUS_BYTE:
> > + tmpval = i2c_smbus_read_byte(fd);
> > + break;
> > + case I2C_SMBUS_WORD_DATA:
> > + tmpval = i2c_smbus_read_word_data(fd, data_addr);
> > + break;
> > + default:
> > + tmpval = i2c_smbus_read_byte_data(fd, data_addr);
> > + }
> > +
> > + if (tmpval < 0)
> > + bb_perror_msg_and_die("Failed to read old value");
> > +
> > + val = (val & mask) | (tmpval & ~mask);
> > +
> > + if (!(opts & opt_y)) {
> > + fprintf(stderr, "Old value 0x%0*x, write mask "
> > + "0x%0*x: Will write 0x%0*x to register "
> > + "0x%02x\n",
> > + mode == I2C_SMBUS_WORD_DATA ? 4 : 2, tmpval,
> > + mode == I2C_SMBUS_WORD_DATA ? 4 : 2, mask,
> > + mode == I2C_SMBUS_WORD_DATA ? 4 : 2, val,
> > + data_addr);
> > +
> > + user_confirm();
> > + }
> > + }
> > +
> > + if (pec)
> > + i2c_set_pec(fd, 1);
> > +
> > + switch (mode) {
> > + case I2C_SMBUS_BYTE:
> > + status = i2c_smbus_write_byte(fd, data_addr);
> > + break;
> > + case I2C_SMBUS_WORD_DATA:
> > + status = i2c_smbus_write_word_data(fd, data_addr, val);
> > + break;
> > + case I2C_SMBUS_BLOCK_DATA:
> > + status = i2c_smbus_write_block_data(fd, data_addr,
> > + blen, block);
> > + break;
> > + case I2C_SMBUS_I2C_BLOCK_DATA:
> > + status = i2c_smbus_write_i2c_block_data(fd, data_addr,
> > + blen, block);
> > + break;
> > + default: /* I2C_SMBUS_BYTE_DATA */
> > + status = i2c_smbus_write_byte_data(fd, data_addr, val);
> > + break;
> > + }
> > + if (status < 0)
> > + bb_perror_msg_and_die("Write failed\n");
> > +
> > + if (pec)
> > + i2c_set_pec(fd, 0); /* Clear PEC */
> > +
> > + /* No readback required - we're done. */
> > + if (!(opts & opt_r))
> > + return 0;
> > +
> > + switch (mode) {
> > + case I2C_SMBUS_BYTE:
> > + status = i2c_smbus_read_byte(fd);
> > + val = data_addr;
> > + break;
> > + case I2C_SMBUS_WORD_DATA:
> > + status = i2c_smbus_read_word_data(fd, data_addr);
> > + break;
> > + default: /* I2C_SMBUS_BYTE_DATA */
> > + status = i2c_smbus_read_byte_data(fd, data_addr);
> > + }
> > +
> > + if (status < 0) {
> > + printf("Warning - readback failed\n");
> > + } else
> > + if (status != val) {
> > + printf("Warning - data mismatch - wrote "
> > + "0x%0*x, read back 0x%0*x\n",
> > + mode == I2C_SMBUS_WORD_DATA ? 4 : 2, val,
> > + mode == I2C_SMBUS_WORD_DATA ? 4 : 2, status);
> > + } else {
> > + printf("Value 0x%0*x written, readback matched\n",
> > + mode == I2C_SMBUS_WORD_DATA ? 4 : 2, val);
> > + }
> > +
> > + return 0;
> > +}
> > +#endif /* ENABLE_I2CSET */
> _______________________________________________
> busybox mailing list
> [email protected]
> http://lists.busybox.net/mailman/listinfo/busybox
>
_______________________________________________
busybox mailing list
[email protected]
http://lists.busybox.net/mailman/listinfo/busybox
