fix(psci): add missing curly braces
This corrects the MISRA violation C2012-15.6:
The body of an iteration-statement or a selection-statement shall
be a compound-statement.
Enclosed statement body within the curly braces.
Change-Id: I8b656f59b445e914dd3f47e3dde83735481a3640
Signed-off-by: Nithin G <nithing@amd.com>
Signed-off-by: Maheedhar Bollapalli <maheedharsai.bollapalli@amd.com>
diff --git a/lib/psci/psci_common.c b/lib/psci/psci_common.c
index 4c2601e..234a195 100644
--- a/lib/psci/psci_common.c
+++ b/lib/psci/psci_common.c
@@ -138,9 +138,9 @@
psci_power_state_t *state_info)
{
/* Check SBZ bits in power state are zero */
- if (psci_check_power_state(power_state) != 0U)
+ if (psci_check_power_state(power_state) != 0U) {
return PSCI_E_INVALID_PARAMS;
-
+ }
assert(psci_plat_pm_ops->validate_power_state != NULL);
/* Validate the power_state using platform pm_ops */
@@ -439,8 +439,9 @@
}
/* Set the the higher levels to RUN */
- for (; lvl <= PLAT_MAX_PWR_LVL; lvl++)
+ for (; lvl <= PLAT_MAX_PWR_LVL; lvl++) {
target_state->pwr_domain_state[lvl] = PSCI_LOCAL_STATE_RUN;
+ }
}
/******************************************************************************
@@ -574,8 +575,9 @@
state_info->pwr_domain_state[lvl] = target_state;
/* Break early if the negotiated target power state is RUN */
- if (is_local_state_run(state_info->pwr_domain_state[lvl]) != 0)
+ if (is_local_state_run(state_info->pwr_domain_state[lvl]) != 0) {
break;
+ }
parent_idx = psci_non_cpu_pd_nodes[parent_idx].parent_node;
}
@@ -757,8 +759,9 @@
int i;
for (i = (int) PLAT_MAX_PWR_LVL; i >= (int) PSCI_CPU_PWR_LVL; i--) {
- if (is_local_state_off(state_info->pwr_domain_state[i]) != 0)
+ if (is_local_state_off(state_info->pwr_domain_state[i]) != 0) {
return (unsigned int) i;
+ }
}
return PSCI_INVALID_PWR_LVL;
@@ -942,8 +945,9 @@
/* Validate the entrypoint using platform psci_ops */
if (psci_plat_pm_ops->validate_ns_entrypoint != NULL) {
rc = psci_plat_pm_ops->validate_ns_entrypoint(entrypoint);
- if (rc != PSCI_E_SUCCESS)
+ if (rc != PSCI_E_SUCCESS) {
return PSCI_E_INVALID_ADDRESS;
+ }
}
/*
@@ -1017,9 +1021,9 @@
* of power management handler and perform the generic, architecture
* and platform specific handling.
*/
- if (psci_get_aff_info_state() == AFF_STATE_ON_PENDING)
+ if (psci_get_aff_info_state() == AFF_STATE_ON_PENDING) {
psci_cpu_on_finish(cpu_idx, &state_info);
- else {
+ } else {
unsigned int max_off_lvl = psci_find_max_off_lvl(&state_info);
assert(max_off_lvl != PSCI_INVALID_PWR_LVL);