[riot-notifications] [RIOT-OS/RIOT] cpu/sam0: re-work i2c driver (#7588)

Dylan Laduranty notifications at github.com
Fri Sep 8 22:22:42 CEST 2017


This PR introduces a common i2c_config struct in `periph_conf.h` for all sam0 based boards. Additionnal I2C SERCOM can be add more easily.
SAML21 can know use SERCOM5 if desired.

I'm wondering if I can remove the init of GCLK_SLOW because it is only for some SMBus timeout (no used yet by RIOT). Should I remove it ? This should not impact the current driver state but I want the community's opinion first :)

I also tried to keep the 80 characters limit per lines.
You can view, comment on, or merge this pull request online at:

  https://github.com/RIOT-OS/RIOT/pull/7588

-- Commit Summary --

  * boards: update all sam0 based boards with shared i2c struct
  * cpu/sam0_common: optimize i2c driver
  * cpu/sam0: add support for run in standby

-- File Changes --

    M boards/arduino-mkr-common/include/periph_conf.h (26)
    M boards/arduino-zero/include/periph_conf.h (26)
    M boards/samd21-xpro/include/periph_conf.h (27)
    M boards/saml21-xpro/include/periph_conf.h (27)
    M boards/samr21-xpro/include/periph_conf.h (28)
    M boards/sodaq-autonomo/include/periph_conf.h (26)
    M cpu/sam0_common/include/periph_cpu_common.h (15)
    M cpu/sam0_common/periph/i2c.c (314)

-- Patch Links --

https://github.com/RIOT-OS/RIOT/pull/7588.patch
https://github.com/RIOT-OS/RIOT/pull/7588.diff

-- 
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/7588
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20170908/74cd420b/attachment.html>


More information about the notifications mailing list