From 58fe703e088be0af934cc0bf31fc60f87fcfde76 Mon Sep 17 00:00:00 2001 From: Nico Huber Date: Wed, 17 Aug 2022 14:43:54 +0200 Subject: [PATCH] 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 Reviewed-on: https://review.coreboot.org/c/coreboot/+/67022 Tested-by: build bot (Jenkins) Reviewed-by: Felix Singer Reviewed-by: Tim Wawrzynczak --- src/device/resource_allocator_v4.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/device/resource_allocator_v4.c b/src/device/resource_allocator_v4.c index fb0320a8eb..e64d8ade2d 100644 --- a/src/device/resource_allocator_v4.c +++ b/src/device/resource_allocator_v4.c @@ -112,12 +112,13 @@ static resource_t effective_limit(const struct resource *const res) * 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 @@ static void compute_bridge_resources(const struct device *bridge, unsigned long * 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); } }