Skip to content

drivers/sx126x/Kconfig: rework model selection#17289

Merged
gschorcht merged 1 commit intoRIOT-OS:masterfrom
leandrolanzieri:pr/drivers/sx126x/kconfig/rework
Nov 30, 2021
Merged

drivers/sx126x/Kconfig: rework model selection#17289
gschorcht merged 1 commit intoRIOT-OS:masterfrom
leandrolanzieri:pr/drivers/sx126x/kconfig/rework

Conversation

@leandrolanzieri
Copy link
Contributor

Contribution description

Split from #17232. This rework how sx126x driver modules are selected, making the entry-point the driver module and using a choice for model selection, in combination with features to default to the correct option.

Testing procedure

  • Check that the new model makes sense and shows up correctly in menuconfig
  • Green CI

Issues/PRs references

#17232

@github-actions github-actions bot added Area: drivers Area: Device drivers Area: Kconfig Area: Kconfig integration Area: LoRa Area: LoRa radio support Area: tests Area: tests and testing framework labels Nov 29, 2021
@fjmolinas fjmolinas added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Nov 29, 2021
Copy link
Contributor

@fjmolinas fjmolinas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense, if Murdock is green lets merge.

@gschorcht gschorcht merged commit 8794b35 into RIOT-OS:master Nov 30, 2021
@leandrolanzieri leandrolanzieri deleted the pr/drivers/sx126x/kconfig/rework branch November 30, 2021 08:40
@leandrolanzieri
Copy link
Contributor Author

Thanks for reviewing!

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: Kconfig Area: Kconfig integration Area: LoRa Area: LoRa radio support Area: tests Area: tests and testing framework CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants