[riot-notifications] [RIOT-OS/RIOT] cpu/nrf5x/nrfmin: fix isr termination (#11395)

Semjon Kerner 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:

  https://github.com/RIOT-OS/RIOT/pull/11395

-- Commit Summary --

  * cpu/nrf5x/nrfmin: fix isr termination

-- File Changes --

    M cpu/nrf5x_common/radio/nrfmin/nrfmin.c (7)

-- Patch Links --

https://github.com/RIOT-OS/RIOT/pull/11395.patch
https://github.com/RIOT-OS/RIOT/pull/11395.diff

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/11395
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190415/e2342309/attachment-0001.html>


More information about the notifications mailing list