[coreboot-gerrit] Change in ...coreboot[master]: google/kahlee: No need to guard with HAVE_ACPI_RESUME

[Richard Spiegel (Code Review)]

Richard Spiegel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31317 )

Change subject: google/kahlee: No need to guard with HAVE_ACPI_RESUME
......................................................................

Patch Set 1: -Code-Review
> Patch Set 1:
>
> Richard, you most likely want to check with Martin or Marshall for my background around AGESA and S3 before commenting further...

I'm still not comfortable on playing with S3. I agree that if ASL code does not advertise S3 then you are probably correct removing these. But it'll add unneeded extra code when S3 is disabled. At least add the message requested by Paul.