[coreboot-gerrit] Patch set updated for coreboot: 1f29668 amd/amdfam10/northbridge.c: Fix FTBFS with CONFIG_PCI_64BIT_PREF_MEM

Timothy Pearson (tpearson@raptorengineeringinc.com) gerrit at coreboot.org
Sun Jan 25 05:05:16 CET 2015


Timothy Pearson (tpearson at raptorengineeringinc.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/8265

-gerrit

commit 1f29668637fa8bbb43114ba2c571ef02aa956912
Author: Timothy Pearson <tpearson at raptorengineeringinc.com>
Date:   Fri Jan 23 20:26:03 2015 -0600

    amd/amdfam10/northbridge.c: Fix FTBFS with CONFIG_PCI_64BIT_PREF_MEM
    
    Remove declaration of unused variable `io`.
    
    Change-Id: I750fc3a135f7634ad16c0f6a1a5bdb16ac702977
    Signed-off-by: Timothy Pearson <tpearson at raptorengineeringinc.com>
---
 src/northbridge/amd/amdfam10/northbridge.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/src/northbridge/amd/amdfam10/northbridge.c b/src/northbridge/amd/amdfam10/northbridge.c
index a8a8155..51cfee7 100644
--- a/src/northbridge/amd/amdfam10/northbridge.c
+++ b/src/northbridge/amd/amdfam10/northbridge.c
@@ -1,6 +1,7 @@
 /*
  * This file is part of the coreboot project.
  *
+ * Copyright (C) 2015 Timothy Pearson <tpearson at raptorengineeringinc.com>, Raptor Engineering
  * Copyright (C) 2007 Advanced Micro Devices, Inc.
  *
  * This program is free software; you can redistribute it and/or modify
@@ -858,7 +859,7 @@ static void setup_uma_memory(void)
 static void amdfam10_domain_set_resources(device_t dev)
 {
 #if CONFIG_PCI_64BIT_PREF_MEM
-	struct resource *io, *mem1, *mem2;
+	struct resource *mem1, *mem2;
 	struct resource *res;
 #endif
 	unsigned long mmio_basek;
@@ -914,7 +915,7 @@ static void amdfam10_domain_set_resources(device_t dev)
 			mem2->base, mem2->limit, mem2->size, mem2->align);
 	}
 
-	for(res = &dev->resource_list; res; res = res->next)
+	for(res = dev->resource_list; res; res = res->next)
 	{
 		res->flags |= IORESOURCE_ASSIGNED;
 		res->flags |= IORESOURCE_STORED;



More information about the coreboot-gerrit mailing list