[coreboot-gerrit] [S] Change in coreboot[main]: drivers/soundwire: Support Realtek ALC712 codec

[Subrata Banik (Code Review)]

Attention is currently required from: Mac Chiang, Pranava Y N, Weimin Wu.

Subrata Banik has posted comments on this change by Weimin Wu. ( https://review.coreboot.org/c/coreboot/+/85571?usp=email )

The change is no longer submittable: All-Comments-Resolved is unsatisfied now.

Change subject: drivers/soundwire: Support Realtek ALC712 codec
......................................................................

Patch Set 2:

(1 comment)

File src/drivers/soundwire/alc711/alc711.c:

https://review.coreboot.org/c/coreboot/+/85571/comment/64fe00e9_aed55513?usp... :
PS2, Line 13: static struct soundwire_multilane alc711_multilane = {
            : };
> Because I don't know how the default value should be configured, and if the chip config does not provide data, then because lane_mapping_count and lane_bus_holder_count are 0, no extra data will be generated, except that one more soundwire_gen_interface_revision(dsd) will be generated. To be honest, I am not sure whether it will cause interference, these need to be confirmed by actual debugging on the board.

can Varun (Intel) able to share a response about how to handle the default data ?