Skip to content

nucleo-l031/l432: fix wrong i2c configuration#12235

Merged
benpicco merged 1 commit into
RIOT-OS:masterfrom
aabadie:pr/boards/nucleo-fix-default-pb6-pb7
Sep 15, 2019
Merged

nucleo-l031/l432: fix wrong i2c configuration#12235
benpicco merged 1 commit into
RIOT-OS:masterfrom
aabadie:pr/boards/nucleo-fix-default-pb6-pb7

Conversation

@aabadie

@aabadie aabadie commented Sep 15, 2019

Copy link
Copy Markdown
Contributor

Contribution description

This PR is fixing the wrong I2C configuration for nucleo-l031k6/nucleo-l432kc introduced by #12141 by mistake.

On those 2 boards, the I2C is connected to PB6/PB7 and not to PB8/PB9.

Testing procedure

Test an I2C sensor on both boards, it should work. In master, it's broken.

Issues/PRs references

Fixes misconfiguration introduced by #12141.

@aabadie aabadie added Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors) CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Area: boards Area: Board ports labels Sep 15, 2019
@aabadie aabadie requested a review from benpicco September 15, 2019 17:32
@mrusme

mrusme commented Sep 15, 2019

Copy link
Copy Markdown

Tested this locally on L432KC, works.

@benpicco benpicco left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed bug introduced by #12141, tested by @mrusme

@benpicco benpicco merged commit ece3130 into RIOT-OS:master Sep 15, 2019
@aabadie aabadie deleted the pr/boards/nucleo-fix-default-pb6-pb7 branch September 16, 2019 08:44
@kb2ma kb2ma added this to the Release 2019.10 milestone Sep 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: boards Area: Board ports CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR 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.

4 participants