-
Notifications
You must be signed in to change notification settings - Fork 6.5k
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
bluetooth: shell: Fix includes #80105
base: main
Are you sure you want to change the base?
bluetooth: shell: Fix includes #80105
Conversation
This allows to build the shell with BT_CTLR_DTM and/or BT_CTLR_ADV_EXT enabled. The issues has been introduced by commit bf897cf (Bluetooth: Shell: Restructure shell files). Signed-off-by: Reto Schneider <reto.schneider@husqvarnagroup.com>
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.
Question:
* For those two settings, would it be okay (tolerated? appreciated?) to add an an extra build in CI "just" to test whether enabling those flags does not break the build?
Please add a new tests
target in testcases.yaml
file with added extra_configs
with CONFIG_BT_CTLR_DTM
for nrf52840dk/nrf52840
board.
Using which board did you discover the issue?
@@ -53,7 +53,7 @@ int cmd_ll_addr_read(const struct shell *sh, size_t argc, char *argv[]) | |||
} | |||
|
|||
#if defined(CONFIG_BT_CTLR_DTM) | |||
#include "../controller/ll_sw/ll_test.h" | |||
#include "../ll_test.h" |
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.
#include "../ll_test.h" | |
#include "controller/ll_sw/ll_test.h" |
@@ -123,7 +123,7 @@ int cmd_test_end(const struct shell *sh, size_t argc, char *argv[]) | |||
#endif /* CONFIG_BT_CTLR_DTM */ | |||
|
|||
#if defined(CONFIG_BT_CTLR_ADV_EXT) | |||
#include "controller/ll_sw/lll.h" | |||
#include "../lll.h" |
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.
#include "../lll.h" | |
#include "controller/ll_sw/lll.h" |
An nRF52840 (or maybe nRF54L15) based out-of-tree board. |
Any proposal on which
Or did you mean to add a directory |
|
This allows to build the shell with BT_CTLR_DTM and/or BT_CTLR_ADV_EXT enabled.
The issues has been introduced by commit
bf897cf (Bluetooth: Shell: Restructure shell files).
Question: