Commit 9638ab7c authored by Daniel Kurtz's avatar Daniel Kurtz Committed by Henrik Rydberg

Input: atmel_mxt_ts - add variable length __mxt_write_reg

The i2c bus requires 4 bytes to do a 1-byte write
(1 byte i2c address + 2 byte offset + 1 byte data).

By taking a length with writes, the driver can amortize transaction
overhead by performing larger transactions where appropriate.

This patch just sets up the new API.  Later patches refactor writes
to take advantage of the larger transactions.
Signed-off-by: default avatarDaniel Kurtz <djkurtz@chromium.org>
Signed-off-by: default avatarHenrik Rydberg <rydberg@euromail.se>
parent 771733e3
...@@ -431,17 +431,24 @@ static int mxt_read_reg(struct i2c_client *client, u16 reg, u8 *val) ...@@ -431,17 +431,24 @@ static int mxt_read_reg(struct i2c_client *client, u16 reg, u8 *val)
return __mxt_read_reg(client, reg, 1, val); return __mxt_read_reg(client, reg, 1, val);
} }
static int mxt_write_reg(struct i2c_client *client, u16 reg, u8 val) static int __mxt_write_reg(struct i2c_client *client, u16 reg, u16 len,
const void *val)
{ {
u8 buf[3]; u8 *buf;
size_t count;
int ret; int ret;
count = len + 2;
buf = kmalloc(count, GFP_KERNEL);
if (!buf)
return -ENOMEM;
buf[0] = reg & 0xff; buf[0] = reg & 0xff;
buf[1] = (reg >> 8) & 0xff; buf[1] = (reg >> 8) & 0xff;
buf[2] = val; memcpy(&buf[2], val, len);
ret = i2c_master_send(client, buf, 3); ret = i2c_master_send(client, buf, count);
if (ret == 3) { if (ret == count) {
ret = 0; ret = 0;
} else { } else {
if (ret >= 0) if (ret >= 0)
...@@ -450,9 +457,15 @@ static int mxt_write_reg(struct i2c_client *client, u16 reg, u8 val) ...@@ -450,9 +457,15 @@ static int mxt_write_reg(struct i2c_client *client, u16 reg, u8 val)
__func__, ret); __func__, ret);
} }
kfree(buf);
return ret; return ret;
} }
static int mxt_write_reg(struct i2c_client *client, u16 reg, u8 val)
{
return __mxt_write_reg(client, reg, 1, &val);
}
static int mxt_read_object_table(struct i2c_client *client, static int mxt_read_object_table(struct i2c_client *client,
u16 reg, u8 *object_buf) u16 reg, u8 *object_buf)
{ {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment