Support two peripherals on F4 hubs

[laurensvalk]

This the follow up to #445 to enable more peripherals. This is only done for BTstack.

I think we can keep Technic/City/Move hub at one peripheral given their limitations.

The first commits are basically just replacing the peripheral_singleton with a pointer to an instance. The logic to make this possible was already done in #445.

Perhaps the more interesting part for reviewing in this PR is this:

 #define HCI_ACL_PAYLOAD_SIZE (1691 + 4)
 #define MAX_ATT_DB_SIZE 512
 #define MAX_NR_BTSTACK_LINK_KEY_DB_MEMORY_ENTRIES  0
-#define MAX_NR_GATT_CLIENTS 1
-#define MAX_NR_HCI_CONNECTIONS 2 // CC2564C can have up to 10 connections
+#define MAX_NR_GATT_CLIENTS 2
+#define MAX_NR_HCI_CONNECTIONS 3 // CC2564C can have up to 10 connections
 #define MAX_NR_HFP_CONNECTIONS 0
 #define MAX_NR_L2CAP_CHANNELS  0
 #define MAX_NR_L2CAP_SERVICES  0
@@ -30,9 +30,9 @@
 #define MAX_NR_RFCOMM_MULTIPLEXERS 0
 #define MAX_NR_RFCOMM_SERVICES 0
 #define MAX_NR_SERVICE_RECORD_ITEMS 0
-#define MAX_NR_SM_LOOKUP_ENTRIES 0
+#define MAX_NR_SM_LOOKUP_ENTRIES 3
 #define MAX_NR_WHITELIST_ENTRIES 0
-#define MAX_NR_LE_DEVICE_DB_ENTRIES 1
+#define MAX_NR_LE_DEVICE_DB_ENTRIES 3

We are deleting the bonds every time so it's probably not necessary to have the lookup entries, but this isn't too expensive.

Bumping the HCI connections and GATT clients does need a fair bit of RAM, so this also does this:

diff --git a/lib/pbio/platform/prime_hub/platform.ld b/lib/pbio/platform/prime_hub/platform.ld
index ba433b5b2..3fb07d7aa 100644
--- a/lib/pbio/platform/prime_hub/platform.ld
+++ b/lib/pbio/platform/prime_hub/platform.ld
@@ -19,4 +19,4 @@ MEMORY
 
 /* Minimal stack size to allow the build to proceed. The actual stack size is whatever is left after
    statically allocated memory. It can be tuned by setting PBDRV_CONFIG_BLOCK_DEVICE_RAM_SIZE. */
-_minimal_stack_size = 16K;
+_minimal_stack_size = 12K;

14K works fine too. I picked 12K because we have that on Essential Hub too and it should still be far more than necessary. This should give us some room for static memory in future developments. (We could also reduce heap instead. We did increase it recently after all to have some heap left even if all slots are full.)

[coveralls]

coverage: 50.339% (-0.07%) from 50.406%
when pulling 076ca37 on peripheral
into 136126d on btstack-scan-connect-bond.

[github-actions]

Download the artifacts for this pull request:

• pr_number.zip
• mpy-cross.zip
• movehub-firmware-build-4596-git6c83738d.zip
• nxt-firmware-build-4596-git6c83738d.zip
• ev3-firmware-build-4596-git6c83738d.zip
• cityhub-firmware-build-4596-git6c83738d.zip
• essentialhub-firmware-build-4596-git6c83738d.zip
• technichub-firmware-build-4596-git6c83738d.zip
• primehub-firmware-build-4596-git6c83738d.zip
• buildhat-firmware-build-4596-git6c83738d.zip

[laurensvalk]

Merged into 076ca37 without changes, but GitHub won't register it due to the different base branch.

[laurensvalk]

There are a few things here that are still using singletons and should be moved into the peripheral instances:

• char write data
• char read data