[S] Change in coreboot[master]: soc/intel/apollolake: Make SATA speed limit configurable

[Felix Held (Code Review)]

Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/75364?usp=email )

Change subject: soc/intel/apollolake: Make SATA speed limit configurable
......................................................................

soc/intel/apollolake: Make SATA speed limit configurable

In cases where there are limitations on the mainboard it can be
necessary to limit the used SATA speed even though both, the SATA
controller and disk drive support a higher speed rate. The FSP parameter
'SpeedLimit' allows to set the speed limit.

It should be noted that Gen 3 equals the default value '0'. This means
that inside FSP the same code is executed.

This patch provides a chip config so that this FSP parameter can be
set as needed in the devicetree on mainboard level.

Change-Id: I9c3eda0649546e3a40eb24a015b7c6efd8f90e0f
Signed-off-by: Mario Scheithauer 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/75364
Reviewed-by: Felix Singer 
Reviewed-by: Jan Samek 
Tested-by: build bot (Jenkins) 
---
M src/soc/intel/apollolake/chip.c
M src/soc/intel/apollolake/chip.h
2 files changed, 11 insertions(+), 0 deletions(-)

Approvals:
  Jan Samek: Looks good to me, approved
  build bot (Jenkins): Verified
  Felix Singer: Looks good to me, approved

diff --git a/src/soc/intel/apollolake/chip.c b/src/soc/intel/apollolake/chip.c
index e7a8168..bd40595 100644
--- a/src/soc/intel/apollolake/chip.c
+++ b/src/soc/intel/apollolake/chip.c
@@ -736,6 +736,7 @@
    /* SATA config */
    if (is_devfn_enabled(PCH_DEVFN_SATA)) {
    	silconfig->SataSalpSupport = !(cfg->DisableSataSalpSupport);
+		silconfig->SpeedLimit = cfg->sata_speed;
    	memcpy(silconfig->SataPortsEnable, cfg->SataPortsEnable,
    		sizeof(silconfig->SataPortsEnable));
    }
diff --git a/src/soc/intel/apollolake/chip.h b/src/soc/intel/apollolake/chip.h
index a92d982..3951880 100644
--- a/src/soc/intel/apollolake/chip.h
+++ b/src/soc/intel/apollolake/chip.h
@@ -25,6 +25,13 @@
    PNP_PERF_POWER,
 };

+enum sata_speed_limit {
+	SATA_DEFAULT = 0,
+	SATA_GEN1,
+	SATA_GEN2,
+	SATA_GEN3
+};
+
 struct soc_intel_apollolake_config {

/* Common structure containing soc config data required by common code*/
@@ -215,6 +222,9 @@

/* Sata Power Optimisation */
    uint8_t SataPwrOptimizeDisable;
+
+	/* SATA speed limit */
+	enum sata_speed_limit sata_speed;
 };

typedef struct soc_intel_apollolake_config config_t;