lkml.org 
[lkml]   [2007]   [Apr]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 37/90] ARM: OMAP: MPUIO wake updates
    Date
    From: David Brownell <dbrownell@users.sourceforge.net>

    GPIO and MPUIO wake updates:

    - Hook MPUIOs into the irq wakeup framework too. This uses a platform
    device to update irq enables during system sleep states, instead of
    a sys_device, since the latter is no longer needed for such things.

    - Also forward enable/disable irq wake requests to the relevant GPIO
    controller, so the top level IRQ dispatcher can (eventually) handle
    these wakeup events automatically if more than one GPIO pin needs to
    be a wakeup event source.

    - Minor tweak to the 24xx non-wakeup gpio stuff: no need to check such
    read-only data under the spinlock.

    This assumes (maybe wrongly?) that only 16xx can do GPIO wakeup; without
    a 15xx I can't test such stuff.

    Also this expects the top level IRQ dispatcher to properly handle requests
    to enable/disable irq wake, which is currently known to be wrong: omap1
    saves the flags but ignores them, omap2 doesn't even save it. (Wakeup
    events are, wrongly, hardwired in the relevant mach-omapX/pm.c file ...)
    So MPUIO irqs won't yet trigger system wakeup.

    Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
    Signed-off-by: Tony Lindgren <tony@atomide.com>
    ---
    arch/arm/plat-omap/gpio.c | 93 ++++++++++++++++++++++++++++++++++++++++-----
    1 files changed, 83 insertions(+), 10 deletions(-)

    diff --git a/arch/arm/plat-omap/gpio.c b/arch/arm/plat-omap/gpio.c
    index 83613f7..7e90cea 100644
    --- a/arch/arm/plat-omap/gpio.c
    +++ b/arch/arm/plat-omap/gpio.c
    @@ -773,29 +773,35 @@ static int _set_gpio_wakeup(struct gpio_bank *bank, int gpio, int enable)
    {
    switch (bank->method) {
    #ifdef CONFIG_ARCH_OMAP16XX
    + case METHOD_MPUIO:
    case METHOD_GPIO_1610:
    spin_lock(&bank->lock);
    - if (enable)
    + if (enable) {
    bank->suspend_wakeup |= (1 << gpio);
    - else
    + enable_irq_wake(bank->irq);
    + } else {
    + disable_irq_wake(bank->irq);
    bank->suspend_wakeup &= ~(1 << gpio);
    + }
    spin_unlock(&bank->lock);
    return 0;
    #endif
    #ifdef CONFIG_ARCH_OMAP24XX
    case METHOD_GPIO_24XX:
    + if (bank->non_wakeup_gpios & (1 << gpio)) {
    + printk(KERN_ERR "Unable to modify wakeup on "
    + "non-wakeup GPIO%d\n",
    + (bank - gpio_bank) * 32 + gpio);
    + return -EINVAL;
    + }
    spin_lock(&bank->lock);
    if (enable) {
    - if (bank->non_wakeup_gpios & (1 << gpio)) {
    - printk(KERN_ERR "Unable to enable wakeup on "
    - "non-wakeup GPIO%d\n",
    - (bank - gpio_bank) * 32 + gpio);
    - spin_unlock(&bank->lock);
    - return -EINVAL;
    - }
    bank->suspend_wakeup |= (1 << gpio);
    - } else
    + enable_irq_wake(bank->irq);
    + } else {
    + disable_irq_wake(bank->irq);
    bank->suspend_wakeup &= ~(1 << gpio);
    + }
    spin_unlock(&bank->lock);
    return 0;
    #endif
    @@ -1111,16 +1117,81 @@ static struct irq_chip mpuio_irq_chip = {
    .mask = mpuio_mask_irq,
    .unmask = mpuio_unmask_irq,
    .set_type = gpio_irq_type,
    +#ifdef CONFIG_ARCH_OMAP16XX
    + /* REVISIT: assuming only 16xx supports MPUIO wake events */
    + .set_wake = gpio_wake_enable,
    +#endif
    };


    #define bank_is_mpuio(bank) ((bank)->method == METHOD_MPUIO)

    +
    +#ifdef CONFIG_ARCH_OMAP16XX
    +
    +#include <linux/platform_device.h>
    +
    +static int omap_mpuio_suspend_late(struct platform_device *pdev, pm_message_t mesg)
    +{
    + struct gpio_bank *bank = platform_get_drvdata(pdev);
    + void __iomem *mask_reg = bank->base + OMAP_MPUIO_GPIO_MASKIT;
    +
    + spin_lock(&bank->lock);
    + bank->saved_wakeup = __raw_readl(mask_reg);
    + __raw_writel(0xffff & ~bank->suspend_wakeup, mask_reg);
    + spin_unlock(&bank->lock);
    +
    + return 0;
    +}
    +
    +static int omap_mpuio_resume_early(struct platform_device *pdev)
    +{
    + struct gpio_bank *bank = platform_get_drvdata(pdev);
    + void __iomem *mask_reg = bank->base + OMAP_MPUIO_GPIO_MASKIT;
    +
    + spin_lock(&bank->lock);
    + __raw_writel(bank->saved_wakeup, mask_reg);
    + spin_unlock(&bank->lock);
    +
    + return 0;
    +}
    +
    +/* use platform_driver for this, now that there's no longer any
    + * point to sys_device (other than not disturbing old code).
    + */
    +static struct platform_driver omap_mpuio_driver = {
    + .suspend_late = omap_mpuio_suspend_late,
    + .resume_early = omap_mpuio_resume_early,
    + .driver = {
    + .name = "mpuio",
    + },
    +};
    +
    +static struct platform_device omap_mpuio_device = {
    + .name = "mpuio",
    + .id = -1,
    + .dev = {
    + .driver = &omap_mpuio_driver.driver,
    + }
    + /* could list the /proc/iomem resources */
    +};
    +
    +static inline void mpuio_init(void)
    +{
    + if (platform_driver_register(&omap_mpuio_driver) == 0)
    + (void) platform_device_register(&omap_mpuio_device);
    +}
    +
    +#else
    +static inline void mpuio_init(void) {}
    +#endif /* 16xx */
    +
    #else

    extern struct irq_chip mpuio_irq_chip;

    #define bank_is_mpuio(bank) 0
    +static inline void mpuio_init(void) {}

    #endif

    @@ -1487,6 +1558,8 @@ static int __init omap_gpio_sysinit(void)
    if (!initialized)
    ret = _omap_gpio_init();

    + mpuio_init();
    +
    #if defined(CONFIG_ARCH_OMAP16XX) || defined(CONFIG_ARCH_OMAP24XX)
    if (cpu_is_omap16xx() || cpu_is_omap24xx()) {
    if (ret == 0) {
    --
    1.4.4.2
    -
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

    \
     
     \ /
      Last update: 2007-04-04 20:17    [W:4.017 / U:0.180 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site