[coreboot-gerrit] Change in coreboot[master]: mb/google/volteer: Add GPIO configuration table for terrador

[Caveh Jalali (Code Review)]

Caveh Jalali has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43439 )

Change subject: mb/google/volteer: Add GPIO configuration table for terrador
......................................................................

Patch Set 2:

(1 comment)

https://review.coreboot.org/c/coreboot/+/43439/2/src/mainboard/google/voltee... 
File src/mainboard/google/volteer/variants/baseboard/devicetree.cb:

https://review.coreboot.org/c/coreboot/+/43439/2/src/mainboard/google/voltee... 
PS2, Line 16: 		option MAX98373_ALC5682I_I2S_TGL_Y 4
1. this new audio DB type should really be defined in program.star before it is used anywhere.
2. if the terrador design only uses a single audio configuration - and that seems to be the case since there is no audio sub-board, then you don't need the probing logic. you can define your GPIOs in gpio.c and define your codec/amp in the devicetree without the probe directive.