[XS] Change in coreboot[main]: mb/google/brya: Fix mux_conn index used by ec/google/chromeec

[Won Chung (Code Review)]

Attention is currently required from: Dinesh Gehlot, Kapil Porwal, Nick Vaccaro, Subrata Banik.

Won Chung has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/81354?usp=email )

Change subject: mb/google/brya: Fix mux_conn index used by ec/google/chromeec
......................................................................

Patch Set 4:

(1 comment)

Patchset:

PS4:
> Interesting, when I did the felwinter project. I already verified all the USB. […]

We see an issue when we apply the first patch in this series where we try to copy _PLD from usb port to mux. PMC mux conn1 uses usbX_port2 and conn2 uses usbX_port3. So, conn1's PLD is to be copied from usbX_port2 and conn2's PLD is to be copied from usbX_port3.

The issue is that `fill_ssdt_typec_device` in `ec_acpi.c` matches conn1 to usbX_port3 and conn2 to usbX_port2, which are reversed. The reason is that given index i, mux_conn[i] is matched with usb port's PLD group token of i+1, so mux_conn[0] is matched with usbX_port2 with group token of 1 and mux_conn[1] is matched with usbX_port3 with group token of 2. Since mux_conn[0] is conn2 and mux_conn[1] is conn1, this matching is swapped.