[coreboot-gerrit] Change in coreboot[master]: sb/intel/bd82x6x/smihandler: Fix xHCI sleep handler

[Angel Pons (Code Review)]

Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41852 )

Change subject: sb/intel/bd82x6x/smihandler: Fix xHCI sleep handler
......................................................................

Patch Set 7:

(1 comment)

https://review.coreboot.org/c/coreboot/+/41852/7/src/southbridge/intel/bd82x... 
File src/southbridge/intel/bd82x6x/smihandler.c:

https://review.coreboot.org/c/coreboot/+/41852/7/src/southbridge/intel/bd82x... 
PS7, Line 107: if (pch_silicon_type() == PCH_TYPE_PPT &&
             : 		    pch_silicon_revision() == PCH_STEP_A0) {
             : 			break;
             : 		}
> This skips over the `pci_or_config16(PCH_XHCI_DEV, 0x74, 0x03);` at the end of this case, but doesn' […]

Just checked, the S3/S4 workaround is indeed specific to PPT A0, but the S5 workaround applies to all steppings.