diff options
| author | Paul Menzel <paulepanter@users.sourceforge.net> | 2013-10-02 21:59:38 +0200 |
|---|---|---|
| committer | Aaron Durbin <adurbin@google.com> | 2013-10-03 15:17:05 +0200 |
| commit | 11b47801b2c3abc3bb8c523b722fafa1103bff45 (patch) | |
| tree | 580dd65d189db666a04f88f82b5ae0e9a294a0d8 /src/cpu/x86/mtrr | |
| parent | b142a5154280c00a3e4bc9d162b31bfe4b665f60 (diff) | |
| download | coreboot-11b47801b2c3abc3bb8c523b722fafa1103bff45.tar.gz coreboot-11b47801b2c3abc3bb8c523b722fafa1103bff45.tar.bz2 coreboot-11b47801b2c3abc3bb8c523b722fafa1103bff45.zip | |
cpu/x86/mtrr/mtrr.c: Remove superfluous assignment to `type_index`
When building coreboot with the Clang static analyzer scan-build,
it reports »Value stored to 'type_index' is never read«. Indeed,
in `memranges_each_entry()` `type_index` is assigned a value
before being read. So remove that line.
Change-Id: I6da2fb8be7157bb98c57281babd4a08ca0d9f7a7
Signed-off-by: Paul Menzel <paulepanter@users.sourceforge.net>
Reviewed-on: http://review.coreboot.org/3953
Tested-by: build bot (Jenkins)
Reviewed-by: Aaron Durbin <adurbin@google.com>
Diffstat (limited to 'src/cpu/x86/mtrr')
| -rw-r--r-- | src/cpu/x86/mtrr/mtrr.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/src/cpu/x86/mtrr/mtrr.c b/src/cpu/x86/mtrr/mtrr.c index 8f1c35ec6eb3..d168978b540e 100644 --- a/src/cpu/x86/mtrr/mtrr.c +++ b/src/cpu/x86/mtrr/mtrr.c @@ -261,7 +261,6 @@ static void calc_fixed_mtrrs(void) desc = &fixed_mtrr_desc[0]; last_desc = &fixed_mtrr_desc[ARRAY_SIZE(fixed_mtrr_desc) - 1]; - type_index = desc->range_index; memranges_each_entry(r, phys_addr_space) { begin = range_entry_base_mtrr_addr(r); |