On Mon, 23 Nov 2020 at 06:17, Ovidiu Panait <ovidiu.pan...@windriver.com> wrote: > > Currently, the following #ifdef construct is used to check whether to run > console_record_init() during pre-relocation init: > defined(CONFIG_CONSOLE_RECORD) && CONFIG_VAL(SYS_MALLOC_F_LEN) > > Introduce CONFIG_CONSOLE_RECORD_INIT_F Kconfig option to get rid of the > complex ifdef check. Also, use IS_ENABLED() instead of #ifdef. > > Signed-off-by: Ovidiu Panait <ovidiu.pan...@windriver.com> > --- > v3 updates: > - use only "default y" for CONSOLE_RECORD_INIT_F > > v2 updates: > - new patch > > common/Kconfig | 8 ++++++++ > common/board_f.c | 7 +++---- > 2 files changed, 11 insertions(+), 4 deletions(-) >
Reviewed-by: Simon Glass <s...@chromium.org> You can keep the review tags for minor revisions.