summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorFurquan Shaikh <furquan@google.com>2020-05-26 12:04:35 -0700
committerFurquan Shaikh <furquan@google.com>2020-05-27 18:57:17 +0000
commit563e614bcde9627b764445dfe2fe7cfca7e99c96 (patch)
tree61017088416443c1ff716801550f0d055f5c98d1
parent55c735a417e75387e988553f9f66d27e19f2364b (diff)
downloadcoreboot-563e614bcde9627b764445dfe2fe7cfca7e99c96.tar.xz
resource_allocator_v4: Fix size of I/O hole at 0x3b0
Addressing comment from CB:41443 that was received after the change landed. memranges_create_hole() takes size as the last parameter. So, the I/O hole created at 0x3b0 needs to set size as 0x3df - 0x3b0 + 1 as 0x3df is the upper limit of that hole. Change-Id: I08fca283436924427e12c6c69edced7e51db42a9 Signed-off-by: Furquan Shaikh <furquan@google.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/41737 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Nico Huber <nico.h@gmx.de> Reviewed-by: Angel Pons <th3fanbus@gmail.com> Reviewed-by: Aaron Durbin <adurbin@chromium.org>
-rw-r--r--src/device/resource_allocator_v4.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/device/resource_allocator_v4.c b/src/device/resource_allocator_v4.c
index ece7150b32..b65fc8bf37 100644
--- a/src/device/resource_allocator_v4.c
+++ b/src/device/resource_allocator_v4.c
@@ -330,7 +330,7 @@ static void constrain_domain_resources(const struct device *domain, struct memra
* Don't allow allocations in the VGA I/O range. PCI has special cases for
* that.
*/
- memranges_create_hole(ranges, 0x3b0, 0x3df);
+ memranges_create_hole(ranges, 0x3b0, 0x3df - 0x3b0 + 1);
/*
* Resource allocator no longer supports the legacy behavior where I/O resource