[coreboot-gerrit] Change in coreboot[master]: soc/intel/common/block/acpi: Add provision for multiple PCI segments

[Subrata Banik (Code Review)]

Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/40335 )

Change subject: soc/intel/common/block/acpi: Add provision for multiple PCI segments
......................................................................

Patch Set 4:

(3 comments)

https://review.coreboot.org/c/coreboot/+/40335/1//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/40335/1//COMMIT_MSG@10 
PS1, Line 10: 1
> Ack

Ack

https://review.coreboot.org/c/coreboot/+/40335/1//COMMIT_MSG@10 
PS1, Line 10: SoC specification [Ice Lake EDS vol 1 chapter 3.18]
> As per EDS, the formula out there

Ack

https://review.coreboot.org/c/coreboot/+/40335/1/src/arch/x86/acpi.c 
File src/arch/x86/acpi.c:

https://review.coreboot.org/c/coreboot/+/40335/1/src/arch/x86/acpi.c@114 
PS1, Line 114: end_bus_number
> got your point, we can have a wrapper here which converts (end / 256) into end_bus_number  and (end  […]

Ack