diff options
| author | Nariman Poushin <nariman.poushin@linaro.org> | 2018-05-01 09:28:40 +0100 |
|---|---|---|
| committer | Nariman Poushin <nariman.poushin@linaro.org> | 2018-06-23 17:22:08 +0100 |
| commit | 9b4c611c4b05fdf03722d2de488ea49beb10a8b7 (patch) | |
| tree | e525e6796bbf2c73574c2f13e4939a40993d1f8f /plat | |
| parent | b70dcbc17222687859ac1cf58d4d7c5cb5c379fe (diff) | |
| download | platform_external_arm-trusted-firmware-9b4c611c4b05fdf03722d2de488ea49beb10a8b7.tar.gz platform_external_arm-trusted-firmware-9b4c611c4b05fdf03722d2de488ea49beb10a8b7.tar.bz2 platform_external_arm-trusted-firmware-9b4c611c4b05fdf03722d2de488ea49beb10a8b7.zip | |
plat/arm: css: Convert CSS_SYSTEM_PWR_STATE from macro to inline function
This is to fix a number of Coverity Scan DEADCODE defects, CID numbers
listed below, as reported from
https://scan.coverity.com/projects/arm-software-arm-trusted-firmware
CID 267023
CID 267022
CID 267020
Change-Id: I2963a799b210149e84ccab5c5b9082267ddfe337
Signed-off-by: Nariman Poushin <nariman.poushin@linaro.org>
Diffstat (limited to 'plat')
| -rw-r--r-- | plat/arm/css/common/css_pm.c | 6 | ||||
| -rw-r--r-- | plat/arm/css/drivers/scp/css_pm_scmi.c | 2 | ||||
| -rw-r--r-- | plat/arm/css/drivers/scp/css_pm_scpi.c | 2 |
3 files changed, 5 insertions, 5 deletions
diff --git a/plat/arm/css/common/css_pm.c b/plat/arm/css/common/css_pm.c index 3df5b7813..986cb0846 100644 --- a/plat/arm/css/common/css_pm.c +++ b/plat/arm/css/common/css_pm.c @@ -96,7 +96,7 @@ static void css_pwr_domain_on_finisher_common( void css_pwr_domain_on_finish(const psci_power_state_t *target_state) { /* Assert that the system power domain need not be initialized */ - assert(CSS_SYSTEM_PWR_STATE(target_state) == ARM_LOCAL_STATE_RUN); + assert(css_system_pwr_state(target_state) == ARM_LOCAL_STATE_RUN); /* Program the gic per-cpu distributor or re-distributor interface */ plat_arm_gic_pcpu_init(); @@ -149,7 +149,7 @@ void css_pwr_domain_suspend(const psci_power_state_t *target_state) css_power_down_common(target_state); /* Perform system domain state saving if issuing system suspend */ - if (CSS_SYSTEM_PWR_STATE(target_state) == ARM_LOCAL_STATE_OFF) { + if (css_system_pwr_state(target_state) == ARM_LOCAL_STATE_OFF) { arm_system_pwr_domain_save(); /* Power off the Redistributor after having saved its context */ @@ -174,7 +174,7 @@ void css_pwr_domain_suspend_finish( return; /* Perform system domain restore if woken up from system suspend */ - if (CSS_SYSTEM_PWR_STATE(target_state) == ARM_LOCAL_STATE_OFF) + if (css_system_pwr_state(target_state) == ARM_LOCAL_STATE_OFF) /* * At this point, the Distributor must be powered on to be ready * to have its state restored. The Redistributor will be powered diff --git a/plat/arm/css/drivers/scp/css_pm_scmi.c b/plat/arm/css/drivers/scp/css_pm_scmi.c index 3a255096e..91ea63a44 100644 --- a/plat/arm/css/drivers/scp/css_pm_scmi.c +++ b/plat/arm/css/drivers/scp/css_pm_scmi.c @@ -87,7 +87,7 @@ void css_scp_suspend(const struct psci_power_state *target_state) ARM_LOCAL_STATE_OFF); /* Check if power down at system power domain level is requested */ - if (CSS_SYSTEM_PWR_STATE(target_state) == ARM_LOCAL_STATE_OFF) { + if (css_system_pwr_state(target_state) == ARM_LOCAL_STATE_OFF) { /* Issue SCMI command for SYSTEM_SUSPEND */ ret = scmi_sys_pwr_state_set(scmi_handle, SCMI_SYS_PWR_FORCEFUL_REQ, diff --git a/plat/arm/css/drivers/scp/css_pm_scpi.c b/plat/arm/css/drivers/scp/css_pm_scpi.c index 23fd80ee9..18e71f6e3 100644 --- a/plat/arm/css/drivers/scp/css_pm_scpi.c +++ b/plat/arm/css/drivers/scp/css_pm_scpi.c @@ -25,7 +25,7 @@ void css_scp_suspend(const struct psci_power_state *target_state) uint32_t system_state = scpi_power_on; /* Check if power down at system power domain level is requested */ - if (CSS_SYSTEM_PWR_STATE(target_state) == ARM_LOCAL_STATE_OFF) + if (css_system_pwr_state(target_state) == ARM_LOCAL_STATE_OFF) system_state = scpi_power_retention; /* Cluster is to be turned off, so disable coherency */ |