[coreboot-gerrit] Change in coreboot[master]: soc/amd/stoneyridge/enable_usbdebug.c: Update pci_ehci_dbg_set_port()

[Richard Spiegel (Code Review)]

Richard Spiegel has posted comments on this change. ( https://review.coreboot.org/28343 )

Change subject: soc/amd/stoneyridge/enable_usbdebug.c: Update pci_ehci_dbg_set_port()
......................................................................

Patch Set 1:

(1 comment)

Missing a file!

https://review.coreboot.org/#/c/28343/1/src/soc/amd/stoneyridge/include/soc/...
File src/soc/amd/stoneyridge/include/soc/southbridge.h:

https://review.coreboot.org/#/c/28343/1/src/soc/amd/stoneyridge/include/soc/...
PS1, Line 311: #define   DEBUG_PORT_MASK		(BIT(16) | BIT(17) | (BIT(18))
> Macros with complex values should be enclosed in parentheses

I don't understand... it is enclosed by parenthesis.