applications: nrf5340_audio: Audio module improvements

[gWacey]

Corrections from use, additional testing and fake function updates.

[NordicBuilder]

Test specification

CI/Jenkins/NRF

• Integration Platforms

CI/Jenkins/integration

Test Module	File based changes	Manually selected	West overwrite

Detailed information of selected test modules

Note: This message is automatically posted and updated by the CI

[NordicBuilder]

You can find the documentation preview for this PR at this link. It will be updated about 10 minutes after the documentation build succeeds.

Note: This comment is automatically posted by the Documentation Publishing GitHub Action.

[andvib]

Looks good! Only minor comments - mostly some missing return checks

[koffes]

I re-requested a review from myself too soon :)

[NordicBuilder]

CI Information

^{To view the history of this post, clich the 'edited' button above}
Build number: 16

Inputs:

Sources:

sdk-nrf: PR head: ae1e1712e26ebf4708815df8f29a3979d9a5b0ee

more details

sdk-nrf:

PR head: ae1e1712e26ebf4708815df8f29a3979d9a5b0ee
merge base: ea2da455203ce6876392d9e4fcc78b1ed537de53
target head (main): b19a7fb5e969027a064b2bc72cd5545d99f14377
Diff

Github labels

Enabled	Name	Description
	ci-disabled	Disable the ci execution
	ci-all-test	Run all of ci, no test spec filtering will be done
	ci-force-downstream	Force execution of downstream even if twister fails
	ci-run-twister	Force run twister
	ci-run-zephyr-twister	Force run zephyr twister

List of changed files detected by CI (16)

include
│  ├── audio_defines.h
│  ├── audio_module
│  │  │ audio_module.h
│  │ data_fifo.h
lib
│  ├── data_fifo
│  │  │ data_fifo.c
subsys
│  ├── audio_module
│  │  ├── CMakeLists.txt
│  │  ├── Kconfig
│  │  │ audio_module.c
tests
│  ├── subsys
│  │  ├── audio_module
│  │  │  ├── CMakeLists.txt
│  │  │  ├── src
│  │  │  │  ├── audio_module_test_common.c
│  │  │  │  ├── audio_module_test_common.h
│  │  │  │  ├── audio_module_test_fakes.c
│  │  │  │  ├── audio_module_test_fakes.h
│  │  │  │  ├── bad_param_test.c
│  │  │  │  ├── functional_test.c
│  │  │  │  │ main.c
│  │  │  │ testcase.yaml

Outputs:

Toolchain

Version: 9338b70c26
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:9338b70c26_81ed5a52d6

Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped; ⚠️ Quarantine

• ◻️ Toolchain - Skipped: existing toolchain is used
• ✅ Build twister
  • sdk-nrf test count: 38
• ✅ Integration tests
  • ✅ test-sdk-audio

Disabled integration tests

• • desktop52_verification
  • doc-internal
  • test_ble_nrf_config
  • test-fw-nrfconnect-apps
  • test-fw-nrfconnect-ble_mesh
  • test-fw-nrfconnect-ble_samples
  • test-fw-nrfconnect-boot
  • test-fw-nrfconnect-chip
  • test-fw-nrfconnect-fem
  • test-fw-nrfconnect-nfc
  • test-fw-nrfconnect-nrf-iot_cloud
  • test-fw-nrfconnect-nrf-iot_libmodem-nrf
  • test-fw-nrfconnect-nrf-iot_lwm2m
  • test-fw-nrfconnect-nrf-iot_mosh
  • test-fw-nrfconnect-nrf-iot_nrf_provisioning
  • test-fw-nrfconnect-nrf-iot_positioning
  • test-fw-nrfconnect-nrf-iot_samples
  • test-fw-nrfconnect-nrf-iot_serial_lte_modem
  • test-fw-nrfconnect-nrf-iot_thingy91
  • test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
  • test-fw-nrfconnect-nrf_crypto
  • test-fw-nrfconnect-proprietary_esb
  • test-fw-nrfconnect-ps
  • test-fw-nrfconnect-rpc
  • test-fw-nrfconnect-rs
  • test-fw-nrfconnect-tfm
  • test-fw-nrfconnect-thread
  • test-fw-nrfconnect-zigbee
  • test-low-level
  • test-sdk-dfu
  • test-sdk-find-my
  • test-sdk-mcuboot
  • test-sdk-pmic-samples
  • test-sdk-sidewalk
  • test-sdk-wifi

Note: This message is automatically posted and updated by the CI

[koffes]

Looks nice :) A couple of topics to discuss.