[PATCH 1/2] ARM: DT: Kirkwood: Use symbolic names from input.h

Jason Cooper jason at lakedaemon.net
Tue Dec 3 16:56:33 EST 2013


On Tue, Dec 03, 2013 at 10:50:05PM +0100, Andrew Lunn wrote:
> On Tue, Dec 03, 2013 at 09:39:02PM +0100, Arnaud Ebalard wrote:
> > Hi Andrew,
> > 
> > Andrew Lunn <andrew at lunn.ch> writes:
> > 
> > > Replace the numeric key value with a symbolic name from
> > > <bt-bindings/input/input.h>
> > >
> > > Signed-off-by: Andrew Lunn <andrew at lunn.ch>
> > > ---
> > >  arch/arm/boot/dts/kirkwood-cloudbox.dts                | 2 +-
> > >  arch/arm/boot/dts/kirkwood-dnskw.dtsi                  | 6 +++---
> > >  arch/arm/boot/dts/kirkwood-ib62x0.dts                  | 4 ++--
> > >  arch/arm/boot/dts/kirkwood-iconnect.dts                | 4 ++--
> > >  arch/arm/boot/dts/kirkwood-iomega_ix2_200.dts          | 6 +++---
> > >  arch/arm/boot/dts/kirkwood-lsxl.dtsi                   | 6 +++---
> > >  arch/arm/boot/dts/kirkwood-mv88f6281gtw-ge.dts         | 4 ++--
> > >  arch/arm/boot/dts/kirkwood-netgear_readynas_duo_v2.dts | 6 +++---
> > 
> > Jason just applied (yesterday IIRC) to mvebu/dt a series which provides
> > cleanup for ReadyNAS .dts files. Among other things, it replaces numeric
> > key values with a symbolic name. And it also does the same for GPIO
> > voltage level. I guess you can remove that specific .dts from your patch.
> 
> I sore your patches. I did it slightly different to you, in that i've
> the includes in the top level kirkwood.dtsi file, where as you put
> them into the .dts files. That makes yours redundant, if we keep to
> this scheme.
> 
> Jason, which do you prefer, includes in every .dts file, or once in
> kirkwood.dtsi. 

I prefer the kernel standard (is it really?) for includes.  Which is
directly in the file needing it.  So, that would mean in both if we use
the macros in both.

thx,

Jason.



More information about the linux-arm-kernel mailing list