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
Fixed missing parentheses and added STM32F042x6 #65
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Change looks good to me - it will be later reviewed and merged by maintainers
any updates to this? Want to use the stm32 core in a workshop, but would be easier if this fix/change is integrated in a new release. |
@@ -49,7 +49,7 @@ uint32_t pinNametoDigitalPin(PinName p); | |||
// Convert an analog pin number Axx to a PinName Pxy | |||
#define analogInputToPinName(p) (digitalPinToPinName(analogInputToDigitalPin(p))) | |||
// All pins could manage EXTI | |||
#define digitalPinToInterrupt(p) ((digitalPinIsValid(p) ? p : NOT_AN_INTERRUPT) | |||
#define digitalPinToInterrupt(p) ((digitalPinIsValid(p) ? p : NOT_AN_INTERRUPT)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Delete the first parenthesis works too and it makes the code more readable.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, that works better :)
@tomvdb |
Hi, @tomvdb, thanks for the fix. I'm on vacation until 31th july. I will do a release with F401 RE next week. |
I've manually merged the PR with minor corrections |
Thanks! Sorry, still meant to fix the suggestions. |
There is an error in pins_arduino.h. The macro is missing a bracket.
digitalPinToInterrupt(p) ((digitalPinIsValid(p) ? p : NOT_AN_INTERRUPT)
I also added the STM32F042x6 startup file define in stm32_def_build.h