Change in ...coreboot[master]: nb/amd/amdfam10/northbridge.c: Remove variable set but not used

[HAOUAS Elyes (Code Review)]

HAOUAS Elyes has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/33271

Change subject: nb/amd/amdfam10/northbridge.c: Remove variable set but not used
......................................................................

nb/amd/amdfam10/northbridge.c: Remove variable set but not used

Change-Id: I02f37e6502f20747c2d61ffa20fec8bdcc08ca32
Signed-off-by: Elyes HAOUAS 
---
M src/northbridge/amd/amdfam10/northbridge.c
1 file changed, 0 insertions(+), 12 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/71/33271/1

diff --git a/src/northbridge/amd/amdfam10/northbridge.c b/src/northbridge/amd/amdfam10/northbridge.c
index a681961..4264fc3 100644
--- a/src/northbridge/amd/amdfam10/northbridge.c
+++ b/src/northbridge/amd/amdfam10/northbridge.c
@@ -209,15 +209,6 @@
    		*/
    		f3xe8 = pci_read_config32(get_node_pci(nodeid, 3), 0xe8);
    		uint8_t internal_node_number = ((f3xe8 & 0xc0000000) >> 30);
-			uint8_t defective_link_number_1;
-			uint8_t defective_link_number_2;
-			if (is_fam15h()) {
-				defective_link_number_1 = 4;	/* Link 0 Sublink 1 */
-				defective_link_number_2 = 7;	/* Link 3 Sublink 1 */
-			} else {
-				defective_link_number_1 = 6;	/* Link 2 Sublink 1 */
-				defective_link_number_2 = 5;	/* Link 1 Sublink 1 */
-			}
    		if (internal_node_number == 0) {
    			/* Node 0 */
    			if (link->link_num == 6)	/* Link 2 Sublink 1 */
@@ -745,7 +736,6 @@
    		uint8_t node;
    		uint8_t interleaved;
    		int8_t range;
-			int8_t max_range;
    		uint8_t max_node;
    		uint64_t max_range_limit;
    		uint32_t dword;
@@ -756,7 +746,6 @@
    		/* Find highest DRAM range (DramLimitAddr) */
    		num_nodes = 0;
    		max_node = 0;
-			max_range = -1;
    		interleaved = 0;
    		max_range_limit = 0;
    		struct device *node_dev;
@@ -782,7 +771,6 @@
    				qword |= (((uint64_t)dword2) & 0xff) << 40;

if (qword > max_range_limit) {
-						max_range = range;
    					max_range_limit = qword;
    					max_node = dword & 0x7;
    				}

[HAOUAS Elyes (Code Review)]

HAOUAS Elyes has abandoned this change. ( https://review.coreboot.org/c/coreboot/+/33271 )

Change subject: nb/amd/amdfam10/northbridge.c: Remove variable set but not used
......................................................................

Abandoned

please see https://review.coreboot.org/c/coreboot/+/33013