Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/46669 )
Change subject: soc/amd/common/acpi: Create platform.asl to define acpi transitions ......................................................................
Patch Set 5: Code-Review+2
(1 comment)
https://review.coreboot.org/c/coreboot/+/46669/5/src/soc/amd/common/acpi/pla... File src/soc/amd/common/acpi/platform.asl:
https://review.coreboot.org/c/coreboot/+/46669/5/src/soc/amd/common/acpi/pla... PS5, Line 8: I think the comment on earlier patchset wasn't visible correctly on gerrit. The space isn't needed before _SB but between Scope and ( and between ) and {.
i.e. Scope (_SB) {