Skip to content
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

mgmt/mcumgr: Strange Kconfig names for MCUMGR_GRP_ZEPHYR_BASIC log levels #43700

Closed
de-nordic opened this issue Mar 11, 2022 · 0 comments · Fixed by #44639
Closed

mgmt/mcumgr: Strange Kconfig names for MCUMGR_GRP_ZEPHYR_BASIC log levels #43700

de-nordic opened this issue Mar 11, 2022 · 0 comments · Fixed by #44639
Assignees
Labels
area: mcumgr bug The issue is a bug, or the PR is fixing a bug priority: low Low impact/importance bug

Comments

@de-nordic
Copy link
Collaborator

Describe the bug
The name resolved from:

module=MGMT_SETTINGS
module-dep=LOG
module-str=SETTINGS
source "${ZEPHYR_BASE}/subsys/logging/Kconfig.template.log_config"

evaluates to CONFIG_MGMT_SETTINGS_LOG_LEVEL... which does not reflect area it serves.

To Reproduce
Just run:

west build -d smp_svr_btt/ -b nrf52840dk_nrf52840 zephyr/samples/subsys/mgmt/mcumgr/smp_svr/ -t menuconfig -- -DOVERLAY_CONFIG=overlay-bt.conf

go to: Sub Systems and OS Services->Device Management->mcumgr support->Command Handlers->Zephyr specific basic group of commands
and see:
image

Expected behavior
The log Kconfig options should reflect command group it serves.

Impact
Confusion.

Environment (please complete the following information):
Still present at a2fe23b

Additional context
The MCUMGR_GRP_ZEPHYR_BASIC could also get renaming.

@de-nordic de-nordic added the bug The issue is a bug, or the PR is fixing a bug label Mar 11, 2022
@mbolivar-nordic mbolivar-nordic added the priority: low Low impact/importance bug label Mar 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: mcumgr bug The issue is a bug, or the PR is fixing a bug priority: low Low impact/importance bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants