On Fri, Oct 13, 2023 at 08:14:57AM -0700, Simon Glass wrote:
Hi Tom,
On Thu, 12 Oct 2023 at 16:11, Tom Rini trini@konsulko.com wrote:
We can remove common.h from most cases of the code here, and only a few places need an additional header instead.
Signed-off-by: Tom Rini trini@konsulko.com
Cc: Angelo Dureghello angelo@kernel-space.org
arch/m68k/cpu/mcf523x/cpu.c | 1 - arch/m68k/cpu/mcf523x/cpu_init.c | 1 - arch/m68k/cpu/mcf523x/interrupts.c | 1 - arch/m68k/cpu/mcf523x/speed.c | 1 - arch/m68k/cpu/mcf52x2/cpu.c | 1 - arch/m68k/cpu/mcf52x2/cpu_init.c | 3 +-- arch/m68k/cpu/mcf52x2/interrupts.c | 1 - arch/m68k/cpu/mcf52x2/speed.c | 1 - arch/m68k/cpu/mcf530x/cpu.c | 1 - arch/m68k/cpu/mcf530x/cpu_init.c | 1 - arch/m68k/cpu/mcf530x/interrupts.c | 1 - arch/m68k/cpu/mcf530x/speed.c | 1 - arch/m68k/cpu/mcf532x/cpu.c | 1 - arch/m68k/cpu/mcf532x/cpu_init.c | 1 - arch/m68k/cpu/mcf532x/interrupts.c | 1 - arch/m68k/cpu/mcf532x/speed.c | 1 - arch/m68k/cpu/mcf5445x/cpu.c | 1 - arch/m68k/cpu/mcf5445x/cpu_init.c | 1 - arch/m68k/cpu/mcf5445x/dspi.c | 1 - arch/m68k/cpu/mcf5445x/interrupts.c | 1 - arch/m68k/cpu/mcf5445x/speed.c | 1 - arch/m68k/cpu/mcf5445x/start.S | 1 - arch/m68k/include/asm/immap.h | 1 + arch/m68k/include/asm/immap_520x.h | 1 + arch/m68k/include/asm/immap_5235.h | 1 + arch/m68k/include/asm/immap_5272.h | 1 + arch/m68k/include/asm/immap_5275.h | 1 + arch/m68k/include/asm/immap_5282.h | 1 + arch/m68k/include/asm/immap_5301x.h | 1 + arch/m68k/include/asm/immap_5307.h | 2 ++ arch/m68k/include/asm/immap_5329.h | 1 + arch/m68k/include/asm/immap_5441x.h | 1 + arch/m68k/lib/bdinfo.c | 3 ++- arch/m68k/lib/bootm.c | 1 - arch/m68k/lib/cache.c | 2 +- arch/m68k/lib/fec.c | 2 +- arch/m68k/lib/interrupts.c | 2 +- arch/m68k/lib/time.c | 1 - arch/m68k/lib/traps.c | 1 - 39 files changed, 17 insertions(+), 30 deletions(-)
I just wondered what we do about '#ifdef CFG_...' which presumably is not included without common.h ? How does that work?
The first header in common.h is config.h which is where many CFG_... instances are.