Duncan Laurie 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. […]
Agreed it does make the alias more useful if we can have it assigned to multiple devices as long as only 1 gets enabled after probing in the end.
maybe sconfig could have the alias as a list of devices, and then the runtime probing could print an error if more than one ends up enabled because it might not be assigned to the expected device.