Tim Wawrzynczak has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45878 )
Change subject: tigerlake mainboards: switch to devtree aliases for PMC MUX connectors ......................................................................
Patch Set 2:
(1 comment)
https://review.coreboot.org/c/coreboot/+/45878/2/src/mainboard/google/voltee... File src/mainboard/google/volteer/variants/volteer/overridetree.cb:
https://review.coreboot.org/c/coreboot/+/45878/2/src/mainboard/google/voltee... PS2, Line 245: conn1
Haha yeah, I did notice that but forgot about it. I think it'll just use the "first" one. […]
Confirmed, it just picked the first one. But what we really wanted here was the ability to set a register's value differently based on fw_config probing, which would require keeping around a list of the possible options and then some machinery to assign them...