[coreboot-gerrit] Change in ...coreboot[master]: nb/amd/pi/00630F01/northbridge.c: Remove variable set but not used

[Patrick Georgi (Code Review)]

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

Change subject: nb/amd/pi/00630F01/northbridge.c: Remove variable set but not used
......................................................................

nb/amd/pi/00630F01/northbridge.c: Remove variable set but not used

Change-Id: Id5e762880ddfcb65872a50e8ffe10d86b3719b5d
Signed-off-by: Elyes HAOUAS 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/33012
Tested-by: build bot (Jenkins) 
Reviewed-by: Marshall Dawson 
---
M src/northbridge/amd/pi/00630F01/northbridge.c
1 file changed, 0 insertions(+), 4 deletions(-)

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

diff --git a/src/northbridge/amd/pi/00630F01/northbridge.c b/src/northbridge/amd/pi/00630F01/northbridge.c
index eca2f0c..309a762 100644
--- a/src/northbridge/amd/pi/00630F01/northbridge.c
+++ b/src/northbridge/amd/pi/00630F01/northbridge.c
@@ -463,11 +463,9 @@
    acpi_header_t *ssdt;
    acpi_header_t *alib;
    acpi_header_t *ivrs;
-	acpi_hest_t *hest;

/* HEST */
    current = ALIGN(current, 8);
-	hest = (acpi_hest_t *)current;
    acpi_write_hest((void *)current, acpi_fill_hest);
    acpi_add_table(rsdp, (void *)current);
    current += ((acpi_header_t *)current)->length;
@@ -680,7 +678,6 @@
    struct bus *link;
 #if CONFIG_HW_MEM_HOLE_SIZEK != 0
    struct hw_mem_hole_info mem_hole;
-	u32 reset_memhole = 1;
 #endif

pci_tolm = 0xffffffffUL;
@@ -711,7 +708,6 @@
    // Use hole_basek as mmio_basek, and we don't need to reset hole anymore
    if ((mem_hole.node_id !=  -1) && (mmio_basek > mem_hole.hole_startk)) {
    	mmio_basek = mem_hole.hole_startk;
-		reset_memhole = 0;
    }
 #endif