[riot-notifications] [RIOT-OS/RIOT] cpu/nrf5x/nrfmin: fix isr termination (#11395)
notifications at github.com
Mon Apr 15 13:27:43 CEST 2019
### Contribution description
The nrfmin driver had a serious bug terminating the interrupt handler in case of an invalid CRC value.
Instead of closing with `cortexm_isr_end();` it just returned, which resulted in unwanted behaviour.
This PR fixes one error mentioned in #10878 and is based on the solution of @pystub
### Testing procedure
Testing should at least involve receiving and then sending packets with a valid CRC. However, it is appreciated if a reviewer makes sure that the behaviour is also correct with invalid CRC.
Make sure to test it using nrfmin with `USEMODULE=nrfmin` on nrf5x boards.
You can view, comment on, or merge this pull request online at:
-- Commit Summary --
* cpu/nrf5x/nrfmin: fix isr termination
-- File Changes --
M cpu/nrf5x_common/radio/nrfmin/nrfmin.c (7)
-- Patch Links --
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the notifications