diff options
| author | Stefan Reinauer <stefan.reinauer@coreboot.org> | 2017-06-27 00:36:15 +0200 |
|---|---|---|
| committer | Stefan Reinauer <stefan.reinauer@coreboot.org> | 2017-06-27 17:02:49 +0000 |
| commit | 67ed2612003577a35ee60ec8aee8763a01637598 (patch) | |
| tree | 1f3d9f6bc335d9f2c493f3cb13ce66dfb1d65b1c | |
| parent | e06e2fdce1f6124c3a31b1ba9a5a9115b25e86ff (diff) | |
amd/amdfam10: Remove dead code
dual_node is never used in that function. And it is never set
correctly either, because the register f3xe8 is never actually
read either. Just remove the whole useless construct.
Change-Id: If316da89bceae6b162f20e4b632276db2d9ef423
Signed-off-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
Reviewed-on: https://review.coreboot.org/20385
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Arthur Heymans <arthur@aheymans.xyz>
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Patrick Georgi <pgeorgi@google.com>
| -rw-r--r-- | src/northbridge/amd/amdfam10/northbridge.c | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/src/northbridge/amd/amdfam10/northbridge.c b/src/northbridge/amd/amdfam10/northbridge.c index a306d25d75..dccd9c6c6f 100644 --- a/src/northbridge/amd/amdfam10/northbridge.c +++ b/src/northbridge/amd/amdfam10/northbridge.c @@ -1693,9 +1693,7 @@ static void detect_and_enable_probe_filter(device_t dev) uint8_t fam15h = 0; uint8_t rev_gte_d = 0; - uint8_t dual_node = 0; unsigned nb_cfg_54; - uint32_t f3xe8; uint32_t family; uint32_t model; @@ -1712,11 +1710,6 @@ static void detect_and_enable_probe_filter(device_t dev) /* Revision D or later */ rev_gte_d = 1; - if (rev_gte_d) - /* Check for dual node capability */ - if (f3xe8 & 0x20000000) - dual_node = 1; - if (rev_gte_d && (sysconf.nodes > 1)) { /* Enable the probe filter */ uint8_t i; |