Karthik Ramasubramanian has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39115 )
Change subject: mb/google/dedede: Add WLAN configuration ......................................................................
Patch Set 11:
(2 comments)
https://review.coreboot.org/c/coreboot/+/39115/5/src/mainboard/google/dedede... File src/mainboard/google/dedede/variants/baseboard/gpio.c:
https://review.coreboot.org/c/coreboot/+/39115/5/src/mainboard/google/dedede... PS5, Line 154: PAD_CFG_GPI_SCI_LOW
I see it routed to I2C_IRQ on WLAN connector, since I2C SCL and SDA on connector is unstuffed, we ar […]
So if I understand correctly it is a way for WLAN connector to interrupt AP regarding some data is ready to be read over I2C. Is that correct?
Since I2C lines are not stuffed, you are suggesting that the Interrupt line is not required.
https://review.coreboot.org/c/coreboot/+/39115/5/src/mainboard/google/dedede... PS5, Line 188: PAD_CFG_NF(GPP_D19, NONE, DEEP, NF2), : /* D20 : WWAN_WLAN_COEX2 */ : PAD_CFG_NF(GPP_D20, NONE, DEEP, NF2), : /* D21 : WWAN_WLAN_COEX3 */ : PAD_CFG_NF(GPP_D21, NONE, DEEP, NF1),
Still I have the follow-up question regarding how the CoEX is managed for M.2 WLAN + WWAN. […]
Got it. Thanks for the confirmation.