diff options
| author | Simon Glass <[email protected]> | 2024-08-22 07:54:56 -0600 |
|---|---|---|
| committer | Tom Rini <[email protected]> | 2024-08-23 15:58:41 -0600 |
| commit | dfc0acd0cc3cd14f64cb6850465ade98ea84b366 (patch) | |
| tree | 91bf8132534a60dd28b1aab0c8bf0f80f8996863 /test/log | |
| parent | 99b23d45cb7f028067a77845e46cb09801e1b9e1 (diff) | |
log: Avoid including function names by default
Unless function names are requested, the logging system should not
compile these into the code. Adjust the macros to handle this.
This means that turning on function names at runtime won't work unless
CONFIG_LOGF_FUNC is enabled. We could perhaps split this into a
separate option if that is a problem.
Enable CONFIG_LOGF_FUNC logging for sandbox since the tests expect the
function names to be included. Fix up the pinmux test which checks a
logging statement.
Signed-off-by: Simon Glass <[email protected]>
Reviewed-by: Quentin Schulz <[email protected]>
Diffstat (limited to 'test/log')
| -rw-r--r-- | test/log/log_test.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/test/log/log_test.c b/test/log/log_test.c index 855353a9c40..7ef89941d42 100644 --- a/test/log/log_test.c +++ b/test/log/log_test.c @@ -452,8 +452,10 @@ int log_test_buffer(struct unit_test_state *uts) /* This one should product no output due to the debug level */ log_buffer(LOGC_BOOT, LOGL_DEBUG, 0, buf, 1, 0x12, 0); - ut_assert_nextline("00000000: 00 11 22 33 44 55 66 77 88 99 aa bb cc dd ee ff ..\"3DUfw........"); - ut_assert_nextline("00000010: 10 00 .."); + ut_assert_nextline( + " log_test_buffer() 00000000: 00 11 22 33 44 55 66 77 88 99 aa bb cc dd ee ff ..\"3DUfw........"); + ut_assert_nextline( + " log_test_buffer() 00000010: 10 00 .."); ut_assert_console_end(); free(buf); |