[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 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'm not sure about the requirements

It depends on the hardware board design.

In my opinion, let's take this as a follow-up. The current Intel driver was assuming maxsleep as S3. If we have to support this in the generic driver, we can always add it back - default as S3 unless provided by mainboard.