Skip to content

Revert "stm32l1/i2c.c: adapt for new periph driver interface"#4428

Merged
OlegHahm merged 1 commit intoRIOT-OS:masterfrom
PeterKietzmann:revert_stm32l1_i2c
Dec 8, 2015
Merged

Revert "stm32l1/i2c.c: adapt for new periph driver interface"#4428
OlegHahm merged 1 commit intoRIOT-OS:masterfrom
PeterKietzmann:revert_stm32l1_i2c

Conversation

@PeterKietzmann
Copy link
Copy Markdown
Member

This reverts commit e9215a6.

It turned out that #4096 did not work correctly. The i2c driver can also be optimized by using the gpio API. Unluckily there is one feature missing for the usage in i2c: the possibility to configure outputs as open drain, as needed for i2c buses. I will discuss this offline and come along with a PR which recovers the wanted improvement of #4096 again then.

@PeterKietzmann PeterKietzmann added Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors) Platform: ARM Platform: This PR/issue effects ARM-based platforms Area: drivers Area: Device drivers labels Dec 7, 2015
@PeterKietzmann PeterKietzmann added this to the Release 2015.12 milestone Dec 7, 2015
@PeterKietzmann PeterKietzmann added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Dec 7, 2015
@OlegHahm
Copy link
Copy Markdown
Member

OlegHahm commented Dec 8, 2015

ACK and go!

OlegHahm added a commit that referenced this pull request Dec 8, 2015
Revert "stm32l1/i2c.c: adapt for new periph driver interface"
@OlegHahm OlegHahm merged commit 4e5facd into RIOT-OS:master Dec 8, 2015
@PeterKietzmann PeterKietzmann deleted the revert_stm32l1_i2c branch February 29, 2016 10:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: drivers Area: Device drivers CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Platform: ARM Platform: This PR/issue effects ARM-based platforms Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants