Michael Niewöhner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/46033 )
Change subject: drivers/wifi: Drop maxsleep parameter from chip config ......................................................................
Patch Set 2:
(1 comment)
https://review.coreboot.org/c/coreboot/+/46033/1/src/drivers/intel/wifi/wifi... File src/drivers/intel/wifi/wifi.c:
https://review.coreboot.org/c/coreboot/+/46033/1/src/drivers/intel/wifi/wifi... PS1, Line 54: By default, all intel wifi chips wake from S3
I don't think 4 is a valid value
4 is mentioned for S4 in this intel doc on page 83: https://www.intel.com/content/dam/www/public/us/en/documents/articles/acpi-c...