Attention is currently required from: Zhi7 Li, Karthik Ramasubramanian. Tim Wawrzynczak has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/56929 )
Change subject: src/drivers/i2c: Add rt5682 driver for codec ALC5682I-VD & VS compatibility ......................................................................
Patch Set 1: Code-Review-1
(1 comment)
File src/drivers/i2c/rt5682/rt5682.c:
https://review.coreboot.org/c/coreboot/+/56929/comment/0983d838_3da1e651 PS1, Line 66: if (fw_config_probe(FW_CONFIG(AUDIO_CODEC_SOURCE, AUDIO_CODEC_ALC5682))) : acpigen_write_name_string("_HID", "10EC5682"); : else if (fw_config_probe(FW_CONFIG(AUDIO_CODEC_SOURCE, AUDIO_CODEC_ALC5682I_VS))) : acpigen_write_name_string("_HID", "RTL5682"); The `fw_config` interface is mainboard specific and cannot go in common driver code.
I think you can solve your problem with just mainboard-specific code, I will leave a comment in the followup patch to demonstrate how.