Commit ba592e28 authored by Sandrine Bailleux's avatar Sandrine Bailleux
Browse files

Fix violations to the coding style

All coding style violations have been fixed in a previous patch and
since then, each individual patch has been checked in this regard.
However, the latest version of the checkpatch.pl script from the Linux
kernel is more advanced and it is able to flag new errors in the
Trusted Firmware codebase. This patch fixes them.

Change-Id: I1f332f2440984be85d36b231bb83260368987077
parent 95d5353c
...@@ -38,7 +38,7 @@ static void unsigned_num_print(unsigned long int unum, unsigned int radix) ...@@ -38,7 +38,7 @@ static void unsigned_num_print(unsigned long int unum, unsigned int radix)
{ {
/* Just need enough space to store 64 bit decimal integer */ /* Just need enough space to store 64 bit decimal integer */
unsigned char num_buf[20]; unsigned char num_buf[20];
int i = 0 , rem; int i = 0, rem;
do { do {
rem = unum % radix; rem = unum % radix;
......
...@@ -64,7 +64,7 @@ static void fvp_program_mailbox(uint64_t mpidr, uint64_t address) ...@@ -64,7 +64,7 @@ static void fvp_program_mailbox(uint64_t mpidr, uint64_t address)
* Function which implements the common FVP specific operations to power down a * Function which implements the common FVP specific operations to power down a
* cpu in response to a CPU_OFF or CPU_SUSPEND request. * cpu in response to a CPU_OFF or CPU_SUSPEND request.
******************************************************************************/ ******************************************************************************/
static void fvp_cpu_pwrdwn_common() static void fvp_cpu_pwrdwn_common(void)
{ {
/* Prevent interrupts from spuriously waking up this cpu */ /* Prevent interrupts from spuriously waking up this cpu */
arm_gic_cpuif_deactivate(); arm_gic_cpuif_deactivate();
...@@ -77,7 +77,7 @@ static void fvp_cpu_pwrdwn_common() ...@@ -77,7 +77,7 @@ static void fvp_cpu_pwrdwn_common()
* Function which implements the common FVP specific operations to power down a * Function which implements the common FVP specific operations to power down a
* cluster in response to a CPU_OFF or CPU_SUSPEND request. * cluster in response to a CPU_OFF or CPU_SUSPEND request.
******************************************************************************/ ******************************************************************************/
static void fvp_cluster_pwrdwn_common() static void fvp_cluster_pwrdwn_common(void)
{ {
uint64_t mpidr = read_mpidr_el1(); uint64_t mpidr = read_mpidr_el1();
......
...@@ -182,7 +182,7 @@ void bl31_platform_setup(void) ...@@ -182,7 +182,7 @@ void bl31_platform_setup(void)
* Perform the very early platform specific architectural setup here. At the * Perform the very early platform specific architectural setup here. At the
* moment this is only intializes the mmu in a quick and dirty way. * moment this is only intializes the mmu in a quick and dirty way.
******************************************************************************/ ******************************************************************************/
void bl31_plat_arch_setup() void bl31_plat_arch_setup(void)
{ {
configure_mmu_el3(BL31_RO_BASE, configure_mmu_el3(BL31_RO_BASE,
(BL31_END - BL31_RO_BASE), (BL31_END - BL31_RO_BASE),
......
...@@ -48,7 +48,7 @@ unsigned int plat_get_aff_state(unsigned int aff_lvl, unsigned long mpidr) ...@@ -48,7 +48,7 @@ unsigned int plat_get_aff_state(unsigned int aff_lvl, unsigned long mpidr)
return aff_lvl <= MPIDR_AFFLVL1 ? PSCI_AFF_PRESENT : PSCI_AFF_ABSENT; return aff_lvl <= MPIDR_AFFLVL1 ? PSCI_AFF_PRESENT : PSCI_AFF_ABSENT;
} }
int plat_setup_topology() int plat_setup_topology(void)
{ {
/* Juno todo: Make topology configurable via SCC */ /* Juno todo: Make topology configurable via SCC */
return 0; return 0;
......
...@@ -58,7 +58,7 @@ void psci_set_suspend_power_state(unsigned int power_state) ...@@ -58,7 +58,7 @@ void psci_set_suspend_power_state(unsigned int power_state)
* powered down during a cpu_suspend call. Returns PSCI_INVALID_DATA if the * powered down during a cpu_suspend call. Returns PSCI_INVALID_DATA if the
* power state is invalid. * power state is invalid.
******************************************************************************/ ******************************************************************************/
int psci_get_suspend_afflvl() int psci_get_suspend_afflvl(void)
{ {
unsigned int power_state; unsigned int power_state;
...@@ -73,7 +73,7 @@ int psci_get_suspend_afflvl() ...@@ -73,7 +73,7 @@ int psci_get_suspend_afflvl()
* parameter saved in the per-cpu data array. Returns PSCI_INVALID_DATA if the * parameter saved in the per-cpu data array. Returns PSCI_INVALID_DATA if the
* power state saved is invalid. * power state saved is invalid.
******************************************************************************/ ******************************************************************************/
int psci_get_suspend_stateid() int psci_get_suspend_stateid(void)
{ {
unsigned int power_state; unsigned int power_state;
......
...@@ -133,7 +133,7 @@ uint32_t psci_get_max_phys_off_afflvl(void) ...@@ -133,7 +133,7 @@ uint32_t psci_get_max_phys_off_afflvl(void)
* been physically powered up. It is expected to be called immediately after * been physically powered up. It is expected to be called immediately after
* reset from assembler code. * reset from assembler code.
******************************************************************************/ ******************************************************************************/
int get_power_on_target_afflvl() int get_power_on_target_afflvl(void)
{ {
int afflvl; int afflvl;
......
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