Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38512 )
Change subject: soc/intel/skylake: Only reserve TPM area for !CONFIG_TPM_CR50 device ......................................................................
Patch Set 5:
(1 comment)
Can you please not merge patches in the middle of the discussion? Was this tested at least? I mean real tested, not oops it still boots.
I have tested this on EVE, Soraka, Hatch and Icelake RVP
Just to be sure we're talking about the same thing. Have you tested if a downstream device can decode MMIO in the given range?
https://review.coreboot.org/c/coreboot/+/38512/3/src/soc/intel/skylake/acpi/... File src/soc/intel/skylake/acpi/systemagent.asl:
https://review.coreboot.org/c/coreboot/+/38512/3/src/soc/intel/skylake/acpi/... PS3, Line 190: 0x00005000)
PCH will always decode any valid fixed range memory but as per PCH fixed memory map, i don't see thi […]
Where is this map?
And if it's not fixed, how is it triggered?