-
Notifications
You must be signed in to change notification settings - Fork 3k
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
TARGET_MCUXpresso_MCUS: fix lp ticker init function #8827
Conversation
In case when lp ticker is already initialized the ticker interrupt should be disabled.
This PR provides implementation of ticker_free() function for the following boards: ARCH_PRO EV_COG_AD3029LZ EV_COG_AD4050LZ K22F K64F K82F KW24D KW41Z LPC546XX NRF51_DK NRF52_DK NUCLEO_F207ZG NUCLEO_F401RE NUCLEO_F429ZI NUCLEO_F746ZG REALTEK_RTL8195AM
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.
Looks good to me
CI started |
Looks fine to me, was this line causing a failure? |
No failure was reported, it was discovered while reviewing code for some other issue. |
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.
Thanks, looks fine.
Test run: SUCCESSSummary: 4 of 4 test jobs passed |
Description
When lp ticker is already initialised, then ticker init function should only disable ticker interrupt. In current implementation
NVIC_EnableIRQ(LPTMR0_IRQn);
is called incorrectly in this case.To fix this issue we will call
lp_ticker_disable_interrupt();
when lp ticker is initialised.Pull request type