diff options
| author | Furquan Shaikh <furquan@google.com> | 2020-05-26 12:04:35 -0700 |
|---|---|---|
| committer | Furquan Shaikh <furquan@google.com> | 2020-05-27 18:57:17 +0000 |
| commit | 563e614bcde9627b764445dfe2fe7cfca7e99c96 (patch) | |
| tree | 61017088416443c1ff716801550f0d055f5c98d1 | |
| parent | 55c735a417e75387e988553f9f66d27e19f2364b (diff) | |
| download | coreboot-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.c | 2 |
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 |