[coreboot-gerrit] Change in coreboot[master]: drivers/soundwire/alc711: Add Realtek ALC711 soundwire device

[Duncan Laurie (Code Review)]

Duncan Laurie has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/46303 )

Change subject: drivers/soundwire/alc711: Add Realtek ALC711 soundwire device
......................................................................

Patch Set 4:

(2 comments)

https://review.coreboot.org/c/coreboot/+/46303/4/src/drivers/soundwire/alc71... 
File src/drivers/soundwire/alc711/alc711.c:

https://review.coreboot.org/c/coreboot/+/46303/4/src/drivers/soundwire/alc71... 
PS4, Line 36: /* Bus frequency must be 1/2/4/8 divider of
            : 	supported input frequencies. */
can go to 96 chars so probably don't need to break this line

https://review.coreboot.org/c/coreboot/+/46303/4/src/drivers/soundwire/alc71... 
PS4, Line 84: 		{
            : 			/* Data Input for Speaker Path */
            : 			.port = 1,
            : 			.source = &alc711_dp1
            : 		}
            : 	}
seems duplicated?  Is there a second port for output?

these should match what is reported in alc711_slave.{sink,source}_port_list