Marshall Dawson has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37547 )
Change subject: northbridge/amd/pi/00660F01: domain_acpi_name function missing in northbridge ......................................................................
Patch Set 3:
(2 comments)
https://review.coreboot.org/c/coreboot/+/37547/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/37547/1//COMMIT_MSG@7 PS1, Line 7: domain_acpi_name function missing in 00660F01 northbridge.
Please rephrase: […]
In PS3 "northbridge/amd/pi/00660F01: domain_acpi_name function missing in northbridge" is a very long subject line. I agree with Paul's suggestion.
https://review.coreboot.org/c/coreboot/+/37547/1//COMMIT_MSG@8 PS1, Line 8:
Please describe the error/problem.
Was this actually required for SATA to boot? Or is the info about SATA unrelated?