Attention is currently required from: Ashish Kumar Mishra, Bob Moragues, Nick Vaccaro, Shelley Chen.
Karthik Ramasubramanian has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/80564?usp=email )
Change subject: mb/google/brox: Handle bluetooth enable on devices ......................................................................
Patch Set 2:
(1 comment)
File src/mainboard/google/brox/variants/brox/overridetree.cb:
https://review.coreboot.org/c/coreboot/+/80564/comment/badc3c56_b89d8a18 : PS2, Line 24: field BT 10 10 : option BT_DISCRETE 0 : option BT_CNVI 1 : end
I also was wondering, but I came to the conclusion that having a separate FW config for bluetooth wo […]
FYI, This bit is already taken for some other purpose as per FW_CONFIG_MASK. It is currently used by ISH_ENABLED/DISABLED.
I would recommend to use WIFI bits. If WIFI_CNVI set cnvi_bt_core = true. Else set cnvi_bt_core = false; Let us define a bit for BT when we cross the bridge. At the moment all our designs use CNVI Wifi + BT combo or PCIe Wifi + BT combo and not a hybrid. As per the FSP configuration, it seems we cannot just use CNVI BT without enabling CNVI WIFI Core.
Again we have to see choose between this CL vs CB:80504