[XS] Change in coreboot[main]: soc/intel/xeon_sp: Remove INCLUDE_PCI_ALL bit for IBL platforms

[Arthur Heymans (Code Review)]

Attention is currently required from: Christian Walter, Jincheng Li, Johnny Lin, Jonathan Zhang, Lean Sheng Tan, Patrick Rudolph, Shuo Liu, Tim Chu.

Arthur Heymans has posted comments on this change by Shuo Liu. ( https://review.coreboot.org/c/coreboot/+/84316?usp=email )

Change subject: soc/intel/xeon_sp: Remove INCLUDE_PCI_ALL bit for IBL platforms
......................................................................

Patch Set 1:

(1 comment)

File src/soc/intel/xeon_sp/uncore_acpi.c:

https://review.coreboot.org/c/coreboot/+/84316/comment/a6e28a85_d5828b6c?usp... :
PS1, Line 271: if (!CONFIG(SOC_INTEL_COMMON_IBL_BASE) && is_dev_on_domain0(iommu)) {
             : 		printk(BIOS_DEBUG, "[Hardware Unit Definition] Flags: 0x%x, PCI Segment Number: 0x%x, "
             : 			"Register Base Address: 0x%x\n",
             : 			DRHD_INCLUDE_PCI_ALL, pcie_seg, reg_base);
             : 		current += acpi_create_dmar_drhd(current, DRHD_INCLUDE_PCI_ALL,
             : 			pcie_seg, reg_base, vtd_probe_bar_size(iommu));
             : 	} else {
             : 		printk(BIOS_DEBUG, "[Hardware Unit Definition] Flags: 0x%x, PCI Segment Number: 0x%x, "
             : 			"Register Base Address: 0x%x\n", 0, pcie_seg, reg_base);
             : 		current += acpi_create_dmar_drhd(current, 0, pcie_seg, reg_base,
             : 			vtd_probe_bar_size(iommu));
             : 	}
can we get rid of the if else statement and change the function input into a variable called flags? That would make it a bit easier to read.