diff mbox

[4/4] ARM: exynos4: remove useless code to save/restore L2 and GIC state

Message ID 1320426228-21746-5-git-send-email-amit.kachhap@linaro.org
State Superseded
Headers show

Commit Message

Amit Daniel Kachhap Nov. 4, 2011, 5:03 p.m. UTC
From: Amit Daniel Kachhap <amit.kachhap@linaro.org>

Following the merge of CPU PM notifiers and L2 resume code, this patch
removes useless code to save and restore L2 and GIC registers.

This is now automatically covered by suspend calls which integrated
CPU PM notifiers and new sleep code that allows to resume L2 before MMU
is turned on.

Signed-off-by: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>
Signed-off-by: Amit Daniel Kachhap <amit.kachhap@linaro.org>
---
 arch/arm/mach-exynos4/pm.c |   86 --------------------------------------------
 1 files changed, 0 insertions(+), 86 deletions(-)

Comments

On 11/04/2011 06:03 PM, amit.kachhap@linaro.org wrote:
> From: Amit Daniel Kachhap <amit.kachhap@linaro.org>
> 
> Following the merge of CPU PM notifiers and L2 resume code, this patch
> removes useless code to save and restore L2 and GIC registers.
> 
> This is now automatically covered by suspend calls which integrated
> CPU PM notifiers and new sleep code that allows to resume L2 before MMU
> is turned on.
> 
> Signed-off-by: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>
> Signed-off-by: Amit Daniel Kachhap <amit.kachhap@linaro.org>
> ---
>  arch/arm/mach-exynos4/pm.c |   86 --------------------------------------------
>  1 files changed, 0 insertions(+), 86 deletions(-)
> 
> diff --git a/arch/arm/mach-exynos4/pm.c b/arch/arm/mach-exynos4/pm.c
> index 62e4f43..7499f14 100644
> --- a/arch/arm/mach-exynos4/pm.c
> +++ b/arch/arm/mach-exynos4/pm.c
> @@ -63,77 +63,6 @@ static struct sleep_save exynos4_vpll_save[] = {
>  };
>  
>  static struct sleep_save exynos4_core_save[] = {
> -	/* GIC side */
> -	SAVE_ITEM(S5P_VA_GIC_CPU + 0x000),
> -	SAVE_ITEM(S5P_VA_GIC_CPU + 0x004),
> -	SAVE_ITEM(S5P_VA_GIC_CPU + 0x008),
> -	SAVE_ITEM(S5P_VA_GIC_CPU + 0x00C),
> -	SAVE_ITEM(S5P_VA_GIC_CPU + 0x014),
> -	SAVE_ITEM(S5P_VA_GIC_CPU + 0x018),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x000),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x004),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x100),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x104),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x108),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x300),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x304),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x308),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x400),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x404),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x408),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x40C),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x410),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x414),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x418),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x41C),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x420),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x424),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x428),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x42C),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x430),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x434),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x438),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x43C),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x440),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x444),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x448),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x44C),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x450),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x454),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x458),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x45C),
> -
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x800),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x804),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x808),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x80C),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x810),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x814),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x818),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x81C),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x820),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x824),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x828),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x82C),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x830),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x834),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x838),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x83C),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x840),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x844),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x848),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x84C),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x850),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x854),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x858),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0x85C),
> -
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0xC00),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0xC04),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0xC08),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0xC0C),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0xC10),
> -	SAVE_ITEM(S5P_VA_GIC_DIST + 0xC14),


This list is not complete anyway, some peripheral devices interrupts do not
work after resume from system suspend to RAM.
Is there any code already handling GIC state during system suspend/resume cycles?
Or you refer to some upcoming patches ?

--
Thanks,
Sylwester
Amit Daniel Kachhap Nov. 11, 2011, 6:28 a.m. UTC | #2
On 4 November 2011 23:03, Sylwester Nawrocki <s.nawrocki@samsung.com> wrote:
> On 11/04/2011 06:03 PM, amit.kachhap@linaro.org wrote:
>> From: Amit Daniel Kachhap <amit.kachhap@linaro.org>
>>
>> Following the merge of CPU PM notifiers and L2 resume code, this patch
>> removes useless code to save and restore L2 and GIC registers.
>>
>> This is now automatically covered by suspend calls which integrated
>> CPU PM notifiers and new sleep code that allows to resume L2 before MMU
>> is turned on.
>>
>> Signed-off-by: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>
>> Signed-off-by: Amit Daniel Kachhap <amit.kachhap@linaro.org>
>> ---
>>  arch/arm/mach-exynos4/pm.c |   86 --------------------------------------------
>>  1 files changed, 0 insertions(+), 86 deletions(-)
>>
>> diff --git a/arch/arm/mach-exynos4/pm.c b/arch/arm/mach-exynos4/pm.c
>> index 62e4f43..7499f14 100644
>> --- a/arch/arm/mach-exynos4/pm.c
>> +++ b/arch/arm/mach-exynos4/pm.c
>> @@ -63,77 +63,6 @@ static struct sleep_save exynos4_vpll_save[] = {
>>  };
>>
>>  static struct sleep_save exynos4_core_save[] = {
>> -     /* GIC side */
>> -     SAVE_ITEM(S5P_VA_GIC_CPU + 0x000),
>> -     SAVE_ITEM(S5P_VA_GIC_CPU + 0x004),
>> -     SAVE_ITEM(S5P_VA_GIC_CPU + 0x008),
>> -     SAVE_ITEM(S5P_VA_GIC_CPU + 0x00C),
>> -     SAVE_ITEM(S5P_VA_GIC_CPU + 0x014),
>> -     SAVE_ITEM(S5P_VA_GIC_CPU + 0x018),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x000),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x004),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x100),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x104),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x108),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x300),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x304),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x308),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x400),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x404),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x408),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x40C),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x410),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x414),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x418),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x41C),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x420),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x424),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x428),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x42C),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x430),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x434),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x438),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x43C),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x440),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x444),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x448),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x44C),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x450),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x454),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x458),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x45C),
>> -
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x800),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x804),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x808),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x80C),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x810),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x814),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x818),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x81C),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x820),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x824),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x828),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x82C),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x830),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x834),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x838),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x83C),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x840),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x844),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x848),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x84C),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x850),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x854),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x858),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x85C),
>> -
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0xC00),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0xC04),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0xC08),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0xC0C),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0xC10),
>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0xC14),
>
>
> This list is not complete anyway, some peripheral devices interrupts do not
> work after resume from system suspend to RAM.
> Is there any code already handling GIC state during system suspend/resume cycles?
> Or you refer to some upcoming patches ?

In my next patch series I have left the GIC save/restore from platform code.

>
> --
> Thanks,
> Sylwester
>
Sylwester Nawrocki Nov. 11, 2011, 11:41 a.m. UTC | #3
On 11/11/2011 07:28 AM, Amit Kachhap wrote:
> On 4 November 2011 23:03, Sylwester Nawrocki<s.nawrocki@samsung.com>  wrote:
>> On 11/04/2011 06:03 PM, amit.kachhap@linaro.org wrote:
>>> From: Amit Daniel Kachhap<amit.kachhap@linaro.org>
>>>
>>> Following the merge of CPU PM notifiers and L2 resume code, this patch
>>> removes useless code to save and restore L2 and GIC registers.
>>>
>>> This is now automatically covered by suspend calls which integrated
>>> CPU PM notifiers and new sleep code that allows to resume L2 before MMU
>>> is turned on.
>>>
>>> Signed-off-by: Lorenzo Pieralisi<lorenzo.pieralisi@arm.com>
>>> Signed-off-by: Amit Daniel Kachhap<amit.kachhap@linaro.org>
>>> ---
>>>   arch/arm/mach-exynos4/pm.c |   86 --------------------------------------------
>>>   1 files changed, 0 insertions(+), 86 deletions(-)
>>>
>>> diff --git a/arch/arm/mach-exynos4/pm.c b/arch/arm/mach-exynos4/pm.c
>>> index 62e4f43..7499f14 100644
>>> --- a/arch/arm/mach-exynos4/pm.c
>>> +++ b/arch/arm/mach-exynos4/pm.c
>>> @@ -63,77 +63,6 @@ static struct sleep_save exynos4_vpll_save[] = {
>>>   };
>>>
>>>   static struct sleep_save exynos4_core_save[] = {
>>> -     /* GIC side */
>>> -     SAVE_ITEM(S5P_VA_GIC_CPU + 0x000),
>>> -     SAVE_ITEM(S5P_VA_GIC_CPU + 0x004),
>>> -     SAVE_ITEM(S5P_VA_GIC_CPU + 0x008),
>>> -     SAVE_ITEM(S5P_VA_GIC_CPU + 0x00C),
>>> -     SAVE_ITEM(S5P_VA_GIC_CPU + 0x014),
>>> -     SAVE_ITEM(S5P_VA_GIC_CPU + 0x018),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x000),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x004),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x100),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x104),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x108),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x300),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x304),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x308),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x400),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x404),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x408),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x40C),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x410),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x414),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x418),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x41C),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x420),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x424),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x428),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x42C),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x430),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x434),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x438),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x43C),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x440),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x444),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x448),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x44C),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x450),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x454),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x458),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x45C),
>>> -
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x800),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x804),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x808),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x80C),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x810),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x814),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x818),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x81C),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x820),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x824),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x828),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x82C),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x830),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x834),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x838),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x83C),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x840),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x844),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x848),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x84C),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x850),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x854),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x858),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0x85C),
>>> -
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0xC00),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0xC04),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0xC08),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0xC0C),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0xC10),
>>> -     SAVE_ITEM(S5P_VA_GIC_DIST + 0xC14),
>>
>>
>> This list is not complete anyway, some peripheral devices interrupts do not
>> work after resume from system suspend to RAM.
>> Is there any code already handling GIC state during system suspend/resume cycles?
>> Or you refer to some upcoming patches ?
> 
> In my next patch series I have left the GIC save/restore from platform code.

OK, although I thought someone is just going to fix the regression introduced with
commit:

commit aab74d3e753649defa52ea43cbec1e91ebb4cc8e
Author: Changhwan Youn <chaos.youn@samsung.com>
Date:   Sat Jul 16 10:49:51 2011 +0900

    ARM: EXYNOS4: Add support external GIC
    
    For full support of power modes, this patch adds implementation
    external GIC on EXYNOS4.
    
    External GIC of Exynos4 cannot support register banking so
    several interrupt related code for CPU1 should be different
    from that of CPU0.
    
    Signed-off-by: Changhwan Youn <chaos.youn@samsung.com>
    Signed-off-by: Kukjin Kim <kgene.kim@samsung.com>

The external GIC has more registers and now not all of them are properly handled during
system sleep/resume. I didn't get around yet to submit a proper solution for that.  
I suspect per CPU distributor and CPU interface register files need to be saved/restored.
I'm not an expert in the GIC internals though, I'm just wondering why can't GIC be left
active during system sleep.

--
Regards,
Sylwester
diff mbox

Patch

diff --git a/arch/arm/mach-exynos4/pm.c b/arch/arm/mach-exynos4/pm.c
index 62e4f43..7499f14 100644
--- a/arch/arm/mach-exynos4/pm.c
+++ b/arch/arm/mach-exynos4/pm.c
@@ -63,77 +63,6 @@  static struct sleep_save exynos4_vpll_save[] = {
 };
 
 static struct sleep_save exynos4_core_save[] = {
-	/* GIC side */
-	SAVE_ITEM(S5P_VA_GIC_CPU + 0x000),
-	SAVE_ITEM(S5P_VA_GIC_CPU + 0x004),
-	SAVE_ITEM(S5P_VA_GIC_CPU + 0x008),
-	SAVE_ITEM(S5P_VA_GIC_CPU + 0x00C),
-	SAVE_ITEM(S5P_VA_GIC_CPU + 0x014),
-	SAVE_ITEM(S5P_VA_GIC_CPU + 0x018),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x000),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x004),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x100),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x104),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x108),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x300),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x304),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x308),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x400),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x404),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x408),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x40C),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x410),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x414),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x418),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x41C),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x420),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x424),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x428),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x42C),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x430),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x434),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x438),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x43C),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x440),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x444),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x448),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x44C),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x450),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x454),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x458),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x45C),
-
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x800),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x804),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x808),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x80C),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x810),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x814),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x818),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x81C),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x820),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x824),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x828),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x82C),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x830),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x834),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x838),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x83C),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x840),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x844),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x848),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x84C),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x850),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x854),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x858),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0x85C),
-
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0xC00),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0xC04),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0xC08),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0xC0C),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0xC10),
-	SAVE_ITEM(S5P_VA_GIC_DIST + 0xC14),
 
 	SAVE_ITEM(S5P_VA_COMBINER_BASE + 0x000),
 	SAVE_ITEM(S5P_VA_COMBINER_BASE + 0x010),
@@ -154,13 +83,6 @@  static struct sleep_save exynos4_core_save[] = {
 	SAVE_ITEM(S5P_SROM_BC3),
 };
 
-static struct sleep_save exynos4_l2cc_save[] = {
-	SAVE_ITEM(S5P_VA_L2CC + L2X0_TAG_LATENCY_CTRL),
-	SAVE_ITEM(S5P_VA_L2CC + L2X0_DATA_LATENCY_CTRL),
-	SAVE_ITEM(S5P_VA_L2CC + L2X0_PREFETCH_CTRL),
-	SAVE_ITEM(S5P_VA_L2CC + L2X0_POWER_CTRL),
-	SAVE_ITEM(S5P_VA_L2CC + L2X0_AUX_CTRL),
-};
 
 /* For Cortex-A9 Diagnostic and Power control register */
 static unsigned int save_arm_register[2];
@@ -181,7 +103,6 @@  static void exynos4_pm_prepare(void)
 	u32 tmp;
 
 	s3c_pm_do_save(exynos4_core_save, ARRAY_SIZE(exynos4_core_save));
-	s3c_pm_do_save(exynos4_l2cc_save, ARRAY_SIZE(exynos4_l2cc_save));
 	s3c_pm_do_save(exynos4_epll_save, ARRAY_SIZE(exynos4_epll_save));
 	s3c_pm_do_save(exynos4_vpll_save, ARRAY_SIZE(exynos4_vpll_save));
 
@@ -397,13 +318,6 @@  static void exynos4_pm_resume(void)
 
 	exynos4_scu_enable(S5P_VA_SCU);
 
-#ifdef CONFIG_CACHE_L2X0
-	s3c_pm_do_restore_core(exynos4_l2cc_save, ARRAY_SIZE(exynos4_l2cc_save));
-	outer_inv_all();
-	/* enable L2X0*/
-	writel_relaxed(1, S5P_VA_L2CC + L2X0_CTRL);
-#endif
-
 early_wakeup:
 	return;
 }