diff options
| author | Jonathan Wright <jonathan.wright@arm.com> | 2018-03-20 14:34:01 +0000 |
|---|---|---|
| committer | Jonathan Wright <jonathan.wright@arm.com> | 2018-03-27 13:38:08 +0100 |
| commit | 2271cb054db6387bbb9215e09ae97a41a13aa4e3 (patch) | |
| tree | 52763b4dfc5c4b241dbedc508cf6e6121935a29d /lib | |
| parent | b7229e4919044a747c500c9d082168aa032eb3bf (diff) | |
| download | platform_external_arm-trusted-firmware-2271cb054db6387bbb9215e09ae97a41a13aa4e3.tar.gz platform_external_arm-trusted-firmware-2271cb054db6387bbb9215e09ae97a41a13aa4e3.tar.bz2 platform_external_arm-trusted-firmware-2271cb054db6387bbb9215e09ae97a41a13aa4e3.zip | |
psci: initialize array fully to comply with MISRA
Initializes each element of the last_cpu_in_non_cpu_pd array in PSCI
stat implementation to -1, the reset value. This satisfies MISRA rule
9.3.
Previously, only the first element of the array was initialized to -1.
Change-Id: I666c71e6c073710c67c6d24c07a219b1feb5b773
Signed-off-by: Jonathan Wright <jonathan.wright@arm.com>
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/psci/psci_stat.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/lib/psci/psci_stat.c b/lib/psci/psci_stat.c index 3e79c5d7c..e925d34ed 100644 --- a/lib/psci/psci_stat.c +++ b/lib/psci/psci_stat.c @@ -24,7 +24,8 @@ typedef struct psci_stat { * Following is used to keep track of the last cpu * that goes to power down in non cpu power domains. */ -static int last_cpu_in_non_cpu_pd[PSCI_NUM_NON_CPU_PWR_DOMAINS] = {-1}; +static int last_cpu_in_non_cpu_pd[PSCI_NUM_NON_CPU_PWR_DOMAINS] = { + [0 ... PSCI_NUM_NON_CPU_PWR_DOMAINS-1] = -1}; /* * Following are used to store PSCI STAT values for @@ -129,6 +130,10 @@ void psci_stats_update_pwr_up(unsigned int end_pwrlvl, * prior to this CPU powering on. */ parent_idx = psci_cpu_pd_nodes[cpu_idx].parent_node; + /* Return early if this is the first power up. */ + if (last_cpu_in_non_cpu_pd[parent_idx] == -1) + return; + for (lvl = PSCI_CPU_PWR_LVL + 1; lvl <= end_pwrlvl; lvl++) { local_state = state_info->pwr_domain_state[lvl]; if (is_local_state_run(local_state)) { |