Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cpu/stm32f1: fix wrong bit operation in SPI init #1953

Merged
merged 1 commit into from
Nov 6, 2014

Conversation

thomaseichinger
Copy link
Member

3 hours for this little beast...

@thomaseichinger thomaseichinger added 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) Impact: major The PR changes a significant part of the code base. It should be reviewed carefully labels Nov 6, 2014
@thomaseichinger thomaseichinger added this to the Release NEXT MAJOR milestone Nov 6, 2014
@OlegHahm
Copy link
Member

OlegHahm commented Nov 6, 2014

What's the effect of this bug?

@haukepetersen
Copy link
Contributor

ACK and go.

haukepetersen added a commit that referenced this pull request Nov 6, 2014
cpu/stm32f1: fix wrong bit operation in SPI init
@haukepetersen haukepetersen merged commit a268afc into RIOT-OS:master Nov 6, 2014
@haukepetersen
Copy link
Contributor

@OlegHahm the effect was that the pins were configured in-correctly if the pin number is > 7. So no effect on the iot-lab but an effect on the agile fox...

@OlegHahm
Copy link
Member

OlegHahm commented Nov 6, 2014

I see. Thanks for explanation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Impact: major The PR changes a significant part of the code base. It should be reviewed carefully 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