Hi Quentin,
On Mon, 12 Aug 2024 at 11:42, Quentin Schulz quentin.schulz@cherry.de wrote:
Hi Simon,
On 8/11/24 2:29 PM, Simon Glass wrote:
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 sjg@chromium.org
(no changes since v2)
You did make some changes though: common/log_syslog.c wasn't patched in v2.
Reviewed-by: Quentin Schulz quentin.schulz@cherry.de
Ah yes, I fixed the v3 changelog in v4 [1]
Regards, Simon
[1] https://patchwork.ozlabs.org/project/uboot/patch/20240822135503.2363706-8-sj...