[coreboot-gerrit] Change in coreboot[master]: soc/intel/common/tcss: Optimize USB-C DP flow and code structure

[Derek Huang (Code Review)]

Attention is currently required from: Tim Wawrzynczak, Paul Menzel, Curtis Chen, Brandon Breitenstein.
Derek Huang has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/57139 )

Change subject: soc/intel/common/tcss: Optimize USB-C DP flow and code structure
......................................................................

Patch Set 23:

(2 comments)

File src/soc/intel/common/block/tcss/tcss.c:

https://review.coreboot.org/c/coreboot/+/57139/comment/e8e96147_c86658f0 
PS22, Line 297: 	port_bitmask = ops->dp_ops.wait_for_connection(WAIT_FOR_DISPLAYPORT_TIMEOUT_MS);
> suggestion: after getting `ops`, return early if ops is NULL

Done

https://review.coreboot.org/c/coreboot/+/57139/comment/1fd2ee57_be99cac7 
PS22, Line 331: if (ret)
              : 			printk(BIOS_ERR, "Port C%zd mux set failed with error %d\n", i, ret);
              : 		else {
              : 			printk(BIOS_INFO, "Port C%zd is configured to DP mode!\n", i);
              : 			return;
              : 		}
> style: if one arm of a conditional requires braces, then all branches should have them

Done