pkg/bplib: Experimental UWB CLA - #22647
Open
RainbowSimon wants to merge 10 commits into
Open
Conversation
This implements an experimental CLA used over UWB with decadriver. The CLA logic itself is not UWB specific, but could apply to all 802.15.4 frames, so this could be generalized in future efforts.
A manual execution test between two boards with decadriver compatible modules.
RainbowSimon
requested review from
MichelRottleuthner,
aabadie and
leandrolanzieri
as code owners
September 1, 2026 11:18
github-actions
Bot
added
Area: doc
crasbe
added
Type: new feature
crasbe
reviewed
Sep 1, 2026
crasbe
left a comment
crasbe
left a comment
Contributor
There was a problem hiding this comment.
Thank you very much again :)
I haven't tested it yet, but some initial comments from the first review round are below.
Don't worry about the failing buildsystem sanity check static test, that will be fixed once #22645 is merged.
Comment on lines
+182
to
+188
| dw3000_hw_init(); | ||
| dw3000_hw_init_interrupt(); | ||
| dw3000_hw_reset(); | ||
|
|
||
| dwt_probe((struct dwt_probe_s *)&dw3000_probe_interf); | ||
| uint32_t dev_id = dwt_readdevid(); | ||
| printf("[deca init] detected device id: %"PRIx32"\n", dev_id); |
Contributor
There was a problem hiding this comment.
Would it make sense to adapt the error checking from the other UWB test here too?
Comment on lines
+24
to
+26
| MSG="Warning: No default decadriver configuration for \"$(BOARD)\"."\ | ||
| "Make sure to set all CFLAGS manually!" | ||
| $(shell $(COLOR_ECHO) "$(COLOR_YELLOW)$(MSG)$(COLOR_RESET)" 1>&2) |
Contributor
There was a problem hiding this comment.
Suggested change
| MSG="Warning: No default decadriver configuration for \"$(BOARD)\"."\ | |
| "Make sure to set all CFLAGS manually!" | |
| $(shell $(COLOR_ECHO) "$(COLOR_YELLOW)$(MSG)$(COLOR_RESET)" 1>&2) | |
| $(call echowarn,("Warning: No default decadriver configuration for \"$(BOARD)\"."\ | |
| "Make sure to set all required CFLAGS manually!")) |
Contributor
Author
There was a problem hiding this comment.
Of course out of scope of this PR but is there a way to put default configurations and detection thereof into decadriver?
Like this every example involving decadriver needs to warn and also have the Makefile.board.dep
Contributor
There was a problem hiding this comment.
The general philosophy is to keep board-dependant stuff out of the general modules and rather keep it in the examples.
Contributor
|
The BOARD_INSUFFICIENT_MEMORY := \
airfy-beacon \
b-l072z-lrwan1 \
blackpill-stm32f103c8 \
blackpill-stm32f103cb \
bluepill-stm32f030c8 \
bluepill-stm32f103c8 \
bluepill-stm32f103cb \
e104-bt5010a-tb \
e104-bt5011a-tb \
hifive1 \
hifive1b \
i-nucleo-lrwan1 \
im880b \
lsn50 \
maple-mini \
nucleo-c031c6 \
nucleo-c071rb \
nucleo-f030r8 \
nucleo-f031k6 \
nucleo-f042k6 \
nucleo-f070rb \
nucleo-f072rb \
nucleo-f103rb \
nucleo-f303k8 \
nucleo-f334r8 \
nucleo-g031k8 \
nucleo-l011k4 \
nucleo-l031k6 \
nucleo-l053r8 \
nucleo-l073rz \
olimexino-stm32 \
samd10-xmini \
saml10-xpro \
saml11-xpro \
slstk3400a \
spark-core \
stk3200 \
stm32f030f4-demo \
stm32f0discovery \
stm32l0538-disco \
yunjia-nrf51822 \
# |
This was referenced Sep 3, 2026
crasbe
added
the
State: needs rebase
Contributor
Author
|
Can I rebase? I think I added all current review points |
Contributor
|
Yes 👍 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Contribution description
This adds an experimental, non standard, CLA (Convergence Layer Adapter) for bplib. This allows sending bundles over UWB using decadriver.
This could be generalized to a 802.15.4 CLA, since the communication itself is not UWB specific.
The other CLAs have an example instead of a test, here I thought this might (in its current state) be a little too specific and experimental to be put into the example folder.
Testing procedure
Added
tests/pkg/bplib_cla_uwbto test this. One needs two boards with decadriver compatible UWB modules.Then, using e.g. the
bp send 0 "Hello World"shell command, one can see the bundle arrive on the other side, meaning it was successfully sent though a BPv7 encoded bundle over the CLA via the UWB PHY.Issues/PRs references
no
Declaration of AI-Tools / LLMs usage:
AI-Tools / LLMs that were used are: