-
Notifications
You must be signed in to change notification settings - Fork 8.3k
drivers: can: use proper namespace for driver-specific Kconfig symbols #100596
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
drivers: can: use proper namespace for driver-specific Kconfig symbols #100596
Conversation
f3d19c2 to
149b3dd
Compare
149b3dd to
f0b9f09
Compare
martinjaeger
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Now that the Kconfig is changed anyway, should it not be plural? _MAX_FILTERS
f0b9f09 to
471c01a
Compare
Use proper namespace for the CAN loopback driver Kconfig symbols, renaming CONFIG_CAN_MAX_FILTER to CONFIG_CAN_LOOPBACK_MAX_FILTERS. This prevents Kconfig symbol clashes between different CAN controller drivers. Signed-off-by: Henrik Brix Andersen <henrik@brixandersen.dk>
Use proper namespace for the CAN native_linux driver Kconfig symbols, renaming CONFIG_CAN_MAX_FILTER to CONFIG_CAN_NATIVE_LINUX_MAX_FILTERS. This prevents Kconfig symbol clashes between different CAN controller drivers. Signed-off-by: Henrik Brix Andersen <henrik@brixandersen.dk>
Use proper namespace for the CAN sja1000 Kconfig symbols, renaming CONFIG_CAN_MAX_FILTER to CONFIG_CAN_SJA1000_MAX_FILTERS. This prevents Kconfig symbol clashes between different CAN controller drivers. Signed-off-by: Henrik Brix Andersen <henrik@brixandersen.dk>
Use proper namespace for the CAN mcp2515 Kconfig symbols, renaming CONFIG_CAN_MAX_FILTER to CONFIG_CAN_MCP2515_MAX_FILTERS. This prevents Kconfig symbol clashes between different CAN controller drivers. Signed-off-by: Henrik Brix Andersen <henrik@brixandersen.dk>
Use proper namespace for the CAN max32 Kconfig symbols, renaming CONFIG_CAN_MAX_FILTER to CONFIG_CAN_MAX32_MAX_FILTERS. This prevents Kconfig symbol clashes between different CAN controller drivers. Signed-off-by: Henrik Brix Andersen <henrik@brixandersen.dk>
Use proper namespace for the CAN mcp251xfd Kconfig symbols, renaming CONFIG_CAN_MAX_FILTER to CONFIG_CAN_MCP251XFD_MAX_FILTERS. This prevents Kconfig symbol clashes between different CAN controller drivers. Signed-off-by: Henrik Brix Andersen <henrik@brixandersen.dk>
Use proper namespace for the MCUX FlexCAN Kconfig symbols, renaming CONFIG_CAN_MAX_FILTER to CONFIG_CAN_MCUX_FLEXCAN_MAX_FILTERS and CONFIG_CAN_MAX_MB to CONFIG_CAN_MCUX_FLEXCAN_MAX_MB. This prevents Kconfig symbol clashes between different CAN controller drivers. Signed-off-by: Henrik Brix Andersen <henrik@brixandersen.dk>
Use proper namespace for the STM32 bxCAN Kconfig symbols, renaming CONFIG_CAN_MAX_STD_ID_FILTER to CONFIG_CAN_STM32_BXCAN_MAX_STD_ID_FILTERS and CONFIG_CAN_MAX_EXT_ID_FILTER to CONFIG_CAN_STM32_BXCAN_MAX_EXT_ID_FILTERS. This prevents Kconfig symbol clashes between different CAN controller drivers. Signed-off-by: Henrik Brix Andersen <henrik@brixandersen.dk>
Use proper namespace for the STM32 FDCAN Kconfig symbols, renaming CONFIG_CAN_MAX_STD_ID_FILTER to CONFIG_CAN_STM32_FDCAN_MAX_STD_ID_FILTERS and CONFIG_CAN_MAX_EXT_ID_FILTER to CONFIG_CAN_STM32_FDCAN_MAX_EXT_ID_FILTERS. This prevents Kconfig symbol clashes between different CAN controller drivers. Signed-off-by: Henrik Brix Andersen <henrik@brixandersen.dk>
Use proper namespace for the CAN xmc4xxx Kconfig symbols, renaming CONFIG_CAN_MAX_FILTER to CONFIG_CAN_XMC4XXX_MAX_FILTERS. This prevents Kconfig symbol clashes between different CAN controller drivers. Signed-off-by: Henrik Brix Andersen <henrik@brixandersen.dk>
Rename Kconfig option CONFIG_CAN_RCAR_MAX_FILTER to CONFIG_CAN_RCAR_MAX_FILTERS to match remaining CAN driver Kconfig option naming. Signed-off-by: Henrik Brix Andersen <henrik@brixandersen.dk>
Increase the default number of software-limited CAN RX filters across the drivers. Some of these were chosen quite conservative, requiring custom configuration for even simple in-tree samples. Users can reduce the number of available RX filters to reduce RAM footprint as needed. Signed-off-by: Henrik Brix Andersen <henrik@brixandersen.dk>
471c01a to
df0074f
Compare
Thanks! Yes, good idea - I've changed it. |
Remove setting of CONFIG_CAN_MAX_FILTER and rely on driver defaults. Signed-off-by: Henrik Brix Andersen <henrik@brixandersen.dk>
Mention the CAN controller driver Kconfig namespace changes in the migration guide. Signed-off-by: Henrik Brix Andersen <henrik@brixandersen.dk>
df0074f to
5d04022
Compare
|



Currently, quite a few CAN controller drivers define Kconfig symbols with identical names (e.g.
CONFIG_CAN_MAX_FILTER) but the ranges and defaults of these symbols differ between drivers, causing clashes and misconfigurations when multiple drivers are enabled concurrently.Rename these driver-specific Kconfig symbols to use proper namespacing and increase their defaults where needed to support typical software needs and in-tree samples.
This problem became evident with the reshuffling of the Kconfig.* include order in 30950b8.