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

Semjon Kerner notifications at github.com
Wed Apr 17 11:37:21 CEST 2019

# Backport of #11395

### 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...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190417/7ae2daa1/attachment.html>

More information about the notifications mailing list