Add support for other compatible ICs about eeprom24x-rs HOT 5 CLOSED

Yeah, I think that’s alright. It’s a difference of opinion and what matters is that it exists and is usable by people. :)

from eeprom24x-rs.

Yeah, I think this makes sense. To be honest, I’m not sure I would define special structure for this and maybe just pass in the memory size.

Argument for: I don’t need to have an array of every size defined and usually the page size is in the part number.

Argument against: guarantees sizes are followed properly?

from eeprom24x-rs.

I see that it would be comfortable for this driver not to have to deal with the page size but if the user passes a too big page size, data will be overwritten. Given an EEPROM IC, I would say that is one of the worst things that can happen.
As a user, I would like the driver to prevent me from doing something potentially bad for myself (there might be use cases for this behavior, but in such an unusual case, I expect an advanced user not to be surprised that he would need to talk to the IC directly to do it).

As you said, the page size is something that is immutable for an IC and I would simply expect a driver to abstract me from such a parameter .

from eeprom24x-rs.

I added support for many other ICs and got rid of the duplicated code via macros.
I published this in release 0.2.0.
However, I will leave this issue open, as I would like to offer a generic API where the user can specify the page size since the device names are not quite consistent. I will probably do this after #5

from eeprom24x-rs.

All this has been fixed in https://github.com/eldruin/eeprom24x-rs/releases/tag/v0.3.0

from eeprom24x-rs.