2015-02-10 03:09:10

by Chanwoo Choi

[permalink] [raw]
Subject: [PATCH] ARM: exynos: Fix wrong hwirq of RTC interrupt for Exynos3250 SoC

This patch fixes wrong hwirq of RTC irq for Exynos3250 SoC. When entering
suspend state, 'enable_irq_wake fail' happen because of the mismatch of RTC hwirq.

[ 429.200937] Freezing user space processes ... (elapsed 0.002 seconds) done.
[ 429.203383] Freezing remaining freezable tasks ... (elapsed 0.000 seconds) done.
[ 429.209914] Suspending console(s) (use no_console_suspend to debug)
[ 429.370685] wake enabled for irq 65
[ 429.370837] wake enabled for irq 64
[ 429.370868] wake enabled for irq 79
[snip]
[ 429.372120] s3c-rtc 10070000.rtc: enable_irq_wake failed

Fixes: a4f582f5c5fe3 (ARM: EXYNOS: Add exynos3250 suspend-to-ram support)
Cc: Kukjin Kim <[email protected]>
Signed-off-by: Chanwoo Choi <[email protected]>
Acked-by: Kyungmin Park <[email protected]>
---
arch/arm/mach-exynos/suspend.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/mach-exynos/suspend.c b/arch/arm/mach-exynos/suspend.c
index d6feef3..2b8bd67 100644
--- a/arch/arm/mach-exynos/suspend.c
+++ b/arch/arm/mach-exynos/suspend.c
@@ -92,8 +92,8 @@ static unsigned int exynos_pmu_spare3;
static u32 exynos_irqwake_intmask = 0xffffffff;

static const struct exynos_wkup_irq exynos3250_wkup_irq[] = {
- { 73, BIT(1) }, /* RTC alarm */
- { 74, BIT(2) }, /* RTC tick */
+ { 105, BIT(1) }, /* RTC alarm */
+ { 106, BIT(2) }, /* RTC tick */
{ /* sentinel */ },
};

--
1.8.5.5


2015-02-10 08:05:11

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH] ARM: exynos: Fix wrong hwirq of RTC interrupt for Exynos3250 SoC

2015-02-10 4:09 GMT+01:00 Chanwoo Choi <[email protected]>:
> This patch fixes wrong hwirq of RTC irq for Exynos3250 SoC. When entering
> suspend state, 'enable_irq_wake fail' happen because of the mismatch of RTC hwirq.
>
> [ 429.200937] Freezing user space processes ... (elapsed 0.002 seconds) done.
> [ 429.203383] Freezing remaining freezable tasks ... (elapsed 0.000 seconds) done.
> [ 429.209914] Suspending console(s) (use no_console_suspend to debug)
> [ 429.370685] wake enabled for irq 65
> [ 429.370837] wake enabled for irq 64
> [ 429.370868] wake enabled for irq 79
> [snip]
> [ 429.372120] s3c-rtc 10070000.rtc: enable_irq_wake failed
>
> Fixes: a4f582f5c5fe3 (ARM: EXYNOS: Add exynos3250 suspend-to-ram support)
> Cc: Kukjin Kim <[email protected]>
> Signed-off-by: Chanwoo Choi <[email protected]>
> Acked-by: Kyungmin Park <[email protected]>
> ---
> arch/arm/mach-exynos/suspend.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)

Yeah, I remember now Jonghwa's patches back in 2014 August and September :)
https://lkml.org/lkml/2014/8/26/40
https://lkml.org/lkml/2014/9/26/92

Anyway now this looks good:
Reviewed-by: Krzysztof Kozlowski <[email protected]>

Best regards,
Krzysztof

>
> diff --git a/arch/arm/mach-exynos/suspend.c b/arch/arm/mach-exynos/suspend.c
> index d6feef3..2b8bd67 100644
> --- a/arch/arm/mach-exynos/suspend.c
> +++ b/arch/arm/mach-exynos/suspend.c
> @@ -92,8 +92,8 @@ static unsigned int exynos_pmu_spare3;
> static u32 exynos_irqwake_intmask = 0xffffffff;
>
> static const struct exynos_wkup_irq exynos3250_wkup_irq[] = {
> - { 73, BIT(1) }, /* RTC alarm */
> - { 74, BIT(2) }, /* RTC tick */
> + { 105, BIT(1) }, /* RTC alarm */
> + { 106, BIT(2) }, /* RTC tick */
> { /* sentinel */ },
> };
>
> --
> 1.8.5.5
>
>
> _______________________________________________
> linux-arm-kernel mailing list
> [email protected]
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

2015-02-26 20:59:15

by Kukjin Kim

[permalink] [raw]
Subject: Re: [PATCH] ARM: exynos: Fix wrong hwirq of RTC interrupt for Exynos3250 SoC

On 02/10/15 17:05, Krzysztof Kozlowski wrote:
> 2015-02-10 4:09 GMT+01:00 Chanwoo Choi <[email protected]>:
>> This patch fixes wrong hwirq of RTC irq for Exynos3250 SoC. When entering
>> suspend state, 'enable_irq_wake fail' happen because of the mismatch of RTC hwirq.
>>
>> [ 429.200937] Freezing user space processes ... (elapsed 0.002 seconds) done.
>> [ 429.203383] Freezing remaining freezable tasks ... (elapsed 0.000 seconds) done.
>> [ 429.209914] Suspending console(s) (use no_console_suspend to debug)
>> [ 429.370685] wake enabled for irq 65
>> [ 429.370837] wake enabled for irq 64
>> [ 429.370868] wake enabled for irq 79
>> [snip]
>> [ 429.372120] s3c-rtc 10070000.rtc: enable_irq_wake failed
>>
>> Fixes: a4f582f5c5fe3 (ARM: EXYNOS: Add exynos3250 suspend-to-ram support)
>> Cc: Kukjin Kim <[email protected]>
>> Signed-off-by: Chanwoo Choi <[email protected]>
>> Acked-by: Kyungmin Park <[email protected]>
>> ---
>> arch/arm/mach-exynos/suspend.c | 4 ++--
>> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> Yeah, I remember now Jonghwa's patches back in 2014 August and September :)
> https://lkml.org/lkml/2014/8/26/40
> https://lkml.org/lkml/2014/9/26/92
>
> Anyway now this looks good:
> Reviewed-by: Krzysztof Kozlowski <[email protected]>
>
Thanks, applied.

- Kukjin