Attention is currently required from: Anil Kumar K, Pranava Y N, Varun Upadhyay.
Subrata Banik has posted comments on this change by Varun Upadhyay. ( https://review.coreboot.org/c/coreboot/+/85074?usp=email )
Change subject: mb/google/fatcat: Add FW_CONFIG Support for ALC721 soundwire ......................................................................
Patch Set 2:
(1 comment)
File src/mainboard/google/fatcat/variants/fatcat/overridetree.cb:
https://review.coreboot.org/c/coreboot/+/85074/comment/aeea75c9_fd20c0fd?usp... : PS2, Line 8: AUDIO_ALC721_SNDW knowing we can have either one AIC between ALC721 or ALC722 while both selects the same driver, do we need a separate FW config ?
can this be handled using ```AUDIO_ALC72x_SNDW``` below code ?
``` chip drivers/intel/soundwire device generic 0 on chip drivers/soundwire/alc711 # SoundWire Link 1 ID 1 register "desc" = ""Headset Codec"" device generic 1.1 on probe AUDIO AUDIO_ALC72x_SNDW end end end end ```
or `# SoundWire Link 3 ID 1` is the meaningful here ?