[i2c] [PATCH v3] New-style I2C and SMBus EEPROM driver (with device_ids)

[Jean Delvare]

And some more nit-picking for the road...

On Thu, 5 Jun 2008 21:31:03 +0200, Wolfram Sang wrote:
> --- /dev/null
> +++ linux-2.6.26-rc4/include/linux/i2c/at24.h
> @@ -0,0 +1,104 @@
> (...)
> + * If you set up a custom eeprom type, please make sure you
> + * have double-checked the parameters. Especially page_size needs extra care,

"Make sure" and "double-checked" are somewhat redundant.

> + * as you risk data loss if your value is bigger than what the chip actually
> + * supports! A typical custom type declaration would look similar to this:
> + *
> + *	struct const at24_platform_data my_eeprom_data {

const struct, in this order.

> + *		AT24_PLATFORM_DATA(byte_len, page_size, flags);
> + *	};

-- 
Jean Delvare