soc/amd/*/chip.h: Correct PSPP Enum Value

It appears the pspp_policy enum is not the same as the FSP definition
currently being used. This means that the incorrect PSPP value setting
would get read by FSP. For Zork programs this meant we actually were
setting links as DXIO_PSPP_BALANCED instead of DXIO_PSPP_POWERSAVE.
This change adds DXIO_PSPP_DISABLED as the first enum value to properly
match the FSP definition and adjusts non AMD Customer Reference Boards
that reference the enum to still send the same value even though it has
now change definitions. If we actually want DXIO_PSPP_POWERSAVE for
those boards that can be adjusted in a future change.

BUG=b:193495634
TEST=Boot to OS with Majolica and Guybrush and run 10G iperf on wifi
with other server on local network.

Change-Id: I287b6d3168697793a2ae8d8e68b4ec824f2ca5ef
Signed-off-by: Matt Papageorge <matthewpapa07@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/56463
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Felix Held <felix-coreboot@felixheld.de>
Reviewed-by: Jason Glenesk <jason.glenesk@gmail.com>

src/mainboard/google/guybrush/variants/baseboard/devicetree.cb

@@ -58,7 +58,7 @@ chip soc/amd/cezanne
 	register "i2c_pad_ctrl_rx_sel[2]" = "I2C_PAD_CTRL_RX_SEL_3_3V" # Audio/SAR
 	register "i2c_pad_ctrl_rx_sel[3]" = "I2C_PAD_CTRL_RX_SEL_1_8V" # GSC
 
-	register "pspp_policy" = "DXIO_PSPP_POWERSAVE"
+	register "pspp_policy" = "DXIO_PSPP_BALANCED"
 
 	register "usb_phy_custom" = "1"
 	register "usb_phy" = "{

src/mainboard/google/zork/variants/baseboard/devicetree_dalboz.cb

@@ -250,7 +250,7 @@ chip soc/amd/picasso
 	register "gpp_clk_config[5]" = "GPP_CLK_OFF"
 	register "gpp_clk_config[6]" = "GPP_CLK_OFF"
 
-	register "pspp_policy" = "DXIO_PSPP_POWERSAVE"
+	register "pspp_policy" = "DXIO_PSPP_BALANCED"
 
 	# See AMD 55570-B1 Table 13: PCI Device ID Assignments.
 	device domain 0 on

src/mainboard/google/zork/variants/baseboard/devicetree_trembyle.cb

@@ -243,7 +243,7 @@ chip soc/amd/picasso
 	register "gpp_clk_config[5]" = "GPP_CLK_OFF"
 	register "gpp_clk_config[6]" = "GPP_CLK_OFF"
 
-	register "pspp_policy" = "DXIO_PSPP_POWERSAVE"
+	register "pspp_policy" = "DXIO_PSPP_BALANCED"
 
 	# See AMD 55570-B1 Table 13: PCI Device ID Assignments.
 	device domain 0 on

src/soc/amd/cezanne/chip.h

@@ -97,7 +97,8 @@ struct soc_amd_cezanne_config {
 
 	/* performance policy for the PCIe links: power consumption vs. link speed */
 	enum {
-		DXIO_PSPP_PERFORMANCE = 0,
+		DXIO_PSPP_DISABLED = 0,
+		DXIO_PSPP_PERFORMANCE,
 		DXIO_PSPP_BALANCED,
 		DXIO_PSPP_POWERSAVE,
 	} pspp_policy;

src/soc/amd/picasso/chip.h

@@ -265,7 +265,8 @@ struct soc_amd_picasso_config {
 
 	/* performance policy for the PCIe links: power consumption vs. link speed */
 	enum {
-		DXIO_PSPP_PERFORMANCE = 0,
+		DXIO_PSPP_DISABLED = 0,
+		DXIO_PSPP_PERFORMANCE,
 		DXIO_PSPP_BALANCED,
 		DXIO_PSPP_POWERSAVE,
 	} pspp_policy;