Skip to content

Conversation

@JordanYates
Copy link
Contributor

Add names to the automatic PHY update choices so the defaults can be updated.

Add names to the automatic PHY update choices so the defaults can be
updated.

Signed-off-by: Jordan Yates <[email protected]>
@JordanYates JordanYates added the Trivial Changes that can be reviewed by anyone, i.e. doc changes, minor build system tweaks, etc. label Dec 2, 2025
@zephyrbot zephyrbot added area: Bluetooth area: Bluetooth Host Bluetooth Host (excluding BR/EDR) labels Dec 2, 2025
@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 2, 2025

Copy link
Member

@jhedberg jhedberg left a comment

Choose a reason for hiding this comment

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

Seems fine to me, but I'd be curious to hear how you prefer to change the defaults and why? The current values are simply to retain as close behavior as possible to what we had before, with the exception that as peripheral you have to explicitly enable the AUTO_PHY_UPDATE option to get this.

@JordanYates
Copy link
Contributor Author

Seems fine to me, but I'd be curious to hear how you prefer to change the defaults and why? The current values are simply to retain as close behavior as possible to what we had before, with the exception that as peripheral you have to explicitly enable the AUTO_PHY_UPDATE option to get this.

The defaults might be same (since AUTO_PHY_UPDATE previously defaulted to y), but right now there is no Kconfig default way to stop any PHY updates from happening (you can't set defaults on choice options, just the choice itself).

choice
prompt "Auto PHY update for central role"
depends on BT_CENTRAL
default BT_AUTO_PHY_CENTRAL_2M

@kartben
Copy link
Contributor

kartben commented Dec 2, 2025

It is worth adding the "new" symbols to the release notes please

@kartben kartben merged commit e401209 into zephyrproject-rtos:main Dec 2, 2025
36 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: Bluetooth Host Bluetooth Host (excluding BR/EDR) area: Bluetooth Trivial Changes that can be reviewed by anyone, i.e. doc changes, minor build system tweaks, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants