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

blink_led sample does not work on most of the nRF boards #15323

Closed
anangl opened this issue Apr 10, 2019 · 2 comments · Fixed by #20369
Closed

blink_led sample does not work on most of the nRF boards #15323

anangl opened this issue Apr 10, 2019 · 2 comments · Fixed by #20369
Assignees
Labels
bug The issue is a bug, or the PR is fixing a bug platform: nRF Nordic nRFx priority: medium Medium impact/importance bug
Milestone

Comments

@anangl
Copy link
Member

anangl commented Apr 10, 2019

The issue was originally reported on slack (#general):

Aaron Tsui 9:15 AM, March 22nd
BTW, As the PWM_CLK lowest is 125khz(16M/128) and COUNTERTOP max is 0x7fff on nrf52, my understanding the nrf52 HW PWM longest period is about 262ms, cannot reach period as 1000ms. right?
@gaute this will cause failure on sample/blink_led, which need a 1000ms period.

Indeed, when trying to run this sample on any board with an nRF52 series SoC, which by default uses the HW PWM driver, one hits the following error:

PWM demo app-blink LED
pwm pin set fails
[00:00:00.008,544] <err> pwm_nrfx: Prescaler for period_cycles 16000000 not found.

[00:00:00.008,544] <err> pwm_nrfx: Incompatible period 16000000

Same happens for boards with nRF91 series SoCs.

For nRF52 the solution could be to use the nRF SW PWM driver, for nRF91 the driver is unavailable.

@anangl anangl added bug The issue is a bug, or the PR is fixing a bug platform: nRF Nordic nRFx labels Apr 10, 2019
@anangl anangl added this to the v1.14.0 milestone Apr 10, 2019
@nashif nashif added the priority: medium Medium impact/importance bug label Apr 10, 2019
@galak galak modified the milestones: v1.14.0, v1.14.1 Apr 16, 2019
@carlescufi
Copy link
Member

@pabigot could you please check if this is still an issue?

@pabigot pabigot added the has-pr label Aug 26, 2019
@pabigot
Copy link
Collaborator

pabigot commented Aug 26, 2019

@carlescufi I believe PR #15287 needs to be rebased and merged first. Right now the build fails on nRF52 because there's no devicetree entry for the required hardware component.

Also the Nordic board definitions seem to be missing the #pwm-cells property which is required to provide the alias to the hardware node (at least using the same approach as in nxp and other boards).

So yes, this still needs work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug The issue is a bug, or the PR is fixing a bug platform: nRF Nordic nRFx priority: medium Medium impact/importance bug
Projects
None yet
5 participants