diff options
| author | Paul Menzel <paulepanter@users.sourceforge.net> | 2017-09-09 11:26:24 +0200 |
|---|---|---|
| committer | Felix Held <felix-coreboot@felixheld.de> | 2018-04-29 18:26:46 +0000 |
| commit | 621abec1e842acd69d122b2b451b35b14d3c5049 (patch) | |
| tree | 5f16b89ad8b5581bd779c7cbe7883d49a4a11359 /src | |
| parent | 03f2536f9d4f9bbdec898deebd8fa7df3b33fabc (diff) | |
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')
| -rw-r--r-- | src/southbridge/amd/cimx/sb800/lpc.c | 6 |
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 */ |