[coreboot-gerrit] Change in coreboot[master]: drivers/wifi: Drop maxsleep parameter from chip config

[Michael Niewöhner (Code Review)]

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
> If you are looking at the PM1_CNT register, then 4 is not a valid value.

Yeah, however, Intel datasheets tend to be very wrong sometimes
> Also, do you know if the mainboard is designed to support wake from wifi(cnvi) in S5?

I'm not sure about the requirements