Re: [PATCH v2 06/14] board_f: m68k: Factor out m68k-specific bdinfo setup

2020-07-16 Thread Simon Glass
Hi Ovidiu, On Wed, 15 Jul 2020 at 08:07, Ovidiu Panait wrote: > > Hi Simon, > > On 15.07.2020 04:05, Simon Glass wrote: > > On Fri, 10 Jul 2020 at 04:25, Ovidiu Panait > > wrote: > >> Factor out m68k-specific bdinfo setup to arch_setup_bdinfo in > >> arch/m68k/lib/bdinfo.c. Also, use if(IS_ENAB

Re: [PATCH v2 06/14] board_f: m68k: Factor out m68k-specific bdinfo setup

2020-07-15 Thread Ovidiu Panait
Hi Simon, On 15.07.2020 04:05, Simon Glass wrote: On Fri, 10 Jul 2020 at 04:25, Ovidiu Panait wrote: Factor out m68k-specific bdinfo setup to arch_setup_bdinfo in arch/m68k/lib/bdinfo.c. Also, use if(IS_ENABLED()) instead of #ifdef where possible. Signed-off-by: Ovidiu Panait --- v2 updates:

Re: [PATCH v2 06/14] board_f: m68k: Factor out m68k-specific bdinfo setup

2020-07-14 Thread Simon Glass
On Fri, 10 Jul 2020 at 04:25, Ovidiu Panait wrote: > > Factor out m68k-specific bdinfo setup to arch_setup_bdinfo in > arch/m68k/lib/bdinfo.c. Also, use if(IS_ENABLED()) instead of #ifdef where > possible. > > Signed-off-by: Ovidiu Panait > --- > v2 updates: > - use "if (IS_ENABLED(CONFIG_SYS_HAS

[PATCH v2 06/14] board_f: m68k: Factor out m68k-specific bdinfo setup

2020-07-10 Thread Ovidiu Panait
Factor out m68k-specific bdinfo setup to arch_setup_bdinfo in arch/m68k/lib/bdinfo.c. Also, use if(IS_ENABLED()) instead of #ifdef where possible. Signed-off-by: Ovidiu Panait --- v2 updates: - use "if (IS_ENABLED(CONFIG_SYS_HAS_SRAM))" instead of "#ifdef CONFIG_SYS_SRAM_BASE" arch/m68k/lib/