Mbed 2 - Mbed OS 5 : Sleep incorrect for Nordic target

[aracosta1]

Description

• Type: Bug

---

Bug

Target
TARGET_NORDIC
TARGET_MCU_NRF51822
TARGET_NRF5

The sleep() function don't take into account the use of the softdevice.
the correct implementation should be similar to the one used on mbed OS 3:

[pan-]

@nvlsianpu Could you look at this ?

[nvlsianpu]

Thanks for yours comment. I will take care of fix it.

[adbridge]

@aracosta1 It looks like your issue should be fixed with the latest release of mbed-os (5.2.0). Could you please verify and close this issue if so ? Thanks.

[aracosta1]

I'm doing some basic test and it seem to work correctly.
I had one warning during a compilation obout sleep.o not unique but now I cannot reproduce te issue anymore.
I will try to do some test also on OS version 2

[aracosta1]

At the moment I cannot test for the OS version 2 because the build system is broken for the latest library version (see bug #3249). I will update when the build system is fixed

[aracosta1]

I had done some test and for me it's working so I close the issue.