[coreboot-gerrit] [XS] Change in coreboot[main]: mb/google/ovis/var/deku: Remove DSP setting

[Subrata Banik (Code Review)]

Attention is currently required from: Derek Huang, Dinesh Gehlot, Eran Mitrani, Jakub Czapiga, Kapil Porwal, Tarun, Tony Huang.

Subrata Banik has posted comments on this change by Tony Huang. ( https://review.coreboot.org/c/coreboot/+/84139?usp=email )

Change subject: mb/google/ovis/var/deku: Remove DSP setting
......................................................................

Patch Set 1:

(1 comment)

File src/mainboard/google/rex/variants/deku/overridetree.cb:

https://review.coreboot.org/c/coreboot/+/84139/comment/3f6077f3_170aae10?usp... :
PS1, Line 51:
> Attach enum in chip.h, any suggested value?
>
> /* iDisp-Link T-Mode 0: 2T, 2: 4T, 3: 8T, 4: 16T */
>    enum {
>    	HDA_TMODE_2T = 0,
>    	HDA_TMODE_4T = 2,
>    	HDA_TMODE_8T = 3,
>    	HDA_TMODE_16T = 4,
>    } pch_hda_idisp_link_tmode;
>    /* iDisp-Link Freq 4: 96MHz, 3: 48MHz. */
>    enum {
>    	HDA_LINKFREQ_48MHZ = 3,
>    	HDA_LINKFREQ_96MHZ = 4,
>    } pch_hda_idisp_link_frequency;

can you please check what is the FSP default value if we don't override. ideally if we set `pch_hda_dsp_enable=0` , other UPDs are mostly remains `don't care` hence, any default value would also work.