fix(plat/css): fix invalid redistributor poweroff

Commit 4d8c18196378824e388cf31ef991ba8fbbb09cbf
introduced an invalid redistributor power off
where we turn off the redistributor without
checking if the system power domain level is
turning off, otherwise we can turn off a
redistributor when other cores or clusters are
sharing it, also if it does indeed needs
powering off during suspend we do it twice.
This change fixes this by checking on the
system power state first then turning off
the redistributor.

Signed-off-by: Waleed Elmelegy <waleed.elmelegy@arm.com>
Change-Id: Id202bc2316ab7c516298fa33ea089ae2e221a933

diff --git a/plat/arm/css/common/css_pm.c b/plat/arm/css/common/css_pm.c
index 9b2639c3541dfb28886765f44f66914281e8fb05..3222226f7123b5d8c469546f714c5724dcd5b7be 100644 (file)
--- a/plat/arm/css/common/css_pm.c
+++ b/plat/arm/css/common/css_pm.c
@@ -130,9 +130,6 @@ static void css_power_down_common(const psci_power_state_t *target_state)
        /* Prevent interrupts from spuriously waking up this cpu */
        plat_arm_gic_cpuif_disable();
 
-       /* Turn redistributor off */
-       plat_arm_gic_redistif_off();
-
        /* Cluster is to be turned off, so disable coherency */
        if (CSS_CLUSTER_PWR_STATE(target_state) == ARM_LOCAL_STATE_OFF) {
                plat_arm_interconnect_exit_coherency();