Re: Looking for ICH SMBUS "Process call" support in i2c-i801 bus driver >

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Jean,

> Not really. What the ICH SMBus can do (using the SMBus process call
> with the I2C_EN bit set) is write a two-byte address and read exactly
> two bytes back. For the AT24C64, this corresponds to a sequential
> random read of size exactly 2 bytes. While I admit it would be possible
> to read from the AT24C64 using this transaction, it would perform
> rather badly: 6 bytes on the wire for 2 bytes of data read. Given how
> large the AT24C64 is, I doubt you will enjoy it, especially with the
> poll-based implementation of i2c-i801: you would need over 8 seconds to
> read the full EEPROM.
>
> Additionally, implementing it would be technically difficult. What the
> ICH implements doesn't match any standard SMBus transaction, and I
> don't know of any other SMBus controller implementing this. I don't
> think we want to define a new functionality bit and helper function for
> a non-standard transaction type only implemented by one controller.
> Alternatively, we could let i2c-i801 claim that it supports raw I2C
> transactions, and only accept this exact type of transaction. But this
> would then require dedicated support in the at24 driver. That's not
> particularly appealing either.
>
> As a matter of fact, the at24 driver only accept EEPROM types up to
> 24C16 on SMBus controllers. For 24C32 and larger, it requires full I2C
> support. Connecting an AT24C64 to an Intel ICH SMBus controller is
> simply bad engineering. You'd rather use one or more AT24C16 instead,
> so you can use standard SMBus transactions and (one address byte) I2C
> block reads.

Thanks much for great input. Let me talk to HW team to see if AT24C16
would be good enough.

Thanks,
Krishna
--
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

[Index of Archives]     [Linux GPIO]     [Linux SPI]     [Linux Hardward Monitoring]     [LM Sensors]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux