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

[Furquan Shaikh (Code Review)]

Furquan Shaikh 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 1:

(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
> What about CNVi? I just checked the vendor ACPI of clevo/l140cu, where this is 0x04. […]

SLP_TYP_S3 is 5 for Intel hardware. I don't think 4 is a valid value. Even for CNVi, this should still be 5 i.e. SLP_TYP_S3. That is what the boards in coreboot currently use. So, this change does not really change the logic.