Change in coreboot[master]: soc/intel/common: Abstract the common TCSS functions

[Subrata Banik (Code Review)]

Attention is currently required from: John Zhao, Tim Wawrzynczak, Paul Menzel, Eric Lai, Angel Pons, Patrick Rudolph, EricR Lai.
Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/62723 )

Change subject: soc/intel/common: Abstract the common TCSS functions
......................................................................

Patch Set 11:

(1 comment)

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

https://review.coreboot.org/c/coreboot/+/62723/comment/1cd6366d_3485be4d 
PS11, Line 34: 	if (config->tcss_ops.valid_tbt_auth) {
             : 		if (!config->tcss_ops.valid_tbt_auth())
             : 			return;
             : 	}
> It seems logically clear NOT to combine them together. If platform presents NULL function pointer, then the valid_tbt_auth() function won't be executed. Otherwise if combining, the NULL pointer function valid_tbt_auth() will be unconditionally executed.

ACK