On Thu, Aug 04, 2011 at 05:00:17PM -0600, Stephen Warren wrote: > In http://www.spinics.net/lists/linux-tegra/msg01731.html, Mark Brown > pointed out that it was a little silly forcing every board or driver > to gpio_request() a GPIO that is later converted to an IRQ, and passed > to request_irq. The first patch in this series instead makes the core > IRQ code perform these calls when appropriate, to avoid duplicating it > everywhere. Interesting, as this is not the case across all the ARM systems, as I believe that many of the s3c24xx do not bother to request the GPIOs before using them as interrupts. > However, this change has the potential for significant regressions; at > least some drivers are already calling gpio_request for GPIOs that are > also used as IRQs. This then causes the gpio_request inside the core IRQ > code to fail, which causes functional regressions. I'm not sure how wide- > spread this issue is, but in testing on NVIDIA Tegra, I found two > instances that needed to be fixed. Perhaps a failure of gpio_request > in the core IRQ code should trigger a WARN rather than returning an > error, to give a grace period for conversion of other code? Do we really need to request the GPIO? I suppose it is useful to stop someone else trying to play with it... Another interesting problem is to do with using an GPIO that has been assigned to an interrupt as a readable resource in the driver, since the I would also consider that the implementor of the IRQs set_type configure the specific GPIO to the correct mode for operating as an interrupt when it is called... this would reduce any problems of passing GPIOs to drivers (especially if device tree is being used) to avoid having to do any arch-specific mode calls. > Stephen Warren (3): > irq: If an IRQ is a GPIO, request and configure it > mmc: tegra: Don't gpio_request GPIOs used as IRQs. > ASoC: jack_add_gpios: Don't gpio_request GPIOs used as IRQs. > > drivers/mmc/host/sdhci-tegra.c | 8 -------- > kernel/irq/manage.c | 25 +++++++++++++++++++++++-- > sound/soc/soc-jack.c | 13 +------------ > 3 files changed, 24 insertions(+), 22 deletions(-) > > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel@xxxxxxxxxxxxxxxxxxx > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel -- Ben Dooks, ben@xxxxxxxxx, http://www.fluff.org/ben/ Large Hadron Colada: A large Pina Colada that makes the universe disappear. -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
