From 67ed2612003577a35ee60ec8aee8763a01637598 Mon Sep 17 00:00:00 2001 From: Stefan Reinauer Date: Tue, 27 Jun 2017 00:36:15 +0200 Subject: amd/amdfam10: Remove dead code MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 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 Reviewed-on: https://review.coreboot.org/20385 Tested-by: build bot (Jenkins) Reviewed-by: Arthur Heymans Reviewed-by: Paul Menzel Reviewed-by: Philippe Mathieu-Daudé Reviewed-by: Patrick Georgi --- src/northbridge/amd/amdfam10/northbridge.c | 7 ------- 1 file changed, 7 deletions(-) (limited to 'src') 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; -- cgit v1.2.3