Hi Bartosz, thanks for the encouragement ! Bartosz wrote: > Also: please have the AT24/at24 prefix for symbols in the driver. These specific symbols are declared static, i.e. they're visible only inside at24.c. Do you think they should still have an at24 prefix ? +#define DECLARE_AT24_PLATDATA(_name, _len, _flags) \ + static const struct at24_platform_data platdata_##_name = { \ + .byte_len = _len, .flags = _flags, \ + }