Arthur Heymans has submitted this change. ( https://review.coreboot.org/c/coreboot/+/67022?usp=email )
(
2 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one. )Change subject: allocator_v4: Remove redundant parameter ......................................................................
allocator_v4: Remove redundant parameter
update_bridge_resource() already gets the type passed as part of the resource.
Change-Id: I6b3c9809caecdd1bad5b98891a00c3392190a3e0 Signed-off-by: Nico Huber nico.h@gmx.de Reviewed-on: https://review.coreboot.org/c/coreboot/+/67022 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Felix Singer service+coreboot-gerrit@felixsinger.de Reviewed-by: Tim Wawrzynczak inforichland@gmail.com --- M src/device/resource_allocator_v4.c 1 file changed, 3 insertions(+), 2 deletions(-)
Approvals: Felix Singer: Looks good to me, but someone else must approve Tim Wawrzynczak: Looks good to me, approved build bot (Jenkins): Verified
diff --git a/src/device/resource_allocator_v4.c b/src/device/resource_allocator_v4.c index fb0320a..e64d8ad 100644 --- a/src/device/resource_allocator_v4.c +++ b/src/device/resource_allocator_v4.c @@ -112,12 +112,13 @@ * allocated base of the bridge resource. */ static void update_bridge_resource(const struct device *bridge, struct resource *bridge_res, - unsigned long type_match, int print_depth) + int print_depth) { const struct device *child; struct resource *child_res; resource_t base; const unsigned long type_mask = IORESOURCE_TYPE_MASK | IORESOURCE_PREFETCH; + const unsigned long type_match = bridge_res->flags & type_mask; struct bus *bus = bridge->link_list;
child_res = NULL; @@ -228,7 +229,7 @@ * Update the window for current bridge resource now that all downstream * requirements are gathered. */ - update_bridge_resource(bridge, res, type_match, print_depth); + update_bridge_resource(bridge, res, print_depth); } }