2018-01-30 15:14 GMT+01:00 johannes <johannes@xxxxxxx>: > On Mon, Jan 29, 2018 at 04:39:15PM +0100, Bartosz Golaszewski wrote: > >> >> What EEPROM model do you need that for? Is it not covered by all the >> >> standard types we already support in the driver? > >> > I need this for a M24M01-D* eeprom. >> > (http://www.st.com/content/ccc/resource/technical/document/datasheet/group0/cb/91/ba/7d/0b/c1/4d/f6/CD00147128/files/CD00147128.pdf/jcr:content/translations/en.CD00147128.pdf) > >> > It is 1 Mbit sized and has 2 byte addresses. > >> > Under another I2C address it offers a additional write lockable page >> > of 256 Byte, but with _2 byte addresses_. I could not correctly read >> > this "second eeprom" with one of the standard eeprom at24* models. >> > [ .. ] > >> This model is compatible with at24cm01. We don't support it yet, but >> instead of a new DT property, just add a new supported model with >> compatible = 'atmel,at24cm01' and AT24_FLAG_ADDR16 to the driver code. > > Hi Bartosz, > > I did the patch as you recommended with a new supported model. > The diff is much smaller now. > > See more info and a question about naming in the patch attached. > > Please send patches using git send-mail. Patchwork didn't even pick that one up. Thanks, Bartosz
