Change in coreboot[master]: soc/intel/tigerlake: Add CNVi Bluetooth flag at devicetree entry

[Cliff Huang (Code Review)]

Attention is currently required from: Bora Guvendik, Anil Kumar K, Furquan Shaikh, Wonkyu Kim, Angel Pons, Bernardo Perez Priego, Srinidhi N Kaushik, Patrick Rudolph.
Cliff Huang has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/50897 )

Change subject: soc/intel/tigerlake: Add CNVi Bluetooth flag at devicetree entry
......................................................................

Patch Set 6:

(3 comments)

File src/soc/intel/tigerlake/chip.h:

https://review.coreboot.org/c/coreboot/+/50897/comment/f60afcb4_28fc8855 
PS6, Line 320: CnviBtCore
> How about `use_cnvi_bluetooth` instead?

Same naming convention is used, like other CNVi variables: CnviMode, CnviBtAudioOffload.

https://review.coreboot.org/c/coreboot/+/50897/comment/5397ad51_a735ca52 
PS6, Line 320: uint8_t
> bool?

In FSP, UINT8 is used. Although 'bool' is also used, most of enable/disable flags in this context use type uint8_t.

File src/soc/intel/tigerlake/fsp_params.c:

https://review.coreboot.org/c/coreboot/+/50897/comment/136adb87_d3fe9c4f 
PS6, Line 357: 	params->CnviBtCore = config->CnviBtCore;
> Can one use CNVi BT without CNVi Wi-Fi?

Good point. This flag is added for tiger lake, let me check with FSP team. Meanwhile, I can look at the FSP source code and do a quick test on my end.