From 99eab34b9ed25065a0b4c3ebd051aa0155c77e6c Mon Sep 17 00:00:00 2001 From: Arthur Heymans Date: Wed, 23 Mar 2022 21:34:20 +0100 Subject: amd/fam*/northbridge.c: Remove unused reset_memhole variable Change-Id: I9231e0399d0b3ac6a608282571fc6d4aefad9dfb Signed-off-by: Arthur Heymans Reviewed-on: https://review.coreboot.org/c/coreboot/+/63042 Tested-by: build bot (Jenkins) Reviewed-by: Elyes Haouas --- src/northbridge/amd/agesa/family14/northbridge.c | 5 +---- src/northbridge/amd/agesa/family15tn/northbridge.c | 5 +---- src/northbridge/amd/agesa/family16kb/northbridge.c | 5 +---- 3 files changed, 3 insertions(+), 12 deletions(-) (limited to 'src/northbridge/amd/agesa') diff --git a/src/northbridge/amd/agesa/family14/northbridge.c b/src/northbridge/amd/agesa/family14/northbridge.c index 77e68d8b24..3507ff709b 100644 --- a/src/northbridge/amd/agesa/family14/northbridge.c +++ b/src/northbridge/amd/agesa/family14/northbridge.c @@ -510,7 +510,6 @@ static void domain_set_resources(struct device *dev) 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; @@ -539,10 +538,8 @@ static void domain_set_resources(struct device *dev) mem_hole = get_hw_mem_hole_info(); // 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)) { + if ((mem_hole.node_id != -1) && (mmio_basek > mem_hole.hole_startk)) mmio_basek = mem_hole.hole_startk; - reset_memhole = 0; - } #endif idx = 0x10; diff --git a/src/northbridge/amd/agesa/family15tn/northbridge.c b/src/northbridge/amd/agesa/family15tn/northbridge.c index 65be7d8a72..d36c9d459d 100644 --- a/src/northbridge/amd/agesa/family15tn/northbridge.c +++ b/src/northbridge/amd/agesa/family15tn/northbridge.c @@ -668,7 +668,6 @@ static void domain_set_resources(struct device *dev) 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; @@ -697,10 +696,8 @@ static void domain_set_resources(struct device *dev) mem_hole = get_hw_mem_hole_info(); // 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)) { + if ((mem_hole.node_id != -1) && (mmio_basek > mem_hole.hole_startk)) mmio_basek = mem_hole.hole_startk; - reset_memhole = 0; - } #endif idx = 0x10; diff --git a/src/northbridge/amd/agesa/family16kb/northbridge.c b/src/northbridge/amd/agesa/family16kb/northbridge.c index 1c630923c7..170a0d2ee2 100644 --- a/src/northbridge/amd/agesa/family16kb/northbridge.c +++ b/src/northbridge/amd/agesa/family16kb/northbridge.c @@ -680,7 +680,6 @@ static void domain_set_resources(struct device *dev) 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; @@ -709,10 +708,8 @@ static void domain_set_resources(struct device *dev) mem_hole = get_hw_mem_hole_info(); // 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)) { + if ((mem_hole.node_id != -1) && (mmio_basek > mem_hole.hole_startk)) mmio_basek = mem_hole.hole_startk; - reset_memhole = 0; - } #endif idx = 0x10; -- cgit v1.2.3