golioth / golioth-zephyr-sdk

Golioth SDK For Zephyr
https://www.golioth.io
Apache License 2.0
65 stars 20 forks source link

samples/test: fix logging warnings #378

Closed szczys closed 1 year ago

szczys commented 1 year ago

change Kconfig selections related to logging.

This fixes the following build warning:

warning: LOG_PROCESS_THREAD_STACK_SIZE (defined at subsys/logging/Kconfig.processing:77) was
assigned the value '2048' but got the value ''. Check these unsatisfied dependencies:
LOG_PROCESS_THREAD (=n), LOG_MODE_DEFERRED (=n), (!LOG_MODE_MINIMAL) (=n). See
http://docs.zephyrproject.org/latest/kconfig.html#CONFIG_LOG_PROCESS_THREAD_STACK_SIZE and/or look
up LOG_PROCESS_THREAD_STACK_SIZE in the menuconfig/guiconfig interface. The Application Development
Primer, Setting Configuration Values, and Kconfig - Tips and Best Practices sections of the manual
might be helpful too.

And prevents ASSERTION FAILED when running the test:

*** Booting Zephyr OS build zephyr-v3.3.0-971-gcf20ff0765ab ***
[00:00:00.364,807] <inf> golioth_system: Initializing
[00:00:00.379,669] <inf> fs_nvs: 8 Sectors of 4096 bytes
[00:00:00.388,458] <inf> fs_nvs: alloc wra: 0, f88
[00:00:00.396,728] <inf> fs_nvs: data wra: 0, b8
Running TESTSUITE test_golioth
===================================================================
[00:00:00.420,440] <inf> golioth_samples: Waiting for interface to be up
[00:00:02.487,731] <inf> wifi_esp_at: AT version: 2.4.0.0(s-4c6eb5e - ESP32 - May 20 2022 03:12:58)
[00:00:02.500,793] <inf> wifi_esp_at: SDK version: qa-test-v4.3.3-20220423
[00:00:02.511,413] <inf> wifi_esp_at: Bin version: 2.4.0(WROVER-32)
[00:00:02.955,932] <inf> wifi_esp_at: ESP Wi-Fi ready
[00:00:02.964,508] <inf> golioth_samples: Connecting to WiFi
[00:00:03.175,140] <inf> golioth_wifi: Connected with status: 0
ASSERTION FAIL [((arch_is_in_isr() == 0) || ((timeout).ticks == (((k_timeout_t) {0})).ticks))] @ WEST_TOPDIR/zephyr/kernel/sem.c:12
github-actions[bot] commented 1 year ago

Visit the preview URL for this PR (updated for commit 6e47797):

https://golioth-zephyr-sdk-doxygen-dev--pr378-szczys-samples-t-e75p5roz.web.app

(expires Mon, 08 May 2023 22:43:48 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: a389eefadf4b4b68a539327b3459dd66c142cf49

mniestroj commented 1 year ago

I'm fine with ignoring checkpatch/gitlint issues in this PR.