[coreboot-gerrit] [XS] Change in coreboot[main]: mb/google/brya: Do not select HAVE_ACPI_RESUME

[Dinesh Gehlot (Code Review)]

Attention is currently required from: Eric Lai, Jayvik Desai, Kapil Porwal, Matt DeVillier, Nick Vaccaro, Paul Menzel, Subrata Banik.

Dinesh Gehlot has posted comments on this change by Dinesh Gehlot. ( https://review.coreboot.org/c/coreboot/+/86422?usp=email )

Change subject: mb/google/brya: Do not select HAVE_ACPI_RESUME
......................................................................

Patch Set 3:

(2 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/86422/comment/6ab6e2cc_628b93b2?usp... :
PS1, Line 9: Brya mainboard does not reliably support S3 entry/exit.
> Please update the commit message then. It says, ACPI S3 could work. […]

Acknowledged

https://review.coreboot.org/c/coreboot/+/86422/comment/8d42aaba_1ba9cabc?usp... :
PS1, Line 10: Also trigger a fail-safe board
            : reset if the system resumes from S3.
> > Sorry for not being clear in my comment. […]

Agree, System is not expected to reach S3 state. The fail safe code has been removed.