From 621abec1e842acd69d122b2b451b35b14d3c5049 Mon Sep 17 00:00:00 2001 From: Paul Menzel Date: Sat, 9 Sep 2017 11:26:24 +0200 Subject: 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 Reviewed-on: https://review.coreboot.org/21463 Tested-by: build bot (Jenkins) Reviewed-by: Felix Held --- src/southbridge/amd/cimx/sb800/lpc.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'src') 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 */ -- cgit v1.2.3