Change in coreboot[master]: binaryPI: Drop CAR teardown without POSTCAR_STAGE

[Kyösti Mälkki (Code Review)]

Kyösti Mälkki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/18526 )

Change subject: binaryPI: Drop CAR teardown without POSTCAR_STAGE
......................................................................

Patch Set 8:

(1 comment)

https://review.coreboot.org/c/coreboot/+/18526/8/src/vendorcode/amd/pi/00630... 
File src/vendorcode/amd/pi/00630F01/binaryPI/gcccar.inc:

https://review.coreboot.org/c/coreboot/+/18526/8/src/vendorcode/amd/pi/00630... 
PS8, Line 920: wbinvd
> Was this used as a very hacky way to make sure cbmem hits ram during romstage? That is definitely na […]

https://review.coreboot.org/c/coreboot/+/18495/5/src/cpu/amd/pi/00670F00/rom...

The same discussion is probably split and repeated at 15 locations. And yes, this wbinvd is why AGESA got its HAVE_ACPI_RESUME disabled for some period of time, you got low-memory corruptions on S3 resume path.