On 17/12/2025 9:39 am, Leo Yan via CoreSight wrote:
Kconfig symbols must not include the CONFIG_ prefix. Remove the CONFIG_ prefix for default values to work.
Fixes: a02509f301c6 ("stm class: Factor out default framing protocol") Fixes: d69d5e83110f ("stm class: Add MIPI SyS-T protocol support") Signed-off-by: Leo Yan leo.yan@arm.com
drivers/hwtracing/stm/Kconfig | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/hwtracing/stm/Kconfig b/drivers/hwtracing/stm/Kconfig index eda6b11d40a1f9ab49a1ec1e6faae8ee178c5ed3..cd7f0b0f3fbebc74775d8835187e49e5bd21d646 100644 --- a/drivers/hwtracing/stm/Kconfig +++ b/drivers/hwtracing/stm/Kconfig @@ -13,7 +13,7 @@ if STM config STM_PROTO_BASIC tristate "Basic STM framing protocol driver"
- default CONFIG_STM
- default STM help This is a simple framing protocol for sending data over STM devices. This was the protocol that the STM framework used
@@ -28,7 +28,7 @@ config STM_PROTO_BASIC config STM_PROTO_SYS_T tristate "MIPI SyS-T STM framing protocol driver"
- default CONFIG_STM
- default STM
The help below here says "If you don't know what this is, say N." which makes me wonder if this should be on by default or not?
Also everything is in an "if STM" block, so "default STM" is a bit redundant. It's not that obvious what the intention was.
James
help This is an implementation of MIPI SyS-T protocol to be used over the STP transport. In addition to the data payload, it
base-commit: 40fbbd64bba6c6e7a72885d2f59b6a3be9991eeb change-id: 20251216-fix_stm_kconfig-a72f40c7612c
Best regards,