En config and irq #230
En config and irq #230
Conversation
NikhitaR-IFX
force-pushed
the
en-ble
branch
from
45c184b to
179b9b6
Compare
Signed-off-by: NikhitaR-IFX <nikhita.rajasekhar@infineon.com>
Signed-off-by: NikhitaR-IFX <nikhita.rajasekhar@infineon.com>
Signed-off-by: NikhitaR-IFX <nikhita.rajasekhar@infineon.com>
Signed-off-by: NikhitaR-IFX <nikhita.rajasekhar@infineon.com>
NikhitaR-IFX
force-pushed
the
en-config-and-irq
branch
from
dde66e7 to
aa566cc
Compare
jaenrig-ifx
left a comment
jaenrig-ifx
left a comment
There was a problem hiding this comment.
BLE details I can not really judge 👍 Good progress. Step by step 🪜
docs/psoc6/quickref.rst
Outdated
| .. method:: BLE.config('param', /) | ||
| BLE.config(*, param=value, ...) | ||
|
|
||
| Among the suggested parameters of the general network WLAN API, for this port, only these are available: |
| addr[5] = (addr[5] & 0x3F) | 0xC0; // MSB[0:1] - 11 - Static Random Address | ||
| break; | ||
| } | ||
| // ToDo: This is in a way supported from MTB BLE stack side but is complicated to map to MPY side. Requires NVRAM to store the keys. Considering this is not an absolutely necessary feature, let's postpone for extension after MVP? |
There was a problem hiding this comment.
From all these key storage requirements, is there an example of how is this done in other cores?
There was a problem hiding this comment.
We have only 3 references. All the ports that have BLE support depend on nimble or btstack and in each following is the implementation:
- extmod/nimble/modbluetooth_nimble.c : IRK is stored in NVM. This might give some explanation
- extmod/btstack/modbluetooth_stack.c : Not supported (https://github.com/Infineon/micropython/blob/ports-psoc6-main/extmod/btstack/modbluetooth_btstack.c#L777)
- ports/zephyr/modbluetooth_zephyr.c : Not supported
We have API's in MTB BLE stack to do this, but just that it is not evaluated yet how to especially with the NVM enablement part.
| raise SystemExit | ||
|
|
||
|
|
||
| def irq_handle(event, data): |
There was a problem hiding this comment.
Can we already add this tests to the HIL? Or to early!?
There was a problem hiding this comment.
The events possible in this handler are the only ones listed here: https://ifx-micropython.readthedocs.io/en/latest/library/bluetooth.html#bluetooth.BLE.irq
This is right now solved in extmod/modbluetooth.c already. We need to at least enable the roles test this irq functionality wise. These are the next tickets I am working on. So I would say, little more time that we get the test in required shape.
Signed-off-by: NikhitaR-IFX <nikhita.rajasekhar@infineon.com>
I understand. I am also trying to add some "design considerations" section for retro and maybe it is easier then to comment :) |
|
I will merge this and in case of any more questions let me know :) |
All good! You are now the BLE expert 👯♀️ |
Summary