[PATCH 19/21] ARM: MVEBU: Simplifiy headers and make local

Andrew Lunn andrew at lunn.ch
Fri Feb 7 04:20:47 EST 2014


On Fri, Feb 07, 2014 at 09:31:04AM +0100, Thomas Petazzoni wrote:
> Dear Andrew Lunn,
> 
> On Fri,  7 Feb 2014 00:42:15 +0100, Andrew Lunn wrote:
> > kirkwood is very nearly fully DT. Remove most of the address
> > definitions from the header files and make it a local header file.
> > 
> > Signed-off-by: Andrew Lunn <andrew at lunn.ch>
> > ---
> >  arch/arm/mach-mvebu/include/mach/bridge-regs.h |  85 ---------------
> >  arch/arm/mach-mvebu/include/mach/kirkwood.h    | 142 -------------------------
> >  arch/arm/mach-mvebu/kirkwood-pm.c              |   2 +-
> >  arch/arm/mach-mvebu/kirkwood.c                 |   2 +-
> >  arch/arm/mach-mvebu/kirkwood.h                 |  22 ++++
> >  5 files changed, 24 insertions(+), 229 deletions(-)
> >  delete mode 100644 arch/arm/mach-mvebu/include/mach/bridge-regs.h
> >  delete mode 100644 arch/arm/mach-mvebu/include/mach/kirkwood.h
> >  create mode 100644 arch/arm/mach-mvebu/kirkwood.h
> 
> Is there any particular reason why this isn't part of PATCH 14/21. I
> was very surprised when reading PATCH 14/21 to see a file named
> "bridge-regs.h" containing Kirkwood-specific definitions added to
> mach-mvebu, and so many address constants being defined, while we are
> DT-based only in mach-mvebu.

Hi Thomas

I wanted to keep 14/21 as a simple move of files, doing the minimal
needed to make it boot. The patches that follow then clean up. I found
that simpler to follow what was going on, since there is nothing
hidden in the move patch.

I can however squash these together if that is what people want to
see.

	Andrew



More information about the linux-arm-kernel mailing list