path: root/plat/rockchip
diff options
authorJustin Chadwell <justin.chadwell@arm.com>2019-09-17 15:21:50 +0100
committerMadhukar Pappireddy <madhukar.pappireddy@arm.com>2019-11-19 08:53:16 -0600
commitb7f6525db642d533327403ff8130eb91190f2e98 (patch)
treefcca63549f0f8c7ab15e706c6995de38aa2290d0 /plat/rockchip
parentb8baa9346aaad24f8da48e408e0dbd5e65619905 (diff)
Enable -Wshadow always
Variable shadowing is, according to the C standard, permitted and valid behaviour. However, allowing a local variable to take the same name as a global one can cause confusion and can make refactoring and bug hunting more difficult. This patch moves -Wshadow from WARNING2 into the general warning group so it is always used. It also fixes all warnings that this introduces by simply renaming the local variable to a new name Change-Id: I6b71bdce6580c6e58b5e0b41e4704ab0aa38576e Signed-off-by: Justin Chadwell <justin.chadwell@arm.com>
Diffstat (limited to 'plat/rockchip')
1 files changed, 3 insertions, 3 deletions
diff --git a/plat/rockchip/rk3399/drivers/dram/dfs.c b/plat/rockchip/rk3399/drivers/dram/dfs.c
index 3b627d287f..816372bfcd 100644
--- a/plat/rockchip/rk3399/drivers/dram/dfs.c
+++ b/plat/rockchip/rk3399/drivers/dram/dfs.c
@@ -1,5 +1,5 @@
- * Copyright (c) 2016, ARM Limited and Contributors. All rights reserved.
+ * Copyright (c) 2016-2019, ARM Limited and Contributors. All rights reserved.
* SPDX-License-Identifier: BSD-3-Clause
@@ -78,10 +78,10 @@ static struct rk3399_sdram_default_config lpddr4_default_config = {
.zqcsi = 0
-static uint32_t get_cs_die_capability(struct rk3399_sdram_params *sdram_config,
+static uint32_t get_cs_die_capability(struct rk3399_sdram_params *ram_config,
uint8_t channel, uint8_t cs)
- struct rk3399_sdram_channel *ch = &sdram_config->ch[channel];
+ struct rk3399_sdram_channel *ch = &ram_config->ch[channel];
uint32_t bandwidth;
uint32_t die_bandwidth;
uint32_t die;