Skip to content

drivers: clock_control: nrf_power_clock: Make isr function global #19673

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

Conversation

nordic-krch
Copy link
Contributor

@nordic-krch nordic-krch commented Oct 8, 2019

Some tests are failing because nrf_power_clock_isr was static.

Apparently, some tests are using irq function (

) which was declared static in the driver.

Fixes #19683.
Signed-off-by: Krzysztof Chruscinski krzysztof.chruscinski@nordicsemi.no

@nordic-krch nordic-krch added area: Clock Control Trivial Changes that can be reviewed by anyone, i.e. doc changes, minor build system tweaks, etc. labels Oct 8, 2019
Copy link
Member

@ioannisg ioannisg left a comment

Choose a reason for hiding this comment

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

OK, but add an inline comment. I think the nrf_rtc_timer has a similar comment there.

Some tests are failing because nrf_power_clock_isr was static.

Signed-off-by: Krzysztof Chruscinski <krzysztof.chruscinski@nordicsemi.no>
@nordic-krch nordic-krch force-pushed the fix_nrf_power_clock_isr_linking branch from 00ab260 to 053a1e3 Compare October 8, 2019 10:30
@nordic-krch
Copy link
Contributor Author

@ioannisg comment was present but at isr implementation. It was earlier declared to be used earlier and declaration was static. I moved the comment there.

@carlescufi carlescufi requested a review from pabigot October 8, 2019 11:31
@ioannisg
Copy link
Member

ioannisg commented Oct 8, 2019

@ioannisg comment was present but at isr implementation. It was earlier declared to be used earlier and declaration was static. I moved the comment there.

Right, that's exactly what I wanted

@ioannisg ioannisg added the Hotfix Fix for issues blocking development, i.e. CI issues, tests failing in CI, etc. label Oct 8, 2019
@carlescufi carlescufi merged commit a3bc875 into zephyrproject-rtos:master Oct 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Clock Control Hotfix Fix for issues blocking development, i.e. CI issues, tests failing in CI, etc. Trivial Changes that can be reviewed by anyone, i.e. doc changes, minor build system tweaks, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

nrf: clock reimplementation breaks test
4 participants