Commit c3710ee7 authored by Caesar Wang's avatar Caesar Wang
Browse files

rockchip/rk3399: fixes the typo and the WARNINGS during suspend/resume



This patch fixes the two things as follows:

1) rk3399_flash_l2_b" seems to be a typo. That's "flush", not "flash".

2) fixes the warnings log.
We always hit the warnings thing during the suspend, as below log:
..
[   51.022334] CPU5: shutdown
[   51.025069] psci: CPU5 killed.
INFO:    sdram_params->ddr_freq = 928000000
WARNING: rk3399_flash_l2_b:reg 28830380,wait

When the L2 completes the clean and invalidate sequence, it asserts the
L2FLUSHDONE signal. The SoC can now deassert L2FLUSHREQ signal and then
the L2 deasserts L2FLUSHDONE.

Then, a loop without a delay isn't really great to measure time. We should
probably add a udelay(10) or so in there and then maybe replace the WARN()
after the loop. In the actual tests, the L2 cache will take ~4ms by
default for big cluster.

In the real world that give 10ms for the enough margin, like the
ddr/cpu/cci frequency and other factors that will affect it.

Change-Id: I55788c897be232bf72e8c7b0e10cf9b06f7aa50d
Signed-off-by: default avatarCaesar Wang <wxt@rock-chips.com>
parent aa5b843f
...@@ -362,19 +362,23 @@ static void pmu_power_domains_resume(void) ...@@ -362,19 +362,23 @@ static void pmu_power_domains_resume(void)
clk_gate_con_restore(); clk_gate_con_restore();
} }
void rk3399_flash_l2_b(void) void rk3399_flush_l2_b(void)
{ {
uint32_t wait_cnt = 0; uint32_t wait_cnt = 0;
mmio_setbits_32(PMU_BASE + PMU_SFT_CON, BIT(L2_FLUSH_REQ_CLUSTER_B)); mmio_setbits_32(PMU_BASE + PMU_SFT_CON, BIT(L2_FLUSH_REQ_CLUSTER_B));
dsb(); dsb();
/*
* The Big cluster flush L2 cache took ~4ms by default, give 10ms for
* the enough margin.
*/
while (!(mmio_read_32(PMU_BASE + PMU_CORE_PWR_ST) & while (!(mmio_read_32(PMU_BASE + PMU_CORE_PWR_ST) &
BIT(L2_FLUSHDONE_CLUSTER_B))) { BIT(L2_FLUSHDONE_CLUSTER_B))) {
wait_cnt++; wait_cnt++;
if (wait_cnt >= MAX_WAIT_COUNT) udelay(10);
WARN("%s:reg %x,wait\n", __func__, if (wait_cnt == 10000 / 10)
mmio_read_32(PMU_BASE + PMU_CORE_PWR_ST)); WARN("L2 cache flush on suspend took longer than 10ms\n");
} }
mmio_clrbits_32(PMU_BASE + PMU_SFT_CON, BIT(L2_FLUSH_REQ_CLUSTER_B)); mmio_clrbits_32(PMU_BASE + PMU_SFT_CON, BIT(L2_FLUSH_REQ_CLUSTER_B));
...@@ -391,7 +395,7 @@ static void pmu_scu_b_pwrdn(void) ...@@ -391,7 +395,7 @@ static void pmu_scu_b_pwrdn(void)
return; return;
} }
rk3399_flash_l2_b(); rk3399_flush_l2_b();
mmio_setbits_32(PMU_BASE + PMU_SFT_CON, BIT(ACINACTM_CLUSTER_B_CFG)); mmio_setbits_32(PMU_BASE + PMU_SFT_CON, BIT(ACINACTM_CLUSTER_B_CFG));
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment