Change in ...coreboot[master]: cpu/amd/quadcore: Remove variable set but not used

[HAOUAS Elyes (Code Review)]

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

Change subject: cpu/amd/quadcore: Remove variable set but not used
......................................................................

cpu/amd/quadcore: Remove variable set but not used

Change-Id: I73f35ea80976ab445c797c4800b1e2fd24d34fdf
Signed-off-by: Elyes HAOUAS 
---
M src/cpu/amd/quadcore/quadcore.c
1 file changed, 3 insertions(+), 3 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/93/32893/1

diff --git a/src/cpu/amd/quadcore/quadcore.c b/src/cpu/amd/quadcore/quadcore.c
index ad4a7ea..125876b 100644
--- a/src/cpu/amd/quadcore/quadcore.c
+++ b/src/cpu/amd/quadcore/quadcore.c
@@ -95,13 +95,13 @@
    	 */

/* Wait for the first core of each compute unit to start... */
-		uint32_t timeout;
    	for (i = 1; i < cores + 1; i++) {
    		if (!(i & 0x1)) {
    			uint32_t ap_apicid =
    			get_boot_apic_id(nodeid, i);
-				timeout = wait_cpu_state(ap_apicid,
-							F10_APSTATE_ASLEEP, F10_APSTATE_ASLEEP);
+				/* Timeout */
+				wait_cpu_state(ap_apicid, F10_APSTATE_ASLEEP,
+						F10_APSTATE_ASLEEP);
    		}
    	}

[Marshall Dawson (Code Review)]

Marshall Dawson has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32893 )

Change subject: cpu/amd/quadcore: Remove variable set but not used
......................................................................

Patch Set 3: Code-Review+2

[Patrick Georgi (Code Review)]

Patrick Georgi has submitted this change and it was merged. ( https://review.coreboot.org/c/coreboot/+/32893 )

Change subject: cpu/amd/quadcore: Remove variable set but not used
......................................................................

cpu/amd/quadcore: Remove variable set but not used

Change-Id: I73f35ea80976ab445c797c4800b1e2fd24d34fdf
Signed-off-by: Elyes HAOUAS 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/32893
Reviewed-by: Marshall Dawson 
Tested-by: build bot (Jenkins) 
---
M src/cpu/amd/quadcore/quadcore.c
1 file changed, 3 insertions(+), 3 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Marshall Dawson: Looks good to me, approved

diff --git a/src/cpu/amd/quadcore/quadcore.c b/src/cpu/amd/quadcore/quadcore.c
index ad4a7ea..125876b 100644
--- a/src/cpu/amd/quadcore/quadcore.c
+++ b/src/cpu/amd/quadcore/quadcore.c
@@ -95,13 +95,13 @@
    	 */

/* Wait for the first core of each compute unit to start... */
-		uint32_t timeout;
    	for (i = 1; i < cores + 1; i++) {
    		if (!(i & 0x1)) {
    			uint32_t ap_apicid =
    			get_boot_apic_id(nodeid, i);
-				timeout = wait_cpu_state(ap_apicid,
-							F10_APSTATE_ASLEEP, F10_APSTATE_ASLEEP);
+				/* Timeout */
+				wait_cpu_state(ap_apicid, F10_APSTATE_ASLEEP,
+						F10_APSTATE_ASLEEP);
    		}
    	}