On Thu, 05 Apr 2012 20:01:43 -0400, Douglas Gilbert wrote:
> On 12-04-05 03:24 AM, Jean Delvare wrote:
> > As the bus driver side implementation of I2C_M_RECV_LEN is heavily
> > tied to SMBus, we can't support received length over 32 bytes, but
> > let's at least support that.
> >
> > In practice, the caller will have to setup a buffer large enough to
> > cover the case where received length byte has value 32, so minimum
> > 32 + 1 = 33 bytes, possibly more if there is a fixed number of bytes
> > added for the specific slave (for example a checksum.)
>
> Either I am misunderstanding how to use this new patch or it is
> broken. After replacing the original patch with this one, setting
> msg->buf[0] to 2, my test program only sees the first two bytes
> of expected data:
> 08 81
> That is down from 8 bytes from the previous patch and 10 bytes
> expected from the SM130.
Does your I2C bus driver process I2C_M_RECV_LEN at all? I bet not.
You'll have to fix that. It's fairly easy, see the reference
implementation in i2c-algo-bit.c:readbytes(). The completely untested
attempt below may do, if not you'll have to fix my code:
---
drivers/i2c/busses/i2c-at91.c | 17 ++++++++++++-----
1 file changed, 12 insertions(+), 5 deletions(-)
--- linux-3.4-rc1.orig/drivers/i2c/busses/i2c-at91.c 2012-04-06
08:27:38.000000000 +0200
+++ linux-3.4-rc1/drivers/i2c/busses/i2c-at91.c 2012-04-06 08:36:47.379360574
+0200
@@ -159,13 +159,14 @@ static short at91_poll_status(unsigned l
return (loop_cntr > 0);
}
-static int xfer_read(struct i2c_adapter *adap, unsigned char *buf, int length)
+static int xfer_read(struct i2c_adapter *adap, unsigned char *buf, int length,
+ int recv_len)
{
int nack_seen = 0;
int sent_stop = 0;
/* Send Start */
- if (1 == length) {
+ if ((1 == length) && !recv_len) {
at91_twi_write(AT91_TWI_CR, AT91_TWI_START | AT91_TWI_STOP);
sent_stop = 1;
} else
@@ -174,7 +175,7 @@ static int xfer_read(struct i2c_adapter
/* Read data */
while (length--) {
/* send Stop before reading last byte (if not already done) */
- if ((0 == length) && (0 == sent_stop))
+ if ((0 == length) && (0 == sent_stop) && !recv_len)
at91_twi_write(AT91_TWI_CR, AT91_TWI_STOP);
if (!at91_poll_status(AT91_TWI_RXRDY, &nack_seen)) {
dev_dbg(&adap->dev, "RXRDY timeout\n");
@@ -184,7 +185,12 @@ static int xfer_read(struct i2c_adapter
/* NACK supplies Stop */
return -EREMOTEIO;
}
- *buf++ = (at91_twi_read(AT91_TWI_RHR) & 0xff);
+ *buf = (at91_twi_read(AT91_TWI_RHR) & 0xff);
+ if (recv_len) {
+ length += *buf;
+ recv_len = 0;
+ }
+ buf++;
}
return 0;
@@ -257,7 +263,8 @@ static int at91_xfer(struct i2c_adapter
if (pmsg->len && pmsg->buf) { /* sanity check */
if (pmsg->flags & I2C_M_RD)
- ret = xfer_read(adap, pmsg->buf, pmsg->len);
+ ret = xfer_read(adap, pmsg->buf, pmsg->len,
+ pmsg->flags & I2C_M_RECV_LEN);
else
ret = xfer_write(adap, pmsg->buf, pmsg->len);
--
Jean Delvare
--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
