Skip to content

ci: report compiler warnings on pull requests - #266

Draft
Ruben-Lohberg wants to merge 1 commit into
mainfrom
expose-build-warnings
Draft

ci: report compiler warnings on pull requests#266
Ruben-Lohberg wants to merge 1 commit into
mainfrom
expose-build-warnings

Conversation

@Ruben-Lohberg

@Ruben-Lohberg Ruben-Lohberg commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

@Ruben-Lohberg Ruben-Lohberg linked an issue Aug 3, 2026 that may be closed by this pull request
FW: Show build warnings in PR #265
Open
@github-actions

github-actions Bot commented Aug 3, 2026 •
edited
Loading

Copy link
Copy Markdown

Build output available:
openearable_v2_firmware.elf.zip
openearable_v2_fota.zip

@github-actions

github-actions Bot commented Aug 3, 2026 •
edited
Loading

Copy link
Copy Markdown

Compiler warnings

The extended-warning build completed successfully.

Project compiler warnings: 317

Show project warnings 
src/audio/le_audio_rx.c:139:41: warning: unused parameter 'dummy1' [-Wunused-parameter]
src/audio/le_audio_rx.c:139:55: warning: unused parameter 'dummy2' [-Wunused-parameter]
src/audio/le_audio_rx.c:139:69: warning: unused parameter 'dummy3' [-Wunused-parameter]
src/audio/audio_system.c:119:34: warning: unused parameter 'arg1' [-Wunused-parameter]
src/audio/audio_system.c:119:46: warning: unused parameter 'arg2' [-Wunused-parameter]
src/audio/audio_system.c:119:58: warning: unused parameter 'arg3' [-Wunused-parameter]
src/audio/audio_datapath.c:261:54: warning: comparison of integer expressions of different signedness: 'int' and 'uint32_t' {aka 'unsigned int'} [-Wsign-compare]
src/audio/audio_datapath.c:266:61: warning: passing argument 1 of 'sdlogger_write_data' from incompatible pointer type [-Wincompatible-pointer-types]
src/audio/audio_datapath.c:227:38: warning: unused variable 'logger_signaled' [-Wunused-variable]
src/audio/audio_datapath.c:277:42: warning: passing argument 1 of 'k_msgq_put' from incompatible pointer type [-Wincompatible-pointer-types]
src/audio/audio_datapath.c:202:31: warning: unused parameter 'arg1' [-Wunused-parameter]
src/audio/audio_datapath.c:202:43: warning: unused parameter 'arg2' [-Wunused-parameter]
src/audio/audio_datapath.c:202:55: warning: unused parameter 'arg3' [-Wunused-parameter]
src/audio/audio_datapath.c:497:13: warning: unused variable 'ret' [-Wunused-variable]
src/audio/audio_datapath.c:657:9: warning: implicit declaration of function 'config_sensor' [-Wimplicit-function-declaration]
src/audio/audio_datapath.c:650:45: warning: unused parameter 'work' [-Wunused-parameter]
src/audio/audio_datapath.c:662:53: warning: unused parameter 'dummy' [-Wunused-parameter]
src/audio/audio_datapath.c:776:67: warning: unused parameter 'count' [-Wunused-parameter]
src/ParseInfo/../SensorManager/BMA580/BMA580_Sensor.h:167:44: warning: missing initializer for member 'bma5_fifo_frame::fifo_avail_len' [-Wmissing-field-initializers]
src/ParseInfo/../SensorManager/BMA580/BMA580_Sensor.h:167:44: warning: missing initializer for member 'bma5_fifo_frame::fifo_avail_frames' [-Wmissing-field-initializers]
src/ParseInfo/../SensorManager/BMA580/BMA580_Sensor.h:167:44: warning: missing initializer for member 'bma5_fifo_frame::acc_byte_start_idx' [-Wmissing-field-initializers]
src/SD_Card/SD_Card_Manager/SD_Card_Manager.h:178:5: warning: missing initializer for member 'fs_mount_t::node' [-Wmissing-field-initializers]
src/SD_Card/SD_Card_Manager/SD_Card_Manager.h:178:5: warning: missing initializer for member 'fs_mount_t::mnt_point' [-Wmissing-field-initializers]
src/SD_Card/SD_Card_Manager/SD_Card_Manager.h:178:5: warning: missing initializer for member 'fs_mount_t::storage_dev' [-Wmissing-field-initializers]
src/SD_Card/SD_Card_Manager/SD_Card_Manager.h:178:5: warning: missing initializer for member 'fs_mount_t::mountp_len' [-Wmissing-field-initializers]
src/SD_Card/SD_Card_Manager/SD_Card_Manager.h:178:5: warning: missing initializer for member 'fs_mount_t::fs' [-Wmissing-field-initializers]
src/SD_Card/SD_Card_Manager/SD_Card_Manager.h:178:5: warning: missing initializer for member 'fs_mount_t::flags' [-Wmissing-field-initializers]
src/SD_Card/SD_Card_Manager/SD_Card_Manager.h:185:5: warning: missing initializer for member 'SDCardManager::tracked_fs_file_t::filep' [-Wmissing-field-initializers]
src/Wire/TWIM.cpp:10:13: warning: unused variable 'ret' [-Wunused-variable]
src/Battery/BQ25120a.cpp:156:14: warning: unused variable 'ret' [-Wunused-variable]
src/Battery/BQ25120a.cpp:163:14: warning: unused variable 'ret' [-Wunused-variable]
src/Battery/BQ25120a.cpp:170:14: warning: unused variable 'ret' [-Wunused-variable]
src/Battery/BQ25120a.cpp:177:14: warning: unused variable 'ret' [-Wunused-variable]
src/Battery/BQ25120a.cpp:209:55: warning: implicit conversion from 'float' to 'double' to match other operand of binary expression [-Wdouble-promotion]
src/Battery/BQ25120a.cpp:203:14: warning: unused variable 'ret' [-Wunused-variable]
src/Battery/BQ25120a.cpp:248:49: warning: implicit conversion from 'float' to 'double' to match other operand of binary expression [-Wdouble-promotion]
src/Battery/BQ25120a.cpp:258:14: warning: unused variable 'ret' [-Wunused-variable]
src/Battery/BQ25120a.cpp:267:14: warning: unused variable 'ret' [-Wunused-variable]
src/Battery/BQ25120a.cpp:282:52: warning: implicit conversion from 'float' to 'double' to match other operand of binary expression [-Wdouble-promotion]
src/Battery/BQ25120a.cpp:308:33: warning: implicit conversion from 'float' to 'double' to match other operand of binary expression [-Wdouble-promotion]
src/Battery/BQ25120a.cpp:291:14: warning: unused variable 'ret' [-Wunused-variable]
src/Battery/BQ25120a.cpp:328:48: warning: implicit conversion from 'float' to 'double' to match other operand of binary expression [-Wdouble-promotion]
src/Battery/BQ25120a.cpp:344:14: warning: unused variable 'ret' [-Wunused-variable]
src/Battery/BQ25120a.cpp:361:36: warning: implicit conversion from 'float' to 'double' to match other operand of binary expression [-Wdouble-promotion]
src/Battery/BQ25120a.cpp:411:14: warning: unused variable 'ret' [-Wunused-variable]
src/Battery/BQ27220.cpp:99:14: warning: unused variable 'ret' [-Wunused-variable]
src/Battery/BQ27220.cpp:138:14: warning: unused variable 'ret' [-Wunused-variable]
src/Battery/BQ27220.cpp:146:14: warning: unused variable 'ret' [-Wunused-variable]
src/Battery/BQ27220.cpp:154:14: warning: unused variable 'ret' [-Wunused-variable]
src/Battery/BQ27220.cpp:160:14: warning: unused variable 'ret' [-Wunused-variable]
src/Battery/BQ27220.cpp:167:14: warning: unused variable 'ret' [-Wunused-variable]
src/Battery/BQ27220.cpp:173:14: warning: unused variable 'ret' [-Wunused-variable]
src/Battery/BQ27220.cpp:179:14: warning: unused variable 'ret' [-Wunused-variable]
src/Battery/BQ27220.cpp:185:14: warning: unused variable 'ret' [-Wunused-variable]
src/Battery/BQ27220.cpp:191:14: warning: unused variable 'ret' [-Wunused-variable]
src/Battery/BQ27220.cpp:197:14: warning: unused variable 'ret' [-Wunused-variable]
src/Battery/BQ27220.cpp:203:14: warning: unused variable 'ret' [-Wunused-variable]
src/Battery/BQ27220.cpp:209:14: warning: unused variable 'ret' [-Wunused-variable]
src/Battery/BQ27220.cpp:215:14: warning: unused variable 'ret' [-Wunused-variable]
src/Battery/BQ27220.cpp:221:14: warning: unused variable 'ret' [-Wunused-variable]
src/Battery/BQ27220.cpp:229:14: warning: unused variable 'ret' [-Wunused-variable]
src/audio/streamctrl.c:539:15: warning: implicit declaration of function 'uicr_sirk_set' [-Wimplicit-function-declaration]
src/audio/streamctrl.c:562:17: warning: declaration of 'type' shadows a parameter [-Wshadow]
src/audio/streamctrl.c:569:34: warning: comparison of integer expressions of different signedness: 'size_t' {aka 'unsigned int'} and 'int' [-Wsign-compare]
src/audio/streamctrl.c:606:33: warning: format '%X' expects argument of type 'unsigned int', but argument 2 has type 'uint64_t' {aka 'long long unsigned int'} [-Wformat=]
src/audio/streamctrl.c:623:57: warning: implicit declaration of function 'bt_hex' [-Wimplicit-function-declaration]
src/audio/streamctrl.c:623:41: warning: format '%s' expects argument of type 'char *', but argument 2 has type 'int' [-Wformat=]
src/audio/streamctrl.c:631:25: warning: implicit declaration of function 'snprintf' [-Wimplicit-function-declaration]
src/audio/streamctrl.c:631:25: warning: incompatible implicit declaration of built-in function 'snprintf' [-Wbuiltin-declaration-mismatch]
src/audio/streamctrl.c:633:29: warning: unused variable 'err' [-Wunused-variable]
src/audio/streamctrl.c:551:9: warning: unused variable 'ret' [-Wunused-variable]
src/audio/streamctrl.c:546:59: warning: unused parameter 'rssi' [-Wunused-parameter]
src/audio/streamctrl.c:546:73: warning: unused parameter 'type' [-Wunused-parameter]
src/audio/streamctrl.c:704:25: warning: implicit declaration of function 'uicr_sirk_get' [-Wimplicit-function-declaration]
src/audio/streamctrl.c:707:39: warning: implicit declaration of function 'uicr_standalone_get' [-Wimplicit-function-declaration]
src/bluetooth/bt_management/conn_interval/conn_intvl_exp_backoff_lin_inc.cpp:22:14: warning: declaration of 'current_interval_units_' shadows a member of 'ConnIntvlExpBackoffLinIncr' [-Wshadow]
src/bluetooth/bt_management/conn_interval/conn_intvl_exp_backoff_lin_inc.cpp:33:60: warning: unused parameter 'elapsed' [-Wunused-parameter]
src/bluetooth/bt_management/bt_mgmt_conn_interval.cpp:149:27: warning: implicit conversion from 'float' to 'double' when passing argument to function [-Wdouble-promotion]
src/bluetooth/bt_management/bt_mgmt_conn_interval.cpp:149:46: warning: implicit conversion from 'float' to 'double' when passing argument to function [-Wdouble-promotion]
src/bluetooth/bt_management/controller_config/bt_mgmt_ctlr_cfg.c:43:51: warning: unused parameter 'work' [-Wunused-parameter]
src/bluetooth/bt_management/controller_config/bt_mgmt_ctlr_cfg.c:55:53: warning: unused parameter 'timer_id' [-Wunused-parameter]
src/bluetooth/bt_management/controller_config/bt_mgmt_ctlr_cfg.c:65:32: warning: unused parameter 'channel_id' [-Wunused-parameter]
src/bluetooth/bt_management/controller_config/bt_mgmt_ctlr_cfg.c:65:50: warning: unused parameter 'user_data' [-Wunused-parameter]
src/Battery/PowerManager.cpp:56:61: warning: unused parameter 'dev' [-Wunused-parameter]
src/Battery/PowerManager.cpp:56:88: warning: unused parameter 'cb' [-Wunused-parameter]
src/Battery/PowerManager.cpp:56:101: warning: unused parameter 'pins' [-Wunused-parameter]
src/Battery/PowerManager.cpp:61:69: warning: unused parameter 'dev' [-Wunused-parameter]
src/Battery/PowerManager.cpp:61:96: warning: unused parameter 'cb' [-Wunused-parameter]
src/Battery/PowerManager.cpp:61:109: warning: unused parameter 'pins' [-Wunused-parameter]
src/Battery/PowerManager.cpp:65:61: warning: unused parameter 'dev' [-Wunused-parameter]
src/Battery/PowerManager.cpp:65:88: warning: unused parameter 'cb' [-Wunused-parameter]
src/Battery/PowerManager.cpp:65:101: warning: unused parameter 'pins' [-Wunused-parameter]
src/Battery/PowerManager.cpp:79:60: warning: unused parameter 'work' [-Wunused-parameter]
src/Battery/PowerManager.cpp:83:61: warning: unused parameter 'work' [-Wunused-parameter]
src/Battery/PowerManager.cpp:89:68: warning: unused parameter 'work' [-Wunused-parameter]
src/Battery/PowerManager.cpp:171:40: warning: implicit conversion from 'float' to 'double' when passing argument to function [-Wdouble-promotion]
src/Battery/PowerManager.cpp:172:50: warning: implicit conversion from 'float' to 'double' when passing argument to function [-Wdouble-promotion]
src/Battery/PowerManager.cpp:173:48: warning: implicit conversion from 'float' to 'double' when passing argument to function [-Wdouble-promotion]
src/Battery/PowerManager.cpp:174:75: warning: implicit conversion from 'float' to 'double' when passing argument to function [-Wdouble-promotion]
src/Battery/PowerManager.cpp:177:31: warning: implicit conversion from 'float' to 'double' to match other operand of binary expression [-Wdouble-promotion]

Showing the first 100 unique project warnings.

Configuration and security warnings: 14

Show configuration warnings 
warning: MBEDTLS_CFG_FILE (defined at /home/runner/work/open-earable-2/open-earable-2/zephyr-
  WARNING: Using default MCUboot signing key file, this file is for debug use
warning: BT_L2CAP_TX_MTU (defined at soc/ambiq/apollo3x/Kconfig.defconfig.apollo3_blue:32,
warning: BT_L2CAP_DYNAMIC_CHANNEL (defined at subsys/bluetooth/host/Kconfig.l2cap:47,
warning: BT_ATT_PREPARE_COUNT (defined at subsys/bluetooth/host/Kconfig.gatt:25,
warning: BT_CTLR_RX_BUFFERS (defined at subsys/bluetooth/controller/Kconfig:234) was assigned the
warning: BT_CTLR_DATA_LENGTH_MAX (defined at /home/runner/work/open-earable-2/open-earable-2/zephyr-
warning: BT_CTLR_ISO_RX_BUFFERS (defined at subsys/bluetooth/controller/Kconfig:245) was assigned
warning: BT_CTLR_ISO_TX_BUFFERS (defined at subsys/bluetooth/controller/Kconfig:259) was assigned
warning: BT_CTLR_PHY_2M (defined at subsys/bluetooth/controller/Kconfig:669) was assigned the value
warning: Experimental symbol NRF5340_AUDIO is enabled.
warning: Experimental symbol AUDIO_BT_MGMT_DFU is enabled.
warning: Experimental symbol SFLOAT is enabled.
      --- WARNING: Using generated NSIB public/private key-pair. ---

The complete warning report contains all 3141 warning lines, including warnings from Zephyr and other dependencies.

View the warning workflow run

@Ruben-Lohberg
Ruben-Lohberg force-pushed the expose-build-warnings branch from 926e5f9 to 692f482 Compare August 3, 2026 18:08
@github-actions

github-actions Bot commented Aug 3, 2026

Copy link
Copy Markdown

Build output available:
openearable_v2_firmware.elf.zip
openearable_v2_fota.zip

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

FW: Show build warnings in PR

1 participant

@Ruben-Lohberg