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>
This commit is contained in:
Marek Behún 2024-06-18 17:34:25 +02:00 committed by Stefan Roese
parent c416e7ad71
commit c173f6268d

View file

@ -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);