[bug report] low_power_pwm.c (Low-Power PWM Example)

[Karel_TandD]

In the Keil Pack nRF_Libraries V3.10 [SDK V10.0.0?] (Low-Power PWM Example)

Not a big Bug, but an inconvenience all the same.

low_power_pwm.c line 165

---

ret_code_t low_power_pwm_start(low_power_pwm_t * p_pwm_instance, uint32_t leds_pin_bit_mask)
{
ASSERT(p_pwm_instance->pwm_state == NRF_DRV_STATE_INITIALIZED);
p_pwm_instance->evt_type = LOW_POWER_PWM_EVENT_PERIOD;
pwm_timeout_handler(p_pwm_instance);
p_pwm_instance->pwm_state = NRF_DRV_STATE_POWERED_ON;
p_pwm_instance->bit_mask = leds_pin_bit_mask; //karel added
return NRF_SUCCESS;
}

---

There is no line to set the bit mask, so I added a line to my own code (BOLD above).
Also in the edit[(Low-Power PWM Example WRONG) (led_softblink_s110_pca10028 RIGHT)] main.c the LEDs functionality is set in/via the
static void leds_init(void) {...}
function.

[Ole Bauck]

I will report this. I cannot see however that the LEDs functionality is set via leds_init(), I cannot find this function.

[Karel_TandD]

Oops.　My mistake. The example was in fact led_softblink_s110_pca10028. main.c leds_init() calls led_softblink_init(&led_sb_init_params), which calls low_power_pwm_init(..) which sets the LEDS as outputs (nrf_gpio_cfg_output(pin_number);)

[Stanley]

Dears, Who knows that the differences between low pwm driver and pwm driver? Is it just for power consumption only?

[Ole Bauck]

Low power pwm uses app_timer to set/clear the pin and is therefore low power, but not accurate. See here for more info.