[coreboot-gerrit] Change in coreboot[master]: soc/intel/xeon_sp/cpx: Enable SATA ports

[Andrey Petrov (Code Review)]

Andrey Petrov has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39918 )

Change subject: soc/intel/xeon_sp/cpx: Enable SATA ports
......................................................................

Patch Set 2:

(1 comment)

https://review.coreboot.org/c/coreboot/+/39918/2/src/soc/intel/xeon_sp/cpx/K... 
File src/soc/intel/xeon_sp/cpx/Kconfig:

https://review.coreboot.org/c/coreboot/+/39918/2/src/soc/intel/xeon_sp/cpx/K... 
PS2, Line 20: SOC_INTEL_COMMON_BLOCK_SATA
> It seems to me that the mask in the common code https://github. […]

good catch. I guess we will have add that logic (number of ports) directly in block/sata/sata.c, since it looks like it is not possible to figure out hardware capabilities run-time. I will work on a patch for this