[coreboot-gerrit] [S] Change in coreboot[main]: acpi: Enable 64bit ECAM resource

[Arthur Heymans (Code Review)]

Attention is currently required from: Angel Pons, Cliff Huang, David Milosevic, Felix Singer, Lance Zhao, Lean Sheng Tan, Martin L Roth, Maximilian Brune, Naresh Solanki, Nico Huber, Patrick Rudolph, Tim Wawrzynczak.

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

Change subject: acpi: Enable 64bit ECAM resource
......................................................................

Patch Set 5:

(1 comment)

File src/acpi/dsdt_top.asl:

https://review.coreboot.org/c/coreboot/+/79096/comment/92b1e8e3_0845e0bc :
PS5, Line 66: 				    0x0000000000000000, // Granularity
> I am not sure but granularity seems to refer to the alignment. Does the ECAM base and limit have no alignment constraints?

On Intel it needs to be aligned to it's size, but IDK on others. However here it's reported as a MinFixed & MaxFixed. You don't care about granularity as you tell the OS not to move it.