Hi, On 05/02/2019 17:41:49+0000, Trent Piepho wrote: > +static const struct nvmem_config isl1208_nvmem_config = { > + .name = "isl1208_nvram", > + .word_size = 1, > + .stride = 1, > + /* .size from chip specific config */ > + .reg_read = isl1208_nvmem_read, > + .reg_write = isl1208_nvmem_write, Can you also set .type here? -- Alexandre Belloni, Bootlin Embedded Linux and Kernel engineering https://bootlin.com