Unverified Commit 72bc6318 authored by Dimitris Papastamos's avatar Dimitris Papastamos Committed by GitHub
Browse files

Merge pull request #1498 from glneo/cache-early-fixes

Early cache enable and coherency fixes
parents 2ee596c4 903f13d3
...@@ -221,9 +221,10 @@ void gicv2_driver_init(const gicv2_driver_data_t *plat_driver_data) ...@@ -221,9 +221,10 @@ void gicv2_driver_init(const gicv2_driver_data_t *plat_driver_data)
* enabled. When the secondary CPU boots up, it initializes the * enabled. When the secondary CPU boots up, it initializes the
* GICC/GICR interface with the caches disabled. Hence flush the * GICC/GICR interface with the caches disabled. Hence flush the
* driver_data to ensure coherency. This is not required if the * driver_data to ensure coherency. This is not required if the
* platform has HW_ASSISTED_COHERENCY enabled. * platform has HW_ASSISTED_COHERENCY or WARMBOOT_ENABLE_DCACHE_EARLY
* enabled.
*/ */
#if !HW_ASSISTED_COHERENCY #if !(HW_ASSISTED_COHERENCY || WARMBOOT_ENABLE_DCACHE_EARLY)
flush_dcache_range((uintptr_t) &driver_data, sizeof(driver_data)); flush_dcache_range((uintptr_t) &driver_data, sizeof(driver_data));
flush_dcache_range((uintptr_t) driver_data, sizeof(*driver_data)); flush_dcache_range((uintptr_t) driver_data, sizeof(*driver_data));
#endif #endif
...@@ -360,7 +361,7 @@ void gicv2_set_pe_target_mask(unsigned int proc_num) ...@@ -360,7 +361,7 @@ void gicv2_set_pe_target_mask(unsigned int proc_num)
if (driver_data->target_masks[proc_num] == 0) { if (driver_data->target_masks[proc_num] == 0) {
driver_data->target_masks[proc_num] = driver_data->target_masks[proc_num] =
gicv2_get_cpuif_id(driver_data->gicd_base); gicv2_get_cpuif_id(driver_data->gicd_base);
#if !HW_ASSISTED_COHERENCY #if !(HW_ASSISTED_COHERENCY || WARMBOOT_ENABLE_DCACHE_EARLY)
/* /*
* PEs only update their own masks. Primary updates it with * PEs only update their own masks. Primary updates it with
* caches on. But because secondaries does it with caches off, * caches on. But because secondaries does it with caches off,
......
...@@ -147,9 +147,10 @@ void gicv3_driver_init(const gicv3_driver_data_t *plat_driver_data) ...@@ -147,9 +147,10 @@ void gicv3_driver_init(const gicv3_driver_data_t *plat_driver_data)
* enabled. When the secondary CPU boots up, it initializes the * enabled. When the secondary CPU boots up, it initializes the
* GICC/GICR interface with the caches disabled. Hence flush the * GICC/GICR interface with the caches disabled. Hence flush the
* driver data to ensure coherency. This is not required if the * driver data to ensure coherency. This is not required if the
* platform has HW_ASSISTED_COHERENCY enabled. * platform has HW_ASSISTED_COHERENCY or WARMBOOT_ENABLE_DCACHE_EARLY
* enabled.
*/ */
#if !HW_ASSISTED_COHERENCY #if !(HW_ASSISTED_COHERENCY || WARMBOOT_ENABLE_DCACHE_EARLY)
flush_dcache_range((uintptr_t) &gicv3_driver_data, flush_dcache_range((uintptr_t) &gicv3_driver_data,
sizeof(gicv3_driver_data)); sizeof(gicv3_driver_data));
flush_dcache_range((uintptr_t) gicv3_driver_data, flush_dcache_range((uintptr_t) gicv3_driver_data,
......
...@@ -267,7 +267,7 @@ static plat_local_state_t *psci_get_req_local_pwr_states(unsigned int pwrlvl, ...@@ -267,7 +267,7 @@ static plat_local_state_t *psci_get_req_local_pwr_states(unsigned int pwrlvl,
static plat_local_state_t get_non_cpu_pd_node_local_state( static plat_local_state_t get_non_cpu_pd_node_local_state(
unsigned int parent_idx) unsigned int parent_idx)
{ {
#if !USE_COHERENT_MEM || !HW_ASSISTED_COHERENCY #if !(USE_COHERENT_MEM || HW_ASSISTED_COHERENCY)
flush_dcache_range( flush_dcache_range(
(uintptr_t) &psci_non_cpu_pd_nodes[parent_idx], (uintptr_t) &psci_non_cpu_pd_nodes[parent_idx],
sizeof(psci_non_cpu_pd_nodes[parent_idx])); sizeof(psci_non_cpu_pd_nodes[parent_idx]));
...@@ -283,7 +283,7 @@ static void set_non_cpu_pd_node_local_state(unsigned int parent_idx, ...@@ -283,7 +283,7 @@ static void set_non_cpu_pd_node_local_state(unsigned int parent_idx,
plat_local_state_t state) plat_local_state_t state)
{ {
psci_non_cpu_pd_nodes[parent_idx].local_state = state; psci_non_cpu_pd_nodes[parent_idx].local_state = state;
#if !USE_COHERENT_MEM || !HW_ASSISTED_COHERENCY #if !(USE_COHERENT_MEM || HW_ASSISTED_COHERENCY)
flush_dcache_range( flush_dcache_range(
(uintptr_t) &psci_non_cpu_pd_nodes[parent_idx], (uintptr_t) &psci_non_cpu_pd_nodes[parent_idx],
sizeof(psci_non_cpu_pd_nodes[parent_idx])); sizeof(psci_non_cpu_pd_nodes[parent_idx]));
......
...@@ -12,7 +12,7 @@ COLD_BOOT_SINGLE_CPU := 1 ...@@ -12,7 +12,7 @@ COLD_BOOT_SINGLE_CPU := 1
PROGRAMMABLE_RESET_ADDRESS:= 1 PROGRAMMABLE_RESET_ADDRESS:= 1
# System coherency is managed in hardware # System coherency is managed in hardware
HW_ASSISTED_COHERENCY := 1 WARMBOOT_ENABLE_DCACHE_EARLY:= 1
USE_COHERENT_MEM := 0 USE_COHERENT_MEM := 0
ERROR_DEPRECATED := 1 ERROR_DEPRECATED := 1
......
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