[PATCH 4/7] m68k: Remove common.h usage

[Simon Glass]

Hi Tom,

On Fri, 13 Oct 2023 at 08:17, Tom Rini <trini at konsulko.com> wrote:
>
> 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 at 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 at konsulko.com>
> > > ---
> > > Cc: Angelo Dureghello <angelo at 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.

OK, so that comes later...

Regards,
Simon