Feature cordio update 20 05

[pan-]

Summary of changes

Update of the Cordio controller and LL stack to the version 20.05. This version of the stack is BT 5.2 certified.
However for resource usage, BT5.2 features are disabled by the mbed os integration.

Impact of changes

From a behavior standpoint, the change should be invisible for users of BLE API. However the latest version of the Cordio stack consumes more flash and more RAM:

• ~5% increase in flash consumption (+10kB).
• ~6% increase in RAM consumption (+1.3kB).

Migration actions required

None

Documentation

None

---

Pull request type

[x] Patch update (Bug fix / Target update / Docs update / Test update / Refactor)
[] Feature update (New feature / Functionality change / New API)
[] Major update (Breaking change E.g. Return code change / API behaviour change)

---

Test results

[] No Tests required for this change (E.g docs only update)
[] Covered by existing mbed-os tests (Greentea or Unittest)
[] Tests / results supplied as part of this PR

---

Reviewers

@donatieng @paul-szczepanek-arm

---

[mergify]

This PR cannot be merged due to conflicts. Please rebase to resolve them.

[ciarmcom]

@pan-, thank you for your changes.
@paul-szczepanek-arm @donatieng @ARMmbed/mbed-os-pan @ARMmbed/mbed-os-maintainers please review.

[0xc0170]

After rebasing, please also update:

Update of the Cordio controller and LL stack to the latest versions.

What version are we getting. Is this just a patch bump (cordio also is just version bump or not) ?

[paul-szczepanek-arm]

I can't edit the description but it's a 20.05 release (old one was 19.2 I think) which is more than a patch, it has new ble 5.2 features but these are not yet enabled.

[ghost]

The changes might be invisible API wise, but I hope the changelog description mentions any differences in resource usage if there are any.

[pan-]

@jrobeson It affects memory consumption, I updated the description with some numbers when all the features are enabled. Depending on the features enabled, impact on code size and RAM usage may be lower.

[pan-]

@0xc0170 I have updated the description and provided some numbers for the visible changes in binary size.
@paul-szczepanek-arm @donatieng Can you review the PR ?

[donatieng]

LGTM (haven't reviewed the stack/LL changes themselves but looked at the integration). @ARMmbed/mbed-os-maintainers would you be able to start the CI while awaiting @paul-szczepanek-arm?

[paul-szczepanek-arm]

I would be reviewing my own code. I can do it, although it would be valuable for someone else to look at it as well.

[mbed-ci]

Test run: SUCCESS

Summary: 6 of 6 test jobs passed
Build number : 1
Build artifacts

[0xc0170]

I'll review shortly so this can get in today if no objections.

[donatieng]

Thanks @0xc0170 !