On Fri, Nov 16, 2012 at 08:32:50PM +0100, Simon Baatz wrote: > Hi Andrew, > > On Tue, Nov 13, 2012 at 04:44:45PM +0100, Andrew Lunn wrote: > > Signed-off-by: Andrew Lunn <and...@lunn.ch> > > --- > > arch/arm/boot/dts/kirkwood-ib62x0.dts | 6 ++++++ > > arch/arm/mach-kirkwood/board-ib62x0.c | 13 ------------- > > 2 files changed, 6 insertions(+), 13 deletions(-) > > > > diff --git a/arch/arm/boot/dts/kirkwood-ib62x0.dts > > b/arch/arm/boot/dts/kirkwood-ib62x0.dts > > index 66794ed..9b0e2e3 100644 > > --- a/arch/arm/boot/dts/kirkwood-ib62x0.dts > > +++ b/arch/arm/boot/dts/kirkwood-ib62x0.dts > > @@ -79,4 +79,10 @@ > > gpios = <&gpio0 27 0>; > > }; > > }; > > + gpio_poweroff { > > + compatible = "gpio-poweroff"; > > + gpios = <&gpio0 24 1>; > > This needs to be: > gpios = <&gpio0 24 0>;
I also got it wrong on Jamie's board! > > If the GPIO is configured as an input, the box shuts down when > initializing the driver. > > With this change: > > Tested-by: Simon Baatz <gmbno...@gmail.com> > > Btw., I also tested it on top of your pinctrl changes, which works > like a charm as well. Thanks for testing that. Andrew _______________________________________________ devicetree-discuss mailing list devicetree-discuss@lists.ozlabs.org https://lists.ozlabs.org/listinfo/devicetree-discuss