Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1443969 > unrolled thread

Re: [RESEND PATCH 11/14] eeprom: at24: add the at24cs series to the list of supported devices

Started byWolfram Sang <wsa@the-dreams.de>
First post2016-07-15 08:40 +0200
Last post2016-07-15 11:00 +0200
Articles 2 — 2 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: [RESEND PATCH 11/14] eeprom: at24: add the at24cs series to the  list of supported devices Wolfram Sang <wsa@the-dreams.de> - 2016-07-15 08:40 +0200
    Re: [RESEND PATCH 11/14] eeprom: at24: add the at24cs series to the  list of supported devices Bartosz Golaszewski <bgolaszewski@baylibre.com> - 2016-07-15 11:00 +0200

#1443969 — Re: [RESEND PATCH 11/14] eeprom: at24: add the at24cs series to the list of supported devices

FromWolfram Sang <wsa@the-dreams.de>
Date2016-07-15 08:40 +0200
SubjectRe: [RESEND PATCH 11/14] eeprom: at24: add the at24cs series to the list of supported devices
Message-ID<rV4WZ-3uz-3@gated-at.bofh.it>

[Multipart message — attachments visible in raw view] — view raw

>  	{ "24c01",	AT24_DEVICE_MAGIC(1024 / 8,	0) },
> +	{ "24cs01",	AT24_DEVICE_MAGIC(128 / 8,

I encoded the size for EEPROMs with "x / 8" originally to make it
visible where the EEPROM name came from, i.e. that is from kilobits not
kilobytes. So, for the serial number I would simply encode the size as
16 and not as (128 / 8), because it has nothing to do with the name. I
can fix this locally if you are okay with the idea?

[toc] | [next] | [standalone]


#1444077

FromBartosz Golaszewski <bgolaszewski@baylibre.com>
Date2016-07-15 11:00 +0200
Message-ID<rV78t-4JM-17@gated-at.bofh.it>
In reply to#1443969
2016-07-15 8:30 GMT+02:00 Wolfram Sang <wsa@the-dreams.de>:
>
>>       { "24c01",      AT24_DEVICE_MAGIC(1024 / 8,     0) },
>> +     { "24cs01",     AT24_DEVICE_MAGIC(128 / 8,
>
> I encoded the size for EEPROMs with "x / 8" originally to make it
> visible where the EEPROM name came from, i.e. that is from kilobits not
> kilobytes. So, for the serial number I would simply encode the size as
> 16 and not as (128 / 8), because it has nothing to do with the name. I
> can fix this locally if you are okay with the idea?
>

Sure, thanks!

Bart

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web