Attention is currently required from: Nico Huber, Jonathan Zhang.
Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/67018 )
Change subject: device/allocator: Allow for multiple domain resources of a type ......................................................................
Patch Set 5:
(1 comment)
File src/device/resource_allocator_v4.c:
https://review.coreboot.org/c/coreboot/+/67018/comment/a2b38bfd_dfb9261f PS1, Line 405: res
Woopsie, `res` is a remnant of the loop over child resources, not a domain resource. I guess we should either not print the resource in cleanup_resource_ranges() or print them all again (i.e. repeat the loop that was added to setup_resource_ranges()). I'd prefer the latter.
Done. I inlined the memrange cleanup for bridges and added and renamed the function to be used for domains where it loops over a resource type.