ARM: s5pv210: Remove legacy setting of external wakeup interrupts
authorKrzysztof Kozlowski <krzk@kernel.org>
Thu, 19 Jul 2018 10:39:35 +0000 (12:39 +0200)
committerJunghoon Kim <jhoon20.kim@samsung.com>
Thu, 14 Feb 2019 05:57:54 +0000 (14:57 +0900)
Since Exynos/S5Pv210 pin-controller driver is taking care about setting
the external wakeup interrupts mask, the legacy code can be removed.

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
Cc: Tomasz Figa <tomasz.figa@gmail.com>
Cc: Sylwester Nawrocki <snawrocki@kernel.org>
Acked-by: Tomasz Figa <tomasz.figa@gmail.com>
Change-Id: Id3dcaa57f6175054e77df3b17688d8d81e0d3797

arch/arm/mach-s5pv210/common.h
arch/arm/mach-s5pv210/pm.c

index 2ad387c1ecf0498f6ebdacbab69d8810199e1cd6..9367228caffcf30c851b73b47e488010ecb40bc0 100644 (file)
@@ -13,7 +13,6 @@
 #define __ARCH_ARM_MACH_S5PV210_COMMON_H
 
 #ifdef CONFIG_PM_SLEEP
-u32 exynos_get_eint_wake_mask(void);
 void s5pv210_cpu_resume(void);
 void s5pv210_pm_init(void);
 #else
index 07cee14a363b05eb1ca99418eb8d3c0074d0b04d..a4b3b0971c2638ca898e2d4ecf3060767e6967d1 100644 (file)
@@ -37,6 +37,11 @@ static struct sleep_save s5pv210_core_save[] = {
  */
 static u32 s5pv210_irqwake_intmask = 0xffffffff;
 
+static u32 s5pv210_read_eint_wakeup_mask(void)
+{
+       return __raw_readl(S5P_EINT_WAKEUP_MASK);
+}
+
 /*
  * Suspend helpers.
  */
@@ -64,8 +69,10 @@ static void s5pv210_pm_prepare(void)
 {
        unsigned int tmp;
 
-       /* Set wake-up mask registers */
-       __raw_writel(exynos_get_eint_wake_mask(), S5P_EINT_WAKEUP_MASK);
+       /*
+        * Set wake-up mask registers
+        * S5P_EINT_WAKEUP_MASK is set by pinctrl driver in late suspend.
+        */
        __raw_writel(s5pv210_irqwake_intmask, S5P_WAKEUP_MASK);
 
        /* ensure at least INFORM0 has the resume address */
@@ -94,6 +101,7 @@ static void s5pv210_pm_prepare(void)
  */
 static int s5pv210_suspend_enter(suspend_state_t state)
 {
+       u32 eint_wakeup_mask = s5pv210_read_eint_wakeup_mask();
        int ret;
 
        s3c_pm_debug_init();
@@ -101,10 +109,10 @@ static int s5pv210_suspend_enter(suspend_state_t state)
        S3C_PMDBG("%s: suspending the system...\n", __func__);
 
        S3C_PMDBG("%s: wakeup masks: %08x,%08x\n", __func__,
-                       s5pv210_irqwake_intmask, exynos_get_eint_wake_mask());
+                       s5pv210_irqwake_intmask, eint_wakeup_mask);
 
        if (s5pv210_irqwake_intmask == -1U
-           && exynos_get_eint_wake_mask() == -1U) {
+           && eint_wakeup_mask == -1U) {
                pr_err("%s: No wake-up sources!\n", __func__);
                pr_err("%s: Aborting sleep\n", __func__);
                return -EINVAL;