[coreboot-gerrit] [L] Change in coreboot[master]: ec/google/chromeec: Add handles for ori, usb role switch

[Caveh Jalali (Code Review)]

Attention is currently required from: Robert Zieba, Tarun Tuli, Subrata Banik, Kapil Porwal, Nick Vaccaro, Prashant Malani, Boris Mittelberg.

Caveh Jalali has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/72370 )

Change subject: ec/google/chromeec: Add handles for ori, usb role switch
......................................................................

Patch Set 4:

(1 comment)

File src/ec/google/chromeec/ec_acpi.c:

https://review.coreboot.org/c/coreboot/+/72370/comment/5b15dff0_5a139011 
PS3, Line 195: .orientation_switch = config->ori_conn[i],
             : 			.usb_role_switch = config->usb_role_conn[i],
> Thanks for confirming. […]

could you add the skyrim case as the next patch so we have some insight
into what's coming up? i didn't see anything interesting USB related
in the existing skyrim device tree.

sounds like part of the problem is that you can't disable
`orientation_switch` or `usb_role_switch` by just explicitly setting
them to NULL in the device tree since that's indistinguishable from not
being present. would it make sense to introduce explicit
`no_orientation_switch`, `no_usb_role_switch` parameters to the
device tree?
`fill_ssdt_typec_device()` will have to check for those.