[coreboot-gerrit] Patch set updated for coreboot: cf19421 resource: Report correct secondary resource window

Kyösti Mälkki (kyosti.malkki@gmail.com) gerrit at coreboot.org
Fri Apr 24 11:59:12 CEST 2015


Kyösti Mälkki (kyosti.malkki at gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/8888

-gerrit

commit cf19421288b7eb3492ea46e65a2360e30b450827
Author: Kyösti Mälkki <kyosti.malkki at gmail.com>
Date:   Mon Mar 23 19:58:23 2015 +0200

    resource: Report correct secondary resource window
    
    Once a bridge window resource is allocated, it becomes the base and limit
    for any resource on the secondary bus. Upper limit was incorrectly
    reported in the log while assigning secondary resources.
    
    Change-Id: I69f0a02aae6d13f77aaa2dace924b8970b23edad
    Signed-off-by: Kyösti Mälkki <kyosti.malkki at gmail.com>
---
 src/device/device.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/device/device.c b/src/device/device.c
index b3b8d24..117bd18 100644
--- a/src/device/device.c
+++ b/src/device/device.c
@@ -525,6 +525,7 @@ static void allocate_resources(struct bus *bus, struct resource *bridge,
 			/* Base must be aligned. */
 			base = round(base, resource->align);
 			resource->base = base;
+			resource->limit = resource->base + resource->size - 1;
 			resource->flags |= IORESOURCE_ASSIGNED;
 			resource->flags &= ~IORESOURCE_STORED;
 			base += resource->size;



More information about the coreboot-gerrit mailing list