diff options
| author | Marek Behún | 2024-06-18 17:34:25 +0200 |
|---|---|---|
| committer | Stefan Roese | 2024-07-08 08:20:58 +0200 |
| commit | c173f6268d405c0cc83d286192dfdd66bad25368 (patch) | |
| tree | 4e9294aac11dfc2a834aa3f75df920f3e849c2a7 | |
| parent | c416e7ad713117a1eeae598146019efd15d2c4a1 (diff) | |
ddr: marvell: a38x: debug: return from ddr3_tip_print_log() early if we won't print anything
Return from ddr3_tip_print_log() early if we won't print anything
anyway.
This way the compiler can optimize away the VALIDATE_IF_ACTIVE() calls
in the for-loop, so if the SILENT_LIB macro is defined, no code is
generated for the rest of the function, which saves some space.
Signed-off-by: Marek Behún <kabel@kernel.org>
| -rw-r--r-- | drivers/ddr/marvell/a38x/ddr3_debug.c | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/drivers/ddr/marvell/a38x/ddr3_debug.c b/drivers/ddr/marvell/a38x/ddr3_debug.c index 9e499cfb992..0374a843878 100644 --- a/drivers/ddr/marvell/a38x/ddr3_debug.c +++ b/drivers/ddr/marvell/a38x/ddr3_debug.c @@ -399,6 +399,15 @@ int ddr3_tip_print_log(u32 dev_num, u32 mem_addr) } #endif /* DDR_VIEWER_TOOL */ + /* return early if we won't print anything anyway */ + if ( +#if defined(SILENT_LIB) + 1 || +#endif + debug_training < DEBUG_LEVEL_INFO) { + return MV_OK; + } + for (if_id = 0; if_id <= MAX_INTERFACE_NUM - 1; if_id++) { VALIDATE_IF_ACTIVE(tm->if_act_mask, if_id); |