Hi Andrew, On 26.02.2016 21:59, Andrew Lunn wrote: > Add a regmap for accessing the EEPROM, and then use that with the > NVMEM framework. Set the NVMEM config structure to enable backward, so > that the 'eeprom' file in sys is provided by the framework. > > Signed-off-by: Andrew Lunn <and...@lunn.ch> > Acked-by: Srinivas Kandagatla <srinivas.kandaga...@linaro.org> > Tested-by: Bartosz Golaszewski <bgolaszew...@baylibre.com> > ---
[snip] > +static int at24_regmap_read(void *context, const void *reg, size_t reg_size, > + void *val, size_t val_size) > +{ > + struct at24_data *at24 = context; > + off_t offset = *(u32 *)reg; > + int err; > + > + err = at24_read(at24, val, offset, val_size); > + if (err) > + return err; > + return 0; return at24_read(at24, val, offset, val_size); Minus 5 LoC. > +} > + > +static int at24_regmap_write(void *context, const void *data, size_t count) > +{ > + struct at24_data *at24 = context; > + const char *buf; > + u32 offset; > + size_t len; > + int err; > + > + memcpy(&offset, data, sizeof(offset)); > + buf = (const char *)data + sizeof(offset); > + len = count - sizeof(offset); > + > + err = at24_write(at24, buf, offset, len); > + if (err) > + return err; > + return 0; return at24_write(at24, buf, offset, len); Minus another 5 LoC. -- With best wishes, Vladimir