Fix nrf52 memory pools

[pan-]

Description

Inclusion of #10666 has broken BLE on NRF52_DK. This patch also reduce the RAM necessary to run BLE on NRF52XXX targets.

• NRF52_DK: 3.3K of RAM saved
• NRF52840_DK: ~39K of RAM saved.

Pull request type

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

Reviewers

@LDong-Arm @paul-szczepanek-arm @donatieng

Release Notes

[ciarmcom]

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

[0xc0170]

5.13.1 labeled (if this should be in the next RC, let us know)

[0xc0170]

5.13.1 labeled (if this should be in the next RC, let us know)

rc2 to address this issue in rc1

cc @adbridge

[0xc0170]

I started the CI job meanwhile

[0xc0170]

CI failed to check-out, we are investigating

cc @ARMmbed/mbed-os-test

[donatieng]

Looks good, thanks @pan-

[cmonr]

CI restarted. S3 issues appear addressed.

[mbed-ci]

Test run: FAILED

Summary: 1 of 7 test jobs failed
Build number : 6
Build artifacts

Failed test jobs:

• jenkins-ci/mbed-os-ci_build-IAR

[bulislaw]

Approved for RC2

[0xc0170]

IAR reports build failures

ToolException: 
    l2cRegCb_t          regCb[L2C_COC_REG_MAX];   /* Registration control blocks */
                        ^
"/builds/ws/mbed-os-ci_build-IAR@4/mbed-os/features/FEATURE_BLE/targets/TARGET_CORDIO/stack/ble-host/sources/stack/l2c/l2c_coc.c",135  Error[Pe070]: incomplete type is not allowed

    l2cChanCb_t         chanCb[L2C_COC_CHAN_MAX]; /* Channel control blocks */
                        ^
"/builds/ws/mbed-os-ci_build-IAR@4/mbed-os/features/FEATURE_BLE/targets/TARGET_CORDIO/stack/ble-host/sources/stack/l2c/l2c_coc.c",137  Error[Pe070]: incomplete type is not allowed

    for (i = 0; i < L2C_COC_REG_MAX; i++, pCb++)

Please review

[0xc0170]

Does L2C_COC_CHAN_MAX need to be 0 ? It produces warnings in comparisons (against 0) plus incomplete type above

[pan-]

Looks like a value of 1 would be more adequate in this PR then.
A value of 0 should be allowed; let see what we can do after the release.

[paul-szczepanek-arm]

does it matter though? we don't use that feature, I assume no allocations take place

[0xc0170]

Once updated, let us know, will restart testing

[pan-]

@0xc0170 Updated.

[0xc0170]

CI restarted

[mbed-ci]

Test run: SUCCESS

Summary: 11 of 11 test jobs passed
Build number : 7
Build artifacts