Aaron Durbin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38548 )
Change subject: soc/intel: Add Intel Xeon Scalable Processor support ......................................................................
Patch Set 54:
(1 comment)
https://review.coreboot.org/c/coreboot/+/38548/54/src/soc/intel/xeon_sp/chip... File src/soc/intel/xeon_sp/chip.c:
https://review.coreboot.org/c/coreboot/+/38548/54/src/soc/intel/xeon_sp/chip... PS54, Line 269: static void assign_bridge_resources(struct iiostack_resource *stack_list, Could someone please provide commentary on what all this resource manipulation is doing in the chipset code? There's a lot of it, and it doesn't seem appropriate. Similarly, there's not much of anything in this file explaining what is going on and why.