Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31140 )
Change subject: lib/hardwaremain: Remove unused acpi_is_wakeup() function ......................................................................
Patch Set 1:
Refer to: https://review.coreboot.org/cgit/coreboot.git/tree/src/arch/x86/acpi_s3.c
The acpi_is_wakeup() function sets the acpi_slp_type global variable, and returns a value based on it. It is necessary to set that global variable correctly (hence a function call is needed), but no value needs to be returned.
I would suggest replacing acpi_is_wakeup() with acpi_handoff_wakeup() instead.
I could see acpi_is_wakeup() function is getting called from some other place as well. i need to see who is calling this.