linux-pm.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] gpio / PM: Replace CONFIG_PM_RUNTIME with CONFIG_PM
@ 2014-12-03  1:50 Rafael J. Wysocki
  2014-12-03  2:33 ` Alexandre Courbot
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Rafael J. Wysocki @ 2014-12-03  1:50 UTC (permalink / raw)
  To: Kevin Hilman
  Cc: Javier Martinez Canillas, Santosh Shilimkar, Linus Walleij,
	Alexandre Courbot, linux-omap, linux-gpio,
	Linux Kernel Mailing List, Linux PM list

From: Rafael J. Wysocki <rafael.j.wysocki@intel.com>

After commit b2b49ccbdd54 (PM: Kconfig: Set PM_RUNTIME if PM_SLEEP is
selected) PM_RUNTIME is always set if PM is set, so #ifdef blocks
depending on CONFIG_PM_RUNTIME may now be changed to depend on
CONFIG_PM.

Replace CONFIG_PM_RUNTIME with CONFIG_PM in drivers/gpio/gpio-omap.c.

Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
---

Note: This depends on commit b2b49ccbdd54 (PM: Kconfig: Set PM_RUNTIME if
PM_SLEEP is selected) which is only in linux-next at the moment (via the
linux-pm tree).

Please let me know if it is OK to take this one into linux-pm.

---
 drivers/gpio/gpio-omap.c |    8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

Index: linux-pm/drivers/gpio/gpio-omap.c
===================================================================
--- linux-pm.orig/drivers/gpio/gpio-omap.c
+++ linux-pm/drivers/gpio/gpio-omap.c
@@ -1259,7 +1259,7 @@ static int omap_gpio_probe(struct platfo
 
 #ifdef CONFIG_ARCH_OMAP2PLUS
 
-#if defined(CONFIG_PM_RUNTIME)
+#if defined(CONFIG_PM)
 static void omap_gpio_restore_context(struct gpio_bank *bank);
 
 static int omap_gpio_runtime_suspend(struct device *dev)
@@ -1440,7 +1440,7 @@ static int omap_gpio_runtime_resume(stru
 
 	return 0;
 }
-#endif /* CONFIG_PM_RUNTIME */
+#endif /* CONFIG_PM */
 
 void omap2_gpio_prepare_for_idle(int pwr_mode)
 {
@@ -1468,7 +1468,7 @@ void omap2_gpio_resume_after_idle(void)
 	}
 }
 
-#if defined(CONFIG_PM_RUNTIME)
+#if defined(CONFIG_PM)
 static void omap_gpio_init_context(struct gpio_bank *p)
 {
 	struct omap_gpio_reg_offs *regs = p->regs;
@@ -1525,7 +1525,7 @@ static void omap_gpio_restore_context(st
 	writel_relaxed(bank->context.irqenable2,
 				bank->base + bank->regs->irqenable2);
 }
-#endif /* CONFIG_PM_RUNTIME */
+#endif /* CONFIG_PM */
 #else
 #define omap_gpio_runtime_suspend NULL
 #define omap_gpio_runtime_resume NULL


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] gpio / PM: Replace CONFIG_PM_RUNTIME with CONFIG_PM
  2014-12-03  1:50 [PATCH] gpio / PM: Replace CONFIG_PM_RUNTIME with CONFIG_PM Rafael J. Wysocki
@ 2014-12-03  2:33 ` Alexandre Courbot
  2014-12-03 11:38 ` Javier Martinez Canillas
  2014-12-03 13:54 ` Linus Walleij
  2 siblings, 0 replies; 4+ messages in thread
From: Alexandre Courbot @ 2014-12-03  2:33 UTC (permalink / raw)
  To: Rafael J. Wysocki
  Cc: Kevin Hilman, Javier Martinez Canillas, Santosh Shilimkar,
	Linus Walleij, linux-omap, linux-gpio, Linux Kernel Mailing List,
	Linux PM list

On Wed, Dec 3, 2014 at 10:50 AM, Rafael J. Wysocki <rjw@rjwysocki.net> wrote:
> From: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
>
> After commit b2b49ccbdd54 (PM: Kconfig: Set PM_RUNTIME if PM_SLEEP is
> selected) PM_RUNTIME is always set if PM is set, so #ifdef blocks
> depending on CONFIG_PM_RUNTIME may now be changed to depend on
> CONFIG_PM.
>
> Replace CONFIG_PM_RUNTIME with CONFIG_PM in drivers/gpio/gpio-omap.c.
>
> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>

Acked-by: Alexandre Courbot <acourbot@nvidia.com>

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] gpio / PM: Replace CONFIG_PM_RUNTIME with CONFIG_PM
  2014-12-03  1:50 [PATCH] gpio / PM: Replace CONFIG_PM_RUNTIME with CONFIG_PM Rafael J. Wysocki
  2014-12-03  2:33 ` Alexandre Courbot
@ 2014-12-03 11:38 ` Javier Martinez Canillas
  2014-12-03 13:54 ` Linus Walleij
  2 siblings, 0 replies; 4+ messages in thread
From: Javier Martinez Canillas @ 2014-12-03 11:38 UTC (permalink / raw)
  To: Rafael J. Wysocki
  Cc: Kevin Hilman, Santosh Shilimkar, Linus Walleij,
	Alexandre Courbot, linux-omap, Linux GPIO List,
	Linux Kernel Mailing List, Linux PM list

Hello Rafael,

On Wed, Dec 3, 2014 at 2:50 AM, Rafael J. Wysocki <rjw@rjwysocki.net> wrote:
> From: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
>
> After commit b2b49ccbdd54 (PM: Kconfig: Set PM_RUNTIME if PM_SLEEP is
> selected) PM_RUNTIME is always set if PM is set, so #ifdef blocks
> depending on CONFIG_PM_RUNTIME may now be changed to depend on
> CONFIG_PM.
>
> Replace CONFIG_PM_RUNTIME with CONFIG_PM in drivers/gpio/gpio-omap.c.
>
> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
> ---

Acked-by: Javier Martinez Canillas <javier@dowhile0.org>

>
> Note: This depends on commit b2b49ccbdd54 (PM: Kconfig: Set PM_RUNTIME if
> PM_SLEEP is selected) which is only in linux-next at the moment (via the
> linux-pm tree).
>
> Please let me know if it is OK to take this one into linux-pm.
>

AFAICT there isn't another patch for gpio-omap on this cycle so I'm OK
if you take this through linux-pm.

Best regards,
Javier

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] gpio / PM: Replace CONFIG_PM_RUNTIME with CONFIG_PM
  2014-12-03  1:50 [PATCH] gpio / PM: Replace CONFIG_PM_RUNTIME with CONFIG_PM Rafael J. Wysocki
  2014-12-03  2:33 ` Alexandre Courbot
  2014-12-03 11:38 ` Javier Martinez Canillas
@ 2014-12-03 13:54 ` Linus Walleij
  2 siblings, 0 replies; 4+ messages in thread
From: Linus Walleij @ 2014-12-03 13:54 UTC (permalink / raw)
  To: Rafael J. Wysocki
  Cc: Kevin Hilman, Javier Martinez Canillas, Santosh Shilimkar,
	Alexandre Courbot, Linux-OMAP, linux-gpio,
	Linux Kernel Mailing List, Linux PM list

On Wed, Dec 3, 2014 at 2:50 AM, Rafael J. Wysocki <rjw@rjwysocki.net> wrote:

> From: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
>
> After commit b2b49ccbdd54 (PM: Kconfig: Set PM_RUNTIME if PM_SLEEP is
> selected) PM_RUNTIME is always set if PM is set, so #ifdef blocks
> depending on CONFIG_PM_RUNTIME may now be changed to depend on
> CONFIG_PM.
>
> Replace CONFIG_PM_RUNTIME with CONFIG_PM in drivers/gpio/gpio-omap.c.
>
> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
> ---
>
> Note: This depends on commit b2b49ccbdd54 (PM: Kconfig: Set PM_RUNTIME if
> PM_SLEEP is selected) which is only in linux-next at the moment (via the
> linux-pm tree).
>
> Please let me know if it is OK to take this one into linux-pm.

Go ahead. Should be no collissions.
Reviewed-by: Linus Walleij <linus.walleij@linaro.org>

Yours,
Linus Walleij

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2014-12-03 13:54 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-12-03  1:50 [PATCH] gpio / PM: Replace CONFIG_PM_RUNTIME with CONFIG_PM Rafael J. Wysocki
2014-12-03  2:33 ` Alexandre Courbot
2014-12-03 11:38 ` Javier Martinez Canillas
2014-12-03 13:54 ` Linus Walleij

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).