On Tue, 21 Feb 2012 08:31:26 +0100, Jean Delvare wrote: > On Mon, 20 Feb 2012 22:45:03 -0500, Douglas Gilbert wrote: > > My illegitimate use case is: > > Sonmicro 13.56 MHz RFID Mifare Module: > > http://www.sonmicro.com/en/downloads/Mifare/ds_SM130.pdf > > http://www.sonmicro.com/en/downloads/Mifare/AN601.pdf > > > > I can make it work by requesting the maximum number of bytes it > > will ever respond with on all reads. > > I have read these two documents quickly and I don't see any use case > for I2C_M_RECV_LEN. The two block reads I see (5.6 READ BLOCK and 5.7 > READ VALUE BLOCK in ds_SM130.pdf) have predefined response lengths of > 18 and 6 bytes respectively. Nowhere I see a read command that would > reply a block length as the first byte. > > Please point me to the page and section of these documents where you > think I2C_M_RECV_LEN is needed. Err, sorry, I see it now. Must have been blind this morning. Unfortunately the length doesn't include the checksum byte, so in practice you always read or write one more byte than the length byte says. This wouldn't be an issue if the checksum was compatible with SMBus PEC, as you would then be able to use the SMBus emulation layer, but it is not. It's really a pity to be so close to SMBus yes be incompatible with it. Shame on the manufacturer. Still, everything isn't lost. Your largest command is 18 byte long, this fits in the SMBus limit (32 bytes) so underlying bus drivers will be happy, if you provide the buffer size they expect (I2C_SMBUS_BLOCK_MAX + 2.) If you were using a kernel driver instead of going through i2c-dev, I think it would work just fine without changing anything to the existing code. You would just allocate a 34-byte buffer and pass len = 2 and flags |= I2C_M_RECV_LEN, and everything would work fine. Which brings me to a question: why are you writing your driver in user-space instead of kernel-space as everybody else is doing? I am also curious which i2c bus driver your system is using? For i2c-dev I have an idea but it's not exactly clean and I'm not sure if it'll work. Unfortunately I have no device supporting I2C_M_RECV_LEN at hand so testing is quite difficult. -- Jean Delvare -- To unsubscribe from this list: send the line "unsubscribe linux-i2c" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
