Skip to content

drivers/sx126x: bind PA config to output power#21687

Merged
crasbe merged 1 commit intoRIOT-OS:masterfrom
fabian18:pr/sx126x_tx_power
Sep 10, 2025
Merged

drivers/sx126x: bind PA config to output power#21687
crasbe merged 1 commit intoRIOT-OS:masterfrom
fabian18:pr/sx126x_tx_power

Conversation

@fabian18
Copy link
Copy Markdown
Contributor

@fabian18 fabian18 commented Aug 28, 2025

Contribution description

Makes it possible to use different transmission powers on sx126x.
I did not necessarily add a field to save the Tx power because this would come in with the radio HAL adaptation.

Testing procedure

DEVELHELP=1 VERBOSE_ASSERT=1 BOARD=adafruit-metro-m4-express USEMODULE+="shell_cmd_ps shield_sx1262" make -C examples/networking/gnrc/gnrc_lorawan flash term PORT=/dev/ttyACM0

Issues/PRs references

Split out from #21675

@github-actions github-actions bot added Area: drivers Area: Device drivers Area: LoRa Area: LoRa radio support labels Aug 28, 2025
@crasbe crasbe added Type: new feature The issue requests / The PR implemements a new feature for RIOT CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Aug 29, 2025
@riot-ci
Copy link
Copy Markdown

riot-ci commented Aug 29, 2025

Murdock results

✔️ PASSED

f54c8ff drivers/sx126x: bind PA config to output power

Success Failures Total Runtime
10515 0 10516 10m:00s

Artifacts

@fabian18 fabian18 requested a review from fjmolinas September 2, 2025 08:15
@kfessel
Copy link
Copy Markdown
Contributor

kfessel commented Sep 4, 2025

please squash

@fabian18 fabian18 added this pull request to the merge queue Sep 8, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Sep 8, 2025
@fabian18 fabian18 added this pull request to the merge queue Sep 10, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Sep 10, 2025
@crasbe crasbe enabled auto-merge September 10, 2025 11:01
@crasbe crasbe added this pull request to the merge queue Sep 10, 2025
@crasbe crasbe removed this pull request from the merge queue due to a manual request Sep 10, 2025
@crasbe crasbe added this pull request to the merge queue Sep 10, 2025
Merged via the queue into RIOT-OS:master with commit ef76a04 Sep 10, 2025
25 checks passed
@benpicco benpicco added this to the Release 2025.10 milestone Dec 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: drivers Area: Device drivers Area: LoRa Area: LoRa radio support CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Type: new feature The issue requests / The PR implemements a new feature for RIOT

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants