[coreboot-gerrit] Change in coreboot[master]: mb/google/dedede/var/boten: Add fw_config probe for ALC5682-VD & VS

[Stanley Wu (Code Review)]

Attention is currently required from: Henry Sun, Paul Menzel, Paul Fagerburg, Karthik Ramasubramanian.
Stanley Wu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/57977 )

Change subject: mb/google/dedede/var/boten: Add fw_config probe for ALC5682-VD & VS
......................................................................

Patch Set 3:

(3 comments)

This change is ready for review.

Commit Message:

https://review.coreboot.org/c/coreboot/+/57977/comment/4b38ce94_2bfa58e7 
PS1, Line 9: ALC5682
> ALC5682I?

Done

File src/mainboard/google/dedede/variants/boten/overridetree.cb:

https://review.coreboot.org/c/coreboot/+/57977/comment/cd71db1f_4fbd6a60 
PS1, Line 2: 41 43
> Please confirm that EC is not using these SSFC bits for other purposes.

Yes, we refer chromium:3021932 and deal with EC to use SSFC bit 9-11 for audio codec scope.

https://review.coreboot.org/c/coreboot/+/57977/comment/99470136_4ea4affb 
PS1, Line 5:
> Nit: Using tab here whereas using space in the previous lines. Recommend to use either tab or space.

Done