On Sun, Jul 5, 2009 at 8:36 PM, Bao, Zheng <[email protected]> wrote:

> 4392 is good. 4395 is not good. I don't exactly which patch causes this
> problem.


Thanks for testing.  I'm sorry it broke.  It looks like I forgot to add
IORESOURCE_BRIDGE for the fam10 resources, so they get skipped by the
allocator.

Could you try this patch?

Signed-off-by: Myles Watson <[email protected]>

Thanks,
Myles
Index: cbv2/src/northbridge/amd/amdfam10/northbridge.c
===================================================================
--- cbv2.orig/src/northbridge/amd/amdfam10/northbridge.c
+++ cbv2/src/northbridge/amd/amdfam10/northbridge.c
@@ -446,7 +446,7 @@ static void amdfam10_link_read_bases(dev
 		resource->align = align;
 		resource->gran	= align;
 		resource->limit = 0xffffUL;
-		resource->flags = IORESOURCE_IO;
+		resource->flags = IORESOURCE_IO | IORESOURCE_BRIDGE;
 	}
 
 	/* Initialize the prefetchable memory constraints on the current bus */
@@ -475,7 +475,7 @@ static void amdfam10_link_read_bases(dev
 		resource->align = log2(HT_MEM_HOST_ALIGN);
 		resource->gran	= log2(HT_MEM_HOST_ALIGN);
 		resource->limit = 0xffffffffffULL;
-		resource->flags = IORESOURCE_MEM;
+		resource->flags = IORESOURCE_MEM | IORESOURCE_BRIDGE;
 
 #if CONFIG_EXT_CONF_SUPPORT == 1
 		if((resource->index & 0x1fff) == 0x1110) { // ext
-- 
coreboot mailing list: [email protected]
http://www.coreboot.org/mailman/listinfo/coreboot

Reply via email to