In ncs 2.4.0, this code was executed in power_manager.c when button pressed while powering down:
In ncs 2.4.0, this code was executed in power_manager.c when button pressed while powering down:
Hi tellg,
I have asked about this behavior change internally and will update you when I get any feedback. Do you mind sharing why you need the wakeup button to work before powering down is completed?
Hieu
I don't need wakeup to work at that point, but if buttonpress happens while powering down, then that button cannot wakeup the system anymore (device reset seems to be needed to fix this). I think the referenced code took care of that situation in ncs 2.4.0, but power down was modified in ncs 2.5.0 to use sys_poweroff, ref https://github.com/nrfconnect/sdk-nrf/commit/b6aa1ab2aae94c5e266c73ef5930d6893cb971d3
Ah I see. That is a problem. I will follow-up with you as soon as I get any information.
Hi tellg,
I have gotten feedback from our team. The issue is indeed with the change to use sys_poweroff(), like you have pointed out.
sys_poweroff() will put the system int power-off right in place. Therefore, additional events, such as the button presses, were not processed.
The developers have registered the issue and will work on an alternative approach. I cannot comment on when that will be available though.
Is this issue critical for you now? If yes, is there any change in NCS 2.5.0 that you need, compared to 2.4.2?
It is not critical, since we have 2 buttons configured for wakeup. The chance of a user pressing both buttons during shutdown in minimal. We have also speeded up our shutdown tasks, so that there is less chance for the issue to happen on our device.
But a later fix will of course be welcomed :-)