summaryrefslogtreecommitdiff
path: root/src/southbridge/amd
diff options
context:
space:
mode:
authorPaul Menzel <paulepanter@users.sourceforge.net>2017-09-09 11:26:24 +0200
committerFelix Held <felix-coreboot@felixheld.de>2018-04-29 18:26:46 +0000
commit621abec1e842acd69d122b2b451b35b14d3c5049 (patch)
tree5f16b89ad8b5581bd779c7cbe7883d49a4a11359 /src/southbridge/amd
parent03f2536f9d4f9bbdec898deebd8fa7df3b33fabc (diff)
downloadcoreboot-621abec1e842acd69d122b2b451b35b14d3c5049.tar.xz
sb/amd/cimx/sb800: Remove unused variable
scan-build from Clang 4.0.1-3 from Debian Sid/unstable warns about the issue below. ``` CC ramstage/southbridge/amd/cimx/sb800/lpc.o src/southbridge/amd/cimx/sb800/lpc.c:102:6: warning: Value stored to 'end' is never read end = resource_end(res); ^ ~~~~~~~~~~~~~~~~~ 1 warning generated. ``` The variable is only used in the commented out print statement. So, remove the unused variable, and directly use the value directly in the print statement. Change-Id: I3f759f6361ffeb07980cb10e17930e11d738a6a7 Signed-off-by: Paul Menzel <paulepanter@users.sourceforge.net> Reviewed-on: https://review.coreboot.org/21463 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Felix Held <felix-coreboot@felixheld.de>
Diffstat (limited to 'src/southbridge/amd')
-rw-r--r--src/southbridge/amd/cimx/sb800/lpc.c6
1 files changed, 3 insertions, 3 deletions
diff --git a/src/southbridge/amd/cimx/sb800/lpc.c b/src/southbridge/amd/cimx/sb800/lpc.c
index 80d721375e..49735583cc 100644
--- a/src/southbridge/amd/cimx/sb800/lpc.c
+++ b/src/southbridge/amd/cimx/sb800/lpc.c
@@ -95,15 +95,15 @@ void lpc_enable_childrens_resources(device_t dev)
&& (child->path.type == DEVICE_PATH_PNP)) {
struct resource *res;
for (res = child->resource_list; res; res = res->next) {
- u32 base, end; /* don't need long long */
+ u32 base; /* don't need long long */
if (!(res->flags & IORESOURCE_IO))
continue;
base = res->base;
- end = resource_end(res);
/*
printk(BIOS_DEBUG, "sb800 lpc decode:%s,
base=0x%08x, end=0x%08x\n",
- dev_path(child), base, end);
+ dev_path(child), base,
+ resource_end(res));
*/
switch (base) {
case 0x60: /* KB */