[S] Change in coreboot[master]: soc/intel/meteorlake: Hook up UPD CnviWifiCore

[Subrata Banik (Code Review)]

Attention is currently required from: Tarun Tuli, Kapil Porwal, Paul Menzel, Eric Lai.

Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/74306 )

Change subject: soc/intel/meteorlake: Hook up UPD CnviWifiCore
......................................................................

Patch Set 3:

(1 comment)

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

https://review.coreboot.org/c/coreboot/+/74306/comment/a8017226_230f763a 
PS3, Line 565: s_cfg->CnviWifiCore = config->cnvi_wifi_core;
> not sure about this, assert is prevent the setting wrong. kind of different idea to force it disable like the CnviBT?

as we would like to create a dependency between CnviMode and CnviWifiCore UPDs, it's better to club both under the same umbrella rather. This would help to ensure that WiFi core can't be enabled without enabling the CNVi itself. `assert` is meaningful for debug only features and when the outcome is not known. In this case, we for sure knw the dependency between those two UPDs.