zephyrproject-rtos/zephyr

LE Audio: Change BSIM test to use LOG instead of printk

Open

#71,302 opened on Apr 10, 2024

View on GitHub
 (1 comment) (0 reactions) (2 assignees)C (15,249 stars) (9,144 forks)batch import
EnhancementGood first issuearea: Bluetootharea: Bluetooth Audio

Description

Is your enhancement proposal related to a problem? Please describe. Currently the LE Audio BSIM are using printk for logging purposes, where the logging system is more suitable.

Describe the solution you'd like Modify all occurrences of printk to an appropriate log level. We do not need to log everything in all cases.

Describe alternatives you've considered N/A

Additional context N/A

Contributor guide

Tech stack
c
Domain
testing
Issue type
test
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
blocked
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
clear
Prerequisites
C programmingZephyr logging subsystemBSIM test framework
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
40
Research direction
Locate all BSIM test files under tests/bsim/le audio that use printk. Replace each occurrence with an appropriate LOG macro (e.g., LOG INF, LOG DBG) based on context. Ensure to include the necessary logging module registration. Refer to existing Zephyr logging documentation for correct log levels.
LE Audio: Change BSIM test to use LOG instead of printk · zephyrproject-rtos/zephyr#71302 | Good First Issue