On Tue, Sep 10, 2013 at 4:14 PM, Aaro Koskinen <[email protected]> wrote:
> On Mon, Sep 09, 2013 at 05:39:58PM +0200, Linus Walleij wrote: >> This series refactor the IOP GPIO driver to move down to the >> GPIO subsystem, instatiate as a platform device, remap its >> I/O address, and kills off the <mach/gpio.h> dependency. > > Patches 1-5 work on N2100. However, when pressing the power button there > is a warning: > > [ 31.995575] reboot: Restarting system > [ 32.023422] ------------[ cut here ]------------ > [ 32.051596] WARNING: CPU: 0 PID: 251 at drivers/gpio/gpiolib.c:160 > gpio_ensure_requested+0x64/0xc0() > [ 32.110436] ---[ end trace 6b2c83f883af871f ]--- Hm, this does not appear right after 1/8 or 2/8 right? I'd rather expect that after 3/8 "ARM: iop32x: read N2100 power key using gpiolib" and then you should have error messages like this in dmesg: could not request reset GPIO Correct? > With patch 6 it starts to crash on first gpio_get_value(), I guess > because gpio-iop platform device is not registered on N2100...? Probably because something is trying to read a GPIO before this device is up, I'll look into this. THANKS! Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
