Duncan Laurie has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44782 )
Change subject: fw_config: Add caching to successfully probed fields ......................................................................
Patch Set 1:
Patch Set 1:
Patch Set 1:
is this problem much different from specifying the underlying USB port numbers in the conn driver? can we add support for a "i2s-port" field to the appropriate coreboot driver for the kernel driver to pick up? we might need to add a skeletal alc5682i.c driver if the max98*.c driver isn't the right place.
actually, soundwire alc5682/max98373 use the generic x.y path specifier, so maybe that's another option to specify the i2s port number needed to pass to the kernel.
Is there already kernel support for an i2s-port field somewhere?
The kernel audio for intel is all based around DMI checks currently, but there is discussion about moving to something more flexible (like audio-graph) in the future. Lots of early discussion in the bug but some of it moved to email later..