rcar_gen3: drivers: Change to restore timer counter value at resume
authorToshiyuki Ogasahara <[email protected]>
Fri, 22 Mar 2019 07:14:00 +0000 (16:14 +0900)
committerMarek Vasut <[email protected]>
Thu, 11 Apr 2019 10:57:03 +0000 (12:57 +0200)
Changed to save and restore cntpct_el0 using memory mapped
register for generic timer when System Suspend and Resume.

Reported by: Volodymyr Babchuk <[email protected]>
Signed-off-by: Toshiyuki Ogasahara <[email protected]>
Signed-off-by: Yoshifumi Hosoya <[email protected]>
Change-Id: I40fd9f5434c4d52b320cd1d20322b9b8e4e67155

drivers/renesas/rcar/console/rcar_printf.c
drivers/renesas/rcar/console/rcar_printf.h
drivers/renesas/rcar/pwrc/call_sram.S
drivers/renesas/rcar/pwrc/pwrc.c
drivers/renesas/rcar/pwrc/pwrc.h
plat/renesas/rcar/plat_pm.c

index 2a6e2c00324122fc8e9acd51220c176b92600004..e75b9f4541618898c29a8c278ee863a80ba945d8 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2015-2017, Renesas Electronics Corporation. All rights reserved.
+ * Copyright (c) 2015-2019, Renesas Electronics Corporation. All rights reserved.
  *
  * SPDX-License-Identifier: BSD-3-Clause
  */
@@ -94,9 +94,6 @@ int32_t rcar_log_init(void)
                             sizeof(t_log->header.head));
                t_log->header.index = 0U;
                t_log->header.size = 0U;
-#ifndef IMAGE_BL2
-               rcar_stack_generic_timer[INDEX_TIMER_COUNT] = 0U;
-#endif
        }
        rcar_lock_init();
 
index bcb00c341f32afb20242720e01812211de90c7e9..5da70e6365d43ead19ac1860c28156c60380e3d9 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2015-2017, Renesas Electronics Corporation. All rights reserved.
+ * Copyright (c) 2015-2019, Renesas Electronics Corporation. All rights reserved.
  *
  * SPDX-License-Identifier: BSD-3-Clause
  */
@@ -12,8 +12,4 @@
 int32_t rcar_set_log_data(int32_t c);
 int32_t rcar_log_init(void);
 
-#if IMAGE_BL31
-extern uint64_t rcar_stack_generic_timer[5];
-#endif
-
 #endif /* RCAR_PRINTF_H */
index 7c96b7ee91c23b183060777775441806a059e9d3..aa8644cb98011a108049b44e15bbd7ca065a4ec4 100644 (file)
@@ -1,21 +1,13 @@
 /*
- * Copyright (c) 2018, Renesas Electronics Corporation. All rights reserved.
+ * Copyright (c) 2015-2019, Renesas Electronics Corporation. All rights reserved.
  *
  * SPDX-License-Identifier: BSD-3-Clause
  */
 
 #include <arch.h>
 #include <asm_macros.S>
-#include "rcar_def.h"
 
 .global rcar_pwrc_switch_stack
-.global rcar_pwrc_save_generic_timer
-.global rcar_pwrc_restore_generic_timer
-
-#define OFFSET_SP_X9_X10       (0x00)
-#define OFFSET_CNTFID0         (0x10)
-#define OFFSET_CNTPCT_EL0      (0x18)
-#define OFFSET_TIMER_COUNT     (0x20)
 
 /*
  * x0 : jump address,
@@ -54,37 +46,3 @@ func rcar_pwrc_switch_stack
        ldp     x29, x30, [sp,#-16]
        ret
 endfunc rcar_pwrc_switch_stack
-
-/* x0 : stack pointer base address */
-func rcar_pwrc_save_generic_timer
-
-       stp     x9, x10, [x0, #OFFSET_SP_X9_X10]
-
-       /* save CNTFID0 and cntpct_el0 */
-       mov_imm x10, (RCAR_CNTC_BASE + CNTFID_OFF)
-       ldr     x9, [x10]
-       mrs     x10, cntpct_el0
-       stp     x9, x10, [x0, #OFFSET_CNTFID0]
-
-       ldp     x9, x10, [x0, #OFFSET_SP_X9_X10]
-
-       ret
-endfunc rcar_pwrc_save_generic_timer
-
-/* x0 : Stack pointer base address */
-func rcar_pwrc_restore_generic_timer
-
-       stp     x9, x10, [x0, #OFFSET_SP_X9_X10]
-
-       /* restore CNTFID0 and cntpct_el0 */
-       ldr     x10, [x0, #OFFSET_CNTFID0]
-       mov_imm x9, (RCAR_CNTC_BASE + CNTFID_OFF)
-       str     x10, [x9]
-       ldp     x9, x10, [x0, #OFFSET_CNTPCT_EL0]
-       add     x9, x9, x10
-       str     x9, [x0, #OFFSET_TIMER_COUNT]
-
-       ldp     x9, x10, [x0, #OFFSET_SP_X9_X10]
-
-       ret
-endfunc rcar_pwrc_restore_generic_timer
index e223f07156b2496b00693b95c17d9a286222fa24..d97e593bbdb82056ca5183b2424c534668b2bc2a 100644 (file)
@@ -13,6 +13,7 @@
 #include <lib/bakery_lock.h>
 #include <lib/mmio.h>
 #include <lib/xlat_tables/xlat_tables_v2.h>
+#include <plat/common/platform.h>
 
 #include "iic_dvfs.h"
 #include "rcar_def.h"
@@ -126,6 +127,14 @@ RCAR_INSTANTIATE_LOCK
 #define        RST_MODEMR                              (RST_BASE + 0x0060U)
 #define        RST_MODEMR_BIT0                         (0x00000001U)
 
+#define RCAR_CNTCR_OFF                         (0x00U)
+#define RCAR_CNTCVL_OFF                                (0x08U)
+#define RCAR_CNTCVU_OFF                                (0x0CU)
+#define RCAR_CNTFID_OFF                                (0x20U)
+
+#define RCAR_CNTCR_EN                          ((uint32_t)1U << 0U)
+#define RCAR_CNTCR_FCREQ(x)                    ((uint32_t)(x) << 8U)
+
 #if PMIC_ROHM_BD9571
 #define        BIT_BKUP_CTRL_OUT                       ((uint8_t)(1U << 4))
 #define        PMIC_BKUP_MODE_CNT                      (0x20U)
@@ -323,6 +332,39 @@ done:
        rcar_lock_release();
 }
 
+static uint64_t rcar_pwrc_saved_cntpct_el0;
+static uint32_t rcar_pwrc_saved_cntfid;
+
+#if RCAR_SYSTEM_SUSPEND
+static void rcar_pwrc_save_timer_state(void)
+{
+       rcar_pwrc_saved_cntpct_el0 = read_cntpct_el0();
+
+       rcar_pwrc_saved_cntfid =
+               mmio_read_32((uintptr_t)(RCAR_CNTC_BASE + RCAR_CNTFID_OFF));
+}
+#endif
+
+void rcar_pwrc_restore_timer_state(void)
+{
+       /* Stop timer before restoring counter value */
+       mmio_write_32((uintptr_t)(RCAR_CNTC_BASE + RCAR_CNTCR_OFF), 0U);
+
+       mmio_write_32((uintptr_t)(RCAR_CNTC_BASE + RCAR_CNTCVL_OFF),
+               (uint32_t)(rcar_pwrc_saved_cntpct_el0 & 0xFFFFFFFFU));
+       mmio_write_32((uintptr_t)(RCAR_CNTC_BASE + RCAR_CNTCVU_OFF),
+               (uint32_t)(rcar_pwrc_saved_cntpct_el0 >> 32U));
+
+       mmio_write_32((uintptr_t)(RCAR_CNTC_BASE + RCAR_CNTFID_OFF),
+               rcar_pwrc_saved_cntfid);
+
+       /* Start generic timer back */
+       write_cntfrq_el0((u_register_t)plat_get_syscnt_freq2());
+
+       mmio_write_32((uintptr_t)(RCAR_CNTC_BASE + RCAR_CNTCR_OFF),
+               (RCAR_CNTCR_FCREQ(0U) | RCAR_CNTCR_EN));
+}
+
 #if !PMIC_ROHM_BD9571
 void rcar_pwrc_system_reset(void)
 {
@@ -640,7 +682,7 @@ void rcar_pwrc_set_suspend_to_ram(void)
                                       DEVICE_SRAM_STACK_SIZE);
        uint32_t sctlr;
 
-       rcar_pwrc_save_generic_timer(rcar_stack_generic_timer);
+       rcar_pwrc_save_timer_state();
 
        /* disable MMU */
        sctlr = (uint32_t) read_sctlr_el3();
index d4d6fc4412d03bee4bdc89efba798fd18770f6cd..cfb35ff92350ceeb99a195b5ca1a3182fc0aba8c 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2015-2017, Renesas Electronics Corporation. All rights reserved.
+ * Copyright (c) 2015-2019, Renesas Electronics Corporation. All rights reserved.
  *
  * SPDX-License-Identifier: BSD-3-Clause
  */
@@ -52,6 +52,7 @@ uint32_t rcar_pwrc_status(uint64_t mpidr);
 uint32_t rcar_pwrc_get_cluster(void);
 uint32_t rcar_pwrc_get_mpidr_cluster(uint64_t mpidr);
 uint32_t rcar_pwrc_get_cpu_num(uint32_t cluster_type);
+void rcar_pwrc_restore_timer_state(void);
 void plat_secondary_reset(void);
 
 void rcar_pwrc_code_copy_to_system_ram(void);
@@ -67,12 +68,8 @@ void rcar_pwrc_init_suspend_to_ram(void);
 void rcar_pwrc_suspend_to_ram(void);
 #endif
 
-extern void rcar_pwrc_save_generic_timer(uint64_t *rcar_stack_generic_timer);
 extern uint32_t rcar_pwrc_switch_stack(uintptr_t jump, uintptr_t stack,
                                       void *arg);
-
-extern uint64_t rcar_stack_generic_timer[5];
-
 #endif
 
 #endif /* PWRC_H */
index f41c172a995910aa943b9ef497924b27252a0931..e678da5dc58597701325a18702058da409660337 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2015-2017, Renesas Electronics Corporation. All rights reserved.
+ * Copyright (c) 2015-2019, Renesas Electronics Corporation. All rights reserved.
  *
  * SPDX-License-Identifier: BSD-3-Clause
  */
@@ -35,8 +35,6 @@
 #define CLUSTER_PWR_STATE(s)   ((s)->pwr_domain_state[MPIDR_AFFLVL1])
 #define CORE_PWR_STATE(s)      ((s)->pwr_domain_state[MPIDR_AFFLVL0])
 
-uint64_t rcar_stack_generic_timer[5] __attribute__ ((section("data")));
-
 extern void rcar_pwrc_restore_generic_timer(uint64_t *stack);
 extern void plat_rcar_gic_driver_init(void);
 extern void plat_rcar_gic_init(void);
@@ -150,11 +148,7 @@ static void rcar_pwr_domain_suspend_finish(const psci_power_state_t
        if (cluster_type == RCAR_CLUSTER_A53A57)
                plat_cci_init();
 
-       rcar_pwrc_restore_generic_timer(rcar_stack_generic_timer);
-
-       /* start generic timer */
-       write_cntfrq_el0(plat_get_syscnt_freq2());
-       mmio_write_32(RCAR_CNTC_BASE + CNTCR_OFF, CNTCR_FCREQ(U(0)) | CNTCR_EN);
+       rcar_pwrc_restore_timer_state();
        rcar_pwrc_setup();
        rcar_pwrc_code_copy_to_system_ram();