Re: [PATCH 02/17] ARM clps711x: Added Cirrus Logic EP7312 CPUvariant

[Russell King - ARM Linux]

On Sat, Apr 21, 2012 at 11:21:49AM +0000, Arnd Bergmann wrote:
> On Saturday 21 April 2012, Alexander Shiyan wrote:
> > 
> > Added register definitions specific to EP7312 and the necessary
> > configuration options.
> > 
> > Signed-off-by: Alexander Shiyan <shc_work@xxxxxxx>
> > ---
> >  arch/arm/include/asm/hardware/ep7312.h         |  106 ++++++++++++++++++++++++
> >  arch/arm/mach-clps711x/Kconfig                 |    9 ++-
> >  arch/arm/mach-clps711x/include/mach/hardware.h |    4 +
> >  drivers/net/irda/Kconfig                       |    3 +-
> >  4 files changed, 119 insertions(+), 3 deletions(-)
> >  create mode 100644 arch/arm/include/asm/hardware/ep7312.h
> 
> I'm missing some historical background here. Why are these files in asm/hardware
> rather than in mach/? Should we move them to the platform directory?

That's where we used to put truely hardware dependent stuff, rather than
machine type stuff when the structure was first implemented.  At that time
we had a number of different mach-* directories wanting to share things
like the acorn IOC/IOMD stuff, and there's also sharing of definitions
between CLPS711x and EP93xx.

That was long before we invented the plat-* stuff - but even that doesn't
actually cover what asm/hardware does - eg, the SA1111 stuff shared
between SA11x0 and PXA.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/