On 09/27/2013 08:08 PM, Kevin Hilman wrote: > Javier Martinez Canillas <javier.martinez@xxxxxxxxxxxxxxx> writes: > >> The GPIO OMAP controller pins can be used as IRQ and GPIO >> independently so is necessary to keep track GPIO pins and >> IRQ lines usage separately to make sure that the bank will >> always be enabled while being used. >> >> Also move gpio_is_input() definition in preparation for the >> next patch that setups the controller's irq_chip driver when >> a caller requests an interrupt line. >> >> Signed-off-by: Javier Martinez Canillas <javier.martinez@xxxxxxxxxxxxxxx> > > I'm fine with this appproach. For both patches: > > Reviewed-by: Kevin Hilman <khilman@xxxxxxxxxx> > > Also, I gave it a spin across a handful of OMAP boards using v3.12-rc2 + > these 2 patches. > > Boot tested successfully with DT boot: > > omap3530/beagle > omap3730/beagle-xm > omap3530/overo (Tobi w/GPIO IRQ networking) > omap3730/overo STORM (w/GPIO IRQ for networking) > am335x/beaglebone > am335x/beaglebone black > omap4430/panda > omap4460/panda-es > omap5912/OSK (omap1) > > I also verified non-DT boot on the OMAP3 platforms that still support > legacy boot. > > So feel free to also add > > Tested-by: Kevin Hilman <khilman@xxxxxxxxxx> > > Thanks for your persistence in getting a fix for this upstream. > > Kevin > Thanks a lot Kevin for testing on so many boards. Linus, Since this patch-set doesn't cause any regression and fix a long standing issue on OMAP, do you think that it would be possible to include on the -rc series as a bugfix or do you prefer to wait until 3.13? Thanks a lot and best regards, Javier -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html