[riot-notifications] [RIOT-OS/RIOT] periph/i2c: handle i2c registers in both little and big endian (#11585)

Federico Pellegrin notifications at github.com
Wed May 29 05:00:40 CEST 2019


@MrKevinWeiss thanks for investigating this! Indeed I was checking more eeproms and all have a big-endian addressing (when needed, so for ones that need 16bit). 
Therefore I agree, until (if ever) we have a case for little-endian we can pull this away not to add useless bytes and more code to maintain.

I will now fix the PR to just adress the fact to have it big endian (this I think is a must as it breaks otherwise on my LE) and correct the PR title as well. Hope it will be fine!


-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/11585#issuecomment-496764748
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190529/8e581c0f/attachment.html>


More information about the notifications mailing list