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

_______________________________________________
i2c mailing list
[email protected]
http://lists.lm-sensors.org/mailman/listinfo/i2c

Reply via email to