[coreboot-gerrit] Change in coreboot[master]: soc/intel: skl,cnl,icl,jsl,tgl: disable usb over-current pin by default

[Michael Niewöhner (Code Review)]

Michael Niewöhner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45112 )

Change subject: soc/intel: skl,cnl,icl,jsl,tgl: disable usb over-current pin by default
......................................................................

Patch Set 2:

(1 comment)

https://review.coreboot.org/c/coreboot/+/45112/2/src/soc/intel/cannonlake/fs... 
File src/soc/intel/cannonlake/fsp_params.c:

https://review.coreboot.org/c/coreboot/+/45112/2/src/soc/intel/cannonlake/fs... 
PS2, Line 376: 		params->Usb2OverCurrentPin[i] = config->usb2_ports[i].ocpin;
> There are multiple ways of doing this different. I'm not sure what the "best" way is. […]

oops... correction for variant 2:

```
for (i = 0; i < ARRAY_SIZE(config->usb2_ports); i++) {
 params->PortUsb20Enable[i] = config->usb2_ports[i].enable;
 if (config->usb2_ports[i].enable)
  params->Usb3OverCurrentPin[i] = config->usb3_ports[i].ocpin;
 else
  params->Usb3OverCurrentPin[i] = 0xff;

...
```