Zhuohao Lee has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43549 )
Change subject: mb/google/volteer/var/voxel: change speaker smart amplifier to ALC1011 ......................................................................
Patch Set 1:
Please fix the build error:
In file included from src/mainboard/google/volteer/fw_config.c:5: src/mainboard/google/volteer/fw_config.c: In function 'fw_config_handle': src/include/fw_config.h:28:17: error: 'FW_CONFIG_FIELD_AUDIO_OPTION_RT1011_ALC5682I_I2S_NAME' undeclared (first use in this function); did you mean 'FW_CONFIG_FIELD_AUDIO_OPTION_MAX98357_ALC5682I_I2S_NAME'? .option_name = FW_CONFIG_FIELD_##__field##_OPTION_##__option##_NAME, \ ^~~~~~~~~~~~~~~~ src/mainboard/google/volteer/fw_config.c:76:22: note: in expansion of macro 'FW_CONFIG' fw_config_probe(FW_CONFIG(AUDIO, RT1011_ALC5682I_I2S))) { ^~~~~~~~~ src/include/fw_config.h:28:17: note: each undeclared identifier is reported only once for each function it appears in .option_name = FW_CONFIG_FIELD_##__field##_OPTION_##__option##_NAME, \ ^~~~~~~~~~~~~~~~ src/mainboard/google/volteer/fw_config.c:76:22: note: in expansion of macro 'FW_CONFIG' fw_config_probe(FW_CONFIG(AUDIO, RT1011_ALC5682I_I2S))) { ^~~~~~~~~ src/include/fw_config.h:30:11: error: 'FW_CONFIG_FIELD_AUDIO_OPTION_RT1011_ALC5682I_I2S_VALUE' undeclared (first use in this function); did you mean 'FW_CONFIG_FIELD_AUDIO_OPTION_MAX98357_ALC5682I_I2S_VALUE'? .value = FW_CONFIG_FIELD_##__field##_OPTION_##__option##_VALUE \ ^~~~~~~~~~~~~~~~ src/mainboard/google/volteer/fw_config.c:76:22: note: in expansion of macro 'FW_CONFIG' fw_config_probe(FW_CONFIG(AUDIO, RT1011_ALC5682I_I2S))) { ^~~~~~~~~ src/mainboard/google/volteer/fw_config.c: At top level: cc1: error: unrecognized command line option '-Wno-address-of-packed-member' [-Werror] cc1: all warnings being treated as errors make[1]: *** [Makefile:362: /cb-build/coreboot-gerrit.0/default/GOOGLE_TERRADOR/ramstage/mainboard/google/volteer/fw_config.o] Error 1 make[1]: Leaving directory '/home/coreboot/slave-root/workspace/coreboot-gerrit'