Revert "Merge pull request #8299 from hasnainvirk/issue_8285"

[cmonr]

Description

Verifying something with CI. New jobs have been failing unittests, but have not been able to locally reproduce issue. Unless CI configuration has changed without warning, this is the only change that could be breaking the job, since it's it's the only LoRa change to be merged.

The particular CI failure did not occur until after the rollup PR was merged (in case anyone is wondering).

Pull request type

[x] Fix
[ ] Refactor
[ ] Target update
[ ] Functionality change
[ ] Docs update
[ ] Test update
[ ] Breaking change

[cmonr]

/morph build

[0xc0170]

@hasnainvirk Can you provide details how this break the master?

[mbed-ci]

Build : SUCCESS

Build number : 3364
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/8436/

Triggering tests

/morph test
/morph export-build
/morph mbed2-build

[mbed-ci]

Exporter Build : SUCCESS

Build number : 2998
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/exporter/8436/

[mbed-ci]

Test : SUCCESS

Build number : 3168
Test logs :http://mbed-os-logs.s3-website-us-west-1.amazonaws.com/?prefix=logs/8436/3168

[hasnainvirk]

Please don't revert. Merge this one instead #8405.
It has commits which fix all the issues with the unit tests which surfaced after the merge and revert of QOS PR.

[cmonr]

@hasnainvirk I can give #8405 some time, but getting the unittest job unblocked is a priority so that we can continue bringing in other PRs.

[NirSonnenschein]

@hasnainvirk @OPpuolitaival , latest word from @0xc0170 is that this needs to be fixed today, if you can get #8405 working today then great, otherwise we will need to revert.

[cmonr]

Fixed with #8405