[coreboot-gerrit] New patch to review for coreboot: bd2ad05 resource: Report correct secondary resource window

Kyösti Mälkki (kyosti.malkki@gmail.com) gerrit at coreboot.org
Tue Mar 24 05:29:43 CET 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 bd2ad05537ece8082ecb4ca77f508f113e01141a
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