* [PATCH] drm/amd/pm: Clean up errors in smu11_driver_if_sienna_cichlid.h
[not found] <20230724082935.9199-1-xujianghui@cdjrlc.com>
@ 2023-07-24 8:30 ` sunran001
0 siblings, 0 replies; 8+ messages in thread
From: sunran001 @ 2023-07-24 8:30 UTC (permalink / raw)
To: alexander.deucher; +Cc: dri-devel, amd-gfx, linux-kernel
Fix the following errors reported by checkpatch:
ERROR: trailing whitespace
ERROR: space prohibited before open square bracket '['
ERROR: space prohibited before that close square bracket ']'
Signed-off-by: Ran Sun <sunran001@208suo.com>
---
.../pmfw_if/smu11_driver_if_sienna_cichlid.h | 386 +++++++++---------
1 file changed, 193 insertions(+), 193 deletions(-)
diff --git
a/drivers/gpu/drm/amd/pm/swsmu/inc/pmfw_if/smu11_driver_if_sienna_cichlid.h
b/drivers/gpu/drm/amd/pm/swsmu/inc/pmfw_if/smu11_driver_if_sienna_cichlid.h
index aa6d29de4002..703ade13d9f4 100644
---
a/drivers/gpu/drm/amd/pm/swsmu/inc/pmfw_if/smu11_driver_if_sienna_cichlid.h
+++
b/drivers/gpu/drm/amd/pm/swsmu/inc/pmfw_if/smu11_driver_if_sienna_cichlid.h
@@ -225,33 +225,33 @@ typedef enum {
#define FW_DSTATE_MP1_WHISPER_MODE_BIT 6
#define FW_DSTATE_SOC_LIV_MIN_BIT 7
#define FW_DSTATE_SOC_PLL_PWRDN_BIT 8
-#define FW_DSTATE_MEM_PLL_PWRDN_BIT 9
+#define FW_DSTATE_MEM_PLL_PWRDN_BIT 9
#define FW_DSTATE_OPTIMIZE_MALL_REFRESH_BIT 10
#define FW_DSTATE_MEM_PSI_BIT 11
#define FW_DSTATE_HSR_NON_STROBE_BIT 12
#define FW_DSTATE_MP0_ENTER_WFI_BIT 13
-#define FW_DSTATE_SOC_ULV_MASK (1 <<
FW_DSTATE_SOC_ULV_BIT )
-#define FW_DSTATE_G6_HSR_MASK (1 <<
FW_DSTATE_G6_HSR_BIT )
-#define FW_DSTATE_G6_PHY_VDDCI_OFF_MASK (1 <<
FW_DSTATE_G6_PHY_VDDCI_OFF_BIT )
-#define FW_DSTATE_MP1_DS_MASK (1 <<
FW_DSTATE_MP1_DS_BIT )
-#define FW_DSTATE_MP0_DS_MASK (1 <<
FW_DSTATE_MP0_DS_BIT )
-#define FW_DSTATE_SMN_DS_MASK (1 <<
FW_DSTATE_SMN_DS_BIT )
-#define FW_DSTATE_MP1_WHISPER_MODE_MASK (1 <<
FW_DSTATE_MP1_WHISPER_MODE_BIT )
-#define FW_DSTATE_SOC_LIV_MIN_MASK (1 <<
FW_DSTATE_SOC_LIV_MIN_BIT )
-#define FW_DSTATE_SOC_PLL_PWRDN_MASK (1 <<
FW_DSTATE_SOC_PLL_PWRDN_BIT )
-#define FW_DSTATE_MEM_PLL_PWRDN_MASK (1 <<
FW_DSTATE_MEM_PLL_PWRDN_BIT )
-#define FW_DSTATE_OPTIMIZE_MALL_REFRESH_MASK (1 <<
FW_DSTATE_OPTIMIZE_MALL_REFRESH_BIT )
-#define FW_DSTATE_MEM_PSI_MASK (1 <<
FW_DSTATE_MEM_PSI_BIT )
-#define FW_DSTATE_HSR_NON_STROBE_MASK (1 <<
FW_DSTATE_HSR_NON_STROBE_BIT )
-#define FW_DSTATE_MP0_ENTER_WFI_MASK (1 <<
FW_DSTATE_MP0_ENTER_WFI_BIT )
+#define FW_DSTATE_SOC_ULV_MASK (1 <<
FW_DSTATE_SOC_ULV_BIT)
+#define FW_DSTATE_G6_HSR_MASK (1 <<
FW_DSTATE_G6_HSR_BIT)
+#define FW_DSTATE_G6_PHY_VDDCI_OFF_MASK (1 <<
FW_DSTATE_G6_PHY_VDDCI_OFF_BIT)
+#define FW_DSTATE_MP1_DS_MASK (1 <<
FW_DSTATE_MP1_DS_BIT)
+#define FW_DSTATE_MP0_DS_MASK (1 <<
FW_DSTATE_MP0_DS_BIT)
+#define FW_DSTATE_SMN_DS_MASK (1 <<
FW_DSTATE_SMN_DS_BIT)
+#define FW_DSTATE_MP1_WHISPER_MODE_MASK (1 <<
FW_DSTATE_MP1_WHISPER_MODE_BIT)
+#define FW_DSTATE_SOC_LIV_MIN_MASK (1 <<
FW_DSTATE_SOC_LIV_MIN_BIT)
+#define FW_DSTATE_SOC_PLL_PWRDN_MASK (1 <<
FW_DSTATE_SOC_PLL_PWRDN_BIT)
+#define FW_DSTATE_MEM_PLL_PWRDN_MASK (1 <<
FW_DSTATE_MEM_PLL_PWRDN_BIT)
+#define FW_DSTATE_OPTIMIZE_MALL_REFRESH_MASK (1 <<
FW_DSTATE_OPTIMIZE_MALL_REFRESH_BIT)
+#define FW_DSTATE_MEM_PSI_MASK (1 <<
FW_DSTATE_MEM_PSI_BIT)
+#define FW_DSTATE_HSR_NON_STROBE_MASK (1 <<
FW_DSTATE_HSR_NON_STROBE_BIT)
+#define FW_DSTATE_MP0_ENTER_WFI_MASK (1 <<
FW_DSTATE_MP0_ENTER_WFI_BIT)
// GFX GPO Feature Contains PACE and DEM sub features
#define GFX_GPO_PACE_BIT 0
#define GFX_GPO_DEM_BIT 1
#define GFX_GPO_PACE_MASK (1 << GFX_GPO_PACE_BIT)
-#define GFX_GPO_DEM_MASK (1 << GFX_GPO_DEM_BIT )
+#define GFX_GPO_DEM_MASK (1 << GFX_GPO_DEM_BIT)
#define GPO_UPDATE_REQ_UCLKDPM_MASK 0x1
#define GPO_UPDATE_REQ_FCLKDPM_MASK 0x2
@@ -312,10 +312,10 @@ typedef enum {
I2C_CONTROLLER_NAME_VR_VDDCI,
I2C_CONTROLLER_NAME_VR_MVDD,
I2C_CONTROLLER_NAME_LIQUID0,
- I2C_CONTROLLER_NAME_LIQUID1,
+ I2C_CONTROLLER_NAME_LIQUID1,
I2C_CONTROLLER_NAME_PLX,
I2C_CONTROLLER_NAME_OTHER,
- I2C_CONTROLLER_NAME_COUNT,
+ I2C_CONTROLLER_NAME_COUNT,
} I2cControllerName_e;
typedef enum {
@@ -325,10 +325,10 @@ typedef enum {
I2C_CONTROLLER_THROTTLER_VR_VDDCI,
I2C_CONTROLLER_THROTTLER_VR_MVDD,
I2C_CONTROLLER_THROTTLER_LIQUID0,
- I2C_CONTROLLER_THROTTLER_LIQUID1,
+ I2C_CONTROLLER_THROTTLER_LIQUID1,
I2C_CONTROLLER_THROTTLER_PLX,
I2C_CONTROLLER_THROTTLER_INA3221,
- I2C_CONTROLLER_THROTTLER_COUNT,
+ I2C_CONTROLLER_THROTTLER_COUNT,
} I2cControllerThrottler_e;
typedef enum {
@@ -336,24 +336,24 @@ typedef enum {
I2C_CONTROLLER_PROTOCOL_VR_IR35217,
I2C_CONTROLLER_PROTOCOL_TMP_TMP102A,
I2C_CONTROLLER_PROTOCOL_INA3221,
- I2C_CONTROLLER_PROTOCOL_COUNT,
+ I2C_CONTROLLER_PROTOCOL_COUNT,
} I2cControllerProtocol_e;
typedef struct {
uint8_t Enabled;
uint8_t Speed;
- uint8_t SlaveAddress;
+ uint8_t SlaveAddress;
uint8_t ControllerPort;
uint8_t ControllerName;
uint8_t ThermalThrotter;
uint8_t I2cProtocol;
- uint8_t PaddingConfig;
+ uint8_t PaddingConfig;
} I2cControllerConfig_t;
typedef enum {
- I2C_PORT_SVD_SCL = 0,
- I2C_PORT_GPIO,
-} I2cPort_e;
+ I2C_PORT_SVD_SCL = 0,
+ I2C_PORT_GPIO,
+} I2cPort_e;
typedef enum {
I2C_SPEED_FAST_50K = 0, //50 Kbits/s
@@ -362,13 +362,13 @@ typedef enum {
I2C_SPEED_FAST_PLUS_1M, //1 Mbits/s (in fast mode)
I2C_SPEED_HIGH_1M, //1 Mbits/s (in high speed mode)
I2C_SPEED_HIGH_2M, //2.3 Mbits/s
- I2C_SPEED_COUNT,
+ I2C_SPEED_COUNT,
} I2cSpeed_e;
typedef enum {
I2C_CMD_READ = 0,
I2C_CMD_WRITE,
- I2C_CMD_COUNT,
+ I2C_CMD_COUNT,
} I2cCmdType_e;
typedef enum {
@@ -426,24 +426,24 @@ typedef enum {
PG_POWER_UP,
} PowerGatingSettings_e;
-typedef struct {
+typedef struct {
uint32_t a; // store in IEEE float format in this variable
uint32_t b; // store in IEEE float format in this variable
uint32_t c; // store in IEEE float format in this variable
} QuadraticInt_t;
-typedef struct {
+typedef struct {
uint32_t a; // store in fixed point, [31:20] signed integer, [19:0]
fractional bits
uint32_t b; // store in fixed point, [31:20] signed integer, [19:0]
fractional bits
uint32_t c; // store in fixed point, [31:20] signed integer, [19:0]
fractional bits
} QuadraticFixedPoint_t;
-typedef struct {
+typedef struct {
uint32_t m; // store in IEEE float format in this variable
uint32_t b; // store in IEEE float format in this variable
} LinearInt_t;
-typedef struct {
+typedef struct {
uint32_t a; // store in IEEE float format in this variable
uint32_t b; // store in IEEE float format in this variable
uint32_t c; // store in IEEE float format in this variable
@@ -455,7 +455,7 @@ typedef enum {
PIECEWISE_LINEAR_FUSED_MODEL = 0,
PIECEWISE_LINEAR_PP_MODEL,
QUADRATIC_PP_MODEL,
- PERPART_PIECEWISE_LINEAR_PP_MODEL,
+ PERPART_PIECEWISE_LINEAR_PP_MODEL,
} DfllDroopModelSelect_e;
typedef struct {
@@ -464,10 +464,10 @@ typedef struct {
}PiecewiseLinearDroopInt_t;
typedef enum {
- GFXCLK_SOURCE_PLL = 0,
- GFXCLK_SOURCE_DFLL,
- GFXCLK_SOURCE_COUNT,
-} GFXCLK_SOURCE_e;
+ GFXCLK_SOURCE_PLL = 0,
+ GFXCLK_SOURCE_DFLL,
+ GFXCLK_SOURCE_COUNT,
+} GFXCLK_SOURCE_e;
//Only Clks that have DPM descriptors are listed here
typedef enum {
@@ -552,18 +552,18 @@ typedef struct {
uint8_t VoltageMode; // 0 - AVFS only, 1-
min(AVFS,SS), 2-SS only
uint8_t SnapToDiscrete; // 0 - Fine grained DPM, 1 -
Discrete DPM
uint8_t NumDiscreteLevels; // Set to 2 (Fmin, Fmax) when
using fine grained DPM, otherwise set to # discrete levels used
- uint8_t Padding;
+ uint8_t Padding;
LinearInt_t ConversionToAvfsClk; // Transfer function to AVFS
Clock (GHz->GHz)
QuadraticInt_t SsCurve; // Slow-slow curve (GHz->V)
uint16_t SsFmin; // Fmin for SS curve. If SS curve
is selected, will use V@SSFmin for F <= Fmin
- uint16_t Padding16;
+ uint16_t Padding16;
} DpmDescriptor_t;
typedef enum {
PPT_THROTTLER_PPT0,
PPT_THROTTLER_PPT1,
PPT_THROTTLER_PPT2,
- PPT_THROTTLER_PPT3,
+ PPT_THROTTLER_PPT3,
PPT_THROTTLER_COUNT
} PPT_THROTTLER_e;
@@ -574,9 +574,9 @@ typedef enum {
TEMP_VR_GFX,
TEMP_VR_MEM0,
TEMP_VR_MEM1,
- TEMP_VR_SOC,
+ TEMP_VR_SOC,
TEMP_LIQUID0,
- TEMP_LIQUID1,
+ TEMP_LIQUID1,
TEMP_PLX,
TEMP_COUNT,
} TEMP_e;
@@ -592,7 +592,7 @@ typedef enum {
CUSTOMER_VARIANT_FALCON,
CUSTOMER_VARIANT_COUNT,
} CUSTOMER_VARIANT_e;
-
+
// Used for 2-step UCLK DPM change workaround
typedef struct {
uint16_t Fmin;
@@ -623,14 +623,14 @@ typedef struct {
// SECTION: Power Configuration
uint8_t TotalPowerConfig; //0-TDP, 1-TGP, 2-TCP Estimated,
3-TCP Measured. Use defines from PwrConfig_e
- uint8_t TotalPowerPadding[3];
+ uint8_t TotalPowerPadding[3];
// SECTION: APCC Settings
uint32_t ApccPlusResidencyLimit;
//SECTION: SMNCLK DPM
- uint16_t SmnclkDpmFreq [NUM_SMNCLK_DPM_LEVELS]; //
in MHz
- uint16_t SmnclkDpmVoltage [NUM_SMNCLK_DPM_LEVELS]; //
mV(Q2)
+ uint16_t SmnclkDpmFreq[NUM_SMNCLK_DPM_LEVELS]; // in MHz
+ uint16_t SmnclkDpmVoltage[NUM_SMNCLK_DPM_LEVELS]; //
mV(Q2)
uint32_t PaddingAPCC;
uint16_t
PerPartDroopVsetGfxDfll[NUM_PIECE_WISE_LINEAR_DROOP_MODEL_VF_POINTS];
//In mV(Q2)
@@ -639,14 +639,14 @@ typedef struct {
// SECTION: Throttler settings
uint32_t ThrottlerControlMask; // See Throtter masks defines
- // SECTION: FW DSTATE Settings
+ // SECTION: FW DSTATE Settings
uint32_t FwDStateMask; // See FW DState masks defines
// SECTION: ULV Settings
uint16_t UlvVoltageOffsetSoc; // In mV(Q2)
uint16_t UlvVoltageOffsetGfx; // In mV(Q2)
- uint16_t MinVoltageUlvGfx; // In mV(Q2) Minimum Voltage ("Vmin")
of VDD_GFX in ULV mode
+ uint16_t MinVoltageUlvGfx; // In mV(Q2) Minimum Voltage ("Vmin")
of VDD_GFX in ULV mode
uint16_t MinVoltageUlvSoc; // In mV(Q2) Minimum Voltage ("Vmin")
of VDD_SOC in ULV mode
uint16_t SocLIVmin; // In mV(Q2) Long Idle Vmin (deep
ULV), for VDD_SOC
@@ -678,62 +678,62 @@ typedef struct {
//SECTION: DPM Config 1
DpmDescriptor_t DpmDescriptor[PPCLK_COUNT];
- uint16_t FreqTableGfx [NUM_GFXCLK_DPM_LEVELS ]; // In
MHz
- uint16_t FreqTableVclk [NUM_VCLK_DPM_LEVELS ]; // In
MHz
- uint16_t FreqTableDclk [NUM_DCLK_DPM_LEVELS ]; // In
MHz
- uint16_t FreqTableSocclk [NUM_SOCCLK_DPM_LEVELS ]; // In
MHz
- uint16_t FreqTableUclk [NUM_UCLK_DPM_LEVELS ]; // In
MHz
- uint16_t FreqTableDcefclk [NUM_DCEFCLK_DPM_LEVELS ]; // In
MHz
- uint16_t FreqTableDispclk [NUM_DISPCLK_DPM_LEVELS ]; // In
MHz
- uint16_t FreqTablePixclk [NUM_PIXCLK_DPM_LEVELS ]; // In
MHz
- uint16_t FreqTablePhyclk [NUM_PHYCLK_DPM_LEVELS ]; // In
MHz
- uint16_t FreqTableDtbclk [NUM_DTBCLK_DPM_LEVELS ]; // In
MHz
- uint16_t FreqTableFclk [NUM_FCLK_DPM_LEVELS ]; // In
MHz
+ uint16_t FreqTableGfx[NUM_GFXCLK_DPM_LEVELS]; // In MHz
+ uint16_t FreqTableVclk[NUM_VCLK_DPM_LEVELS]; // In MHz
+ uint16_t FreqTableDclk[NUM_DCLK_DPM_LEVELS]; // In MHz
+ uint16_t FreqTableSocclk[NUM_SOCCLK_DPM_LEVELS]; // In MHz
+ uint16_t FreqTableUclk[NUM_UCLK_DPM_LEVELS]; // In MHz
+ uint16_t FreqTableDcefclk[NUM_DCEFCLK_DPM_LEVELS]; // In
MHz
+ uint16_t FreqTableDispclk[NUM_DISPCLK_DPM_LEVELS]; // In
MHz
+ uint16_t FreqTablePixclk[NUM_PIXCLK_DPM_LEVELS]; // In MHz
+ uint16_t FreqTablePhyclk[NUM_PHYCLK_DPM_LEVELS]; // In MHz
+ uint16_t FreqTableDtbclk[NUM_DTBCLK_DPM_LEVELS]; // In MHz
+ uint16_t FreqTableFclk[NUM_FCLK_DPM_LEVELS]; // In MHz
uint32_t Paddingclks;
DroopInt_t
PerPartDroopModelGfxDfll[NUM_PIECE_WISE_LINEAR_DROOP_MODEL_VF_POINTS];
//GHz ->Vstore in IEEE float format
- uint32_t DcModeMaxFreq [PPCLK_COUNT ]; // In
MHz
+ uint32_t DcModeMaxFreq[PPCLK_COUNT]; // In MHz
- uint8_t FreqTableUclkDiv [NUM_UCLK_DPM_LEVELS ]; //
0:Div-1, 1:Div-1/2, 2:Div-1/4, 3:Div-1/8
+ uint8_t FreqTableUclkDiv[NUM_UCLK_DPM_LEVELS]; // 0:Div-1,
1:Div-1/2, 2:Div-1/4, 3:Div-1/8
// Used for MALL performance boost
uint16_t FclkBoostFreq; // In
Mhz
uint16_t FclkParamPadding;
// SECTION: DPM Config 2
- uint16_t Mp0clkFreq [NUM_MP0CLK_DPM_LEVELS]; // in
MHz
- uint16_t Mp0DpmVoltage [NUM_MP0CLK_DPM_LEVELS]; //
mV(Q2)
- uint16_t MemVddciVoltage [NUM_UCLK_DPM_LEVELS]; //
mV(Q2)
- uint16_t MemMvddVoltage [NUM_UCLK_DPM_LEVELS]; //
mV(Q2)
+ uint16_t Mp0clkFreq[NUM_MP0CLK_DPM_LEVELS]; // in MHz
+ uint16_t Mp0DpmVoltage[NUM_MP0CLK_DPM_LEVELS]; // mV(Q2)
+ uint16_t MemVddciVoltage[NUM_UCLK_DPM_LEVELS]; //
mV(Q2)
+ uint16_t MemMvddVoltage[NUM_UCLK_DPM_LEVELS]; // mV(Q2)
// GFXCLK DPM
uint16_t GfxclkFgfxoffEntry; // in Mhz
- uint16_t GfxclkFinit; // in Mhz
- uint16_t GfxclkFidle; // in MHz
+ uint16_t GfxclkFinit; // in Mhz
+ uint16_t GfxclkFidle; // in MHz
uint8_t GfxclkSource; // 0 = PLL, 1 = DFLL
uint8_t GfxclkPadding;
- // GFX GPO
+ // GFX GPO
uint8_t GfxGpoSubFeatureMask; // bit 0 = PACE, bit 1 = DEM
uint8_t GfxGpoEnabledWorkPolicyMask; //Any policy that GPO
can be enabled
uint8_t GfxGpoDisabledWorkPolicyMask; //Any policy that GPO
can be disabled
uint8_t GfxGpoPadding[1];
uint32_t GfxGpoVotingAllow; //For indicating which feature
changes should result in a GPO table recalculation
- uint32_t GfxGpoPadding32[4];
+ uint32_t GfxGpoPadding32[4];
uint16_t GfxDcsFopt; // Optimal GFXCLK for DCS in
Mhz
uint16_t GfxDcsFclkFopt; // Optimal FCLK for DCS in Mhz
uint16_t GfxDcsUclkFopt; // Optimal UCLK for DCS in Mhz
-
- uint16_t DcsGfxOffVoltage; //Voltage in mV(Q2) applied to
VDDGFX when entering DCS GFXOFF phase
+
+ uint16_t DcsGfxOffVoltage; //Voltage in mV(Q2) applied to
VDDGFX when entering DCS GFXOFF phase
uint16_t DcsMinGfxOffTime; //Minimum amount of time PMFW
shuts GFX OFF as part of GFX DCS phase
uint16_t DcsMaxGfxOffTime; //Maximum amount of time PMFW
can shut GFX OFF as part of GFX DCS phase at a stretch.
uint32_t DcsMinCreditAccum; //Min amount of positive credit
accumulation before waking GFX up as part of DCS.
- uint16_t DcsExitHysteresis; //The min amount of time power
credit accumulator should have a value > 0 before SMU exits the DCS
throttling phase.
+ uint16_t DcsExitHysteresis; //The min amount of time power
credit accumulator should have a value > 0 before SMU exits the DCS
throttling phase.
uint16_t DcsTimeout; //This is the amount of time
SMU FW waits for RLC to put GFX into GFXOFF before reverting to the
fallback mechanism of throttling GFXCLK to Fmin.
uint32_t DcsParamPadding[5];
@@ -743,9 +743,9 @@ typedef struct {
// UCLK section
uint8_t LowestUclkReservedForUlv; // Set this to 1 if UCLK DPM0
is reserved for ULV-mode only
uint8_t PaddingMem[3];
-
- uint8_t UclkDpmPstates [NUM_UCLK_DPM_LEVELS]; // 4 DPM
states, 0-P0, 1-P1, 2-P2, 3-P3.
-
+
+ uint8_t UclkDpmPstates[NUM_UCLK_DPM_LEVELS]; // 4 DPM
states, 0-P0, 1-P1, 2-P2, 3-P3.
+
// Used for 2-Step UCLK change workaround
UclkDpmChangeRange_t UclkDpmSrcFreqRange; // In Mhz
UclkDpmChangeRange_t UclkDpmTargFreqRange; // In Mhz
@@ -755,8 +755,8 @@ typedef struct {
// Link DPM Settings
uint8_t PcieGenSpeed[NUM_LINK_LEVELS]; ///<
0:PciE-gen1 1:PciE-gen2 2:PciE-gen3 3:PciE-gen4
uint8_t PcieLaneCount[NUM_LINK_LEVELS]; ///< 1=x1,
2=x2, 3=x4, 4=x8, 5=x12, 6=x16
- uint16_t LclkFreq[NUM_LINK_LEVELS];
-
+ uint16_t LclkFreq[NUM_LINK_LEVELS];
+
// SECTION: Fan Control
uint16_t FanStopTemp; //Celcius
uint16_t FanStartTemp; //Celcius
@@ -767,36 +767,36 @@ typedef struct {
uint16_t FanAcousticLimitRpm;
uint16_t FanThrottlingRpm;
uint16_t FanMaximumRpm;
- uint16_t MGpuFanBoostLimitRpm;
+ uint16_t MGpuFanBoostLimitRpm;
uint16_t FanTargetTemperature;
uint16_t FanTargetGfxclk;
uint16_t FanPadding16;
uint8_t FanTempInputSelect;
uint8_t FanPadding;
- uint8_t FanZeroRpmEnable;
+ uint8_t FanZeroRpmEnable;
uint8_t FanTachEdgePerRev;
-
+
// The following are AFC override parameters. Leave at 0 to use FW
defaults.
int16_t FuzzyFan_ErrorSetDelta;
int16_t FuzzyFan_ErrorRateSetDelta;
int16_t FuzzyFan_PwmSetDelta;
uint16_t FuzzyFan_Reserved;
- // SECTION: AVFS
+ // SECTION: AVFS
// Overrides
uint8_t OverrideAvfsGb[AVFS_VOLTAGE_COUNT];
- uint8_t dBtcGbGfxDfllModelSelect; //0 -> fused piece-wise
model, 1 -> piece-wise linear(PPTable), 2 -> quadratic model(PPTable)
+ uint8_t dBtcGbGfxDfllModelSelect; //0 -> fused piece-wise
model, 1 -> piece-wise linear(PPTable), 2 -> quadratic model(PPTable)
uint8_t Padding8_Avfs;
- QuadraticInt_t qAvfsGb[AVFS_VOLTAGE_COUNT]; // GHz->V
Override of fused curve
+ QuadraticInt_t qAvfsGb[AVFS_VOLTAGE_COUNT]; // GHz->V
Override of fused curve
DroopInt_t dBtcGbGfxPll; // GHz->V BtcGb
DroopInt_t dBtcGbGfxDfll; // GHz->V BtcGb
DroopInt_t dBtcGbSoc; // GHz->V BtcGb
- LinearInt_t qAgingGb[AVFS_VOLTAGE_COUNT]; // GHz->V
+ LinearInt_t qAgingGb[AVFS_VOLTAGE_COUNT]; // GHz->V
PiecewiseLinearDroopInt_t PiecewiseLinearDroopIntGfxDfll; //GHz
->Vstore in IEEE float format
- QuadraticInt_t qStaticVoltageOffset[AVFS_VOLTAGE_COUNT]; // GHz->V
+ QuadraticInt_t qStaticVoltageOffset[AVFS_VOLTAGE_COUNT]; // GHz->V
uint16_t DcTol[AVFS_VOLTAGE_COUNT]; // mV Q2
@@ -807,17 +807,17 @@ typedef struct {
uint16_t DcBtcMax[AVFS_VOLTAGE_COUNT]; // mV Q2
uint16_t DcBtcGb[AVFS_VOLTAGE_COUNT]; // mV Q2
-
+
// SECTION: XGMI
uint8_t XgmiDpmPstates[NUM_XGMI_LEVELS]; // 2 DPM states,
high and low. 0-P0, 1-P1, 2-P2, 3-P3.
uint8_t XgmiDpmSpare[2];
// SECTION: Advanced Options
uint32_t DebugOverrides;
- QuadraticInt_t ReservedEquation0;
- QuadraticInt_t ReservedEquation1;
- QuadraticInt_t ReservedEquation2;
- QuadraticInt_t ReservedEquation3;
+ QuadraticInt_t ReservedEquation0;
+ QuadraticInt_t ReservedEquation1;
+ QuadraticInt_t ReservedEquation2;
+ QuadraticInt_t ReservedEquation3;
// SECTION: Sku Reserved
uint8_t CustomerVariant;
@@ -825,17 +825,17 @@ typedef struct {
//VC BTC parameters are only applicable to VDD_GFX domain
uint8_t VcBtcEnabled;
uint16_t VcBtcVminT0; // T0_VMIN
- uint16_t VcBtcFixedVminAgingOffset; //
FIXED_VMIN_AGING_OFFSET
- uint16_t VcBtcVmin2PsmDegrationGb; //
VMIN_TO_PSM_DEGRADATION_GB
+ uint16_t VcBtcFixedVminAgingOffset; //
FIXED_VMIN_AGING_OFFSET
+ uint16_t VcBtcVmin2PsmDegrationGb; //
VMIN_TO_PSM_DEGRADATION_GB
uint32_t VcBtcPsmA; // A_PSM
uint32_t VcBtcPsmB; // B_PSM
uint32_t VcBtcVminA; // A_VMIN
- uint32_t VcBtcVminB; // B_VMIN
-
+ uint32_t VcBtcVminB; // B_VMIN
+
//GPIO Board feature
uint16_t LedGpio; //GeneriA GPIO flag used to
control the radeon LEDs
- uint16_t GfxPowerStagesGpio; //Genlk_vsync GPIO flag used to
control gfx power stages
-
+ uint16_t GfxPowerStagesGpio; //Genlk_vsync GPIO flag used to
control gfx power stages
+
uint32_t SkuReserved[8];
@@ -845,7 +845,7 @@ typedef struct {
uint32_t GamingClk[6];
// SECTION: I2C Control
- I2cControllerConfig_t I2cControllers[NUM_I2C_CONTROLLERS];
+ I2cControllerConfig_t I2cControllers[NUM_I2C_CONTROLLERS];
uint8_t GpioScl; // GPIO Number for SCL Line, used only for
CKSVII2C1
uint8_t GpioSda; // GPIO Number for SDA Line, used only for
CKSVII2C1
@@ -875,21 +875,21 @@ typedef struct {
uint16_t Mem0MaxCurrent; // in Amps
int8_t Mem0Offset; // in Amps
uint8_t Padding_TelemetryMem0;
-
+
uint16_t Mem1MaxCurrent; // in Amps
int8_t Mem1Offset; // in Amps
uint8_t Padding_TelemetryMem1;
uint32_t MvddRatio; // This is used for MVDD Svi2 Div Ratio
workaround. It has 16 fractional bits (Q16.16)
-
+
// SECTION: GPIO Settings
uint8_t AcDcGpio; // GPIO pin configured for AC/DC
switching
uint8_t AcDcPolarity; // GPIO polarity for AC/DC switching
uint8_t VR0HotGpio; // GPIO pin configured for VR0 HOT
event
uint8_t VR0HotPolarity; // GPIO polarity for VR0 HOT event
- uint8_t VR1HotGpio; // GPIO pin configured for VR1 HOT
event
- uint8_t VR1HotPolarity; // GPIO polarity for VR1 HOT event
+ uint8_t VR1HotGpio; // GPIO pin configured for VR1 HOT
event
+ uint8_t VR1HotPolarity; // GPIO polarity for VR1 HOT event
uint8_t GthrGpio; // GPIO pin configured for GTHR Event
uint8_t GthrPolarity; // replace GPIO polarity for GTHR
@@ -914,7 +914,7 @@ typedef struct {
uint8_t DfllGfxclkSpreadEnabled; // on or off
uint8_t DfllGfxclkSpreadPercent; // Q4.4
uint16_t DfllGfxclkSpreadFreq; // kHz
-
+
// UCLK Spread Spectrum
uint16_t UclkSpreadPadding;
uint16_t UclkSpreadFreq; // kHz
@@ -923,23 +923,23 @@ typedef struct {
uint8_t FclkSpreadEnabled; // on or off
uint8_t FclkSpreadPercent; // Q4.4
uint16_t FclkSpreadFreq; // kHz
-
+
// Section: Memory Config
- uint32_t MemoryChannelEnabled; // For DRAM use only, Max 32
channels enabled bit mask.
-
+ uint32_t MemoryChannelEnabled; // For DRAM use only, Max 32
channels enabled bit mask.
+
uint8_t DramBitWidth; // For DRAM use only. See Dram Bit width
type defines
uint8_t PaddingMem1[3];
// Section: Total Board Power
uint16_t TotalBoardPower; //Only needed for TCP Estimated
case, where TCP = TGP+Total Board Power
- uint16_t BoardPowerPadding;
-
+ uint16_t BoardPowerPadding;
+
// SECTION: XGMI Training
- uint8_t XgmiLinkSpeed [NUM_XGMI_PSTATE_LEVELS];
- uint8_t XgmiLinkWidth [NUM_XGMI_PSTATE_LEVELS];
+ uint8_t XgmiLinkSpeed[NUM_XGMI_PSTATE_LEVELS];
+ uint8_t XgmiLinkWidth[NUM_XGMI_PSTATE_LEVELS];
- uint16_t XgmiFclkFreq [NUM_XGMI_PSTATE_LEVELS];
- uint16_t XgmiSocVoltage [NUM_XGMI_PSTATE_LEVELS];
+ uint16_t XgmiFclkFreq[NUM_XGMI_PSTATE_LEVELS];
+ uint16_t XgmiSocVoltage[NUM_XGMI_PSTATE_LEVELS];
// SECTION: UMC feature flags
uint8_t HsrEnabled;
@@ -947,7 +947,7 @@ typedef struct {
uint8_t PaddingUmcFlags[2];
// UCLK Spread Spectrum
- uint8_t UclkSpreadPercent[16];
+ uint8_t UclkSpreadPercent[16];
// SECTION: Board Reserved
uint32_t BoardReserved[11];
@@ -983,14 +983,14 @@ typedef struct {
// SECTION: Power Configuration
uint8_t TotalPowerConfig; //0-TDP, 1-TGP, 2-TCP Estimated,
3-TCP Measured. Use defines from PwrConfig_e
- uint8_t TotalPowerPadding[3];
+ uint8_t TotalPowerPadding[3];
// SECTION: APCC Settings
uint32_t ApccPlusResidencyLimit;
//SECTION: SMNCLK DPM
- uint16_t SmnclkDpmFreq [NUM_SMNCLK_DPM_LEVELS]; //
in MHz
- uint16_t SmnclkDpmVoltage [NUM_SMNCLK_DPM_LEVELS]; //
mV(Q2)
+ uint16_t SmnclkDpmFreq[NUM_SMNCLK_DPM_LEVELS]; // in MHz
+ uint16_t SmnclkDpmVoltage[NUM_SMNCLK_DPM_LEVELS]; //
mV(Q2)
uint32_t PaddingAPCC;
uint16_t
PerPartDroopVsetGfxDfll[NUM_PIECE_WISE_LINEAR_DROOP_MODEL_VF_POINTS];
//In mV(Q2)
@@ -999,14 +999,14 @@ typedef struct {
// SECTION: Throttler settings
uint32_t ThrottlerControlMask; // See Throtter masks defines
- // SECTION: FW DSTATE Settings
+ // SECTION: FW DSTATE Settings
uint32_t FwDStateMask; // See FW DState masks defines
// SECTION: ULV Settings
uint16_t UlvVoltageOffsetSoc; // In mV(Q2)
uint16_t UlvVoltageOffsetGfx; // In mV(Q2)
- uint16_t MinVoltageUlvGfx; // In mV(Q2) Minimum Voltage ("Vmin")
of VDD_GFX in ULV mode
+ uint16_t MinVoltageUlvGfx; // In mV(Q2) Minimum Voltage ("Vmin")
of VDD_GFX in ULV mode
uint16_t MinVoltageUlvSoc; // In mV(Q2) Minimum Voltage ("Vmin")
of VDD_SOC in ULV mode
uint16_t SocLIVmin;
@@ -1038,42 +1038,42 @@ typedef struct {
//SECTION: DPM Config 1
DpmDescriptor_t DpmDescriptor[PPCLK_COUNT];
- uint16_t FreqTableGfx [NUM_GFXCLK_DPM_LEVELS ]; // In
MHz
- uint16_t FreqTableVclk [NUM_VCLK_DPM_LEVELS ]; // In
MHz
- uint16_t FreqTableDclk [NUM_DCLK_DPM_LEVELS ]; // In
MHz
- uint16_t FreqTableSocclk [NUM_SOCCLK_DPM_LEVELS ]; // In
MHz
- uint16_t FreqTableUclk [NUM_UCLK_DPM_LEVELS ]; // In
MHz
- uint16_t FreqTableDcefclk [NUM_DCEFCLK_DPM_LEVELS ]; // In
MHz
- uint16_t FreqTableDispclk [NUM_DISPCLK_DPM_LEVELS ]; // In
MHz
- uint16_t FreqTablePixclk [NUM_PIXCLK_DPM_LEVELS ]; // In
MHz
- uint16_t FreqTablePhyclk [NUM_PHYCLK_DPM_LEVELS ]; // In
MHz
- uint16_t FreqTableDtbclk [NUM_DTBCLK_DPM_LEVELS ]; // In
MHz
- uint16_t FreqTableFclk [NUM_FCLK_DPM_LEVELS ]; // In
MHz
+ uint16_t FreqTableGfx[NUM_GFXCLK_DPM_LEVELS]; // In MHz
+ uint16_t FreqTableVclk[NUM_VCLK_DPM_LEVELS]; // In MHz
+ uint16_t FreqTableDclk[NUM_DCLK_DPM_LEVELS]; // In MHz
+ uint16_t FreqTableSocclk[NUM_SOCCLK_DPM_LEVELS]; // In MHz
+ uint16_t FreqTableUclk[NUM_UCLK_DPM_LEVELS]; // In MHz
+ uint16_t FreqTableDcefclk[NUM_DCEFCLK_DPM_LEVELS]; // In
MHz
+ uint16_t FreqTableDispclk[NUM_DISPCLK_DPM_LEVELS]; // In
MHz
+ uint16_t FreqTablePixclk[NUM_PIXCLK_DPM_LEVELS]; // In MHz
+ uint16_t FreqTablePhyclk[NUM_PHYCLK_DPM_LEVELS]; // In MHz
+ uint16_t FreqTableDtbclk[NUM_DTBCLK_DPM_LEVELS]; // In MHz
+ uint16_t FreqTableFclk[NUM_FCLK_DPM_LEVELS]; // In MHz
uint32_t Paddingclks;
DroopInt_t
PerPartDroopModelGfxDfll[NUM_PIECE_WISE_LINEAR_DROOP_MODEL_VF_POINTS];
//GHz ->Vstore in IEEE float format
- uint32_t DcModeMaxFreq [PPCLK_COUNT ]; // In
MHz
+ uint32_t DcModeMaxFreq[PPCLK_COUNT]; // In MHz
- uint8_t FreqTableUclkDiv [NUM_UCLK_DPM_LEVELS ]; //
0:Div-1, 1:Div-1/2, 2:Div-1/4, 3:Div-1/8
+ uint8_t FreqTableUclkDiv[NUM_UCLK_DPM_LEVELS]; // 0:Div-1,
1:Div-1/2, 2:Div-1/4, 3:Div-1/8
// Used for MALL performance boost
uint16_t FclkBoostFreq; // In
Mhz
uint16_t FclkParamPadding;
// SECTION: DPM Config 2
- uint16_t Mp0clkFreq [NUM_MP0CLK_DPM_LEVELS]; // in
MHz
- uint16_t Mp0DpmVoltage [NUM_MP0CLK_DPM_LEVELS]; //
mV(Q2)
- uint16_t MemVddciVoltage [NUM_UCLK_DPM_LEVELS]; //
mV(Q2)
- uint16_t MemMvddVoltage [NUM_UCLK_DPM_LEVELS]; //
mV(Q2)
+ uint16_t Mp0clkFreq[NUM_MP0CLK_DPM_LEVELS]; // in MHz
+ uint16_t Mp0DpmVoltage[NUM_MP0CLK_DPM_LEVELS]; // mV(Q2)
+ uint16_t MemVddciVoltage[NUM_UCLK_DPM_LEVELS]; //
mV(Q2)
+ uint16_t MemMvddVoltage[NUM_UCLK_DPM_LEVELS]; // mV(Q2)
// GFXCLK DPM
uint16_t GfxclkFgfxoffEntry; // in Mhz
- uint16_t GfxclkFinit; // in Mhz
- uint16_t GfxclkFidle; // in MHz
+ uint16_t GfxclkFinit; // in Mhz
+ uint16_t GfxclkFidle; // in MHz
uint8_t GfxclkSource; // 0 = PLL, 1 = DFLL
uint8_t GfxclkPadding;
- // GFX GPO
+ // GFX GPO
uint8_t GfxGpoSubFeatureMask; // bit 0 = PACE, bit 1 = DEM
uint8_t GfxGpoEnabledWorkPolicyMask; //Any policy that GPO
can be enabled
uint8_t GfxGpoDisabledWorkPolicyMask; //Any policy that GPO
can be disabled
@@ -1086,14 +1086,14 @@ typedef struct {
uint16_t GfxDcsFclkFopt; // Optimal FCLK for DCS in Mhz
uint16_t GfxDcsUclkFopt; // Optimal UCLK for DCS in Mhz
- uint16_t DcsGfxOffVoltage; //Voltage in mV(Q2) applied to
VDDGFX when entering DCS GFXOFF phase
+ uint16_t DcsGfxOffVoltage; //Voltage in mV(Q2) applied to
VDDGFX when entering DCS GFXOFF phase
uint16_t DcsMinGfxOffTime; //Minimum amount of time PMFW
shuts GFX OFF as part of GFX DCS phase
uint16_t DcsMaxGfxOffTime; //Maximum amount of time PMFW
can shut GFX OFF as part of GFX DCS phase at a stretch.
uint32_t DcsMinCreditAccum; //Min amount of positive credit
accumulation before waking GFX up as part of DCS.
- uint16_t DcsExitHysteresis; //The min amount of time power
credit accumulator should have a value > 0 before SMU exits the DCS
throttling phase.
+ uint16_t DcsExitHysteresis; //The min amount of time power
credit accumulator should have a value > 0 before SMU exits the DCS
throttling phase.
uint16_t DcsTimeout; //This is the amount of time
SMU FW waits for RLC to put GFX into GFXOFF before reverting to the
fallback mechanism of throttling GFXCLK to Fmin.
uint32_t DcsParamPadding[5];
@@ -1104,7 +1104,7 @@ typedef struct {
uint8_t LowestUclkReservedForUlv; // Set this to 1 if UCLK DPM0
is reserved for ULV-mode only
uint8_t PaddingMem[3];
- uint8_t UclkDpmPstates [NUM_UCLK_DPM_LEVELS]; // 4 DPM
states, 0-P0, 1-P1, 2-P2, 3-P3.
+ uint8_t UclkDpmPstates[NUM_UCLK_DPM_LEVELS]; // 4 DPM
states, 0-P0, 1-P1, 2-P2, 3-P3.
// Used for 2-Step UCLK change workaround
UclkDpmChangeRange_t UclkDpmSrcFreqRange; // In Mhz
@@ -1115,7 +1115,7 @@ typedef struct {
// Link DPM Settings
uint8_t PcieGenSpeed[NUM_LINK_LEVELS]; ///<
0:PciE-gen1 1:PciE-gen2 2:PciE-gen3 3:PciE-gen4
uint8_t PcieLaneCount[NUM_LINK_LEVELS]; ///< 1=x1,
2=x2, 3=x4, 4=x8, 5=x12, 6=x16
- uint16_t LclkFreq[NUM_LINK_LEVELS];
+ uint16_t LclkFreq[NUM_LINK_LEVELS];
// SECTION: Fan Control
uint16_t FanStopTemp; //Celcius
@@ -1133,7 +1133,7 @@ typedef struct {
uint16_t FanPadding16;
uint8_t FanTempInputSelect;
uint8_t FanPadding;
- uint8_t FanZeroRpmEnable;
+ uint8_t FanZeroRpmEnable;
uint8_t FanTachEdgePerRev;
// The following are AFC override parameters. Leave at 0 to use FW
defaults.
@@ -1142,21 +1142,21 @@ typedef struct {
int16_t FuzzyFan_PwmSetDelta;
uint16_t FuzzyFan_Reserved;
- // SECTION: AVFS
+ // SECTION: AVFS
// Overrides
uint8_t OverrideAvfsGb[AVFS_VOLTAGE_COUNT];
- uint8_t dBtcGbGfxDfllModelSelect; //0 -> fused piece-wise
model, 1 -> piece-wise linear(PPTable), 2 -> quadratic model(PPTable)
+ uint8_t dBtcGbGfxDfllModelSelect; //0 -> fused piece-wise
model, 1 -> piece-wise linear(PPTable), 2 -> quadratic model(PPTable)
uint8_t Padding8_Avfs;
- QuadraticInt_t qAvfsGb[AVFS_VOLTAGE_COUNT]; // GHz->V
Override of fused curve
+ QuadraticInt_t qAvfsGb[AVFS_VOLTAGE_COUNT]; // GHz->V
Override of fused curve
DroopInt_t dBtcGbGfxPll; // GHz->V BtcGb
DroopInt_t dBtcGbGfxDfll; // GHz->V BtcGb
DroopInt_t dBtcGbSoc; // GHz->V BtcGb
- LinearInt_t qAgingGb[AVFS_VOLTAGE_COUNT]; // GHz->V
+ LinearInt_t qAgingGb[AVFS_VOLTAGE_COUNT]; // GHz->V
PiecewiseLinearDroopInt_t PiecewiseLinearDroopIntGfxDfll; //GHz
->Vstore in IEEE float format
- QuadraticInt_t qStaticVoltageOffset[AVFS_VOLTAGE_COUNT]; // GHz->V
+ QuadraticInt_t qStaticVoltageOffset[AVFS_VOLTAGE_COUNT]; // GHz->V
uint16_t DcTol[AVFS_VOLTAGE_COUNT]; // mV Q2
@@ -1185,16 +1185,16 @@ typedef struct {
//VC BTC parameters are only applicable to VDD_GFX domain
uint8_t VcBtcEnabled;
uint16_t VcBtcVminT0; // T0_VMIN
- uint16_t VcBtcFixedVminAgingOffset; //
FIXED_VMIN_AGING_OFFSET
- uint16_t VcBtcVmin2PsmDegrationGb; //
VMIN_TO_PSM_DEGRADATION_GB
+ uint16_t VcBtcFixedVminAgingOffset; //
FIXED_VMIN_AGING_OFFSET
+ uint16_t VcBtcVmin2PsmDegrationGb; //
VMIN_TO_PSM_DEGRADATION_GB
uint32_t VcBtcPsmA; // A_PSM
uint32_t VcBtcPsmB; // B_PSM
uint32_t VcBtcVminA; // A_VMIN
- uint32_t VcBtcVminB; // B_VMIN
+ uint32_t VcBtcVminB; // B_VMIN
//GPIO Board feature
uint16_t LedGpio; //GeneriA GPIO flag used to
control the radeon LEDs
- uint16_t GfxPowerStagesGpio; //Genlk_vsync GPIO flag used to
control gfx power stages
+ uint16_t GfxPowerStagesGpio; //Genlk_vsync GPIO flag used to
control gfx power stages
uint32_t SkuReserved[63];
@@ -1206,7 +1206,7 @@ typedef struct {
uint32_t GamingClk[6];
// SECTION: I2C Control
- I2cControllerConfig_t I2cControllers[NUM_I2C_CONTROLLERS];
+ I2cControllerConfig_t I2cControllers[NUM_I2C_CONTROLLERS];
uint8_t GpioScl; // GPIO Number for SCL Line, used only for
CKSVII2C1
uint8_t GpioSda; // GPIO Number for SDA Line, used only for
CKSVII2C1
@@ -1249,8 +1249,8 @@ typedef struct {
uint8_t VR0HotGpio; // GPIO pin configured for VR0 HOT
event
uint8_t VR0HotPolarity; // GPIO polarity for VR0 HOT event
- uint8_t VR1HotGpio; // GPIO pin configured for VR1 HOT
event
- uint8_t VR1HotPolarity; // GPIO polarity for VR1 HOT event
+ uint8_t VR1HotGpio; // GPIO pin configured for VR1 HOT
event
+ uint8_t VR1HotPolarity; // GPIO polarity for VR1 HOT event
uint8_t GthrGpio; // GPIO pin configured for GTHR Event
uint8_t GthrPolarity; // replace GPIO polarity for GTHR
@@ -1286,21 +1286,21 @@ typedef struct {
uint16_t FclkSpreadFreq; // kHz
// Section: Memory Config
- uint32_t MemoryChannelEnabled; // For DRAM use only, Max 32
channels enabled bit mask.
+ uint32_t MemoryChannelEnabled; // For DRAM use only, Max 32
channels enabled bit mask.
uint8_t DramBitWidth; // For DRAM use only. See Dram Bit width
type defines
uint8_t PaddingMem1[3];
// Section: Total Board Power
uint16_t TotalBoardPower; //Only needed for TCP Estimated
case, where TCP = TGP+Total Board Power
- uint16_t BoardPowerPadding;
+ uint16_t BoardPowerPadding;
// SECTION: XGMI Training
- uint8_t XgmiLinkSpeed [NUM_XGMI_PSTATE_LEVELS];
- uint8_t XgmiLinkWidth [NUM_XGMI_PSTATE_LEVELS];
+ uint8_t XgmiLinkSpeed[NUM_XGMI_PSTATE_LEVELS];
+ uint8_t XgmiLinkWidth[NUM_XGMI_PSTATE_LEVELS];
- uint16_t XgmiFclkFreq [NUM_XGMI_PSTATE_LEVELS];
- uint16_t XgmiSocVoltage [NUM_XGMI_PSTATE_LEVELS];
+ uint16_t XgmiFclkFreq[NUM_XGMI_PSTATE_LEVELS];
+ uint16_t XgmiSocVoltage[NUM_XGMI_PSTATE_LEVELS];
// SECTION: UMC feature flags
uint8_t HsrEnabled;
@@ -1328,15 +1328,15 @@ typedef struct {
uint16_t UclkAverageLpfTau;
uint16_t GfxActivityLpfTau;
uint16_t UclkActivityLpfTau;
- uint16_t SocketPowerLpfTau;
+ uint16_t SocketPowerLpfTau;
uint16_t VcnClkAverageLpfTau;
- uint16_t padding16;
+ uint16_t padding16;
} DriverSmuConfig_t;
typedef struct {
DriverSmuConfig_t DriverSmuConfig;
- uint32_t Spare[7];
+ uint32_t Spare[7];
// Padding - ignore
uint32_t MmHubPadding[8]; // SMU internal use
} DriverSmuConfigExternal_t;
@@ -1345,14 +1345,14 @@ typedef struct {
uint16_t GfxclkFmin; // MHz
uint16_t GfxclkFmax; // MHz
QuadraticInt_t CustomGfxVfCurve; // a: mV/MHz^2, b:
mv/MHz, c: mV
- uint16_t CustomCurveFmin; // MHz
- uint16_t UclkFmin; // MHz
+ uint16_t CustomCurveFmin; // MHz
+ uint16_t UclkFmin; // MHz
uint16_t UclkFmax; // MHz
int16_t OverDrivePct; // %
uint16_t FanMaximumRpm;
uint16_t FanMinimumPwm;
uint16_t FanAcousticLimitRpm;
- uint16_t FanTargetTemperature; // Degree Celcius
+ uint16_t FanTargetTemperature; // Degree Celcius
uint8_t FanLinearPwmPoints[NUM_OD_FAN_MAX_POINTS];
uint8_t FanLinearTempPoints[NUM_OD_FAN_MAX_POINTS];
uint16_t MaxOpTemp; // Degree Celcius
@@ -1361,13 +1361,13 @@ typedef struct {
uint8_t FanZeroRpmStopTemp;
uint8_t FanMode;
uint8_t Padding[1];
-} OverDriveTable_t;
+} OverDriveTable_t;
typedef struct {
OverDriveTable_t OverDriveTable;
- uint32_t Spare[8];
+ uint32_t Spare[8];
- uint32_t MmHubPadding[8]; // SMU internal use
+ uint32_t MmHubPadding[8]; // SMU internal use
} OverDriveTableExternal_t;
typedef struct {
@@ -1380,7 +1380,7 @@ typedef struct {
uint16_t AverageUclkFrequencyPreDs ;
uint16_t AverageUclkFrequencyPostDs ;
-
+
uint16_t AverageGfxActivity ;
uint16_t AverageUclkActivity ;
uint8_t CurrSocVoltageOffset ;
@@ -1393,14 +1393,14 @@ typedef struct {
uint16_t TemperatureMem ;
uint16_t TemperatureVrGfx ;
uint16_t TemperatureVrMem0 ;
- uint16_t TemperatureVrMem1 ;
+ uint16_t TemperatureVrMem1 ;
uint16_t TemperatureVrSoc ;
uint16_t TemperatureLiquid0 ;
- uint16_t TemperatureLiquid1 ;
+ uint16_t TemperatureLiquid1 ;
uint16_t TemperaturePlx ;
uint16_t Padding16 ;
- uint32_t ThrottlerStatus ;
-
+ uint32_t ThrottlerStatus ;
+
uint8_t LinkDpmLevel;
uint8_t CurrFanPwm;
uint16_t CurrFanSpeed;
@@ -1416,7 +1416,7 @@ typedef struct {
uint16_t AverageVclk0Frequency ;
uint16_t AverageDclk0Frequency ;
uint16_t AverageVclk1Frequency ;
- uint16_t AverageDclk1Frequency ;
+ uint16_t AverageDclk1Frequency ;
uint16_t VcnActivityPercentage ; //place holder, David N. to provide
full sequence
uint8_t PcieRate ;
uint8_t PcieWidth ;
@@ -1614,7 +1614,7 @@ typedef struct {
uint32_t Spare[1];
// Padding - ignore
- uint32_t MmHubPadding[8]; // SMU internal use
+ uint32_t MmHubPadding[8]; // SMU internal use
} SmuMetricsExternal_t;
typedef struct {
@@ -1622,7 +1622,7 @@ typedef struct {
uint16_t MaxClock; // This is either DCEFCLK or SOCCLK (in MHz)
uint16_t MinUclk;
uint16_t MaxUclk;
-
+
uint8_t WmSetting;
uint8_t Flags;
uint8_t Padding[2];
@@ -1658,7 +1658,7 @@ typedef struct {
typedef struct {
uint16_t avgPsmCount[67];
uint16_t minPsmCount[67];
- float avgPsmVoltage[67];
+ float avgPsmVoltage[67];
float minPsmVoltage[67];
} AvfsDebugTable_t;
@@ -1673,7 +1673,7 @@ typedef struct {
uint8_t Padding;
uint8_t AvfsEn[AVFS_VOLTAGE_COUNT];
-
+
uint8_t OverrideVFT[AVFS_VOLTAGE_COUNT];
uint8_t OverrideAvfsGb[AVFS_VOLTAGE_COUNT];
@@ -1729,7 +1729,7 @@ typedef struct {
uint8_t Gfx_IdleHystLimit;
uint8_t Gfx_FPS;
uint8_t Gfx_MinActiveFreqType;
- uint8_t Gfx_BoosterFreqType;
+ uint8_t Gfx_BoosterFreqType;
uint8_t Gfx_MinFreqStep; // Minimum delta between
current and target frequeny in order for FW to change clock.
uint16_t Gfx_MinActiveFreq; // MHz
uint16_t Gfx_BoosterFreq; // MHz
@@ -1744,7 +1744,7 @@ typedef struct {
uint8_t Fclk_IdleHystLimit;
uint8_t Fclk_FPS;
uint8_t Fclk_MinActiveFreqType;
- uint8_t Fclk_BoosterFreqType;
+ uint8_t Fclk_BoosterFreqType;
uint8_t Fclk_MinFreqStep; // Minimum delta between
current and target frequeny in order for FW to change clock.
uint16_t Fclk_MinActiveFreq; // MHz
uint16_t Fclk_BoosterFreq; // MHz
@@ -1780,19 +1780,19 @@ typedef struct {
typedef struct {
DpmActivityMonitorCoeffInt_t DpmActivityMonitorCoeffInt;
- uint32_t MmHubPadding[8]; // SMU internal use
+ uint32_t MmHubPadding[8]; // SMU internal use
} DpmActivityMonitorCoeffIntExternal_t;
// Workload bits
-#define WORKLOAD_PPLIB_DEFAULT_BIT 0
-#define WORKLOAD_PPLIB_FULL_SCREEN_3D_BIT 1
-#define WORKLOAD_PPLIB_POWER_SAVING_BIT 2
-#define WORKLOAD_PPLIB_VIDEO_BIT 3
-#define WORKLOAD_PPLIB_VR_BIT 4
-#define WORKLOAD_PPLIB_COMPUTE_BIT 5
-#define WORKLOAD_PPLIB_CUSTOM_BIT 6
-#define WORKLOAD_PPLIB_W3D_BIT 7
-#define WORKLOAD_PPLIB_COUNT 8
+#define WORKLOAD_PPLIB_DEFAULT_BIT 0
+#define WORKLOAD_PPLIB_FULL_SCREEN_3D_BIT 1
+#define WORKLOAD_PPLIB_POWER_SAVING_BIT 2
+#define WORKLOAD_PPLIB_VIDEO_BIT 3
+#define WORKLOAD_PPLIB_VR_BIT 4
+#define WORKLOAD_PPLIB_COMPUTE_BIT 5
+#define WORKLOAD_PPLIB_CUSTOM_BIT 6
+#define WORKLOAD_PPLIB_W3D_BIT 7
+#define WORKLOAD_PPLIB_COUNT 8
// These defines are used with the following messages:
@@ -1824,8 +1824,8 @@ typedef struct {
typedef struct {
RlcPaceFlopsPerByteOverride_t RlcPaceFlopsPerByteOverride;
-
- uint32_t MmHubPadding[8]; // SMU internal use
+
+ uint32_t MmHubPadding[8]; // SMU internal use
} RlcPaceFlopsPerByteOverrideExternal_t;
// These defines are used with the SMC_MSG_SetUclkFastSwitch message.
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH] drm/amd/pm: Clean up errors in smu11_driver_if_sienna_cichlid.h
@ 2023-07-24 8:30 ` sunran001
0 siblings, 0 replies; 8+ messages in thread
From: sunran001 @ 2023-07-24 8:30 UTC (permalink / raw)
To: alexander.deucher; +Cc: amd-gfx, dri-devel, linux-kernel
Fix the following errors reported by checkpatch:
ERROR: trailing whitespace
ERROR: space prohibited before open square bracket '['
ERROR: space prohibited before that close square bracket ']'
Signed-off-by: Ran Sun <sunran001@208suo.com>
---
.../pmfw_if/smu11_driver_if_sienna_cichlid.h | 386 +++++++++---------
1 file changed, 193 insertions(+), 193 deletions(-)
diff --git
a/drivers/gpu/drm/amd/pm/swsmu/inc/pmfw_if/smu11_driver_if_sienna_cichlid.h
b/drivers/gpu/drm/amd/pm/swsmu/inc/pmfw_if/smu11_driver_if_sienna_cichlid.h
index aa6d29de4002..703ade13d9f4 100644
---
a/drivers/gpu/drm/amd/pm/swsmu/inc/pmfw_if/smu11_driver_if_sienna_cichlid.h
+++
b/drivers/gpu/drm/amd/pm/swsmu/inc/pmfw_if/smu11_driver_if_sienna_cichlid.h
@@ -225,33 +225,33 @@ typedef enum {
#define FW_DSTATE_MP1_WHISPER_MODE_BIT 6
#define FW_DSTATE_SOC_LIV_MIN_BIT 7
#define FW_DSTATE_SOC_PLL_PWRDN_BIT 8
-#define FW_DSTATE_MEM_PLL_PWRDN_BIT 9
+#define FW_DSTATE_MEM_PLL_PWRDN_BIT 9
#define FW_DSTATE_OPTIMIZE_MALL_REFRESH_BIT 10
#define FW_DSTATE_MEM_PSI_BIT 11
#define FW_DSTATE_HSR_NON_STROBE_BIT 12
#define FW_DSTATE_MP0_ENTER_WFI_BIT 13
-#define FW_DSTATE_SOC_ULV_MASK (1 <<
FW_DSTATE_SOC_ULV_BIT )
-#define FW_DSTATE_G6_HSR_MASK (1 <<
FW_DSTATE_G6_HSR_BIT )
-#define FW_DSTATE_G6_PHY_VDDCI_OFF_MASK (1 <<
FW_DSTATE_G6_PHY_VDDCI_OFF_BIT )
-#define FW_DSTATE_MP1_DS_MASK (1 <<
FW_DSTATE_MP1_DS_BIT )
-#define FW_DSTATE_MP0_DS_MASK (1 <<
FW_DSTATE_MP0_DS_BIT )
-#define FW_DSTATE_SMN_DS_MASK (1 <<
FW_DSTATE_SMN_DS_BIT )
-#define FW_DSTATE_MP1_WHISPER_MODE_MASK (1 <<
FW_DSTATE_MP1_WHISPER_MODE_BIT )
-#define FW_DSTATE_SOC_LIV_MIN_MASK (1 <<
FW_DSTATE_SOC_LIV_MIN_BIT )
-#define FW_DSTATE_SOC_PLL_PWRDN_MASK (1 <<
FW_DSTATE_SOC_PLL_PWRDN_BIT )
-#define FW_DSTATE_MEM_PLL_PWRDN_MASK (1 <<
FW_DSTATE_MEM_PLL_PWRDN_BIT )
-#define FW_DSTATE_OPTIMIZE_MALL_REFRESH_MASK (1 <<
FW_DSTATE_OPTIMIZE_MALL_REFRESH_BIT )
-#define FW_DSTATE_MEM_PSI_MASK (1 <<
FW_DSTATE_MEM_PSI_BIT )
-#define FW_DSTATE_HSR_NON_STROBE_MASK (1 <<
FW_DSTATE_HSR_NON_STROBE_BIT )
-#define FW_DSTATE_MP0_ENTER_WFI_MASK (1 <<
FW_DSTATE_MP0_ENTER_WFI_BIT )
+#define FW_DSTATE_SOC_ULV_MASK (1 <<
FW_DSTATE_SOC_ULV_BIT)
+#define FW_DSTATE_G6_HSR_MASK (1 <<
FW_DSTATE_G6_HSR_BIT)
+#define FW_DSTATE_G6_PHY_VDDCI_OFF_MASK (1 <<
FW_DSTATE_G6_PHY_VDDCI_OFF_BIT)
+#define FW_DSTATE_MP1_DS_MASK (1 <<
FW_DSTATE_MP1_DS_BIT)
+#define FW_DSTATE_MP0_DS_MASK (1 <<
FW_DSTATE_MP0_DS_BIT)
+#define FW_DSTATE_SMN_DS_MASK (1 <<
FW_DSTATE_SMN_DS_BIT)
+#define FW_DSTATE_MP1_WHISPER_MODE_MASK (1 <<
FW_DSTATE_MP1_WHISPER_MODE_BIT)
+#define FW_DSTATE_SOC_LIV_MIN_MASK (1 <<
FW_DSTATE_SOC_LIV_MIN_BIT)
+#define FW_DSTATE_SOC_PLL_PWRDN_MASK (1 <<
FW_DSTATE_SOC_PLL_PWRDN_BIT)
+#define FW_DSTATE_MEM_PLL_PWRDN_MASK (1 <<
FW_DSTATE_MEM_PLL_PWRDN_BIT)
+#define FW_DSTATE_OPTIMIZE_MALL_REFRESH_MASK (1 <<
FW_DSTATE_OPTIMIZE_MALL_REFRESH_BIT)
+#define FW_DSTATE_MEM_PSI_MASK (1 <<
FW_DSTATE_MEM_PSI_BIT)
+#define FW_DSTATE_HSR_NON_STROBE_MASK (1 <<
FW_DSTATE_HSR_NON_STROBE_BIT)
+#define FW_DSTATE_MP0_ENTER_WFI_MASK (1 <<
FW_DSTATE_MP0_ENTER_WFI_BIT)
// GFX GPO Feature Contains PACE and DEM sub features
#define GFX_GPO_PACE_BIT 0
#define GFX_GPO_DEM_BIT 1
#define GFX_GPO_PACE_MASK (1 << GFX_GPO_PACE_BIT)
-#define GFX_GPO_DEM_MASK (1 << GFX_GPO_DEM_BIT )
+#define GFX_GPO_DEM_MASK (1 << GFX_GPO_DEM_BIT)
#define GPO_UPDATE_REQ_UCLKDPM_MASK 0x1
#define GPO_UPDATE_REQ_FCLKDPM_MASK 0x2
@@ -312,10 +312,10 @@ typedef enum {
I2C_CONTROLLER_NAME_VR_VDDCI,
I2C_CONTROLLER_NAME_VR_MVDD,
I2C_CONTROLLER_NAME_LIQUID0,
- I2C_CONTROLLER_NAME_LIQUID1,
+ I2C_CONTROLLER_NAME_LIQUID1,
I2C_CONTROLLER_NAME_PLX,
I2C_CONTROLLER_NAME_OTHER,
- I2C_CONTROLLER_NAME_COUNT,
+ I2C_CONTROLLER_NAME_COUNT,
} I2cControllerName_e;
typedef enum {
@@ -325,10 +325,10 @@ typedef enum {
I2C_CONTROLLER_THROTTLER_VR_VDDCI,
I2C_CONTROLLER_THROTTLER_VR_MVDD,
I2C_CONTROLLER_THROTTLER_LIQUID0,
- I2C_CONTROLLER_THROTTLER_LIQUID1,
+ I2C_CONTROLLER_THROTTLER_LIQUID1,
I2C_CONTROLLER_THROTTLER_PLX,
I2C_CONTROLLER_THROTTLER_INA3221,
- I2C_CONTROLLER_THROTTLER_COUNT,
+ I2C_CONTROLLER_THROTTLER_COUNT,
} I2cControllerThrottler_e;
typedef enum {
@@ -336,24 +336,24 @@ typedef enum {
I2C_CONTROLLER_PROTOCOL_VR_IR35217,
I2C_CONTROLLER_PROTOCOL_TMP_TMP102A,
I2C_CONTROLLER_PROTOCOL_INA3221,
- I2C_CONTROLLER_PROTOCOL_COUNT,
+ I2C_CONTROLLER_PROTOCOL_COUNT,
} I2cControllerProtocol_e;
typedef struct {
uint8_t Enabled;
uint8_t Speed;
- uint8_t SlaveAddress;
+ uint8_t SlaveAddress;
uint8_t ControllerPort;
uint8_t ControllerName;
uint8_t ThermalThrotter;
uint8_t I2cProtocol;
- uint8_t PaddingConfig;
+ uint8_t PaddingConfig;
} I2cControllerConfig_t;
typedef enum {
- I2C_PORT_SVD_SCL = 0,
- I2C_PORT_GPIO,
-} I2cPort_e;
+ I2C_PORT_SVD_SCL = 0,
+ I2C_PORT_GPIO,
+} I2cPort_e;
typedef enum {
I2C_SPEED_FAST_50K = 0, //50 Kbits/s
@@ -362,13 +362,13 @@ typedef enum {
I2C_SPEED_FAST_PLUS_1M, //1 Mbits/s (in fast mode)
I2C_SPEED_HIGH_1M, //1 Mbits/s (in high speed mode)
I2C_SPEED_HIGH_2M, //2.3 Mbits/s
- I2C_SPEED_COUNT,
+ I2C_SPEED_COUNT,
} I2cSpeed_e;
typedef enum {
I2C_CMD_READ = 0,
I2C_CMD_WRITE,
- I2C_CMD_COUNT,
+ I2C_CMD_COUNT,
} I2cCmdType_e;
typedef enum {
@@ -426,24 +426,24 @@ typedef enum {
PG_POWER_UP,
} PowerGatingSettings_e;
-typedef struct {
+typedef struct {
uint32_t a; // store in IEEE float format in this variable
uint32_t b; // store in IEEE float format in this variable
uint32_t c; // store in IEEE float format in this variable
} QuadraticInt_t;
-typedef struct {
+typedef struct {
uint32_t a; // store in fixed point, [31:20] signed integer, [19:0]
fractional bits
uint32_t b; // store in fixed point, [31:20] signed integer, [19:0]
fractional bits
uint32_t c; // store in fixed point, [31:20] signed integer, [19:0]
fractional bits
} QuadraticFixedPoint_t;
-typedef struct {
+typedef struct {
uint32_t m; // store in IEEE float format in this variable
uint32_t b; // store in IEEE float format in this variable
} LinearInt_t;
-typedef struct {
+typedef struct {
uint32_t a; // store in IEEE float format in this variable
uint32_t b; // store in IEEE float format in this variable
uint32_t c; // store in IEEE float format in this variable
@@ -455,7 +455,7 @@ typedef enum {
PIECEWISE_LINEAR_FUSED_MODEL = 0,
PIECEWISE_LINEAR_PP_MODEL,
QUADRATIC_PP_MODEL,
- PERPART_PIECEWISE_LINEAR_PP_MODEL,
+ PERPART_PIECEWISE_LINEAR_PP_MODEL,
} DfllDroopModelSelect_e;
typedef struct {
@@ -464,10 +464,10 @@ typedef struct {
}PiecewiseLinearDroopInt_t;
typedef enum {
- GFXCLK_SOURCE_PLL = 0,
- GFXCLK_SOURCE_DFLL,
- GFXCLK_SOURCE_COUNT,
-} GFXCLK_SOURCE_e;
+ GFXCLK_SOURCE_PLL = 0,
+ GFXCLK_SOURCE_DFLL,
+ GFXCLK_SOURCE_COUNT,
+} GFXCLK_SOURCE_e;
//Only Clks that have DPM descriptors are listed here
typedef enum {
@@ -552,18 +552,18 @@ typedef struct {
uint8_t VoltageMode; // 0 - AVFS only, 1-
min(AVFS,SS), 2-SS only
uint8_t SnapToDiscrete; // 0 - Fine grained DPM, 1 -
Discrete DPM
uint8_t NumDiscreteLevels; // Set to 2 (Fmin, Fmax) when
using fine grained DPM, otherwise set to # discrete levels used
- uint8_t Padding;
+ uint8_t Padding;
LinearInt_t ConversionToAvfsClk; // Transfer function to AVFS
Clock (GHz->GHz)
QuadraticInt_t SsCurve; // Slow-slow curve (GHz->V)
uint16_t SsFmin; // Fmin for SS curve. If SS curve
is selected, will use V@SSFmin for F <= Fmin
- uint16_t Padding16;
+ uint16_t Padding16;
} DpmDescriptor_t;
typedef enum {
PPT_THROTTLER_PPT0,
PPT_THROTTLER_PPT1,
PPT_THROTTLER_PPT2,
- PPT_THROTTLER_PPT3,
+ PPT_THROTTLER_PPT3,
PPT_THROTTLER_COUNT
} PPT_THROTTLER_e;
@@ -574,9 +574,9 @@ typedef enum {
TEMP_VR_GFX,
TEMP_VR_MEM0,
TEMP_VR_MEM1,
- TEMP_VR_SOC,
+ TEMP_VR_SOC,
TEMP_LIQUID0,
- TEMP_LIQUID1,
+ TEMP_LIQUID1,
TEMP_PLX,
TEMP_COUNT,
} TEMP_e;
@@ -592,7 +592,7 @@ typedef enum {
CUSTOMER_VARIANT_FALCON,
CUSTOMER_VARIANT_COUNT,
} CUSTOMER_VARIANT_e;
-
+
// Used for 2-step UCLK DPM change workaround
typedef struct {
uint16_t Fmin;
@@ -623,14 +623,14 @@ typedef struct {
// SECTION: Power Configuration
uint8_t TotalPowerConfig; //0-TDP, 1-TGP, 2-TCP Estimated,
3-TCP Measured. Use defines from PwrConfig_e
- uint8_t TotalPowerPadding[3];
+ uint8_t TotalPowerPadding[3];
// SECTION: APCC Settings
uint32_t ApccPlusResidencyLimit;
//SECTION: SMNCLK DPM
- uint16_t SmnclkDpmFreq [NUM_SMNCLK_DPM_LEVELS]; //
in MHz
- uint16_t SmnclkDpmVoltage [NUM_SMNCLK_DPM_LEVELS]; //
mV(Q2)
+ uint16_t SmnclkDpmFreq[NUM_SMNCLK_DPM_LEVELS]; // in MHz
+ uint16_t SmnclkDpmVoltage[NUM_SMNCLK_DPM_LEVELS]; //
mV(Q2)
uint32_t PaddingAPCC;
uint16_t
PerPartDroopVsetGfxDfll[NUM_PIECE_WISE_LINEAR_DROOP_MODEL_VF_POINTS];
//In mV(Q2)
@@ -639,14 +639,14 @@ typedef struct {
// SECTION: Throttler settings
uint32_t ThrottlerControlMask; // See Throtter masks defines
- // SECTION: FW DSTATE Settings
+ // SECTION: FW DSTATE Settings
uint32_t FwDStateMask; // See FW DState masks defines
// SECTION: ULV Settings
uint16_t UlvVoltageOffsetSoc; // In mV(Q2)
uint16_t UlvVoltageOffsetGfx; // In mV(Q2)
- uint16_t MinVoltageUlvGfx; // In mV(Q2) Minimum Voltage ("Vmin")
of VDD_GFX in ULV mode
+ uint16_t MinVoltageUlvGfx; // In mV(Q2) Minimum Voltage ("Vmin")
of VDD_GFX in ULV mode
uint16_t MinVoltageUlvSoc; // In mV(Q2) Minimum Voltage ("Vmin")
of VDD_SOC in ULV mode
uint16_t SocLIVmin; // In mV(Q2) Long Idle Vmin (deep
ULV), for VDD_SOC
@@ -678,62 +678,62 @@ typedef struct {
//SECTION: DPM Config 1
DpmDescriptor_t DpmDescriptor[PPCLK_COUNT];
- uint16_t FreqTableGfx [NUM_GFXCLK_DPM_LEVELS ]; // In
MHz
- uint16_t FreqTableVclk [NUM_VCLK_DPM_LEVELS ]; // In
MHz
- uint16_t FreqTableDclk [NUM_DCLK_DPM_LEVELS ]; // In
MHz
- uint16_t FreqTableSocclk [NUM_SOCCLK_DPM_LEVELS ]; // In
MHz
- uint16_t FreqTableUclk [NUM_UCLK_DPM_LEVELS ]; // In
MHz
- uint16_t FreqTableDcefclk [NUM_DCEFCLK_DPM_LEVELS ]; // In
MHz
- uint16_t FreqTableDispclk [NUM_DISPCLK_DPM_LEVELS ]; // In
MHz
- uint16_t FreqTablePixclk [NUM_PIXCLK_DPM_LEVELS ]; // In
MHz
- uint16_t FreqTablePhyclk [NUM_PHYCLK_DPM_LEVELS ]; // In
MHz
- uint16_t FreqTableDtbclk [NUM_DTBCLK_DPM_LEVELS ]; // In
MHz
- uint16_t FreqTableFclk [NUM_FCLK_DPM_LEVELS ]; // In
MHz
+ uint16_t FreqTableGfx[NUM_GFXCLK_DPM_LEVELS]; // In MHz
+ uint16_t FreqTableVclk[NUM_VCLK_DPM_LEVELS]; // In MHz
+ uint16_t FreqTableDclk[NUM_DCLK_DPM_LEVELS]; // In MHz
+ uint16_t FreqTableSocclk[NUM_SOCCLK_DPM_LEVELS]; // In MHz
+ uint16_t FreqTableUclk[NUM_UCLK_DPM_LEVELS]; // In MHz
+ uint16_t FreqTableDcefclk[NUM_DCEFCLK_DPM_LEVELS]; // In
MHz
+ uint16_t FreqTableDispclk[NUM_DISPCLK_DPM_LEVELS]; // In
MHz
+ uint16_t FreqTablePixclk[NUM_PIXCLK_DPM_LEVELS]; // In MHz
+ uint16_t FreqTablePhyclk[NUM_PHYCLK_DPM_LEVELS]; // In MHz
+ uint16_t FreqTableDtbclk[NUM_DTBCLK_DPM_LEVELS]; // In MHz
+ uint16_t FreqTableFclk[NUM_FCLK_DPM_LEVELS]; // In MHz
uint32_t Paddingclks;
DroopInt_t
PerPartDroopModelGfxDfll[NUM_PIECE_WISE_LINEAR_DROOP_MODEL_VF_POINTS];
//GHz ->Vstore in IEEE float format
- uint32_t DcModeMaxFreq [PPCLK_COUNT ]; // In
MHz
+ uint32_t DcModeMaxFreq[PPCLK_COUNT]; // In MHz
- uint8_t FreqTableUclkDiv [NUM_UCLK_DPM_LEVELS ]; //
0:Div-1, 1:Div-1/2, 2:Div-1/4, 3:Div-1/8
+ uint8_t FreqTableUclkDiv[NUM_UCLK_DPM_LEVELS]; // 0:Div-1,
1:Div-1/2, 2:Div-1/4, 3:Div-1/8
// Used for MALL performance boost
uint16_t FclkBoostFreq; // In
Mhz
uint16_t FclkParamPadding;
// SECTION: DPM Config 2
- uint16_t Mp0clkFreq [NUM_MP0CLK_DPM_LEVELS]; // in
MHz
- uint16_t Mp0DpmVoltage [NUM_MP0CLK_DPM_LEVELS]; //
mV(Q2)
- uint16_t MemVddciVoltage [NUM_UCLK_DPM_LEVELS]; //
mV(Q2)
- uint16_t MemMvddVoltage [NUM_UCLK_DPM_LEVELS]; //
mV(Q2)
+ uint16_t Mp0clkFreq[NUM_MP0CLK_DPM_LEVELS]; // in MHz
+ uint16_t Mp0DpmVoltage[NUM_MP0CLK_DPM_LEVELS]; // mV(Q2)
+ uint16_t MemVddciVoltage[NUM_UCLK_DPM_LEVELS]; //
mV(Q2)
+ uint16_t MemMvddVoltage[NUM_UCLK_DPM_LEVELS]; // mV(Q2)
// GFXCLK DPM
uint16_t GfxclkFgfxoffEntry; // in Mhz
- uint16_t GfxclkFinit; // in Mhz
- uint16_t GfxclkFidle; // in MHz
+ uint16_t GfxclkFinit; // in Mhz
+ uint16_t GfxclkFidle; // in MHz
uint8_t GfxclkSource; // 0 = PLL, 1 = DFLL
uint8_t GfxclkPadding;
- // GFX GPO
+ // GFX GPO
uint8_t GfxGpoSubFeatureMask; // bit 0 = PACE, bit 1 = DEM
uint8_t GfxGpoEnabledWorkPolicyMask; //Any policy that GPO
can be enabled
uint8_t GfxGpoDisabledWorkPolicyMask; //Any policy that GPO
can be disabled
uint8_t GfxGpoPadding[1];
uint32_t GfxGpoVotingAllow; //For indicating which feature
changes should result in a GPO table recalculation
- uint32_t GfxGpoPadding32[4];
+ uint32_t GfxGpoPadding32[4];
uint16_t GfxDcsFopt; // Optimal GFXCLK for DCS in
Mhz
uint16_t GfxDcsFclkFopt; // Optimal FCLK for DCS in Mhz
uint16_t GfxDcsUclkFopt; // Optimal UCLK for DCS in Mhz
-
- uint16_t DcsGfxOffVoltage; //Voltage in mV(Q2) applied to
VDDGFX when entering DCS GFXOFF phase
+
+ uint16_t DcsGfxOffVoltage; //Voltage in mV(Q2) applied to
VDDGFX when entering DCS GFXOFF phase
uint16_t DcsMinGfxOffTime; //Minimum amount of time PMFW
shuts GFX OFF as part of GFX DCS phase
uint16_t DcsMaxGfxOffTime; //Maximum amount of time PMFW
can shut GFX OFF as part of GFX DCS phase at a stretch.
uint32_t DcsMinCreditAccum; //Min amount of positive credit
accumulation before waking GFX up as part of DCS.
- uint16_t DcsExitHysteresis; //The min amount of time power
credit accumulator should have a value > 0 before SMU exits the DCS
throttling phase.
+ uint16_t DcsExitHysteresis; //The min amount of time power
credit accumulator should have a value > 0 before SMU exits the DCS
throttling phase.
uint16_t DcsTimeout; //This is the amount of time
SMU FW waits for RLC to put GFX into GFXOFF before reverting to the
fallback mechanism of throttling GFXCLK to Fmin.
uint32_t DcsParamPadding[5];
@@ -743,9 +743,9 @@ typedef struct {
// UCLK section
uint8_t LowestUclkReservedForUlv; // Set this to 1 if UCLK DPM0
is reserved for ULV-mode only
uint8_t PaddingMem[3];
-
- uint8_t UclkDpmPstates [NUM_UCLK_DPM_LEVELS]; // 4 DPM
states, 0-P0, 1-P1, 2-P2, 3-P3.
-
+
+ uint8_t UclkDpmPstates[NUM_UCLK_DPM_LEVELS]; // 4 DPM
states, 0-P0, 1-P1, 2-P2, 3-P3.
+
// Used for 2-Step UCLK change workaround
UclkDpmChangeRange_t UclkDpmSrcFreqRange; // In Mhz
UclkDpmChangeRange_t UclkDpmTargFreqRange; // In Mhz
@@ -755,8 +755,8 @@ typedef struct {
// Link DPM Settings
uint8_t PcieGenSpeed[NUM_LINK_LEVELS]; ///<
0:PciE-gen1 1:PciE-gen2 2:PciE-gen3 3:PciE-gen4
uint8_t PcieLaneCount[NUM_LINK_LEVELS]; ///< 1=x1,
2=x2, 3=x4, 4=x8, 5=x12, 6=x16
- uint16_t LclkFreq[NUM_LINK_LEVELS];
-
+ uint16_t LclkFreq[NUM_LINK_LEVELS];
+
// SECTION: Fan Control
uint16_t FanStopTemp; //Celcius
uint16_t FanStartTemp; //Celcius
@@ -767,36 +767,36 @@ typedef struct {
uint16_t FanAcousticLimitRpm;
uint16_t FanThrottlingRpm;
uint16_t FanMaximumRpm;
- uint16_t MGpuFanBoostLimitRpm;
+ uint16_t MGpuFanBoostLimitRpm;
uint16_t FanTargetTemperature;
uint16_t FanTargetGfxclk;
uint16_t FanPadding16;
uint8_t FanTempInputSelect;
uint8_t FanPadding;
- uint8_t FanZeroRpmEnable;
+ uint8_t FanZeroRpmEnable;
uint8_t FanTachEdgePerRev;
-
+
// The following are AFC override parameters. Leave at 0 to use FW
defaults.
int16_t FuzzyFan_ErrorSetDelta;
int16_t FuzzyFan_ErrorRateSetDelta;
int16_t FuzzyFan_PwmSetDelta;
uint16_t FuzzyFan_Reserved;
- // SECTION: AVFS
+ // SECTION: AVFS
// Overrides
uint8_t OverrideAvfsGb[AVFS_VOLTAGE_COUNT];
- uint8_t dBtcGbGfxDfllModelSelect; //0 -> fused piece-wise
model, 1 -> piece-wise linear(PPTable), 2 -> quadratic model(PPTable)
+ uint8_t dBtcGbGfxDfllModelSelect; //0 -> fused piece-wise
model, 1 -> piece-wise linear(PPTable), 2 -> quadratic model(PPTable)
uint8_t Padding8_Avfs;
- QuadraticInt_t qAvfsGb[AVFS_VOLTAGE_COUNT]; // GHz->V
Override of fused curve
+ QuadraticInt_t qAvfsGb[AVFS_VOLTAGE_COUNT]; // GHz->V
Override of fused curve
DroopInt_t dBtcGbGfxPll; // GHz->V BtcGb
DroopInt_t dBtcGbGfxDfll; // GHz->V BtcGb
DroopInt_t dBtcGbSoc; // GHz->V BtcGb
- LinearInt_t qAgingGb[AVFS_VOLTAGE_COUNT]; // GHz->V
+ LinearInt_t qAgingGb[AVFS_VOLTAGE_COUNT]; // GHz->V
PiecewiseLinearDroopInt_t PiecewiseLinearDroopIntGfxDfll; //GHz
->Vstore in IEEE float format
- QuadraticInt_t qStaticVoltageOffset[AVFS_VOLTAGE_COUNT]; // GHz->V
+ QuadraticInt_t qStaticVoltageOffset[AVFS_VOLTAGE_COUNT]; // GHz->V
uint16_t DcTol[AVFS_VOLTAGE_COUNT]; // mV Q2
@@ -807,17 +807,17 @@ typedef struct {
uint16_t DcBtcMax[AVFS_VOLTAGE_COUNT]; // mV Q2
uint16_t DcBtcGb[AVFS_VOLTAGE_COUNT]; // mV Q2
-
+
// SECTION: XGMI
uint8_t XgmiDpmPstates[NUM_XGMI_LEVELS]; // 2 DPM states,
high and low. 0-P0, 1-P1, 2-P2, 3-P3.
uint8_t XgmiDpmSpare[2];
// SECTION: Advanced Options
uint32_t DebugOverrides;
- QuadraticInt_t ReservedEquation0;
- QuadraticInt_t ReservedEquation1;
- QuadraticInt_t ReservedEquation2;
- QuadraticInt_t ReservedEquation3;
+ QuadraticInt_t ReservedEquation0;
+ QuadraticInt_t ReservedEquation1;
+ QuadraticInt_t ReservedEquation2;
+ QuadraticInt_t ReservedEquation3;
// SECTION: Sku Reserved
uint8_t CustomerVariant;
@@ -825,17 +825,17 @@ typedef struct {
//VC BTC parameters are only applicable to VDD_GFX domain
uint8_t VcBtcEnabled;
uint16_t VcBtcVminT0; // T0_VMIN
- uint16_t VcBtcFixedVminAgingOffset; //
FIXED_VMIN_AGING_OFFSET
- uint16_t VcBtcVmin2PsmDegrationGb; //
VMIN_TO_PSM_DEGRADATION_GB
+ uint16_t VcBtcFixedVminAgingOffset; //
FIXED_VMIN_AGING_OFFSET
+ uint16_t VcBtcVmin2PsmDegrationGb; //
VMIN_TO_PSM_DEGRADATION_GB
uint32_t VcBtcPsmA; // A_PSM
uint32_t VcBtcPsmB; // B_PSM
uint32_t VcBtcVminA; // A_VMIN
- uint32_t VcBtcVminB; // B_VMIN
-
+ uint32_t VcBtcVminB; // B_VMIN
+
//GPIO Board feature
uint16_t LedGpio; //GeneriA GPIO flag used to
control the radeon LEDs
- uint16_t GfxPowerStagesGpio; //Genlk_vsync GPIO flag used to
control gfx power stages
-
+ uint16_t GfxPowerStagesGpio; //Genlk_vsync GPIO flag used to
control gfx power stages
+
uint32_t SkuReserved[8];
@@ -845,7 +845,7 @@ typedef struct {
uint32_t GamingClk[6];
// SECTION: I2C Control
- I2cControllerConfig_t I2cControllers[NUM_I2C_CONTROLLERS];
+ I2cControllerConfig_t I2cControllers[NUM_I2C_CONTROLLERS];
uint8_t GpioScl; // GPIO Number for SCL Line, used only for
CKSVII2C1
uint8_t GpioSda; // GPIO Number for SDA Line, used only for
CKSVII2C1
@@ -875,21 +875,21 @@ typedef struct {
uint16_t Mem0MaxCurrent; // in Amps
int8_t Mem0Offset; // in Amps
uint8_t Padding_TelemetryMem0;
-
+
uint16_t Mem1MaxCurrent; // in Amps
int8_t Mem1Offset; // in Amps
uint8_t Padding_TelemetryMem1;
uint32_t MvddRatio; // This is used for MVDD Svi2 Div Ratio
workaround. It has 16 fractional bits (Q16.16)
-
+
// SECTION: GPIO Settings
uint8_t AcDcGpio; // GPIO pin configured for AC/DC
switching
uint8_t AcDcPolarity; // GPIO polarity for AC/DC switching
uint8_t VR0HotGpio; // GPIO pin configured for VR0 HOT
event
uint8_t VR0HotPolarity; // GPIO polarity for VR0 HOT event
- uint8_t VR1HotGpio; // GPIO pin configured for VR1 HOT
event
- uint8_t VR1HotPolarity; // GPIO polarity for VR1 HOT event
+ uint8_t VR1HotGpio; // GPIO pin configured for VR1 HOT
event
+ uint8_t VR1HotPolarity; // GPIO polarity for VR1 HOT event
uint8_t GthrGpio; // GPIO pin configured for GTHR Event
uint8_t GthrPolarity; // replace GPIO polarity for GTHR
@@ -914,7 +914,7 @@ typedef struct {
uint8_t DfllGfxclkSpreadEnabled; // on or off
uint8_t DfllGfxclkSpreadPercent; // Q4.4
uint16_t DfllGfxclkSpreadFreq; // kHz
-
+
// UCLK Spread Spectrum
uint16_t UclkSpreadPadding;
uint16_t UclkSpreadFreq; // kHz
@@ -923,23 +923,23 @@ typedef struct {
uint8_t FclkSpreadEnabled; // on or off
uint8_t FclkSpreadPercent; // Q4.4
uint16_t FclkSpreadFreq; // kHz
-
+
// Section: Memory Config
- uint32_t MemoryChannelEnabled; // For DRAM use only, Max 32
channels enabled bit mask.
-
+ uint32_t MemoryChannelEnabled; // For DRAM use only, Max 32
channels enabled bit mask.
+
uint8_t DramBitWidth; // For DRAM use only. See Dram Bit width
type defines
uint8_t PaddingMem1[3];
// Section: Total Board Power
uint16_t TotalBoardPower; //Only needed for TCP Estimated
case, where TCP = TGP+Total Board Power
- uint16_t BoardPowerPadding;
-
+ uint16_t BoardPowerPadding;
+
// SECTION: XGMI Training
- uint8_t XgmiLinkSpeed [NUM_XGMI_PSTATE_LEVELS];
- uint8_t XgmiLinkWidth [NUM_XGMI_PSTATE_LEVELS];
+ uint8_t XgmiLinkSpeed[NUM_XGMI_PSTATE_LEVELS];
+ uint8_t XgmiLinkWidth[NUM_XGMI_PSTATE_LEVELS];
- uint16_t XgmiFclkFreq [NUM_XGMI_PSTATE_LEVELS];
- uint16_t XgmiSocVoltage [NUM_XGMI_PSTATE_LEVELS];
+ uint16_t XgmiFclkFreq[NUM_XGMI_PSTATE_LEVELS];
+ uint16_t XgmiSocVoltage[NUM_XGMI_PSTATE_LEVELS];
// SECTION: UMC feature flags
uint8_t HsrEnabled;
@@ -947,7 +947,7 @@ typedef struct {
uint8_t PaddingUmcFlags[2];
// UCLK Spread Spectrum
- uint8_t UclkSpreadPercent[16];
+ uint8_t UclkSpreadPercent[16];
// SECTION: Board Reserved
uint32_t BoardReserved[11];
@@ -983,14 +983,14 @@ typedef struct {
// SECTION: Power Configuration
uint8_t TotalPowerConfig; //0-TDP, 1-TGP, 2-TCP Estimated,
3-TCP Measured. Use defines from PwrConfig_e
- uint8_t TotalPowerPadding[3];
+ uint8_t TotalPowerPadding[3];
// SECTION: APCC Settings
uint32_t ApccPlusResidencyLimit;
//SECTION: SMNCLK DPM
- uint16_t SmnclkDpmFreq [NUM_SMNCLK_DPM_LEVELS]; //
in MHz
- uint16_t SmnclkDpmVoltage [NUM_SMNCLK_DPM_LEVELS]; //
mV(Q2)
+ uint16_t SmnclkDpmFreq[NUM_SMNCLK_DPM_LEVELS]; // in MHz
+ uint16_t SmnclkDpmVoltage[NUM_SMNCLK_DPM_LEVELS]; //
mV(Q2)
uint32_t PaddingAPCC;
uint16_t
PerPartDroopVsetGfxDfll[NUM_PIECE_WISE_LINEAR_DROOP_MODEL_VF_POINTS];
//In mV(Q2)
@@ -999,14 +999,14 @@ typedef struct {
// SECTION: Throttler settings
uint32_t ThrottlerControlMask; // See Throtter masks defines
- // SECTION: FW DSTATE Settings
+ // SECTION: FW DSTATE Settings
uint32_t FwDStateMask; // See FW DState masks defines
// SECTION: ULV Settings
uint16_t UlvVoltageOffsetSoc; // In mV(Q2)
uint16_t UlvVoltageOffsetGfx; // In mV(Q2)
- uint16_t MinVoltageUlvGfx; // In mV(Q2) Minimum Voltage ("Vmin")
of VDD_GFX in ULV mode
+ uint16_t MinVoltageUlvGfx; // In mV(Q2) Minimum Voltage ("Vmin")
of VDD_GFX in ULV mode
uint16_t MinVoltageUlvSoc; // In mV(Q2) Minimum Voltage ("Vmin")
of VDD_SOC in ULV mode
uint16_t SocLIVmin;
@@ -1038,42 +1038,42 @@ typedef struct {
//SECTION: DPM Config 1
DpmDescriptor_t DpmDescriptor[PPCLK_COUNT];
- uint16_t FreqTableGfx [NUM_GFXCLK_DPM_LEVELS ]; // In
MHz
- uint16_t FreqTableVclk [NUM_VCLK_DPM_LEVELS ]; // In
MHz
- uint16_t FreqTableDclk [NUM_DCLK_DPM_LEVELS ]; // In
MHz
- uint16_t FreqTableSocclk [NUM_SOCCLK_DPM_LEVELS ]; // In
MHz
- uint16_t FreqTableUclk [NUM_UCLK_DPM_LEVELS ]; // In
MHz
- uint16_t FreqTableDcefclk [NUM_DCEFCLK_DPM_LEVELS ]; // In
MHz
- uint16_t FreqTableDispclk [NUM_DISPCLK_DPM_LEVELS ]; // In
MHz
- uint16_t FreqTablePixclk [NUM_PIXCLK_DPM_LEVELS ]; // In
MHz
- uint16_t FreqTablePhyclk [NUM_PHYCLK_DPM_LEVELS ]; // In
MHz
- uint16_t FreqTableDtbclk [NUM_DTBCLK_DPM_LEVELS ]; // In
MHz
- uint16_t FreqTableFclk [NUM_FCLK_DPM_LEVELS ]; // In
MHz
+ uint16_t FreqTableGfx[NUM_GFXCLK_DPM_LEVELS]; // In MHz
+ uint16_t FreqTableVclk[NUM_VCLK_DPM_LEVELS]; // In MHz
+ uint16_t FreqTableDclk[NUM_DCLK_DPM_LEVELS]; // In MHz
+ uint16_t FreqTableSocclk[NUM_SOCCLK_DPM_LEVELS]; // In MHz
+ uint16_t FreqTableUclk[NUM_UCLK_DPM_LEVELS]; // In MHz
+ uint16_t FreqTableDcefclk[NUM_DCEFCLK_DPM_LEVELS]; // In
MHz
+ uint16_t FreqTableDispclk[NUM_DISPCLK_DPM_LEVELS]; // In
MHz
+ uint16_t FreqTablePixclk[NUM_PIXCLK_DPM_LEVELS]; // In MHz
+ uint16_t FreqTablePhyclk[NUM_PHYCLK_DPM_LEVELS]; // In MHz
+ uint16_t FreqTableDtbclk[NUM_DTBCLK_DPM_LEVELS]; // In MHz
+ uint16_t FreqTableFclk[NUM_FCLK_DPM_LEVELS]; // In MHz
uint32_t Paddingclks;
DroopInt_t
PerPartDroopModelGfxDfll[NUM_PIECE_WISE_LINEAR_DROOP_MODEL_VF_POINTS];
//GHz ->Vstore in IEEE float format
- uint32_t DcModeMaxFreq [PPCLK_COUNT ]; // In
MHz
+ uint32_t DcModeMaxFreq[PPCLK_COUNT]; // In MHz
- uint8_t FreqTableUclkDiv [NUM_UCLK_DPM_LEVELS ]; //
0:Div-1, 1:Div-1/2, 2:Div-1/4, 3:Div-1/8
+ uint8_t FreqTableUclkDiv[NUM_UCLK_DPM_LEVELS]; // 0:Div-1,
1:Div-1/2, 2:Div-1/4, 3:Div-1/8
// Used for MALL performance boost
uint16_t FclkBoostFreq; // In
Mhz
uint16_t FclkParamPadding;
// SECTION: DPM Config 2
- uint16_t Mp0clkFreq [NUM_MP0CLK_DPM_LEVELS]; // in
MHz
- uint16_t Mp0DpmVoltage [NUM_MP0CLK_DPM_LEVELS]; //
mV(Q2)
- uint16_t MemVddciVoltage [NUM_UCLK_DPM_LEVELS]; //
mV(Q2)
- uint16_t MemMvddVoltage [NUM_UCLK_DPM_LEVELS]; //
mV(Q2)
+ uint16_t Mp0clkFreq[NUM_MP0CLK_DPM_LEVELS]; // in MHz
+ uint16_t Mp0DpmVoltage[NUM_MP0CLK_DPM_LEVELS]; // mV(Q2)
+ uint16_t MemVddciVoltage[NUM_UCLK_DPM_LEVELS]; //
mV(Q2)
+ uint16_t MemMvddVoltage[NUM_UCLK_DPM_LEVELS]; // mV(Q2)
// GFXCLK DPM
uint16_t GfxclkFgfxoffEntry; // in Mhz
- uint16_t GfxclkFinit; // in Mhz
- uint16_t GfxclkFidle; // in MHz
+ uint16_t GfxclkFinit; // in Mhz
+ uint16_t GfxclkFidle; // in MHz
uint8_t GfxclkSource; // 0 = PLL, 1 = DFLL
uint8_t GfxclkPadding;
- // GFX GPO
+ // GFX GPO
uint8_t GfxGpoSubFeatureMask; // bit 0 = PACE, bit 1 = DEM
uint8_t GfxGpoEnabledWorkPolicyMask; //Any policy that GPO
can be enabled
uint8_t GfxGpoDisabledWorkPolicyMask; //Any policy that GPO
can be disabled
@@ -1086,14 +1086,14 @@ typedef struct {
uint16_t GfxDcsFclkFopt; // Optimal FCLK for DCS in Mhz
uint16_t GfxDcsUclkFopt; // Optimal UCLK for DCS in Mhz
- uint16_t DcsGfxOffVoltage; //Voltage in mV(Q2) applied to
VDDGFX when entering DCS GFXOFF phase
+ uint16_t DcsGfxOffVoltage; //Voltage in mV(Q2) applied to
VDDGFX when entering DCS GFXOFF phase
uint16_t DcsMinGfxOffTime; //Minimum amount of time PMFW
shuts GFX OFF as part of GFX DCS phase
uint16_t DcsMaxGfxOffTime; //Maximum amount of time PMFW
can shut GFX OFF as part of GFX DCS phase at a stretch.
uint32_t DcsMinCreditAccum; //Min amount of positive credit
accumulation before waking GFX up as part of DCS.
- uint16_t DcsExitHysteresis; //The min amount of time power
credit accumulator should have a value > 0 before SMU exits the DCS
throttling phase.
+ uint16_t DcsExitHysteresis; //The min amount of time power
credit accumulator should have a value > 0 before SMU exits the DCS
throttling phase.
uint16_t DcsTimeout; //This is the amount of time
SMU FW waits for RLC to put GFX into GFXOFF before reverting to the
fallback mechanism of throttling GFXCLK to Fmin.
uint32_t DcsParamPadding[5];
@@ -1104,7 +1104,7 @@ typedef struct {
uint8_t LowestUclkReservedForUlv; // Set this to 1 if UCLK DPM0
is reserved for ULV-mode only
uint8_t PaddingMem[3];
- uint8_t UclkDpmPstates [NUM_UCLK_DPM_LEVELS]; // 4 DPM
states, 0-P0, 1-P1, 2-P2, 3-P3.
+ uint8_t UclkDpmPstates[NUM_UCLK_DPM_LEVELS]; // 4 DPM
states, 0-P0, 1-P1, 2-P2, 3-P3.
// Used for 2-Step UCLK change workaround
UclkDpmChangeRange_t UclkDpmSrcFreqRange; // In Mhz
@@ -1115,7 +1115,7 @@ typedef struct {
// Link DPM Settings
uint8_t PcieGenSpeed[NUM_LINK_LEVELS]; ///<
0:PciE-gen1 1:PciE-gen2 2:PciE-gen3 3:PciE-gen4
uint8_t PcieLaneCount[NUM_LINK_LEVELS]; ///< 1=x1,
2=x2, 3=x4, 4=x8, 5=x12, 6=x16
- uint16_t LclkFreq[NUM_LINK_LEVELS];
+ uint16_t LclkFreq[NUM_LINK_LEVELS];
// SECTION: Fan Control
uint16_t FanStopTemp; //Celcius
@@ -1133,7 +1133,7 @@ typedef struct {
uint16_t FanPadding16;
uint8_t FanTempInputSelect;
uint8_t FanPadding;
- uint8_t FanZeroRpmEnable;
+ uint8_t FanZeroRpmEnable;
uint8_t FanTachEdgePerRev;
// The following are AFC override parameters. Leave at 0 to use FW
defaults.
@@ -1142,21 +1142,21 @@ typedef struct {
int16_t FuzzyFan_PwmSetDelta;
uint16_t FuzzyFan_Reserved;
- // SECTION: AVFS
+ // SECTION: AVFS
// Overrides
uint8_t OverrideAvfsGb[AVFS_VOLTAGE_COUNT];
- uint8_t dBtcGbGfxDfllModelSelect; //0 -> fused piece-wise
model, 1 -> piece-wise linear(PPTable), 2 -> quadratic model(PPTable)
+ uint8_t dBtcGbGfxDfllModelSelect; //0 -> fused piece-wise
model, 1 -> piece-wise linear(PPTable), 2 -> quadratic model(PPTable)
uint8_t Padding8_Avfs;
- QuadraticInt_t qAvfsGb[AVFS_VOLTAGE_COUNT]; // GHz->V
Override of fused curve
+ QuadraticInt_t qAvfsGb[AVFS_VOLTAGE_COUNT]; // GHz->V
Override of fused curve
DroopInt_t dBtcGbGfxPll; // GHz->V BtcGb
DroopInt_t dBtcGbGfxDfll; // GHz->V BtcGb
DroopInt_t dBtcGbSoc; // GHz->V BtcGb
- LinearInt_t qAgingGb[AVFS_VOLTAGE_COUNT]; // GHz->V
+ LinearInt_t qAgingGb[AVFS_VOLTAGE_COUNT]; // GHz->V
PiecewiseLinearDroopInt_t PiecewiseLinearDroopIntGfxDfll; //GHz
->Vstore in IEEE float format
- QuadraticInt_t qStaticVoltageOffset[AVFS_VOLTAGE_COUNT]; // GHz->V
+ QuadraticInt_t qStaticVoltageOffset[AVFS_VOLTAGE_COUNT]; // GHz->V
uint16_t DcTol[AVFS_VOLTAGE_COUNT]; // mV Q2
@@ -1185,16 +1185,16 @@ typedef struct {
//VC BTC parameters are only applicable to VDD_GFX domain
uint8_t VcBtcEnabled;
uint16_t VcBtcVminT0; // T0_VMIN
- uint16_t VcBtcFixedVminAgingOffset; //
FIXED_VMIN_AGING_OFFSET
- uint16_t VcBtcVmin2PsmDegrationGb; //
VMIN_TO_PSM_DEGRADATION_GB
+ uint16_t VcBtcFixedVminAgingOffset; //
FIXED_VMIN_AGING_OFFSET
+ uint16_t VcBtcVmin2PsmDegrationGb; //
VMIN_TO_PSM_DEGRADATION_GB
uint32_t VcBtcPsmA; // A_PSM
uint32_t VcBtcPsmB; // B_PSM
uint32_t VcBtcVminA; // A_VMIN
- uint32_t VcBtcVminB; // B_VMIN
+ uint32_t VcBtcVminB; // B_VMIN
//GPIO Board feature
uint16_t LedGpio; //GeneriA GPIO flag used to
control the radeon LEDs
- uint16_t GfxPowerStagesGpio; //Genlk_vsync GPIO flag used to
control gfx power stages
+ uint16_t GfxPowerStagesGpio; //Genlk_vsync GPIO flag used to
control gfx power stages
uint32_t SkuReserved[63];
@@ -1206,7 +1206,7 @@ typedef struct {
uint32_t GamingClk[6];
// SECTION: I2C Control
- I2cControllerConfig_t I2cControllers[NUM_I2C_CONTROLLERS];
+ I2cControllerConfig_t I2cControllers[NUM_I2C_CONTROLLERS];
uint8_t GpioScl; // GPIO Number for SCL Line, used only for
CKSVII2C1
uint8_t GpioSda; // GPIO Number for SDA Line, used only for
CKSVII2C1
@@ -1249,8 +1249,8 @@ typedef struct {
uint8_t VR0HotGpio; // GPIO pin configured for VR0 HOT
event
uint8_t VR0HotPolarity; // GPIO polarity for VR0 HOT event
- uint8_t VR1HotGpio; // GPIO pin configured for VR1 HOT
event
- uint8_t VR1HotPolarity; // GPIO polarity for VR1 HOT event
+ uint8_t VR1HotGpio; // GPIO pin configured for VR1 HOT
event
+ uint8_t VR1HotPolarity; // GPIO polarity for VR1 HOT event
uint8_t GthrGpio; // GPIO pin configured for GTHR Event
uint8_t GthrPolarity; // replace GPIO polarity for GTHR
@@ -1286,21 +1286,21 @@ typedef struct {
uint16_t FclkSpreadFreq; // kHz
// Section: Memory Config
- uint32_t MemoryChannelEnabled; // For DRAM use only, Max 32
channels enabled bit mask.
+ uint32_t MemoryChannelEnabled; // For DRAM use only, Max 32
channels enabled bit mask.
uint8_t DramBitWidth; // For DRAM use only. See Dram Bit width
type defines
uint8_t PaddingMem1[3];
// Section: Total Board Power
uint16_t TotalBoardPower; //Only needed for TCP Estimated
case, where TCP = TGP+Total Board Power
- uint16_t BoardPowerPadding;
+ uint16_t BoardPowerPadding;
// SECTION: XGMI Training
- uint8_t XgmiLinkSpeed [NUM_XGMI_PSTATE_LEVELS];
- uint8_t XgmiLinkWidth [NUM_XGMI_PSTATE_LEVELS];
+ uint8_t XgmiLinkSpeed[NUM_XGMI_PSTATE_LEVELS];
+ uint8_t XgmiLinkWidth[NUM_XGMI_PSTATE_LEVELS];
- uint16_t XgmiFclkFreq [NUM_XGMI_PSTATE_LEVELS];
- uint16_t XgmiSocVoltage [NUM_XGMI_PSTATE_LEVELS];
+ uint16_t XgmiFclkFreq[NUM_XGMI_PSTATE_LEVELS];
+ uint16_t XgmiSocVoltage[NUM_XGMI_PSTATE_LEVELS];
// SECTION: UMC feature flags
uint8_t HsrEnabled;
@@ -1328,15 +1328,15 @@ typedef struct {
uint16_t UclkAverageLpfTau;
uint16_t GfxActivityLpfTau;
uint16_t UclkActivityLpfTau;
- uint16_t SocketPowerLpfTau;
+ uint16_t SocketPowerLpfTau;
uint16_t VcnClkAverageLpfTau;
- uint16_t padding16;
+ uint16_t padding16;
} DriverSmuConfig_t;
typedef struct {
DriverSmuConfig_t DriverSmuConfig;
- uint32_t Spare[7];
+ uint32_t Spare[7];
// Padding - ignore
uint32_t MmHubPadding[8]; // SMU internal use
} DriverSmuConfigExternal_t;
@@ -1345,14 +1345,14 @@ typedef struct {
uint16_t GfxclkFmin; // MHz
uint16_t GfxclkFmax; // MHz
QuadraticInt_t CustomGfxVfCurve; // a: mV/MHz^2, b:
mv/MHz, c: mV
- uint16_t CustomCurveFmin; // MHz
- uint16_t UclkFmin; // MHz
+ uint16_t CustomCurveFmin; // MHz
+ uint16_t UclkFmin; // MHz
uint16_t UclkFmax; // MHz
int16_t OverDrivePct; // %
uint16_t FanMaximumRpm;
uint16_t FanMinimumPwm;
uint16_t FanAcousticLimitRpm;
- uint16_t FanTargetTemperature; // Degree Celcius
+ uint16_t FanTargetTemperature; // Degree Celcius
uint8_t FanLinearPwmPoints[NUM_OD_FAN_MAX_POINTS];
uint8_t FanLinearTempPoints[NUM_OD_FAN_MAX_POINTS];
uint16_t MaxOpTemp; // Degree Celcius
@@ -1361,13 +1361,13 @@ typedef struct {
uint8_t FanZeroRpmStopTemp;
uint8_t FanMode;
uint8_t Padding[1];
-} OverDriveTable_t;
+} OverDriveTable_t;
typedef struct {
OverDriveTable_t OverDriveTable;
- uint32_t Spare[8];
+ uint32_t Spare[8];
- uint32_t MmHubPadding[8]; // SMU internal use
+ uint32_t MmHubPadding[8]; // SMU internal use
} OverDriveTableExternal_t;
typedef struct {
@@ -1380,7 +1380,7 @@ typedef struct {
uint16_t AverageUclkFrequencyPreDs ;
uint16_t AverageUclkFrequencyPostDs ;
-
+
uint16_t AverageGfxActivity ;
uint16_t AverageUclkActivity ;
uint8_t CurrSocVoltageOffset ;
@@ -1393,14 +1393,14 @@ typedef struct {
uint16_t TemperatureMem ;
uint16_t TemperatureVrGfx ;
uint16_t TemperatureVrMem0 ;
- uint16_t TemperatureVrMem1 ;
+ uint16_t TemperatureVrMem1 ;
uint16_t TemperatureVrSoc ;
uint16_t TemperatureLiquid0 ;
- uint16_t TemperatureLiquid1 ;
+ uint16_t TemperatureLiquid1 ;
uint16_t TemperaturePlx ;
uint16_t Padding16 ;
- uint32_t ThrottlerStatus ;
-
+ uint32_t ThrottlerStatus ;
+
uint8_t LinkDpmLevel;
uint8_t CurrFanPwm;
uint16_t CurrFanSpeed;
@@ -1416,7 +1416,7 @@ typedef struct {
uint16_t AverageVclk0Frequency ;
uint16_t AverageDclk0Frequency ;
uint16_t AverageVclk1Frequency ;
- uint16_t AverageDclk1Frequency ;
+ uint16_t AverageDclk1Frequency ;
uint16_t VcnActivityPercentage ; //place holder, David N. to provide
full sequence
uint8_t PcieRate ;
uint8_t PcieWidth ;
@@ -1614,7 +1614,7 @@ typedef struct {
uint32_t Spare[1];
// Padding - ignore
- uint32_t MmHubPadding[8]; // SMU internal use
+ uint32_t MmHubPadding[8]; // SMU internal use
} SmuMetricsExternal_t;
typedef struct {
@@ -1622,7 +1622,7 @@ typedef struct {
uint16_t MaxClock; // This is either DCEFCLK or SOCCLK (in MHz)
uint16_t MinUclk;
uint16_t MaxUclk;
-
+
uint8_t WmSetting;
uint8_t Flags;
uint8_t Padding[2];
@@ -1658,7 +1658,7 @@ typedef struct {
typedef struct {
uint16_t avgPsmCount[67];
uint16_t minPsmCount[67];
- float avgPsmVoltage[67];
+ float avgPsmVoltage[67];
float minPsmVoltage[67];
} AvfsDebugTable_t;
@@ -1673,7 +1673,7 @@ typedef struct {
uint8_t Padding;
uint8_t AvfsEn[AVFS_VOLTAGE_COUNT];
-
+
uint8_t OverrideVFT[AVFS_VOLTAGE_COUNT];
uint8_t OverrideAvfsGb[AVFS_VOLTAGE_COUNT];
@@ -1729,7 +1729,7 @@ typedef struct {
uint8_t Gfx_IdleHystLimit;
uint8_t Gfx_FPS;
uint8_t Gfx_MinActiveFreqType;
- uint8_t Gfx_BoosterFreqType;
+ uint8_t Gfx_BoosterFreqType;
uint8_t Gfx_MinFreqStep; // Minimum delta between
current and target frequeny in order for FW to change clock.
uint16_t Gfx_MinActiveFreq; // MHz
uint16_t Gfx_BoosterFreq; // MHz
@@ -1744,7 +1744,7 @@ typedef struct {
uint8_t Fclk_IdleHystLimit;
uint8_t Fclk_FPS;
uint8_t Fclk_MinActiveFreqType;
- uint8_t Fclk_BoosterFreqType;
+ uint8_t Fclk_BoosterFreqType;
uint8_t Fclk_MinFreqStep; // Minimum delta between
current and target frequeny in order for FW to change clock.
uint16_t Fclk_MinActiveFreq; // MHz
uint16_t Fclk_BoosterFreq; // MHz
@@ -1780,19 +1780,19 @@ typedef struct {
typedef struct {
DpmActivityMonitorCoeffInt_t DpmActivityMonitorCoeffInt;
- uint32_t MmHubPadding[8]; // SMU internal use
+ uint32_t MmHubPadding[8]; // SMU internal use
} DpmActivityMonitorCoeffIntExternal_t;
// Workload bits
-#define WORKLOAD_PPLIB_DEFAULT_BIT 0
-#define WORKLOAD_PPLIB_FULL_SCREEN_3D_BIT 1
-#define WORKLOAD_PPLIB_POWER_SAVING_BIT 2
-#define WORKLOAD_PPLIB_VIDEO_BIT 3
-#define WORKLOAD_PPLIB_VR_BIT 4
-#define WORKLOAD_PPLIB_COMPUTE_BIT 5
-#define WORKLOAD_PPLIB_CUSTOM_BIT 6
-#define WORKLOAD_PPLIB_W3D_BIT 7
-#define WORKLOAD_PPLIB_COUNT 8
+#define WORKLOAD_PPLIB_DEFAULT_BIT 0
+#define WORKLOAD_PPLIB_FULL_SCREEN_3D_BIT 1
+#define WORKLOAD_PPLIB_POWER_SAVING_BIT 2
+#define WORKLOAD_PPLIB_VIDEO_BIT 3
+#define WORKLOAD_PPLIB_VR_BIT 4
+#define WORKLOAD_PPLIB_COMPUTE_BIT 5
+#define WORKLOAD_PPLIB_CUSTOM_BIT 6
+#define WORKLOAD_PPLIB_W3D_BIT 7
+#define WORKLOAD_PPLIB_COUNT 8
// These defines are used with the following messages:
@@ -1824,8 +1824,8 @@ typedef struct {
typedef struct {
RlcPaceFlopsPerByteOverride_t RlcPaceFlopsPerByteOverride;
-
- uint32_t MmHubPadding[8]; // SMU internal use
+
+ uint32_t MmHubPadding[8]; // SMU internal use
} RlcPaceFlopsPerByteOverrideExternal_t;
// These defines are used with the SMC_MSG_SetUclkFastSwitch message.
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] drm/amd/pm: Clean up errors in smu11_driver_if_sienna_cichlid.h
2023-07-24 8:30 ` sunran001
@ 2023-07-24 21:51 ` Alex Deucher
-1 siblings, 0 replies; 8+ messages in thread
From: Alex Deucher @ 2023-07-24 21:51 UTC (permalink / raw)
To: sunran001; +Cc: alexander.deucher, dri-devel, amd-gfx, linux-kernel
This doesn't apply cleanly.
Alex
On Mon, Jul 24, 2023 at 4:30 AM <sunran001@208suo.com> wrote:
>
> Fix the following errors reported by checkpatch:
>
> ERROR: trailing whitespace
> ERROR: space prohibited before open square bracket '['
> ERROR: space prohibited before that close square bracket ']'
>
> Signed-off-by: Ran Sun <sunran001@208suo.com>
> ---
> .../pmfw_if/smu11_driver_if_sienna_cichlid.h | 386 +++++++++---------
> 1 file changed, 193 insertions(+), 193 deletions(-)
>
> diff --git
> a/drivers/gpu/drm/amd/pm/swsmu/inc/pmfw_if/smu11_driver_if_sienna_cichlid.h
> b/drivers/gpu/drm/amd/pm/swsmu/inc/pmfw_if/smu11_driver_if_sienna_cichlid.h
> index aa6d29de4002..703ade13d9f4 100644
> ---
> a/drivers/gpu/drm/amd/pm/swsmu/inc/pmfw_if/smu11_driver_if_sienna_cichlid.h
> +++
> b/drivers/gpu/drm/amd/pm/swsmu/inc/pmfw_if/smu11_driver_if_sienna_cichlid.h
> @@ -225,33 +225,33 @@ typedef enum {
> #define FW_DSTATE_MP1_WHISPER_MODE_BIT 6
> #define FW_DSTATE_SOC_LIV_MIN_BIT 7
> #define FW_DSTATE_SOC_PLL_PWRDN_BIT 8
> -#define FW_DSTATE_MEM_PLL_PWRDN_BIT 9
> +#define FW_DSTATE_MEM_PLL_PWRDN_BIT 9
> #define FW_DSTATE_OPTIMIZE_MALL_REFRESH_BIT 10
> #define FW_DSTATE_MEM_PSI_BIT 11
> #define FW_DSTATE_HSR_NON_STROBE_BIT 12
> #define FW_DSTATE_MP0_ENTER_WFI_BIT 13
>
> -#define FW_DSTATE_SOC_ULV_MASK (1 <<
> FW_DSTATE_SOC_ULV_BIT )
> -#define FW_DSTATE_G6_HSR_MASK (1 <<
> FW_DSTATE_G6_HSR_BIT )
> -#define FW_DSTATE_G6_PHY_VDDCI_OFF_MASK (1 <<
> FW_DSTATE_G6_PHY_VDDCI_OFF_BIT )
> -#define FW_DSTATE_MP1_DS_MASK (1 <<
> FW_DSTATE_MP1_DS_BIT )
> -#define FW_DSTATE_MP0_DS_MASK (1 <<
> FW_DSTATE_MP0_DS_BIT )
> -#define FW_DSTATE_SMN_DS_MASK (1 <<
> FW_DSTATE_SMN_DS_BIT )
> -#define FW_DSTATE_MP1_WHISPER_MODE_MASK (1 <<
> FW_DSTATE_MP1_WHISPER_MODE_BIT )
> -#define FW_DSTATE_SOC_LIV_MIN_MASK (1 <<
> FW_DSTATE_SOC_LIV_MIN_BIT )
> -#define FW_DSTATE_SOC_PLL_PWRDN_MASK (1 <<
> FW_DSTATE_SOC_PLL_PWRDN_BIT )
> -#define FW_DSTATE_MEM_PLL_PWRDN_MASK (1 <<
> FW_DSTATE_MEM_PLL_PWRDN_BIT )
> -#define FW_DSTATE_OPTIMIZE_MALL_REFRESH_MASK (1 <<
> FW_DSTATE_OPTIMIZE_MALL_REFRESH_BIT )
> -#define FW_DSTATE_MEM_PSI_MASK (1 <<
> FW_DSTATE_MEM_PSI_BIT )
> -#define FW_DSTATE_HSR_NON_STROBE_MASK (1 <<
> FW_DSTATE_HSR_NON_STROBE_BIT )
> -#define FW_DSTATE_MP0_ENTER_WFI_MASK (1 <<
> FW_DSTATE_MP0_ENTER_WFI_BIT )
> +#define FW_DSTATE_SOC_ULV_MASK (1 <<
> FW_DSTATE_SOC_ULV_BIT)
> +#define FW_DSTATE_G6_HSR_MASK (1 <<
> FW_DSTATE_G6_HSR_BIT)
> +#define FW_DSTATE_G6_PHY_VDDCI_OFF_MASK (1 <<
> FW_DSTATE_G6_PHY_VDDCI_OFF_BIT)
> +#define FW_DSTATE_MP1_DS_MASK (1 <<
> FW_DSTATE_MP1_DS_BIT)
> +#define FW_DSTATE_MP0_DS_MASK (1 <<
> FW_DSTATE_MP0_DS_BIT)
> +#define FW_DSTATE_SMN_DS_MASK (1 <<
> FW_DSTATE_SMN_DS_BIT)
> +#define FW_DSTATE_MP1_WHISPER_MODE_MASK (1 <<
> FW_DSTATE_MP1_WHISPER_MODE_BIT)
> +#define FW_DSTATE_SOC_LIV_MIN_MASK (1 <<
> FW_DSTATE_SOC_LIV_MIN_BIT)
> +#define FW_DSTATE_SOC_PLL_PWRDN_MASK (1 <<
> FW_DSTATE_SOC_PLL_PWRDN_BIT)
> +#define FW_DSTATE_MEM_PLL_PWRDN_MASK (1 <<
> FW_DSTATE_MEM_PLL_PWRDN_BIT)
> +#define FW_DSTATE_OPTIMIZE_MALL_REFRESH_MASK (1 <<
> FW_DSTATE_OPTIMIZE_MALL_REFRESH_BIT)
> +#define FW_DSTATE_MEM_PSI_MASK (1 <<
> FW_DSTATE_MEM_PSI_BIT)
> +#define FW_DSTATE_HSR_NON_STROBE_MASK (1 <<
> FW_DSTATE_HSR_NON_STROBE_BIT)
> +#define FW_DSTATE_MP0_ENTER_WFI_MASK (1 <<
> FW_DSTATE_MP0_ENTER_WFI_BIT)
>
> // GFX GPO Feature Contains PACE and DEM sub features
> #define GFX_GPO_PACE_BIT 0
> #define GFX_GPO_DEM_BIT 1
>
> #define GFX_GPO_PACE_MASK (1 << GFX_GPO_PACE_BIT)
> -#define GFX_GPO_DEM_MASK (1 << GFX_GPO_DEM_BIT )
> +#define GFX_GPO_DEM_MASK (1 << GFX_GPO_DEM_BIT)
>
> #define GPO_UPDATE_REQ_UCLKDPM_MASK 0x1
> #define GPO_UPDATE_REQ_FCLKDPM_MASK 0x2
> @@ -312,10 +312,10 @@ typedef enum {
> I2C_CONTROLLER_NAME_VR_VDDCI,
> I2C_CONTROLLER_NAME_VR_MVDD,
> I2C_CONTROLLER_NAME_LIQUID0,
> - I2C_CONTROLLER_NAME_LIQUID1,
> + I2C_CONTROLLER_NAME_LIQUID1,
> I2C_CONTROLLER_NAME_PLX,
> I2C_CONTROLLER_NAME_OTHER,
> - I2C_CONTROLLER_NAME_COUNT,
> + I2C_CONTROLLER_NAME_COUNT,
> } I2cControllerName_e;
>
> typedef enum {
> @@ -325,10 +325,10 @@ typedef enum {
> I2C_CONTROLLER_THROTTLER_VR_VDDCI,
> I2C_CONTROLLER_THROTTLER_VR_MVDD,
> I2C_CONTROLLER_THROTTLER_LIQUID0,
> - I2C_CONTROLLER_THROTTLER_LIQUID1,
> + I2C_CONTROLLER_THROTTLER_LIQUID1,
> I2C_CONTROLLER_THROTTLER_PLX,
> I2C_CONTROLLER_THROTTLER_INA3221,
> - I2C_CONTROLLER_THROTTLER_COUNT,
> + I2C_CONTROLLER_THROTTLER_COUNT,
> } I2cControllerThrottler_e;
>
> typedef enum {
> @@ -336,24 +336,24 @@ typedef enum {
> I2C_CONTROLLER_PROTOCOL_VR_IR35217,
> I2C_CONTROLLER_PROTOCOL_TMP_TMP102A,
> I2C_CONTROLLER_PROTOCOL_INA3221,
> - I2C_CONTROLLER_PROTOCOL_COUNT,
> + I2C_CONTROLLER_PROTOCOL_COUNT,
> } I2cControllerProtocol_e;
>
> typedef struct {
> uint8_t Enabled;
> uint8_t Speed;
> - uint8_t SlaveAddress;
> + uint8_t SlaveAddress;
> uint8_t ControllerPort;
> uint8_t ControllerName;
> uint8_t ThermalThrotter;
> uint8_t I2cProtocol;
> - uint8_t PaddingConfig;
> + uint8_t PaddingConfig;
> } I2cControllerConfig_t;
>
> typedef enum {
> - I2C_PORT_SVD_SCL = 0,
> - I2C_PORT_GPIO,
> -} I2cPort_e;
> + I2C_PORT_SVD_SCL = 0,
> + I2C_PORT_GPIO,
> +} I2cPort_e;
>
> typedef enum {
> I2C_SPEED_FAST_50K = 0, //50 Kbits/s
> @@ -362,13 +362,13 @@ typedef enum {
> I2C_SPEED_FAST_PLUS_1M, //1 Mbits/s (in fast mode)
> I2C_SPEED_HIGH_1M, //1 Mbits/s (in high speed mode)
> I2C_SPEED_HIGH_2M, //2.3 Mbits/s
> - I2C_SPEED_COUNT,
> + I2C_SPEED_COUNT,
> } I2cSpeed_e;
>
> typedef enum {
> I2C_CMD_READ = 0,
> I2C_CMD_WRITE,
> - I2C_CMD_COUNT,
> + I2C_CMD_COUNT,
> } I2cCmdType_e;
>
> typedef enum {
> @@ -426,24 +426,24 @@ typedef enum {
> PG_POWER_UP,
> } PowerGatingSettings_e;
>
> -typedef struct {
> +typedef struct {
> uint32_t a; // store in IEEE float format in this variable
> uint32_t b; // store in IEEE float format in this variable
> uint32_t c; // store in IEEE float format in this variable
> } QuadraticInt_t;
>
> -typedef struct {
> +typedef struct {
> uint32_t a; // store in fixed point, [31:20] signed integer, [19:0]
> fractional bits
> uint32_t b; // store in fixed point, [31:20] signed integer, [19:0]
> fractional bits
> uint32_t c; // store in fixed point, [31:20] signed integer, [19:0]
> fractional bits
> } QuadraticFixedPoint_t;
>
> -typedef struct {
> +typedef struct {
> uint32_t m; // store in IEEE float format in this variable
> uint32_t b; // store in IEEE float format in this variable
> } LinearInt_t;
>
> -typedef struct {
> +typedef struct {
> uint32_t a; // store in IEEE float format in this variable
> uint32_t b; // store in IEEE float format in this variable
> uint32_t c; // store in IEEE float format in this variable
> @@ -455,7 +455,7 @@ typedef enum {
> PIECEWISE_LINEAR_FUSED_MODEL = 0,
> PIECEWISE_LINEAR_PP_MODEL,
> QUADRATIC_PP_MODEL,
> - PERPART_PIECEWISE_LINEAR_PP_MODEL,
> + PERPART_PIECEWISE_LINEAR_PP_MODEL,
> } DfllDroopModelSelect_e;
>
> typedef struct {
> @@ -464,10 +464,10 @@ typedef struct {
> }PiecewiseLinearDroopInt_t;
>
> typedef enum {
> - GFXCLK_SOURCE_PLL = 0,
> - GFXCLK_SOURCE_DFLL,
> - GFXCLK_SOURCE_COUNT,
> -} GFXCLK_SOURCE_e;
> + GFXCLK_SOURCE_PLL = 0,
> + GFXCLK_SOURCE_DFLL,
> + GFXCLK_SOURCE_COUNT,
> +} GFXCLK_SOURCE_e;
>
> //Only Clks that have DPM descriptors are listed here
> typedef enum {
> @@ -552,18 +552,18 @@ typedef struct {
> uint8_t VoltageMode; // 0 - AVFS only, 1-
> min(AVFS,SS), 2-SS only
> uint8_t SnapToDiscrete; // 0 - Fine grained DPM, 1 -
> Discrete DPM
> uint8_t NumDiscreteLevels; // Set to 2 (Fmin, Fmax) when
> using fine grained DPM, otherwise set to # discrete levels used
> - uint8_t Padding;
> + uint8_t Padding;
> LinearInt_t ConversionToAvfsClk; // Transfer function to AVFS
> Clock (GHz->GHz)
> QuadraticInt_t SsCurve; // Slow-slow curve (GHz->V)
> uint16_t SsFmin; // Fmin for SS curve. If SS curve
> is selected, will use V@SSFmin for F <= Fmin
> - uint16_t Padding16;
> + uint16_t Padding16;
> } DpmDescriptor_t;
>
> typedef enum {
> PPT_THROTTLER_PPT0,
> PPT_THROTTLER_PPT1,
> PPT_THROTTLER_PPT2,
> - PPT_THROTTLER_PPT3,
> + PPT_THROTTLER_PPT3,
> PPT_THROTTLER_COUNT
> } PPT_THROTTLER_e;
>
> @@ -574,9 +574,9 @@ typedef enum {
> TEMP_VR_GFX,
> TEMP_VR_MEM0,
> TEMP_VR_MEM1,
> - TEMP_VR_SOC,
> + TEMP_VR_SOC,
> TEMP_LIQUID0,
> - TEMP_LIQUID1,
> + TEMP_LIQUID1,
> TEMP_PLX,
> TEMP_COUNT,
> } TEMP_e;
> @@ -592,7 +592,7 @@ typedef enum {
> CUSTOMER_VARIANT_FALCON,
> CUSTOMER_VARIANT_COUNT,
> } CUSTOMER_VARIANT_e;
> -
> +
> // Used for 2-step UCLK DPM change workaround
> typedef struct {
> uint16_t Fmin;
> @@ -623,14 +623,14 @@ typedef struct {
>
> // SECTION: Power Configuration
> uint8_t TotalPowerConfig; //0-TDP, 1-TGP, 2-TCP Estimated,
> 3-TCP Measured. Use defines from PwrConfig_e
> - uint8_t TotalPowerPadding[3];
> + uint8_t TotalPowerPadding[3];
>
> // SECTION: APCC Settings
> uint32_t ApccPlusResidencyLimit;
>
> //SECTION: SMNCLK DPM
> - uint16_t SmnclkDpmFreq [NUM_SMNCLK_DPM_LEVELS]; //
> in MHz
> - uint16_t SmnclkDpmVoltage [NUM_SMNCLK_DPM_LEVELS]; //
> mV(Q2)
> + uint16_t SmnclkDpmFreq[NUM_SMNCLK_DPM_LEVELS]; // in MHz
> + uint16_t SmnclkDpmVoltage[NUM_SMNCLK_DPM_LEVELS]; //
> mV(Q2)
>
> uint32_t PaddingAPCC;
> uint16_t
> PerPartDroopVsetGfxDfll[NUM_PIECE_WISE_LINEAR_DROOP_MODEL_VF_POINTS];
> //In mV(Q2)
> @@ -639,14 +639,14 @@ typedef struct {
> // SECTION: Throttler settings
> uint32_t ThrottlerControlMask; // See Throtter masks defines
>
> - // SECTION: FW DSTATE Settings
> + // SECTION: FW DSTATE Settings
> uint32_t FwDStateMask; // See FW DState masks defines
>
> // SECTION: ULV Settings
> uint16_t UlvVoltageOffsetSoc; // In mV(Q2)
> uint16_t UlvVoltageOffsetGfx; // In mV(Q2)
>
> - uint16_t MinVoltageUlvGfx; // In mV(Q2) Minimum Voltage ("Vmin")
> of VDD_GFX in ULV mode
> + uint16_t MinVoltageUlvGfx; // In mV(Q2) Minimum Voltage ("Vmin")
> of VDD_GFX in ULV mode
> uint16_t MinVoltageUlvSoc; // In mV(Q2) Minimum Voltage ("Vmin")
> of VDD_SOC in ULV mode
>
> uint16_t SocLIVmin; // In mV(Q2) Long Idle Vmin (deep
> ULV), for VDD_SOC
> @@ -678,62 +678,62 @@ typedef struct {
> //SECTION: DPM Config 1
> DpmDescriptor_t DpmDescriptor[PPCLK_COUNT];
>
> - uint16_t FreqTableGfx [NUM_GFXCLK_DPM_LEVELS ]; // In
> MHz
> - uint16_t FreqTableVclk [NUM_VCLK_DPM_LEVELS ]; // In
> MHz
> - uint16_t FreqTableDclk [NUM_DCLK_DPM_LEVELS ]; // In
> MHz
> - uint16_t FreqTableSocclk [NUM_SOCCLK_DPM_LEVELS ]; // In
> MHz
> - uint16_t FreqTableUclk [NUM_UCLK_DPM_LEVELS ]; // In
> MHz
> - uint16_t FreqTableDcefclk [NUM_DCEFCLK_DPM_LEVELS ]; // In
> MHz
> - uint16_t FreqTableDispclk [NUM_DISPCLK_DPM_LEVELS ]; // In
> MHz
> - uint16_t FreqTablePixclk [NUM_PIXCLK_DPM_LEVELS ]; // In
> MHz
> - uint16_t FreqTablePhyclk [NUM_PHYCLK_DPM_LEVELS ]; // In
> MHz
> - uint16_t FreqTableDtbclk [NUM_DTBCLK_DPM_LEVELS ]; // In
> MHz
> - uint16_t FreqTableFclk [NUM_FCLK_DPM_LEVELS ]; // In
> MHz
> + uint16_t FreqTableGfx[NUM_GFXCLK_DPM_LEVELS]; // In MHz
> + uint16_t FreqTableVclk[NUM_VCLK_DPM_LEVELS]; // In MHz
> + uint16_t FreqTableDclk[NUM_DCLK_DPM_LEVELS]; // In MHz
> + uint16_t FreqTableSocclk[NUM_SOCCLK_DPM_LEVELS]; // In MHz
> + uint16_t FreqTableUclk[NUM_UCLK_DPM_LEVELS]; // In MHz
> + uint16_t FreqTableDcefclk[NUM_DCEFCLK_DPM_LEVELS]; // In
> MHz
> + uint16_t FreqTableDispclk[NUM_DISPCLK_DPM_LEVELS]; // In
> MHz
> + uint16_t FreqTablePixclk[NUM_PIXCLK_DPM_LEVELS]; // In MHz
> + uint16_t FreqTablePhyclk[NUM_PHYCLK_DPM_LEVELS]; // In MHz
> + uint16_t FreqTableDtbclk[NUM_DTBCLK_DPM_LEVELS]; // In MHz
> + uint16_t FreqTableFclk[NUM_FCLK_DPM_LEVELS]; // In MHz
> uint32_t Paddingclks;
>
> DroopInt_t
> PerPartDroopModelGfxDfll[NUM_PIECE_WISE_LINEAR_DROOP_MODEL_VF_POINTS];
> //GHz ->Vstore in IEEE float format
>
> - uint32_t DcModeMaxFreq [PPCLK_COUNT ]; // In
> MHz
> + uint32_t DcModeMaxFreq[PPCLK_COUNT]; // In MHz
>
> - uint8_t FreqTableUclkDiv [NUM_UCLK_DPM_LEVELS ]; //
> 0:Div-1, 1:Div-1/2, 2:Div-1/4, 3:Div-1/8
> + uint8_t FreqTableUclkDiv[NUM_UCLK_DPM_LEVELS]; // 0:Div-1,
> 1:Div-1/2, 2:Div-1/4, 3:Div-1/8
>
> // Used for MALL performance boost
> uint16_t FclkBoostFreq; // In
> Mhz
> uint16_t FclkParamPadding;
>
> // SECTION: DPM Config 2
> - uint16_t Mp0clkFreq [NUM_MP0CLK_DPM_LEVELS]; // in
> MHz
> - uint16_t Mp0DpmVoltage [NUM_MP0CLK_DPM_LEVELS]; //
> mV(Q2)
> - uint16_t MemVddciVoltage [NUM_UCLK_DPM_LEVELS]; //
> mV(Q2)
> - uint16_t MemMvddVoltage [NUM_UCLK_DPM_LEVELS]; //
> mV(Q2)
> + uint16_t Mp0clkFreq[NUM_MP0CLK_DPM_LEVELS]; // in MHz
> + uint16_t Mp0DpmVoltage[NUM_MP0CLK_DPM_LEVELS]; // mV(Q2)
> + uint16_t MemVddciVoltage[NUM_UCLK_DPM_LEVELS]; //
> mV(Q2)
> + uint16_t MemMvddVoltage[NUM_UCLK_DPM_LEVELS]; // mV(Q2)
> // GFXCLK DPM
> uint16_t GfxclkFgfxoffEntry; // in Mhz
> - uint16_t GfxclkFinit; // in Mhz
> - uint16_t GfxclkFidle; // in MHz
> + uint16_t GfxclkFinit; // in Mhz
> + uint16_t GfxclkFidle; // in MHz
> uint8_t GfxclkSource; // 0 = PLL, 1 = DFLL
> uint8_t GfxclkPadding;
>
> - // GFX GPO
> + // GFX GPO
> uint8_t GfxGpoSubFeatureMask; // bit 0 = PACE, bit 1 = DEM
> uint8_t GfxGpoEnabledWorkPolicyMask; //Any policy that GPO
> can be enabled
> uint8_t GfxGpoDisabledWorkPolicyMask; //Any policy that GPO
> can be disabled
> uint8_t GfxGpoPadding[1];
> uint32_t GfxGpoVotingAllow; //For indicating which feature
> changes should result in a GPO table recalculation
>
> - uint32_t GfxGpoPadding32[4];
> + uint32_t GfxGpoPadding32[4];
>
> uint16_t GfxDcsFopt; // Optimal GFXCLK for DCS in
> Mhz
> uint16_t GfxDcsFclkFopt; // Optimal FCLK for DCS in Mhz
> uint16_t GfxDcsUclkFopt; // Optimal UCLK for DCS in Mhz
> -
> - uint16_t DcsGfxOffVoltage; //Voltage in mV(Q2) applied to
> VDDGFX when entering DCS GFXOFF phase
> +
> + uint16_t DcsGfxOffVoltage; //Voltage in mV(Q2) applied to
> VDDGFX when entering DCS GFXOFF phase
>
> uint16_t DcsMinGfxOffTime; //Minimum amount of time PMFW
> shuts GFX OFF as part of GFX DCS phase
> uint16_t DcsMaxGfxOffTime; //Maximum amount of time PMFW
> can shut GFX OFF as part of GFX DCS phase at a stretch.
>
> uint32_t DcsMinCreditAccum; //Min amount of positive credit
> accumulation before waking GFX up as part of DCS.
>
> - uint16_t DcsExitHysteresis; //The min amount of time power
> credit accumulator should have a value > 0 before SMU exits the DCS
> throttling phase.
> + uint16_t DcsExitHysteresis; //The min amount of time power
> credit accumulator should have a value > 0 before SMU exits the DCS
> throttling phase.
> uint16_t DcsTimeout; //This is the amount of time
> SMU FW waits for RLC to put GFX into GFXOFF before reverting to the
> fallback mechanism of throttling GFXCLK to Fmin.
>
> uint32_t DcsParamPadding[5];
> @@ -743,9 +743,9 @@ typedef struct {
> // UCLK section
> uint8_t LowestUclkReservedForUlv; // Set this to 1 if UCLK DPM0
> is reserved for ULV-mode only
> uint8_t PaddingMem[3];
> -
> - uint8_t UclkDpmPstates [NUM_UCLK_DPM_LEVELS]; // 4 DPM
> states, 0-P0, 1-P1, 2-P2, 3-P3.
> -
> +
> + uint8_t UclkDpmPstates[NUM_UCLK_DPM_LEVELS]; // 4 DPM
> states, 0-P0, 1-P1, 2-P2, 3-P3.
> +
> // Used for 2-Step UCLK change workaround
> UclkDpmChangeRange_t UclkDpmSrcFreqRange; // In Mhz
> UclkDpmChangeRange_t UclkDpmTargFreqRange; // In Mhz
> @@ -755,8 +755,8 @@ typedef struct {
> // Link DPM Settings
> uint8_t PcieGenSpeed[NUM_LINK_LEVELS]; ///<
> 0:PciE-gen1 1:PciE-gen2 2:PciE-gen3 3:PciE-gen4
> uint8_t PcieLaneCount[NUM_LINK_LEVELS]; ///< 1=x1,
> 2=x2, 3=x4, 4=x8, 5=x12, 6=x16
> - uint16_t LclkFreq[NUM_LINK_LEVELS];
> -
> + uint16_t LclkFreq[NUM_LINK_LEVELS];
> +
> // SECTION: Fan Control
> uint16_t FanStopTemp; //Celcius
> uint16_t FanStartTemp; //Celcius
> @@ -767,36 +767,36 @@ typedef struct {
> uint16_t FanAcousticLimitRpm;
> uint16_t FanThrottlingRpm;
> uint16_t FanMaximumRpm;
> - uint16_t MGpuFanBoostLimitRpm;
> + uint16_t MGpuFanBoostLimitRpm;
> uint16_t FanTargetTemperature;
> uint16_t FanTargetGfxclk;
> uint16_t FanPadding16;
> uint8_t FanTempInputSelect;
> uint8_t FanPadding;
> - uint8_t FanZeroRpmEnable;
> + uint8_t FanZeroRpmEnable;
> uint8_t FanTachEdgePerRev;
> -
> +
> // The following are AFC override parameters. Leave at 0 to use FW
> defaults.
> int16_t FuzzyFan_ErrorSetDelta;
> int16_t FuzzyFan_ErrorRateSetDelta;
> int16_t FuzzyFan_PwmSetDelta;
> uint16_t FuzzyFan_Reserved;
>
> - // SECTION: AVFS
> + // SECTION: AVFS
> // Overrides
> uint8_t OverrideAvfsGb[AVFS_VOLTAGE_COUNT];
> - uint8_t dBtcGbGfxDfllModelSelect; //0 -> fused piece-wise
> model, 1 -> piece-wise linear(PPTable), 2 -> quadratic model(PPTable)
> + uint8_t dBtcGbGfxDfllModelSelect; //0 -> fused piece-wise
> model, 1 -> piece-wise linear(PPTable), 2 -> quadratic model(PPTable)
> uint8_t Padding8_Avfs;
>
> - QuadraticInt_t qAvfsGb[AVFS_VOLTAGE_COUNT]; // GHz->V
> Override of fused curve
> + QuadraticInt_t qAvfsGb[AVFS_VOLTAGE_COUNT]; // GHz->V
> Override of fused curve
> DroopInt_t dBtcGbGfxPll; // GHz->V BtcGb
> DroopInt_t dBtcGbGfxDfll; // GHz->V BtcGb
> DroopInt_t dBtcGbSoc; // GHz->V BtcGb
> - LinearInt_t qAgingGb[AVFS_VOLTAGE_COUNT]; // GHz->V
> + LinearInt_t qAgingGb[AVFS_VOLTAGE_COUNT]; // GHz->V
>
> PiecewiseLinearDroopInt_t PiecewiseLinearDroopIntGfxDfll; //GHz
> ->Vstore in IEEE float format
>
> - QuadraticInt_t qStaticVoltageOffset[AVFS_VOLTAGE_COUNT]; // GHz->V
> + QuadraticInt_t qStaticVoltageOffset[AVFS_VOLTAGE_COUNT]; // GHz->V
>
> uint16_t DcTol[AVFS_VOLTAGE_COUNT]; // mV Q2
>
> @@ -807,17 +807,17 @@ typedef struct {
> uint16_t DcBtcMax[AVFS_VOLTAGE_COUNT]; // mV Q2
>
> uint16_t DcBtcGb[AVFS_VOLTAGE_COUNT]; // mV Q2
> -
> +
> // SECTION: XGMI
> uint8_t XgmiDpmPstates[NUM_XGMI_LEVELS]; // 2 DPM states,
> high and low. 0-P0, 1-P1, 2-P2, 3-P3.
> uint8_t XgmiDpmSpare[2];
>
> // SECTION: Advanced Options
> uint32_t DebugOverrides;
> - QuadraticInt_t ReservedEquation0;
> - QuadraticInt_t ReservedEquation1;
> - QuadraticInt_t ReservedEquation2;
> - QuadraticInt_t ReservedEquation3;
> + QuadraticInt_t ReservedEquation0;
> + QuadraticInt_t ReservedEquation1;
> + QuadraticInt_t ReservedEquation2;
> + QuadraticInt_t ReservedEquation3;
>
> // SECTION: Sku Reserved
> uint8_t CustomerVariant;
> @@ -825,17 +825,17 @@ typedef struct {
> //VC BTC parameters are only applicable to VDD_GFX domain
> uint8_t VcBtcEnabled;
> uint16_t VcBtcVminT0; // T0_VMIN
> - uint16_t VcBtcFixedVminAgingOffset; //
> FIXED_VMIN_AGING_OFFSET
> - uint16_t VcBtcVmin2PsmDegrationGb; //
> VMIN_TO_PSM_DEGRADATION_GB
> + uint16_t VcBtcFixedVminAgingOffset; //
> FIXED_VMIN_AGING_OFFSET
> + uint16_t VcBtcVmin2PsmDegrationGb; //
> VMIN_TO_PSM_DEGRADATION_GB
> uint32_t VcBtcPsmA; // A_PSM
> uint32_t VcBtcPsmB; // B_PSM
> uint32_t VcBtcVminA; // A_VMIN
> - uint32_t VcBtcVminB; // B_VMIN
> -
> + uint32_t VcBtcVminB; // B_VMIN
> +
> //GPIO Board feature
> uint16_t LedGpio; //GeneriA GPIO flag used to
> control the radeon LEDs
> - uint16_t GfxPowerStagesGpio; //Genlk_vsync GPIO flag used to
> control gfx power stages
> -
> + uint16_t GfxPowerStagesGpio; //Genlk_vsync GPIO flag used to
> control gfx power stages
> +
> uint32_t SkuReserved[8];
>
>
> @@ -845,7 +845,7 @@ typedef struct {
> uint32_t GamingClk[6];
>
> // SECTION: I2C Control
> - I2cControllerConfig_t I2cControllers[NUM_I2C_CONTROLLERS];
> + I2cControllerConfig_t I2cControllers[NUM_I2C_CONTROLLERS];
>
> uint8_t GpioScl; // GPIO Number for SCL Line, used only for
> CKSVII2C1
> uint8_t GpioSda; // GPIO Number for SDA Line, used only for
> CKSVII2C1
> @@ -875,21 +875,21 @@ typedef struct {
> uint16_t Mem0MaxCurrent; // in Amps
> int8_t Mem0Offset; // in Amps
> uint8_t Padding_TelemetryMem0;
> -
> +
> uint16_t Mem1MaxCurrent; // in Amps
> int8_t Mem1Offset; // in Amps
> uint8_t Padding_TelemetryMem1;
>
> uint32_t MvddRatio; // This is used for MVDD Svi2 Div Ratio
> workaround. It has 16 fractional bits (Q16.16)
> -
> +
> // SECTION: GPIO Settings
> uint8_t AcDcGpio; // GPIO pin configured for AC/DC
> switching
> uint8_t AcDcPolarity; // GPIO polarity for AC/DC switching
> uint8_t VR0HotGpio; // GPIO pin configured for VR0 HOT
> event
> uint8_t VR0HotPolarity; // GPIO polarity for VR0 HOT event
>
> - uint8_t VR1HotGpio; // GPIO pin configured for VR1 HOT
> event
> - uint8_t VR1HotPolarity; // GPIO polarity for VR1 HOT event
> + uint8_t VR1HotGpio; // GPIO pin configured for VR1 HOT
> event
> + uint8_t VR1HotPolarity; // GPIO polarity for VR1 HOT event
> uint8_t GthrGpio; // GPIO pin configured for GTHR Event
> uint8_t GthrPolarity; // replace GPIO polarity for GTHR
>
> @@ -914,7 +914,7 @@ typedef struct {
> uint8_t DfllGfxclkSpreadEnabled; // on or off
> uint8_t DfllGfxclkSpreadPercent; // Q4.4
> uint16_t DfllGfxclkSpreadFreq; // kHz
> -
> +
> // UCLK Spread Spectrum
> uint16_t UclkSpreadPadding;
> uint16_t UclkSpreadFreq; // kHz
> @@ -923,23 +923,23 @@ typedef struct {
> uint8_t FclkSpreadEnabled; // on or off
> uint8_t FclkSpreadPercent; // Q4.4
> uint16_t FclkSpreadFreq; // kHz
> -
> +
> // Section: Memory Config
> - uint32_t MemoryChannelEnabled; // For DRAM use only, Max 32
> channels enabled bit mask.
> -
> + uint32_t MemoryChannelEnabled; // For DRAM use only, Max 32
> channels enabled bit mask.
> +
> uint8_t DramBitWidth; // For DRAM use only. See Dram Bit width
> type defines
> uint8_t PaddingMem1[3];
>
> // Section: Total Board Power
> uint16_t TotalBoardPower; //Only needed for TCP Estimated
> case, where TCP = TGP+Total Board Power
> - uint16_t BoardPowerPadding;
> -
> + uint16_t BoardPowerPadding;
> +
> // SECTION: XGMI Training
> - uint8_t XgmiLinkSpeed [NUM_XGMI_PSTATE_LEVELS];
> - uint8_t XgmiLinkWidth [NUM_XGMI_PSTATE_LEVELS];
> + uint8_t XgmiLinkSpeed[NUM_XGMI_PSTATE_LEVELS];
> + uint8_t XgmiLinkWidth[NUM_XGMI_PSTATE_LEVELS];
>
> - uint16_t XgmiFclkFreq [NUM_XGMI_PSTATE_LEVELS];
> - uint16_t XgmiSocVoltage [NUM_XGMI_PSTATE_LEVELS];
> + uint16_t XgmiFclkFreq[NUM_XGMI_PSTATE_LEVELS];
> + uint16_t XgmiSocVoltage[NUM_XGMI_PSTATE_LEVELS];
>
> // SECTION: UMC feature flags
> uint8_t HsrEnabled;
> @@ -947,7 +947,7 @@ typedef struct {
> uint8_t PaddingUmcFlags[2];
>
> // UCLK Spread Spectrum
> - uint8_t UclkSpreadPercent[16];
> + uint8_t UclkSpreadPercent[16];
>
> // SECTION: Board Reserved
> uint32_t BoardReserved[11];
> @@ -983,14 +983,14 @@ typedef struct {
>
> // SECTION: Power Configuration
> uint8_t TotalPowerConfig; //0-TDP, 1-TGP, 2-TCP Estimated,
> 3-TCP Measured. Use defines from PwrConfig_e
> - uint8_t TotalPowerPadding[3];
> + uint8_t TotalPowerPadding[3];
>
> // SECTION: APCC Settings
> uint32_t ApccPlusResidencyLimit;
>
> //SECTION: SMNCLK DPM
> - uint16_t SmnclkDpmFreq [NUM_SMNCLK_DPM_LEVELS]; //
> in MHz
> - uint16_t SmnclkDpmVoltage [NUM_SMNCLK_DPM_LEVELS]; //
> mV(Q2)
> + uint16_t SmnclkDpmFreq[NUM_SMNCLK_DPM_LEVELS]; // in MHz
> + uint16_t SmnclkDpmVoltage[NUM_SMNCLK_DPM_LEVELS]; //
> mV(Q2)
>
> uint32_t PaddingAPCC;
> uint16_t
> PerPartDroopVsetGfxDfll[NUM_PIECE_WISE_LINEAR_DROOP_MODEL_VF_POINTS];
> //In mV(Q2)
> @@ -999,14 +999,14 @@ typedef struct {
> // SECTION: Throttler settings
> uint32_t ThrottlerControlMask; // See Throtter masks defines
>
> - // SECTION: FW DSTATE Settings
> + // SECTION: FW DSTATE Settings
> uint32_t FwDStateMask; // See FW DState masks defines
>
> // SECTION: ULV Settings
> uint16_t UlvVoltageOffsetSoc; // In mV(Q2)
> uint16_t UlvVoltageOffsetGfx; // In mV(Q2)
>
> - uint16_t MinVoltageUlvGfx; // In mV(Q2) Minimum Voltage ("Vmin")
> of VDD_GFX in ULV mode
> + uint16_t MinVoltageUlvGfx; // In mV(Q2) Minimum Voltage ("Vmin")
> of VDD_GFX in ULV mode
> uint16_t MinVoltageUlvSoc; // In mV(Q2) Minimum Voltage ("Vmin")
> of VDD_SOC in ULV mode
>
> uint16_t SocLIVmin;
> @@ -1038,42 +1038,42 @@ typedef struct {
> //SECTION: DPM Config 1
> DpmDescriptor_t DpmDescriptor[PPCLK_COUNT];
>
> - uint16_t FreqTableGfx [NUM_GFXCLK_DPM_LEVELS ]; // In
> MHz
> - uint16_t FreqTableVclk [NUM_VCLK_DPM_LEVELS ]; // In
> MHz
> - uint16_t FreqTableDclk [NUM_DCLK_DPM_LEVELS ]; // In
> MHz
> - uint16_t FreqTableSocclk [NUM_SOCCLK_DPM_LEVELS ]; // In
> MHz
> - uint16_t FreqTableUclk [NUM_UCLK_DPM_LEVELS ]; // In
> MHz
> - uint16_t FreqTableDcefclk [NUM_DCEFCLK_DPM_LEVELS ]; // In
> MHz
> - uint16_t FreqTableDispclk [NUM_DISPCLK_DPM_LEVELS ]; // In
> MHz
> - uint16_t FreqTablePixclk [NUM_PIXCLK_DPM_LEVELS ]; // In
> MHz
> - uint16_t FreqTablePhyclk [NUM_PHYCLK_DPM_LEVELS ]; // In
> MHz
> - uint16_t FreqTableDtbclk [NUM_DTBCLK_DPM_LEVELS ]; // In
> MHz
> - uint16_t FreqTableFclk [NUM_FCLK_DPM_LEVELS ]; // In
> MHz
> + uint16_t FreqTableGfx[NUM_GFXCLK_DPM_LEVELS]; // In MHz
> + uint16_t FreqTableVclk[NUM_VCLK_DPM_LEVELS]; // In MHz
> + uint16_t FreqTableDclk[NUM_DCLK_DPM_LEVELS]; // In MHz
> + uint16_t FreqTableSocclk[NUM_SOCCLK_DPM_LEVELS]; // In MHz
> + uint16_t FreqTableUclk[NUM_UCLK_DPM_LEVELS]; // In MHz
> + uint16_t FreqTableDcefclk[NUM_DCEFCLK_DPM_LEVELS]; // In
> MHz
> + uint16_t FreqTableDispclk[NUM_DISPCLK_DPM_LEVELS]; // In
> MHz
> + uint16_t FreqTablePixclk[NUM_PIXCLK_DPM_LEVELS]; // In MHz
> + uint16_t FreqTablePhyclk[NUM_PHYCLK_DPM_LEVELS]; // In MHz
> + uint16_t FreqTableDtbclk[NUM_DTBCLK_DPM_LEVELS]; // In MHz
> + uint16_t FreqTableFclk[NUM_FCLK_DPM_LEVELS]; // In MHz
> uint32_t Paddingclks;
>
> DroopInt_t
> PerPartDroopModelGfxDfll[NUM_PIECE_WISE_LINEAR_DROOP_MODEL_VF_POINTS];
> //GHz ->Vstore in IEEE float format
>
> - uint32_t DcModeMaxFreq [PPCLK_COUNT ]; // In
> MHz
> + uint32_t DcModeMaxFreq[PPCLK_COUNT]; // In MHz
>
> - uint8_t FreqTableUclkDiv [NUM_UCLK_DPM_LEVELS ]; //
> 0:Div-1, 1:Div-1/2, 2:Div-1/4, 3:Div-1/8
> + uint8_t FreqTableUclkDiv[NUM_UCLK_DPM_LEVELS]; // 0:Div-1,
> 1:Div-1/2, 2:Div-1/4, 3:Div-1/8
>
> // Used for MALL performance boost
> uint16_t FclkBoostFreq; // In
> Mhz
> uint16_t FclkParamPadding;
>
> // SECTION: DPM Config 2
> - uint16_t Mp0clkFreq [NUM_MP0CLK_DPM_LEVELS]; // in
> MHz
> - uint16_t Mp0DpmVoltage [NUM_MP0CLK_DPM_LEVELS]; //
> mV(Q2)
> - uint16_t MemVddciVoltage [NUM_UCLK_DPM_LEVELS]; //
> mV(Q2)
> - uint16_t MemMvddVoltage [NUM_UCLK_DPM_LEVELS]; //
> mV(Q2)
> + uint16_t Mp0clkFreq[NUM_MP0CLK_DPM_LEVELS]; // in MHz
> + uint16_t Mp0DpmVoltage[NUM_MP0CLK_DPM_LEVELS]; // mV(Q2)
> + uint16_t MemVddciVoltage[NUM_UCLK_DPM_LEVELS]; //
> mV(Q2)
> + uint16_t MemMvddVoltage[NUM_UCLK_DPM_LEVELS]; // mV(Q2)
> // GFXCLK DPM
> uint16_t GfxclkFgfxoffEntry; // in Mhz
> - uint16_t GfxclkFinit; // in Mhz
> - uint16_t GfxclkFidle; // in MHz
> + uint16_t GfxclkFinit; // in Mhz
> + uint16_t GfxclkFidle; // in MHz
> uint8_t GfxclkSource; // 0 = PLL, 1 = DFLL
> uint8_t GfxclkPadding;
>
> - // GFX GPO
> + // GFX GPO
> uint8_t GfxGpoSubFeatureMask; // bit 0 = PACE, bit 1 = DEM
> uint8_t GfxGpoEnabledWorkPolicyMask; //Any policy that GPO
> can be enabled
> uint8_t GfxGpoDisabledWorkPolicyMask; //Any policy that GPO
> can be disabled
> @@ -1086,14 +1086,14 @@ typedef struct {
> uint16_t GfxDcsFclkFopt; // Optimal FCLK for DCS in Mhz
> uint16_t GfxDcsUclkFopt; // Optimal UCLK for DCS in Mhz
>
> - uint16_t DcsGfxOffVoltage; //Voltage in mV(Q2) applied to
> VDDGFX when entering DCS GFXOFF phase
> + uint16_t DcsGfxOffVoltage; //Voltage in mV(Q2) applied to
> VDDGFX when entering DCS GFXOFF phase
>
> uint16_t DcsMinGfxOffTime; //Minimum amount of time PMFW
> shuts GFX OFF as part of GFX DCS phase
> uint16_t DcsMaxGfxOffTime; //Maximum amount of time PMFW
> can shut GFX OFF as part of GFX DCS phase at a stretch.
>
> uint32_t DcsMinCreditAccum; //Min amount of positive credit
> accumulation before waking GFX up as part of DCS.
>
> - uint16_t DcsExitHysteresis; //The min amount of time power
> credit accumulator should have a value > 0 before SMU exits the DCS
> throttling phase.
> + uint16_t DcsExitHysteresis; //The min amount of time power
> credit accumulator should have a value > 0 before SMU exits the DCS
> throttling phase.
> uint16_t DcsTimeout; //This is the amount of time
> SMU FW waits for RLC to put GFX into GFXOFF before reverting to the
> fallback mechanism of throttling GFXCLK to Fmin.
>
> uint32_t DcsParamPadding[5];
> @@ -1104,7 +1104,7 @@ typedef struct {
> uint8_t LowestUclkReservedForUlv; // Set this to 1 if UCLK DPM0
> is reserved for ULV-mode only
> uint8_t PaddingMem[3];
>
> - uint8_t UclkDpmPstates [NUM_UCLK_DPM_LEVELS]; // 4 DPM
> states, 0-P0, 1-P1, 2-P2, 3-P3.
> + uint8_t UclkDpmPstates[NUM_UCLK_DPM_LEVELS]; // 4 DPM
> states, 0-P0, 1-P1, 2-P2, 3-P3.
>
> // Used for 2-Step UCLK change workaround
> UclkDpmChangeRange_t UclkDpmSrcFreqRange; // In Mhz
> @@ -1115,7 +1115,7 @@ typedef struct {
> // Link DPM Settings
> uint8_t PcieGenSpeed[NUM_LINK_LEVELS]; ///<
> 0:PciE-gen1 1:PciE-gen2 2:PciE-gen3 3:PciE-gen4
> uint8_t PcieLaneCount[NUM_LINK_LEVELS]; ///< 1=x1,
> 2=x2, 3=x4, 4=x8, 5=x12, 6=x16
> - uint16_t LclkFreq[NUM_LINK_LEVELS];
> + uint16_t LclkFreq[NUM_LINK_LEVELS];
>
> // SECTION: Fan Control
> uint16_t FanStopTemp; //Celcius
> @@ -1133,7 +1133,7 @@ typedef struct {
> uint16_t FanPadding16;
> uint8_t FanTempInputSelect;
> uint8_t FanPadding;
> - uint8_t FanZeroRpmEnable;
> + uint8_t FanZeroRpmEnable;
> uint8_t FanTachEdgePerRev;
>
> // The following are AFC override parameters. Leave at 0 to use FW
> defaults.
> @@ -1142,21 +1142,21 @@ typedef struct {
> int16_t FuzzyFan_PwmSetDelta;
> uint16_t FuzzyFan_Reserved;
>
> - // SECTION: AVFS
> + // SECTION: AVFS
> // Overrides
> uint8_t OverrideAvfsGb[AVFS_VOLTAGE_COUNT];
> - uint8_t dBtcGbGfxDfllModelSelect; //0 -> fused piece-wise
> model, 1 -> piece-wise linear(PPTable), 2 -> quadratic model(PPTable)
> + uint8_t dBtcGbGfxDfllModelSelect; //0 -> fused piece-wise
> model, 1 -> piece-wise linear(PPTable), 2 -> quadratic model(PPTable)
> uint8_t Padding8_Avfs;
>
> - QuadraticInt_t qAvfsGb[AVFS_VOLTAGE_COUNT]; // GHz->V
> Override of fused curve
> + QuadraticInt_t qAvfsGb[AVFS_VOLTAGE_COUNT]; // GHz->V
> Override of fused curve
> DroopInt_t dBtcGbGfxPll; // GHz->V BtcGb
> DroopInt_t dBtcGbGfxDfll; // GHz->V BtcGb
> DroopInt_t dBtcGbSoc; // GHz->V BtcGb
> - LinearInt_t qAgingGb[AVFS_VOLTAGE_COUNT]; // GHz->V
> + LinearInt_t qAgingGb[AVFS_VOLTAGE_COUNT]; // GHz->V
>
> PiecewiseLinearDroopInt_t PiecewiseLinearDroopIntGfxDfll; //GHz
> ->Vstore in IEEE float format
>
> - QuadraticInt_t qStaticVoltageOffset[AVFS_VOLTAGE_COUNT]; // GHz->V
> + QuadraticInt_t qStaticVoltageOffset[AVFS_VOLTAGE_COUNT]; // GHz->V
>
> uint16_t DcTol[AVFS_VOLTAGE_COUNT]; // mV Q2
>
> @@ -1185,16 +1185,16 @@ typedef struct {
> //VC BTC parameters are only applicable to VDD_GFX domain
> uint8_t VcBtcEnabled;
> uint16_t VcBtcVminT0; // T0_VMIN
> - uint16_t VcBtcFixedVminAgingOffset; //
> FIXED_VMIN_AGING_OFFSET
> - uint16_t VcBtcVmin2PsmDegrationGb; //
> VMIN_TO_PSM_DEGRADATION_GB
> + uint16_t VcBtcFixedVminAgingOffset; //
> FIXED_VMIN_AGING_OFFSET
> + uint16_t VcBtcVmin2PsmDegrationGb; //
> VMIN_TO_PSM_DEGRADATION_GB
> uint32_t VcBtcPsmA; // A_PSM
> uint32_t VcBtcPsmB; // B_PSM
> uint32_t VcBtcVminA; // A_VMIN
> - uint32_t VcBtcVminB; // B_VMIN
> + uint32_t VcBtcVminB; // B_VMIN
>
> //GPIO Board feature
> uint16_t LedGpio; //GeneriA GPIO flag used to
> control the radeon LEDs
> - uint16_t GfxPowerStagesGpio; //Genlk_vsync GPIO flag used to
> control gfx power stages
> + uint16_t GfxPowerStagesGpio; //Genlk_vsync GPIO flag used to
> control gfx power stages
>
> uint32_t SkuReserved[63];
>
> @@ -1206,7 +1206,7 @@ typedef struct {
> uint32_t GamingClk[6];
>
> // SECTION: I2C Control
> - I2cControllerConfig_t I2cControllers[NUM_I2C_CONTROLLERS];
> + I2cControllerConfig_t I2cControllers[NUM_I2C_CONTROLLERS];
>
> uint8_t GpioScl; // GPIO Number for SCL Line, used only for
> CKSVII2C1
> uint8_t GpioSda; // GPIO Number for SDA Line, used only for
> CKSVII2C1
> @@ -1249,8 +1249,8 @@ typedef struct {
> uint8_t VR0HotGpio; // GPIO pin configured for VR0 HOT
> event
> uint8_t VR0HotPolarity; // GPIO polarity for VR0 HOT event
>
> - uint8_t VR1HotGpio; // GPIO pin configured for VR1 HOT
> event
> - uint8_t VR1HotPolarity; // GPIO polarity for VR1 HOT event
> + uint8_t VR1HotGpio; // GPIO pin configured for VR1 HOT
> event
> + uint8_t VR1HotPolarity; // GPIO polarity for VR1 HOT event
> uint8_t GthrGpio; // GPIO pin configured for GTHR Event
> uint8_t GthrPolarity; // replace GPIO polarity for GTHR
>
> @@ -1286,21 +1286,21 @@ typedef struct {
> uint16_t FclkSpreadFreq; // kHz
>
> // Section: Memory Config
> - uint32_t MemoryChannelEnabled; // For DRAM use only, Max 32
> channels enabled bit mask.
> + uint32_t MemoryChannelEnabled; // For DRAM use only, Max 32
> channels enabled bit mask.
>
> uint8_t DramBitWidth; // For DRAM use only. See Dram Bit width
> type defines
> uint8_t PaddingMem1[3];
>
> // Section: Total Board Power
> uint16_t TotalBoardPower; //Only needed for TCP Estimated
> case, where TCP = TGP+Total Board Power
> - uint16_t BoardPowerPadding;
> + uint16_t BoardPowerPadding;
>
> // SECTION: XGMI Training
> - uint8_t XgmiLinkSpeed [NUM_XGMI_PSTATE_LEVELS];
> - uint8_t XgmiLinkWidth [NUM_XGMI_PSTATE_LEVELS];
> + uint8_t XgmiLinkSpeed[NUM_XGMI_PSTATE_LEVELS];
> + uint8_t XgmiLinkWidth[NUM_XGMI_PSTATE_LEVELS];
>
> - uint16_t XgmiFclkFreq [NUM_XGMI_PSTATE_LEVELS];
> - uint16_t XgmiSocVoltage [NUM_XGMI_PSTATE_LEVELS];
> + uint16_t XgmiFclkFreq[NUM_XGMI_PSTATE_LEVELS];
> + uint16_t XgmiSocVoltage[NUM_XGMI_PSTATE_LEVELS];
>
> // SECTION: UMC feature flags
> uint8_t HsrEnabled;
> @@ -1328,15 +1328,15 @@ typedef struct {
> uint16_t UclkAverageLpfTau;
> uint16_t GfxActivityLpfTau;
> uint16_t UclkActivityLpfTau;
> - uint16_t SocketPowerLpfTau;
> + uint16_t SocketPowerLpfTau;
> uint16_t VcnClkAverageLpfTau;
> - uint16_t padding16;
> + uint16_t padding16;
> } DriverSmuConfig_t;
>
> typedef struct {
> DriverSmuConfig_t DriverSmuConfig;
>
> - uint32_t Spare[7];
> + uint32_t Spare[7];
> // Padding - ignore
> uint32_t MmHubPadding[8]; // SMU internal use
> } DriverSmuConfigExternal_t;
> @@ -1345,14 +1345,14 @@ typedef struct {
> uint16_t GfxclkFmin; // MHz
> uint16_t GfxclkFmax; // MHz
> QuadraticInt_t CustomGfxVfCurve; // a: mV/MHz^2, b:
> mv/MHz, c: mV
> - uint16_t CustomCurveFmin; // MHz
> - uint16_t UclkFmin; // MHz
> + uint16_t CustomCurveFmin; // MHz
> + uint16_t UclkFmin; // MHz
> uint16_t UclkFmax; // MHz
> int16_t OverDrivePct; // %
> uint16_t FanMaximumRpm;
> uint16_t FanMinimumPwm;
> uint16_t FanAcousticLimitRpm;
> - uint16_t FanTargetTemperature; // Degree Celcius
> + uint16_t FanTargetTemperature; // Degree Celcius
> uint8_t FanLinearPwmPoints[NUM_OD_FAN_MAX_POINTS];
> uint8_t FanLinearTempPoints[NUM_OD_FAN_MAX_POINTS];
> uint16_t MaxOpTemp; // Degree Celcius
> @@ -1361,13 +1361,13 @@ typedef struct {
> uint8_t FanZeroRpmStopTemp;
> uint8_t FanMode;
> uint8_t Padding[1];
> -} OverDriveTable_t;
> +} OverDriveTable_t;
>
> typedef struct {
> OverDriveTable_t OverDriveTable;
> - uint32_t Spare[8];
> + uint32_t Spare[8];
>
> - uint32_t MmHubPadding[8]; // SMU internal use
> + uint32_t MmHubPadding[8]; // SMU internal use
> } OverDriveTableExternal_t;
>
> typedef struct {
> @@ -1380,7 +1380,7 @@ typedef struct {
> uint16_t AverageUclkFrequencyPreDs ;
> uint16_t AverageUclkFrequencyPostDs ;
>
> -
> +
> uint16_t AverageGfxActivity ;
> uint16_t AverageUclkActivity ;
> uint8_t CurrSocVoltageOffset ;
> @@ -1393,14 +1393,14 @@ typedef struct {
> uint16_t TemperatureMem ;
> uint16_t TemperatureVrGfx ;
> uint16_t TemperatureVrMem0 ;
> - uint16_t TemperatureVrMem1 ;
> + uint16_t TemperatureVrMem1 ;
> uint16_t TemperatureVrSoc ;
> uint16_t TemperatureLiquid0 ;
> - uint16_t TemperatureLiquid1 ;
> + uint16_t TemperatureLiquid1 ;
> uint16_t TemperaturePlx ;
> uint16_t Padding16 ;
> - uint32_t ThrottlerStatus ;
> -
> + uint32_t ThrottlerStatus ;
> +
> uint8_t LinkDpmLevel;
> uint8_t CurrFanPwm;
> uint16_t CurrFanSpeed;
> @@ -1416,7 +1416,7 @@ typedef struct {
> uint16_t AverageVclk0Frequency ;
> uint16_t AverageDclk0Frequency ;
> uint16_t AverageVclk1Frequency ;
> - uint16_t AverageDclk1Frequency ;
> + uint16_t AverageDclk1Frequency ;
> uint16_t VcnActivityPercentage ; //place holder, David N. to provide
> full sequence
> uint8_t PcieRate ;
> uint8_t PcieWidth ;
> @@ -1614,7 +1614,7 @@ typedef struct {
> uint32_t Spare[1];
>
> // Padding - ignore
> - uint32_t MmHubPadding[8]; // SMU internal use
> + uint32_t MmHubPadding[8]; // SMU internal use
> } SmuMetricsExternal_t;
>
> typedef struct {
> @@ -1622,7 +1622,7 @@ typedef struct {
> uint16_t MaxClock; // This is either DCEFCLK or SOCCLK (in MHz)
> uint16_t MinUclk;
> uint16_t MaxUclk;
> -
> +
> uint8_t WmSetting;
> uint8_t Flags;
> uint8_t Padding[2];
> @@ -1658,7 +1658,7 @@ typedef struct {
> typedef struct {
> uint16_t avgPsmCount[67];
> uint16_t minPsmCount[67];
> - float avgPsmVoltage[67];
> + float avgPsmVoltage[67];
> float minPsmVoltage[67];
> } AvfsDebugTable_t;
>
> @@ -1673,7 +1673,7 @@ typedef struct {
> uint8_t Padding;
>
> uint8_t AvfsEn[AVFS_VOLTAGE_COUNT];
> -
> +
> uint8_t OverrideVFT[AVFS_VOLTAGE_COUNT];
> uint8_t OverrideAvfsGb[AVFS_VOLTAGE_COUNT];
>
> @@ -1729,7 +1729,7 @@ typedef struct {
> uint8_t Gfx_IdleHystLimit;
> uint8_t Gfx_FPS;
> uint8_t Gfx_MinActiveFreqType;
> - uint8_t Gfx_BoosterFreqType;
> + uint8_t Gfx_BoosterFreqType;
> uint8_t Gfx_MinFreqStep; // Minimum delta between
> current and target frequeny in order for FW to change clock.
> uint16_t Gfx_MinActiveFreq; // MHz
> uint16_t Gfx_BoosterFreq; // MHz
> @@ -1744,7 +1744,7 @@ typedef struct {
> uint8_t Fclk_IdleHystLimit;
> uint8_t Fclk_FPS;
> uint8_t Fclk_MinActiveFreqType;
> - uint8_t Fclk_BoosterFreqType;
> + uint8_t Fclk_BoosterFreqType;
> uint8_t Fclk_MinFreqStep; // Minimum delta between
> current and target frequeny in order for FW to change clock.
> uint16_t Fclk_MinActiveFreq; // MHz
> uint16_t Fclk_BoosterFreq; // MHz
> @@ -1780,19 +1780,19 @@ typedef struct {
>
> typedef struct {
> DpmActivityMonitorCoeffInt_t DpmActivityMonitorCoeffInt;
> - uint32_t MmHubPadding[8]; // SMU internal use
> + uint32_t MmHubPadding[8]; // SMU internal use
> } DpmActivityMonitorCoeffIntExternal_t;
>
> // Workload bits
> -#define WORKLOAD_PPLIB_DEFAULT_BIT 0
> -#define WORKLOAD_PPLIB_FULL_SCREEN_3D_BIT 1
> -#define WORKLOAD_PPLIB_POWER_SAVING_BIT 2
> -#define WORKLOAD_PPLIB_VIDEO_BIT 3
> -#define WORKLOAD_PPLIB_VR_BIT 4
> -#define WORKLOAD_PPLIB_COMPUTE_BIT 5
> -#define WORKLOAD_PPLIB_CUSTOM_BIT 6
> -#define WORKLOAD_PPLIB_W3D_BIT 7
> -#define WORKLOAD_PPLIB_COUNT 8
> +#define WORKLOAD_PPLIB_DEFAULT_BIT 0
> +#define WORKLOAD_PPLIB_FULL_SCREEN_3D_BIT 1
> +#define WORKLOAD_PPLIB_POWER_SAVING_BIT 2
> +#define WORKLOAD_PPLIB_VIDEO_BIT 3
> +#define WORKLOAD_PPLIB_VR_BIT 4
> +#define WORKLOAD_PPLIB_COMPUTE_BIT 5
> +#define WORKLOAD_PPLIB_CUSTOM_BIT 6
> +#define WORKLOAD_PPLIB_W3D_BIT 7
> +#define WORKLOAD_PPLIB_COUNT 8
>
>
> // These defines are used with the following messages:
> @@ -1824,8 +1824,8 @@ typedef struct {
>
> typedef struct {
> RlcPaceFlopsPerByteOverride_t RlcPaceFlopsPerByteOverride;
> -
> - uint32_t MmHubPadding[8]; // SMU internal use
> +
> + uint32_t MmHubPadding[8]; // SMU internal use
> } RlcPaceFlopsPerByteOverrideExternal_t;
>
> // These defines are used with the SMC_MSG_SetUclkFastSwitch message.
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] drm/amd/pm: Clean up errors in smu11_driver_if_sienna_cichlid.h
@ 2023-07-24 21:51 ` Alex Deucher
0 siblings, 0 replies; 8+ messages in thread
From: Alex Deucher @ 2023-07-24 21:51 UTC (permalink / raw)
To: sunran001; +Cc: alexander.deucher, amd-gfx, dri-devel, linux-kernel
This doesn't apply cleanly.
Alex
On Mon, Jul 24, 2023 at 4:30 AM <sunran001@208suo.com> wrote:
>
> Fix the following errors reported by checkpatch:
>
> ERROR: trailing whitespace
> ERROR: space prohibited before open square bracket '['
> ERROR: space prohibited before that close square bracket ']'
>
> Signed-off-by: Ran Sun <sunran001@208suo.com>
> ---
> .../pmfw_if/smu11_driver_if_sienna_cichlid.h | 386 +++++++++---------
> 1 file changed, 193 insertions(+), 193 deletions(-)
>
> diff --git
> a/drivers/gpu/drm/amd/pm/swsmu/inc/pmfw_if/smu11_driver_if_sienna_cichlid.h
> b/drivers/gpu/drm/amd/pm/swsmu/inc/pmfw_if/smu11_driver_if_sienna_cichlid.h
> index aa6d29de4002..703ade13d9f4 100644
> ---
> a/drivers/gpu/drm/amd/pm/swsmu/inc/pmfw_if/smu11_driver_if_sienna_cichlid.h
> +++
> b/drivers/gpu/drm/amd/pm/swsmu/inc/pmfw_if/smu11_driver_if_sienna_cichlid.h
> @@ -225,33 +225,33 @@ typedef enum {
> #define FW_DSTATE_MP1_WHISPER_MODE_BIT 6
> #define FW_DSTATE_SOC_LIV_MIN_BIT 7
> #define FW_DSTATE_SOC_PLL_PWRDN_BIT 8
> -#define FW_DSTATE_MEM_PLL_PWRDN_BIT 9
> +#define FW_DSTATE_MEM_PLL_PWRDN_BIT 9
> #define FW_DSTATE_OPTIMIZE_MALL_REFRESH_BIT 10
> #define FW_DSTATE_MEM_PSI_BIT 11
> #define FW_DSTATE_HSR_NON_STROBE_BIT 12
> #define FW_DSTATE_MP0_ENTER_WFI_BIT 13
>
> -#define FW_DSTATE_SOC_ULV_MASK (1 <<
> FW_DSTATE_SOC_ULV_BIT )
> -#define FW_DSTATE_G6_HSR_MASK (1 <<
> FW_DSTATE_G6_HSR_BIT )
> -#define FW_DSTATE_G6_PHY_VDDCI_OFF_MASK (1 <<
> FW_DSTATE_G6_PHY_VDDCI_OFF_BIT )
> -#define FW_DSTATE_MP1_DS_MASK (1 <<
> FW_DSTATE_MP1_DS_BIT )
> -#define FW_DSTATE_MP0_DS_MASK (1 <<
> FW_DSTATE_MP0_DS_BIT )
> -#define FW_DSTATE_SMN_DS_MASK (1 <<
> FW_DSTATE_SMN_DS_BIT )
> -#define FW_DSTATE_MP1_WHISPER_MODE_MASK (1 <<
> FW_DSTATE_MP1_WHISPER_MODE_BIT )
> -#define FW_DSTATE_SOC_LIV_MIN_MASK (1 <<
> FW_DSTATE_SOC_LIV_MIN_BIT )
> -#define FW_DSTATE_SOC_PLL_PWRDN_MASK (1 <<
> FW_DSTATE_SOC_PLL_PWRDN_BIT )
> -#define FW_DSTATE_MEM_PLL_PWRDN_MASK (1 <<
> FW_DSTATE_MEM_PLL_PWRDN_BIT )
> -#define FW_DSTATE_OPTIMIZE_MALL_REFRESH_MASK (1 <<
> FW_DSTATE_OPTIMIZE_MALL_REFRESH_BIT )
> -#define FW_DSTATE_MEM_PSI_MASK (1 <<
> FW_DSTATE_MEM_PSI_BIT )
> -#define FW_DSTATE_HSR_NON_STROBE_MASK (1 <<
> FW_DSTATE_HSR_NON_STROBE_BIT )
> -#define FW_DSTATE_MP0_ENTER_WFI_MASK (1 <<
> FW_DSTATE_MP0_ENTER_WFI_BIT )
> +#define FW_DSTATE_SOC_ULV_MASK (1 <<
> FW_DSTATE_SOC_ULV_BIT)
> +#define FW_DSTATE_G6_HSR_MASK (1 <<
> FW_DSTATE_G6_HSR_BIT)
> +#define FW_DSTATE_G6_PHY_VDDCI_OFF_MASK (1 <<
> FW_DSTATE_G6_PHY_VDDCI_OFF_BIT)
> +#define FW_DSTATE_MP1_DS_MASK (1 <<
> FW_DSTATE_MP1_DS_BIT)
> +#define FW_DSTATE_MP0_DS_MASK (1 <<
> FW_DSTATE_MP0_DS_BIT)
> +#define FW_DSTATE_SMN_DS_MASK (1 <<
> FW_DSTATE_SMN_DS_BIT)
> +#define FW_DSTATE_MP1_WHISPER_MODE_MASK (1 <<
> FW_DSTATE_MP1_WHISPER_MODE_BIT)
> +#define FW_DSTATE_SOC_LIV_MIN_MASK (1 <<
> FW_DSTATE_SOC_LIV_MIN_BIT)
> +#define FW_DSTATE_SOC_PLL_PWRDN_MASK (1 <<
> FW_DSTATE_SOC_PLL_PWRDN_BIT)
> +#define FW_DSTATE_MEM_PLL_PWRDN_MASK (1 <<
> FW_DSTATE_MEM_PLL_PWRDN_BIT)
> +#define FW_DSTATE_OPTIMIZE_MALL_REFRESH_MASK (1 <<
> FW_DSTATE_OPTIMIZE_MALL_REFRESH_BIT)
> +#define FW_DSTATE_MEM_PSI_MASK (1 <<
> FW_DSTATE_MEM_PSI_BIT)
> +#define FW_DSTATE_HSR_NON_STROBE_MASK (1 <<
> FW_DSTATE_HSR_NON_STROBE_BIT)
> +#define FW_DSTATE_MP0_ENTER_WFI_MASK (1 <<
> FW_DSTATE_MP0_ENTER_WFI_BIT)
>
> // GFX GPO Feature Contains PACE and DEM sub features
> #define GFX_GPO_PACE_BIT 0
> #define GFX_GPO_DEM_BIT 1
>
> #define GFX_GPO_PACE_MASK (1 << GFX_GPO_PACE_BIT)
> -#define GFX_GPO_DEM_MASK (1 << GFX_GPO_DEM_BIT )
> +#define GFX_GPO_DEM_MASK (1 << GFX_GPO_DEM_BIT)
>
> #define GPO_UPDATE_REQ_UCLKDPM_MASK 0x1
> #define GPO_UPDATE_REQ_FCLKDPM_MASK 0x2
> @@ -312,10 +312,10 @@ typedef enum {
> I2C_CONTROLLER_NAME_VR_VDDCI,
> I2C_CONTROLLER_NAME_VR_MVDD,
> I2C_CONTROLLER_NAME_LIQUID0,
> - I2C_CONTROLLER_NAME_LIQUID1,
> + I2C_CONTROLLER_NAME_LIQUID1,
> I2C_CONTROLLER_NAME_PLX,
> I2C_CONTROLLER_NAME_OTHER,
> - I2C_CONTROLLER_NAME_COUNT,
> + I2C_CONTROLLER_NAME_COUNT,
> } I2cControllerName_e;
>
> typedef enum {
> @@ -325,10 +325,10 @@ typedef enum {
> I2C_CONTROLLER_THROTTLER_VR_VDDCI,
> I2C_CONTROLLER_THROTTLER_VR_MVDD,
> I2C_CONTROLLER_THROTTLER_LIQUID0,
> - I2C_CONTROLLER_THROTTLER_LIQUID1,
> + I2C_CONTROLLER_THROTTLER_LIQUID1,
> I2C_CONTROLLER_THROTTLER_PLX,
> I2C_CONTROLLER_THROTTLER_INA3221,
> - I2C_CONTROLLER_THROTTLER_COUNT,
> + I2C_CONTROLLER_THROTTLER_COUNT,
> } I2cControllerThrottler_e;
>
> typedef enum {
> @@ -336,24 +336,24 @@ typedef enum {
> I2C_CONTROLLER_PROTOCOL_VR_IR35217,
> I2C_CONTROLLER_PROTOCOL_TMP_TMP102A,
> I2C_CONTROLLER_PROTOCOL_INA3221,
> - I2C_CONTROLLER_PROTOCOL_COUNT,
> + I2C_CONTROLLER_PROTOCOL_COUNT,
> } I2cControllerProtocol_e;
>
> typedef struct {
> uint8_t Enabled;
> uint8_t Speed;
> - uint8_t SlaveAddress;
> + uint8_t SlaveAddress;
> uint8_t ControllerPort;
> uint8_t ControllerName;
> uint8_t ThermalThrotter;
> uint8_t I2cProtocol;
> - uint8_t PaddingConfig;
> + uint8_t PaddingConfig;
> } I2cControllerConfig_t;
>
> typedef enum {
> - I2C_PORT_SVD_SCL = 0,
> - I2C_PORT_GPIO,
> -} I2cPort_e;
> + I2C_PORT_SVD_SCL = 0,
> + I2C_PORT_GPIO,
> +} I2cPort_e;
>
> typedef enum {
> I2C_SPEED_FAST_50K = 0, //50 Kbits/s
> @@ -362,13 +362,13 @@ typedef enum {
> I2C_SPEED_FAST_PLUS_1M, //1 Mbits/s (in fast mode)
> I2C_SPEED_HIGH_1M, //1 Mbits/s (in high speed mode)
> I2C_SPEED_HIGH_2M, //2.3 Mbits/s
> - I2C_SPEED_COUNT,
> + I2C_SPEED_COUNT,
> } I2cSpeed_e;
>
> typedef enum {
> I2C_CMD_READ = 0,
> I2C_CMD_WRITE,
> - I2C_CMD_COUNT,
> + I2C_CMD_COUNT,
> } I2cCmdType_e;
>
> typedef enum {
> @@ -426,24 +426,24 @@ typedef enum {
> PG_POWER_UP,
> } PowerGatingSettings_e;
>
> -typedef struct {
> +typedef struct {
> uint32_t a; // store in IEEE float format in this variable
> uint32_t b; // store in IEEE float format in this variable
> uint32_t c; // store in IEEE float format in this variable
> } QuadraticInt_t;
>
> -typedef struct {
> +typedef struct {
> uint32_t a; // store in fixed point, [31:20] signed integer, [19:0]
> fractional bits
> uint32_t b; // store in fixed point, [31:20] signed integer, [19:0]
> fractional bits
> uint32_t c; // store in fixed point, [31:20] signed integer, [19:0]
> fractional bits
> } QuadraticFixedPoint_t;
>
> -typedef struct {
> +typedef struct {
> uint32_t m; // store in IEEE float format in this variable
> uint32_t b; // store in IEEE float format in this variable
> } LinearInt_t;
>
> -typedef struct {
> +typedef struct {
> uint32_t a; // store in IEEE float format in this variable
> uint32_t b; // store in IEEE float format in this variable
> uint32_t c; // store in IEEE float format in this variable
> @@ -455,7 +455,7 @@ typedef enum {
> PIECEWISE_LINEAR_FUSED_MODEL = 0,
> PIECEWISE_LINEAR_PP_MODEL,
> QUADRATIC_PP_MODEL,
> - PERPART_PIECEWISE_LINEAR_PP_MODEL,
> + PERPART_PIECEWISE_LINEAR_PP_MODEL,
> } DfllDroopModelSelect_e;
>
> typedef struct {
> @@ -464,10 +464,10 @@ typedef struct {
> }PiecewiseLinearDroopInt_t;
>
> typedef enum {
> - GFXCLK_SOURCE_PLL = 0,
> - GFXCLK_SOURCE_DFLL,
> - GFXCLK_SOURCE_COUNT,
> -} GFXCLK_SOURCE_e;
> + GFXCLK_SOURCE_PLL = 0,
> + GFXCLK_SOURCE_DFLL,
> + GFXCLK_SOURCE_COUNT,
> +} GFXCLK_SOURCE_e;
>
> //Only Clks that have DPM descriptors are listed here
> typedef enum {
> @@ -552,18 +552,18 @@ typedef struct {
> uint8_t VoltageMode; // 0 - AVFS only, 1-
> min(AVFS,SS), 2-SS only
> uint8_t SnapToDiscrete; // 0 - Fine grained DPM, 1 -
> Discrete DPM
> uint8_t NumDiscreteLevels; // Set to 2 (Fmin, Fmax) when
> using fine grained DPM, otherwise set to # discrete levels used
> - uint8_t Padding;
> + uint8_t Padding;
> LinearInt_t ConversionToAvfsClk; // Transfer function to AVFS
> Clock (GHz->GHz)
> QuadraticInt_t SsCurve; // Slow-slow curve (GHz->V)
> uint16_t SsFmin; // Fmin for SS curve. If SS curve
> is selected, will use V@SSFmin for F <= Fmin
> - uint16_t Padding16;
> + uint16_t Padding16;
> } DpmDescriptor_t;
>
> typedef enum {
> PPT_THROTTLER_PPT0,
> PPT_THROTTLER_PPT1,
> PPT_THROTTLER_PPT2,
> - PPT_THROTTLER_PPT3,
> + PPT_THROTTLER_PPT3,
> PPT_THROTTLER_COUNT
> } PPT_THROTTLER_e;
>
> @@ -574,9 +574,9 @@ typedef enum {
> TEMP_VR_GFX,
> TEMP_VR_MEM0,
> TEMP_VR_MEM1,
> - TEMP_VR_SOC,
> + TEMP_VR_SOC,
> TEMP_LIQUID0,
> - TEMP_LIQUID1,
> + TEMP_LIQUID1,
> TEMP_PLX,
> TEMP_COUNT,
> } TEMP_e;
> @@ -592,7 +592,7 @@ typedef enum {
> CUSTOMER_VARIANT_FALCON,
> CUSTOMER_VARIANT_COUNT,
> } CUSTOMER_VARIANT_e;
> -
> +
> // Used for 2-step UCLK DPM change workaround
> typedef struct {
> uint16_t Fmin;
> @@ -623,14 +623,14 @@ typedef struct {
>
> // SECTION: Power Configuration
> uint8_t TotalPowerConfig; //0-TDP, 1-TGP, 2-TCP Estimated,
> 3-TCP Measured. Use defines from PwrConfig_e
> - uint8_t TotalPowerPadding[3];
> + uint8_t TotalPowerPadding[3];
>
> // SECTION: APCC Settings
> uint32_t ApccPlusResidencyLimit;
>
> //SECTION: SMNCLK DPM
> - uint16_t SmnclkDpmFreq [NUM_SMNCLK_DPM_LEVELS]; //
> in MHz
> - uint16_t SmnclkDpmVoltage [NUM_SMNCLK_DPM_LEVELS]; //
> mV(Q2)
> + uint16_t SmnclkDpmFreq[NUM_SMNCLK_DPM_LEVELS]; // in MHz
> + uint16_t SmnclkDpmVoltage[NUM_SMNCLK_DPM_LEVELS]; //
> mV(Q2)
>
> uint32_t PaddingAPCC;
> uint16_t
> PerPartDroopVsetGfxDfll[NUM_PIECE_WISE_LINEAR_DROOP_MODEL_VF_POINTS];
> //In mV(Q2)
> @@ -639,14 +639,14 @@ typedef struct {
> // SECTION: Throttler settings
> uint32_t ThrottlerControlMask; // See Throtter masks defines
>
> - // SECTION: FW DSTATE Settings
> + // SECTION: FW DSTATE Settings
> uint32_t FwDStateMask; // See FW DState masks defines
>
> // SECTION: ULV Settings
> uint16_t UlvVoltageOffsetSoc; // In mV(Q2)
> uint16_t UlvVoltageOffsetGfx; // In mV(Q2)
>
> - uint16_t MinVoltageUlvGfx; // In mV(Q2) Minimum Voltage ("Vmin")
> of VDD_GFX in ULV mode
> + uint16_t MinVoltageUlvGfx; // In mV(Q2) Minimum Voltage ("Vmin")
> of VDD_GFX in ULV mode
> uint16_t MinVoltageUlvSoc; // In mV(Q2) Minimum Voltage ("Vmin")
> of VDD_SOC in ULV mode
>
> uint16_t SocLIVmin; // In mV(Q2) Long Idle Vmin (deep
> ULV), for VDD_SOC
> @@ -678,62 +678,62 @@ typedef struct {
> //SECTION: DPM Config 1
> DpmDescriptor_t DpmDescriptor[PPCLK_COUNT];
>
> - uint16_t FreqTableGfx [NUM_GFXCLK_DPM_LEVELS ]; // In
> MHz
> - uint16_t FreqTableVclk [NUM_VCLK_DPM_LEVELS ]; // In
> MHz
> - uint16_t FreqTableDclk [NUM_DCLK_DPM_LEVELS ]; // In
> MHz
> - uint16_t FreqTableSocclk [NUM_SOCCLK_DPM_LEVELS ]; // In
> MHz
> - uint16_t FreqTableUclk [NUM_UCLK_DPM_LEVELS ]; // In
> MHz
> - uint16_t FreqTableDcefclk [NUM_DCEFCLK_DPM_LEVELS ]; // In
> MHz
> - uint16_t FreqTableDispclk [NUM_DISPCLK_DPM_LEVELS ]; // In
> MHz
> - uint16_t FreqTablePixclk [NUM_PIXCLK_DPM_LEVELS ]; // In
> MHz
> - uint16_t FreqTablePhyclk [NUM_PHYCLK_DPM_LEVELS ]; // In
> MHz
> - uint16_t FreqTableDtbclk [NUM_DTBCLK_DPM_LEVELS ]; // In
> MHz
> - uint16_t FreqTableFclk [NUM_FCLK_DPM_LEVELS ]; // In
> MHz
> + uint16_t FreqTableGfx[NUM_GFXCLK_DPM_LEVELS]; // In MHz
> + uint16_t FreqTableVclk[NUM_VCLK_DPM_LEVELS]; // In MHz
> + uint16_t FreqTableDclk[NUM_DCLK_DPM_LEVELS]; // In MHz
> + uint16_t FreqTableSocclk[NUM_SOCCLK_DPM_LEVELS]; // In MHz
> + uint16_t FreqTableUclk[NUM_UCLK_DPM_LEVELS]; // In MHz
> + uint16_t FreqTableDcefclk[NUM_DCEFCLK_DPM_LEVELS]; // In
> MHz
> + uint16_t FreqTableDispclk[NUM_DISPCLK_DPM_LEVELS]; // In
> MHz
> + uint16_t FreqTablePixclk[NUM_PIXCLK_DPM_LEVELS]; // In MHz
> + uint16_t FreqTablePhyclk[NUM_PHYCLK_DPM_LEVELS]; // In MHz
> + uint16_t FreqTableDtbclk[NUM_DTBCLK_DPM_LEVELS]; // In MHz
> + uint16_t FreqTableFclk[NUM_FCLK_DPM_LEVELS]; // In MHz
> uint32_t Paddingclks;
>
> DroopInt_t
> PerPartDroopModelGfxDfll[NUM_PIECE_WISE_LINEAR_DROOP_MODEL_VF_POINTS];
> //GHz ->Vstore in IEEE float format
>
> - uint32_t DcModeMaxFreq [PPCLK_COUNT ]; // In
> MHz
> + uint32_t DcModeMaxFreq[PPCLK_COUNT]; // In MHz
>
> - uint8_t FreqTableUclkDiv [NUM_UCLK_DPM_LEVELS ]; //
> 0:Div-1, 1:Div-1/2, 2:Div-1/4, 3:Div-1/8
> + uint8_t FreqTableUclkDiv[NUM_UCLK_DPM_LEVELS]; // 0:Div-1,
> 1:Div-1/2, 2:Div-1/4, 3:Div-1/8
>
> // Used for MALL performance boost
> uint16_t FclkBoostFreq; // In
> Mhz
> uint16_t FclkParamPadding;
>
> // SECTION: DPM Config 2
> - uint16_t Mp0clkFreq [NUM_MP0CLK_DPM_LEVELS]; // in
> MHz
> - uint16_t Mp0DpmVoltage [NUM_MP0CLK_DPM_LEVELS]; //
> mV(Q2)
> - uint16_t MemVddciVoltage [NUM_UCLK_DPM_LEVELS]; //
> mV(Q2)
> - uint16_t MemMvddVoltage [NUM_UCLK_DPM_LEVELS]; //
> mV(Q2)
> + uint16_t Mp0clkFreq[NUM_MP0CLK_DPM_LEVELS]; // in MHz
> + uint16_t Mp0DpmVoltage[NUM_MP0CLK_DPM_LEVELS]; // mV(Q2)
> + uint16_t MemVddciVoltage[NUM_UCLK_DPM_LEVELS]; //
> mV(Q2)
> + uint16_t MemMvddVoltage[NUM_UCLK_DPM_LEVELS]; // mV(Q2)
> // GFXCLK DPM
> uint16_t GfxclkFgfxoffEntry; // in Mhz
> - uint16_t GfxclkFinit; // in Mhz
> - uint16_t GfxclkFidle; // in MHz
> + uint16_t GfxclkFinit; // in Mhz
> + uint16_t GfxclkFidle; // in MHz
> uint8_t GfxclkSource; // 0 = PLL, 1 = DFLL
> uint8_t GfxclkPadding;
>
> - // GFX GPO
> + // GFX GPO
> uint8_t GfxGpoSubFeatureMask; // bit 0 = PACE, bit 1 = DEM
> uint8_t GfxGpoEnabledWorkPolicyMask; //Any policy that GPO
> can be enabled
> uint8_t GfxGpoDisabledWorkPolicyMask; //Any policy that GPO
> can be disabled
> uint8_t GfxGpoPadding[1];
> uint32_t GfxGpoVotingAllow; //For indicating which feature
> changes should result in a GPO table recalculation
>
> - uint32_t GfxGpoPadding32[4];
> + uint32_t GfxGpoPadding32[4];
>
> uint16_t GfxDcsFopt; // Optimal GFXCLK for DCS in
> Mhz
> uint16_t GfxDcsFclkFopt; // Optimal FCLK for DCS in Mhz
> uint16_t GfxDcsUclkFopt; // Optimal UCLK for DCS in Mhz
> -
> - uint16_t DcsGfxOffVoltage; //Voltage in mV(Q2) applied to
> VDDGFX when entering DCS GFXOFF phase
> +
> + uint16_t DcsGfxOffVoltage; //Voltage in mV(Q2) applied to
> VDDGFX when entering DCS GFXOFF phase
>
> uint16_t DcsMinGfxOffTime; //Minimum amount of time PMFW
> shuts GFX OFF as part of GFX DCS phase
> uint16_t DcsMaxGfxOffTime; //Maximum amount of time PMFW
> can shut GFX OFF as part of GFX DCS phase at a stretch.
>
> uint32_t DcsMinCreditAccum; //Min amount of positive credit
> accumulation before waking GFX up as part of DCS.
>
> - uint16_t DcsExitHysteresis; //The min amount of time power
> credit accumulator should have a value > 0 before SMU exits the DCS
> throttling phase.
> + uint16_t DcsExitHysteresis; //The min amount of time power
> credit accumulator should have a value > 0 before SMU exits the DCS
> throttling phase.
> uint16_t DcsTimeout; //This is the amount of time
> SMU FW waits for RLC to put GFX into GFXOFF before reverting to the
> fallback mechanism of throttling GFXCLK to Fmin.
>
> uint32_t DcsParamPadding[5];
> @@ -743,9 +743,9 @@ typedef struct {
> // UCLK section
> uint8_t LowestUclkReservedForUlv; // Set this to 1 if UCLK DPM0
> is reserved for ULV-mode only
> uint8_t PaddingMem[3];
> -
> - uint8_t UclkDpmPstates [NUM_UCLK_DPM_LEVELS]; // 4 DPM
> states, 0-P0, 1-P1, 2-P2, 3-P3.
> -
> +
> + uint8_t UclkDpmPstates[NUM_UCLK_DPM_LEVELS]; // 4 DPM
> states, 0-P0, 1-P1, 2-P2, 3-P3.
> +
> // Used for 2-Step UCLK change workaround
> UclkDpmChangeRange_t UclkDpmSrcFreqRange; // In Mhz
> UclkDpmChangeRange_t UclkDpmTargFreqRange; // In Mhz
> @@ -755,8 +755,8 @@ typedef struct {
> // Link DPM Settings
> uint8_t PcieGenSpeed[NUM_LINK_LEVELS]; ///<
> 0:PciE-gen1 1:PciE-gen2 2:PciE-gen3 3:PciE-gen4
> uint8_t PcieLaneCount[NUM_LINK_LEVELS]; ///< 1=x1,
> 2=x2, 3=x4, 4=x8, 5=x12, 6=x16
> - uint16_t LclkFreq[NUM_LINK_LEVELS];
> -
> + uint16_t LclkFreq[NUM_LINK_LEVELS];
> +
> // SECTION: Fan Control
> uint16_t FanStopTemp; //Celcius
> uint16_t FanStartTemp; //Celcius
> @@ -767,36 +767,36 @@ typedef struct {
> uint16_t FanAcousticLimitRpm;
> uint16_t FanThrottlingRpm;
> uint16_t FanMaximumRpm;
> - uint16_t MGpuFanBoostLimitRpm;
> + uint16_t MGpuFanBoostLimitRpm;
> uint16_t FanTargetTemperature;
> uint16_t FanTargetGfxclk;
> uint16_t FanPadding16;
> uint8_t FanTempInputSelect;
> uint8_t FanPadding;
> - uint8_t FanZeroRpmEnable;
> + uint8_t FanZeroRpmEnable;
> uint8_t FanTachEdgePerRev;
> -
> +
> // The following are AFC override parameters. Leave at 0 to use FW
> defaults.
> int16_t FuzzyFan_ErrorSetDelta;
> int16_t FuzzyFan_ErrorRateSetDelta;
> int16_t FuzzyFan_PwmSetDelta;
> uint16_t FuzzyFan_Reserved;
>
> - // SECTION: AVFS
> + // SECTION: AVFS
> // Overrides
> uint8_t OverrideAvfsGb[AVFS_VOLTAGE_COUNT];
> - uint8_t dBtcGbGfxDfllModelSelect; //0 -> fused piece-wise
> model, 1 -> piece-wise linear(PPTable), 2 -> quadratic model(PPTable)
> + uint8_t dBtcGbGfxDfllModelSelect; //0 -> fused piece-wise
> model, 1 -> piece-wise linear(PPTable), 2 -> quadratic model(PPTable)
> uint8_t Padding8_Avfs;
>
> - QuadraticInt_t qAvfsGb[AVFS_VOLTAGE_COUNT]; // GHz->V
> Override of fused curve
> + QuadraticInt_t qAvfsGb[AVFS_VOLTAGE_COUNT]; // GHz->V
> Override of fused curve
> DroopInt_t dBtcGbGfxPll; // GHz->V BtcGb
> DroopInt_t dBtcGbGfxDfll; // GHz->V BtcGb
> DroopInt_t dBtcGbSoc; // GHz->V BtcGb
> - LinearInt_t qAgingGb[AVFS_VOLTAGE_COUNT]; // GHz->V
> + LinearInt_t qAgingGb[AVFS_VOLTAGE_COUNT]; // GHz->V
>
> PiecewiseLinearDroopInt_t PiecewiseLinearDroopIntGfxDfll; //GHz
> ->Vstore in IEEE float format
>
> - QuadraticInt_t qStaticVoltageOffset[AVFS_VOLTAGE_COUNT]; // GHz->V
> + QuadraticInt_t qStaticVoltageOffset[AVFS_VOLTAGE_COUNT]; // GHz->V
>
> uint16_t DcTol[AVFS_VOLTAGE_COUNT]; // mV Q2
>
> @@ -807,17 +807,17 @@ typedef struct {
> uint16_t DcBtcMax[AVFS_VOLTAGE_COUNT]; // mV Q2
>
> uint16_t DcBtcGb[AVFS_VOLTAGE_COUNT]; // mV Q2
> -
> +
> // SECTION: XGMI
> uint8_t XgmiDpmPstates[NUM_XGMI_LEVELS]; // 2 DPM states,
> high and low. 0-P0, 1-P1, 2-P2, 3-P3.
> uint8_t XgmiDpmSpare[2];
>
> // SECTION: Advanced Options
> uint32_t DebugOverrides;
> - QuadraticInt_t ReservedEquation0;
> - QuadraticInt_t ReservedEquation1;
> - QuadraticInt_t ReservedEquation2;
> - QuadraticInt_t ReservedEquation3;
> + QuadraticInt_t ReservedEquation0;
> + QuadraticInt_t ReservedEquation1;
> + QuadraticInt_t ReservedEquation2;
> + QuadraticInt_t ReservedEquation3;
>
> // SECTION: Sku Reserved
> uint8_t CustomerVariant;
> @@ -825,17 +825,17 @@ typedef struct {
> //VC BTC parameters are only applicable to VDD_GFX domain
> uint8_t VcBtcEnabled;
> uint16_t VcBtcVminT0; // T0_VMIN
> - uint16_t VcBtcFixedVminAgingOffset; //
> FIXED_VMIN_AGING_OFFSET
> - uint16_t VcBtcVmin2PsmDegrationGb; //
> VMIN_TO_PSM_DEGRADATION_GB
> + uint16_t VcBtcFixedVminAgingOffset; //
> FIXED_VMIN_AGING_OFFSET
> + uint16_t VcBtcVmin2PsmDegrationGb; //
> VMIN_TO_PSM_DEGRADATION_GB
> uint32_t VcBtcPsmA; // A_PSM
> uint32_t VcBtcPsmB; // B_PSM
> uint32_t VcBtcVminA; // A_VMIN
> - uint32_t VcBtcVminB; // B_VMIN
> -
> + uint32_t VcBtcVminB; // B_VMIN
> +
> //GPIO Board feature
> uint16_t LedGpio; //GeneriA GPIO flag used to
> control the radeon LEDs
> - uint16_t GfxPowerStagesGpio; //Genlk_vsync GPIO flag used to
> control gfx power stages
> -
> + uint16_t GfxPowerStagesGpio; //Genlk_vsync GPIO flag used to
> control gfx power stages
> +
> uint32_t SkuReserved[8];
>
>
> @@ -845,7 +845,7 @@ typedef struct {
> uint32_t GamingClk[6];
>
> // SECTION: I2C Control
> - I2cControllerConfig_t I2cControllers[NUM_I2C_CONTROLLERS];
> + I2cControllerConfig_t I2cControllers[NUM_I2C_CONTROLLERS];
>
> uint8_t GpioScl; // GPIO Number for SCL Line, used only for
> CKSVII2C1
> uint8_t GpioSda; // GPIO Number for SDA Line, used only for
> CKSVII2C1
> @@ -875,21 +875,21 @@ typedef struct {
> uint16_t Mem0MaxCurrent; // in Amps
> int8_t Mem0Offset; // in Amps
> uint8_t Padding_TelemetryMem0;
> -
> +
> uint16_t Mem1MaxCurrent; // in Amps
> int8_t Mem1Offset; // in Amps
> uint8_t Padding_TelemetryMem1;
>
> uint32_t MvddRatio; // This is used for MVDD Svi2 Div Ratio
> workaround. It has 16 fractional bits (Q16.16)
> -
> +
> // SECTION: GPIO Settings
> uint8_t AcDcGpio; // GPIO pin configured for AC/DC
> switching
> uint8_t AcDcPolarity; // GPIO polarity for AC/DC switching
> uint8_t VR0HotGpio; // GPIO pin configured for VR0 HOT
> event
> uint8_t VR0HotPolarity; // GPIO polarity for VR0 HOT event
>
> - uint8_t VR1HotGpio; // GPIO pin configured for VR1 HOT
> event
> - uint8_t VR1HotPolarity; // GPIO polarity for VR1 HOT event
> + uint8_t VR1HotGpio; // GPIO pin configured for VR1 HOT
> event
> + uint8_t VR1HotPolarity; // GPIO polarity for VR1 HOT event
> uint8_t GthrGpio; // GPIO pin configured for GTHR Event
> uint8_t GthrPolarity; // replace GPIO polarity for GTHR
>
> @@ -914,7 +914,7 @@ typedef struct {
> uint8_t DfllGfxclkSpreadEnabled; // on or off
> uint8_t DfllGfxclkSpreadPercent; // Q4.4
> uint16_t DfllGfxclkSpreadFreq; // kHz
> -
> +
> // UCLK Spread Spectrum
> uint16_t UclkSpreadPadding;
> uint16_t UclkSpreadFreq; // kHz
> @@ -923,23 +923,23 @@ typedef struct {
> uint8_t FclkSpreadEnabled; // on or off
> uint8_t FclkSpreadPercent; // Q4.4
> uint16_t FclkSpreadFreq; // kHz
> -
> +
> // Section: Memory Config
> - uint32_t MemoryChannelEnabled; // For DRAM use only, Max 32
> channels enabled bit mask.
> -
> + uint32_t MemoryChannelEnabled; // For DRAM use only, Max 32
> channels enabled bit mask.
> +
> uint8_t DramBitWidth; // For DRAM use only. See Dram Bit width
> type defines
> uint8_t PaddingMem1[3];
>
> // Section: Total Board Power
> uint16_t TotalBoardPower; //Only needed for TCP Estimated
> case, where TCP = TGP+Total Board Power
> - uint16_t BoardPowerPadding;
> -
> + uint16_t BoardPowerPadding;
> +
> // SECTION: XGMI Training
> - uint8_t XgmiLinkSpeed [NUM_XGMI_PSTATE_LEVELS];
> - uint8_t XgmiLinkWidth [NUM_XGMI_PSTATE_LEVELS];
> + uint8_t XgmiLinkSpeed[NUM_XGMI_PSTATE_LEVELS];
> + uint8_t XgmiLinkWidth[NUM_XGMI_PSTATE_LEVELS];
>
> - uint16_t XgmiFclkFreq [NUM_XGMI_PSTATE_LEVELS];
> - uint16_t XgmiSocVoltage [NUM_XGMI_PSTATE_LEVELS];
> + uint16_t XgmiFclkFreq[NUM_XGMI_PSTATE_LEVELS];
> + uint16_t XgmiSocVoltage[NUM_XGMI_PSTATE_LEVELS];
>
> // SECTION: UMC feature flags
> uint8_t HsrEnabled;
> @@ -947,7 +947,7 @@ typedef struct {
> uint8_t PaddingUmcFlags[2];
>
> // UCLK Spread Spectrum
> - uint8_t UclkSpreadPercent[16];
> + uint8_t UclkSpreadPercent[16];
>
> // SECTION: Board Reserved
> uint32_t BoardReserved[11];
> @@ -983,14 +983,14 @@ typedef struct {
>
> // SECTION: Power Configuration
> uint8_t TotalPowerConfig; //0-TDP, 1-TGP, 2-TCP Estimated,
> 3-TCP Measured. Use defines from PwrConfig_e
> - uint8_t TotalPowerPadding[3];
> + uint8_t TotalPowerPadding[3];
>
> // SECTION: APCC Settings
> uint32_t ApccPlusResidencyLimit;
>
> //SECTION: SMNCLK DPM
> - uint16_t SmnclkDpmFreq [NUM_SMNCLK_DPM_LEVELS]; //
> in MHz
> - uint16_t SmnclkDpmVoltage [NUM_SMNCLK_DPM_LEVELS]; //
> mV(Q2)
> + uint16_t SmnclkDpmFreq[NUM_SMNCLK_DPM_LEVELS]; // in MHz
> + uint16_t SmnclkDpmVoltage[NUM_SMNCLK_DPM_LEVELS]; //
> mV(Q2)
>
> uint32_t PaddingAPCC;
> uint16_t
> PerPartDroopVsetGfxDfll[NUM_PIECE_WISE_LINEAR_DROOP_MODEL_VF_POINTS];
> //In mV(Q2)
> @@ -999,14 +999,14 @@ typedef struct {
> // SECTION: Throttler settings
> uint32_t ThrottlerControlMask; // See Throtter masks defines
>
> - // SECTION: FW DSTATE Settings
> + // SECTION: FW DSTATE Settings
> uint32_t FwDStateMask; // See FW DState masks defines
>
> // SECTION: ULV Settings
> uint16_t UlvVoltageOffsetSoc; // In mV(Q2)
> uint16_t UlvVoltageOffsetGfx; // In mV(Q2)
>
> - uint16_t MinVoltageUlvGfx; // In mV(Q2) Minimum Voltage ("Vmin")
> of VDD_GFX in ULV mode
> + uint16_t MinVoltageUlvGfx; // In mV(Q2) Minimum Voltage ("Vmin")
> of VDD_GFX in ULV mode
> uint16_t MinVoltageUlvSoc; // In mV(Q2) Minimum Voltage ("Vmin")
> of VDD_SOC in ULV mode
>
> uint16_t SocLIVmin;
> @@ -1038,42 +1038,42 @@ typedef struct {
> //SECTION: DPM Config 1
> DpmDescriptor_t DpmDescriptor[PPCLK_COUNT];
>
> - uint16_t FreqTableGfx [NUM_GFXCLK_DPM_LEVELS ]; // In
> MHz
> - uint16_t FreqTableVclk [NUM_VCLK_DPM_LEVELS ]; // In
> MHz
> - uint16_t FreqTableDclk [NUM_DCLK_DPM_LEVELS ]; // In
> MHz
> - uint16_t FreqTableSocclk [NUM_SOCCLK_DPM_LEVELS ]; // In
> MHz
> - uint16_t FreqTableUclk [NUM_UCLK_DPM_LEVELS ]; // In
> MHz
> - uint16_t FreqTableDcefclk [NUM_DCEFCLK_DPM_LEVELS ]; // In
> MHz
> - uint16_t FreqTableDispclk [NUM_DISPCLK_DPM_LEVELS ]; // In
> MHz
> - uint16_t FreqTablePixclk [NUM_PIXCLK_DPM_LEVELS ]; // In
> MHz
> - uint16_t FreqTablePhyclk [NUM_PHYCLK_DPM_LEVELS ]; // In
> MHz
> - uint16_t FreqTableDtbclk [NUM_DTBCLK_DPM_LEVELS ]; // In
> MHz
> - uint16_t FreqTableFclk [NUM_FCLK_DPM_LEVELS ]; // In
> MHz
> + uint16_t FreqTableGfx[NUM_GFXCLK_DPM_LEVELS]; // In MHz
> + uint16_t FreqTableVclk[NUM_VCLK_DPM_LEVELS]; // In MHz
> + uint16_t FreqTableDclk[NUM_DCLK_DPM_LEVELS]; // In MHz
> + uint16_t FreqTableSocclk[NUM_SOCCLK_DPM_LEVELS]; // In MHz
> + uint16_t FreqTableUclk[NUM_UCLK_DPM_LEVELS]; // In MHz
> + uint16_t FreqTableDcefclk[NUM_DCEFCLK_DPM_LEVELS]; // In
> MHz
> + uint16_t FreqTableDispclk[NUM_DISPCLK_DPM_LEVELS]; // In
> MHz
> + uint16_t FreqTablePixclk[NUM_PIXCLK_DPM_LEVELS]; // In MHz
> + uint16_t FreqTablePhyclk[NUM_PHYCLK_DPM_LEVELS]; // In MHz
> + uint16_t FreqTableDtbclk[NUM_DTBCLK_DPM_LEVELS]; // In MHz
> + uint16_t FreqTableFclk[NUM_FCLK_DPM_LEVELS]; // In MHz
> uint32_t Paddingclks;
>
> DroopInt_t
> PerPartDroopModelGfxDfll[NUM_PIECE_WISE_LINEAR_DROOP_MODEL_VF_POINTS];
> //GHz ->Vstore in IEEE float format
>
> - uint32_t DcModeMaxFreq [PPCLK_COUNT ]; // In
> MHz
> + uint32_t DcModeMaxFreq[PPCLK_COUNT]; // In MHz
>
> - uint8_t FreqTableUclkDiv [NUM_UCLK_DPM_LEVELS ]; //
> 0:Div-1, 1:Div-1/2, 2:Div-1/4, 3:Div-1/8
> + uint8_t FreqTableUclkDiv[NUM_UCLK_DPM_LEVELS]; // 0:Div-1,
> 1:Div-1/2, 2:Div-1/4, 3:Div-1/8
>
> // Used for MALL performance boost
> uint16_t FclkBoostFreq; // In
> Mhz
> uint16_t FclkParamPadding;
>
> // SECTION: DPM Config 2
> - uint16_t Mp0clkFreq [NUM_MP0CLK_DPM_LEVELS]; // in
> MHz
> - uint16_t Mp0DpmVoltage [NUM_MP0CLK_DPM_LEVELS]; //
> mV(Q2)
> - uint16_t MemVddciVoltage [NUM_UCLK_DPM_LEVELS]; //
> mV(Q2)
> - uint16_t MemMvddVoltage [NUM_UCLK_DPM_LEVELS]; //
> mV(Q2)
> + uint16_t Mp0clkFreq[NUM_MP0CLK_DPM_LEVELS]; // in MHz
> + uint16_t Mp0DpmVoltage[NUM_MP0CLK_DPM_LEVELS]; // mV(Q2)
> + uint16_t MemVddciVoltage[NUM_UCLK_DPM_LEVELS]; //
> mV(Q2)
> + uint16_t MemMvddVoltage[NUM_UCLK_DPM_LEVELS]; // mV(Q2)
> // GFXCLK DPM
> uint16_t GfxclkFgfxoffEntry; // in Mhz
> - uint16_t GfxclkFinit; // in Mhz
> - uint16_t GfxclkFidle; // in MHz
> + uint16_t GfxclkFinit; // in Mhz
> + uint16_t GfxclkFidle; // in MHz
> uint8_t GfxclkSource; // 0 = PLL, 1 = DFLL
> uint8_t GfxclkPadding;
>
> - // GFX GPO
> + // GFX GPO
> uint8_t GfxGpoSubFeatureMask; // bit 0 = PACE, bit 1 = DEM
> uint8_t GfxGpoEnabledWorkPolicyMask; //Any policy that GPO
> can be enabled
> uint8_t GfxGpoDisabledWorkPolicyMask; //Any policy that GPO
> can be disabled
> @@ -1086,14 +1086,14 @@ typedef struct {
> uint16_t GfxDcsFclkFopt; // Optimal FCLK for DCS in Mhz
> uint16_t GfxDcsUclkFopt; // Optimal UCLK for DCS in Mhz
>
> - uint16_t DcsGfxOffVoltage; //Voltage in mV(Q2) applied to
> VDDGFX when entering DCS GFXOFF phase
> + uint16_t DcsGfxOffVoltage; //Voltage in mV(Q2) applied to
> VDDGFX when entering DCS GFXOFF phase
>
> uint16_t DcsMinGfxOffTime; //Minimum amount of time PMFW
> shuts GFX OFF as part of GFX DCS phase
> uint16_t DcsMaxGfxOffTime; //Maximum amount of time PMFW
> can shut GFX OFF as part of GFX DCS phase at a stretch.
>
> uint32_t DcsMinCreditAccum; //Min amount of positive credit
> accumulation before waking GFX up as part of DCS.
>
> - uint16_t DcsExitHysteresis; //The min amount of time power
> credit accumulator should have a value > 0 before SMU exits the DCS
> throttling phase.
> + uint16_t DcsExitHysteresis; //The min amount of time power
> credit accumulator should have a value > 0 before SMU exits the DCS
> throttling phase.
> uint16_t DcsTimeout; //This is the amount of time
> SMU FW waits for RLC to put GFX into GFXOFF before reverting to the
> fallback mechanism of throttling GFXCLK to Fmin.
>
> uint32_t DcsParamPadding[5];
> @@ -1104,7 +1104,7 @@ typedef struct {
> uint8_t LowestUclkReservedForUlv; // Set this to 1 if UCLK DPM0
> is reserved for ULV-mode only
> uint8_t PaddingMem[3];
>
> - uint8_t UclkDpmPstates [NUM_UCLK_DPM_LEVELS]; // 4 DPM
> states, 0-P0, 1-P1, 2-P2, 3-P3.
> + uint8_t UclkDpmPstates[NUM_UCLK_DPM_LEVELS]; // 4 DPM
> states, 0-P0, 1-P1, 2-P2, 3-P3.
>
> // Used for 2-Step UCLK change workaround
> UclkDpmChangeRange_t UclkDpmSrcFreqRange; // In Mhz
> @@ -1115,7 +1115,7 @@ typedef struct {
> // Link DPM Settings
> uint8_t PcieGenSpeed[NUM_LINK_LEVELS]; ///<
> 0:PciE-gen1 1:PciE-gen2 2:PciE-gen3 3:PciE-gen4
> uint8_t PcieLaneCount[NUM_LINK_LEVELS]; ///< 1=x1,
> 2=x2, 3=x4, 4=x8, 5=x12, 6=x16
> - uint16_t LclkFreq[NUM_LINK_LEVELS];
> + uint16_t LclkFreq[NUM_LINK_LEVELS];
>
> // SECTION: Fan Control
> uint16_t FanStopTemp; //Celcius
> @@ -1133,7 +1133,7 @@ typedef struct {
> uint16_t FanPadding16;
> uint8_t FanTempInputSelect;
> uint8_t FanPadding;
> - uint8_t FanZeroRpmEnable;
> + uint8_t FanZeroRpmEnable;
> uint8_t FanTachEdgePerRev;
>
> // The following are AFC override parameters. Leave at 0 to use FW
> defaults.
> @@ -1142,21 +1142,21 @@ typedef struct {
> int16_t FuzzyFan_PwmSetDelta;
> uint16_t FuzzyFan_Reserved;
>
> - // SECTION: AVFS
> + // SECTION: AVFS
> // Overrides
> uint8_t OverrideAvfsGb[AVFS_VOLTAGE_COUNT];
> - uint8_t dBtcGbGfxDfllModelSelect; //0 -> fused piece-wise
> model, 1 -> piece-wise linear(PPTable), 2 -> quadratic model(PPTable)
> + uint8_t dBtcGbGfxDfllModelSelect; //0 -> fused piece-wise
> model, 1 -> piece-wise linear(PPTable), 2 -> quadratic model(PPTable)
> uint8_t Padding8_Avfs;
>
> - QuadraticInt_t qAvfsGb[AVFS_VOLTAGE_COUNT]; // GHz->V
> Override of fused curve
> + QuadraticInt_t qAvfsGb[AVFS_VOLTAGE_COUNT]; // GHz->V
> Override of fused curve
> DroopInt_t dBtcGbGfxPll; // GHz->V BtcGb
> DroopInt_t dBtcGbGfxDfll; // GHz->V BtcGb
> DroopInt_t dBtcGbSoc; // GHz->V BtcGb
> - LinearInt_t qAgingGb[AVFS_VOLTAGE_COUNT]; // GHz->V
> + LinearInt_t qAgingGb[AVFS_VOLTAGE_COUNT]; // GHz->V
>
> PiecewiseLinearDroopInt_t PiecewiseLinearDroopIntGfxDfll; //GHz
> ->Vstore in IEEE float format
>
> - QuadraticInt_t qStaticVoltageOffset[AVFS_VOLTAGE_COUNT]; // GHz->V
> + QuadraticInt_t qStaticVoltageOffset[AVFS_VOLTAGE_COUNT]; // GHz->V
>
> uint16_t DcTol[AVFS_VOLTAGE_COUNT]; // mV Q2
>
> @@ -1185,16 +1185,16 @@ typedef struct {
> //VC BTC parameters are only applicable to VDD_GFX domain
> uint8_t VcBtcEnabled;
> uint16_t VcBtcVminT0; // T0_VMIN
> - uint16_t VcBtcFixedVminAgingOffset; //
> FIXED_VMIN_AGING_OFFSET
> - uint16_t VcBtcVmin2PsmDegrationGb; //
> VMIN_TO_PSM_DEGRADATION_GB
> + uint16_t VcBtcFixedVminAgingOffset; //
> FIXED_VMIN_AGING_OFFSET
> + uint16_t VcBtcVmin2PsmDegrationGb; //
> VMIN_TO_PSM_DEGRADATION_GB
> uint32_t VcBtcPsmA; // A_PSM
> uint32_t VcBtcPsmB; // B_PSM
> uint32_t VcBtcVminA; // A_VMIN
> - uint32_t VcBtcVminB; // B_VMIN
> + uint32_t VcBtcVminB; // B_VMIN
>
> //GPIO Board feature
> uint16_t LedGpio; //GeneriA GPIO flag used to
> control the radeon LEDs
> - uint16_t GfxPowerStagesGpio; //Genlk_vsync GPIO flag used to
> control gfx power stages
> + uint16_t GfxPowerStagesGpio; //Genlk_vsync GPIO flag used to
> control gfx power stages
>
> uint32_t SkuReserved[63];
>
> @@ -1206,7 +1206,7 @@ typedef struct {
> uint32_t GamingClk[6];
>
> // SECTION: I2C Control
> - I2cControllerConfig_t I2cControllers[NUM_I2C_CONTROLLERS];
> + I2cControllerConfig_t I2cControllers[NUM_I2C_CONTROLLERS];
>
> uint8_t GpioScl; // GPIO Number for SCL Line, used only for
> CKSVII2C1
> uint8_t GpioSda; // GPIO Number for SDA Line, used only for
> CKSVII2C1
> @@ -1249,8 +1249,8 @@ typedef struct {
> uint8_t VR0HotGpio; // GPIO pin configured for VR0 HOT
> event
> uint8_t VR0HotPolarity; // GPIO polarity for VR0 HOT event
>
> - uint8_t VR1HotGpio; // GPIO pin configured for VR1 HOT
> event
> - uint8_t VR1HotPolarity; // GPIO polarity for VR1 HOT event
> + uint8_t VR1HotGpio; // GPIO pin configured for VR1 HOT
> event
> + uint8_t VR1HotPolarity; // GPIO polarity for VR1 HOT event
> uint8_t GthrGpio; // GPIO pin configured for GTHR Event
> uint8_t GthrPolarity; // replace GPIO polarity for GTHR
>
> @@ -1286,21 +1286,21 @@ typedef struct {
> uint16_t FclkSpreadFreq; // kHz
>
> // Section: Memory Config
> - uint32_t MemoryChannelEnabled; // For DRAM use only, Max 32
> channels enabled bit mask.
> + uint32_t MemoryChannelEnabled; // For DRAM use only, Max 32
> channels enabled bit mask.
>
> uint8_t DramBitWidth; // For DRAM use only. See Dram Bit width
> type defines
> uint8_t PaddingMem1[3];
>
> // Section: Total Board Power
> uint16_t TotalBoardPower; //Only needed for TCP Estimated
> case, where TCP = TGP+Total Board Power
> - uint16_t BoardPowerPadding;
> + uint16_t BoardPowerPadding;
>
> // SECTION: XGMI Training
> - uint8_t XgmiLinkSpeed [NUM_XGMI_PSTATE_LEVELS];
> - uint8_t XgmiLinkWidth [NUM_XGMI_PSTATE_LEVELS];
> + uint8_t XgmiLinkSpeed[NUM_XGMI_PSTATE_LEVELS];
> + uint8_t XgmiLinkWidth[NUM_XGMI_PSTATE_LEVELS];
>
> - uint16_t XgmiFclkFreq [NUM_XGMI_PSTATE_LEVELS];
> - uint16_t XgmiSocVoltage [NUM_XGMI_PSTATE_LEVELS];
> + uint16_t XgmiFclkFreq[NUM_XGMI_PSTATE_LEVELS];
> + uint16_t XgmiSocVoltage[NUM_XGMI_PSTATE_LEVELS];
>
> // SECTION: UMC feature flags
> uint8_t HsrEnabled;
> @@ -1328,15 +1328,15 @@ typedef struct {
> uint16_t UclkAverageLpfTau;
> uint16_t GfxActivityLpfTau;
> uint16_t UclkActivityLpfTau;
> - uint16_t SocketPowerLpfTau;
> + uint16_t SocketPowerLpfTau;
> uint16_t VcnClkAverageLpfTau;
> - uint16_t padding16;
> + uint16_t padding16;
> } DriverSmuConfig_t;
>
> typedef struct {
> DriverSmuConfig_t DriverSmuConfig;
>
> - uint32_t Spare[7];
> + uint32_t Spare[7];
> // Padding - ignore
> uint32_t MmHubPadding[8]; // SMU internal use
> } DriverSmuConfigExternal_t;
> @@ -1345,14 +1345,14 @@ typedef struct {
> uint16_t GfxclkFmin; // MHz
> uint16_t GfxclkFmax; // MHz
> QuadraticInt_t CustomGfxVfCurve; // a: mV/MHz^2, b:
> mv/MHz, c: mV
> - uint16_t CustomCurveFmin; // MHz
> - uint16_t UclkFmin; // MHz
> + uint16_t CustomCurveFmin; // MHz
> + uint16_t UclkFmin; // MHz
> uint16_t UclkFmax; // MHz
> int16_t OverDrivePct; // %
> uint16_t FanMaximumRpm;
> uint16_t FanMinimumPwm;
> uint16_t FanAcousticLimitRpm;
> - uint16_t FanTargetTemperature; // Degree Celcius
> + uint16_t FanTargetTemperature; // Degree Celcius
> uint8_t FanLinearPwmPoints[NUM_OD_FAN_MAX_POINTS];
> uint8_t FanLinearTempPoints[NUM_OD_FAN_MAX_POINTS];
> uint16_t MaxOpTemp; // Degree Celcius
> @@ -1361,13 +1361,13 @@ typedef struct {
> uint8_t FanZeroRpmStopTemp;
> uint8_t FanMode;
> uint8_t Padding[1];
> -} OverDriveTable_t;
> +} OverDriveTable_t;
>
> typedef struct {
> OverDriveTable_t OverDriveTable;
> - uint32_t Spare[8];
> + uint32_t Spare[8];
>
> - uint32_t MmHubPadding[8]; // SMU internal use
> + uint32_t MmHubPadding[8]; // SMU internal use
> } OverDriveTableExternal_t;
>
> typedef struct {
> @@ -1380,7 +1380,7 @@ typedef struct {
> uint16_t AverageUclkFrequencyPreDs ;
> uint16_t AverageUclkFrequencyPostDs ;
>
> -
> +
> uint16_t AverageGfxActivity ;
> uint16_t AverageUclkActivity ;
> uint8_t CurrSocVoltageOffset ;
> @@ -1393,14 +1393,14 @@ typedef struct {
> uint16_t TemperatureMem ;
> uint16_t TemperatureVrGfx ;
> uint16_t TemperatureVrMem0 ;
> - uint16_t TemperatureVrMem1 ;
> + uint16_t TemperatureVrMem1 ;
> uint16_t TemperatureVrSoc ;
> uint16_t TemperatureLiquid0 ;
> - uint16_t TemperatureLiquid1 ;
> + uint16_t TemperatureLiquid1 ;
> uint16_t TemperaturePlx ;
> uint16_t Padding16 ;
> - uint32_t ThrottlerStatus ;
> -
> + uint32_t ThrottlerStatus ;
> +
> uint8_t LinkDpmLevel;
> uint8_t CurrFanPwm;
> uint16_t CurrFanSpeed;
> @@ -1416,7 +1416,7 @@ typedef struct {
> uint16_t AverageVclk0Frequency ;
> uint16_t AverageDclk0Frequency ;
> uint16_t AverageVclk1Frequency ;
> - uint16_t AverageDclk1Frequency ;
> + uint16_t AverageDclk1Frequency ;
> uint16_t VcnActivityPercentage ; //place holder, David N. to provide
> full sequence
> uint8_t PcieRate ;
> uint8_t PcieWidth ;
> @@ -1614,7 +1614,7 @@ typedef struct {
> uint32_t Spare[1];
>
> // Padding - ignore
> - uint32_t MmHubPadding[8]; // SMU internal use
> + uint32_t MmHubPadding[8]; // SMU internal use
> } SmuMetricsExternal_t;
>
> typedef struct {
> @@ -1622,7 +1622,7 @@ typedef struct {
> uint16_t MaxClock; // This is either DCEFCLK or SOCCLK (in MHz)
> uint16_t MinUclk;
> uint16_t MaxUclk;
> -
> +
> uint8_t WmSetting;
> uint8_t Flags;
> uint8_t Padding[2];
> @@ -1658,7 +1658,7 @@ typedef struct {
> typedef struct {
> uint16_t avgPsmCount[67];
> uint16_t minPsmCount[67];
> - float avgPsmVoltage[67];
> + float avgPsmVoltage[67];
> float minPsmVoltage[67];
> } AvfsDebugTable_t;
>
> @@ -1673,7 +1673,7 @@ typedef struct {
> uint8_t Padding;
>
> uint8_t AvfsEn[AVFS_VOLTAGE_COUNT];
> -
> +
> uint8_t OverrideVFT[AVFS_VOLTAGE_COUNT];
> uint8_t OverrideAvfsGb[AVFS_VOLTAGE_COUNT];
>
> @@ -1729,7 +1729,7 @@ typedef struct {
> uint8_t Gfx_IdleHystLimit;
> uint8_t Gfx_FPS;
> uint8_t Gfx_MinActiveFreqType;
> - uint8_t Gfx_BoosterFreqType;
> + uint8_t Gfx_BoosterFreqType;
> uint8_t Gfx_MinFreqStep; // Minimum delta between
> current and target frequeny in order for FW to change clock.
> uint16_t Gfx_MinActiveFreq; // MHz
> uint16_t Gfx_BoosterFreq; // MHz
> @@ -1744,7 +1744,7 @@ typedef struct {
> uint8_t Fclk_IdleHystLimit;
> uint8_t Fclk_FPS;
> uint8_t Fclk_MinActiveFreqType;
> - uint8_t Fclk_BoosterFreqType;
> + uint8_t Fclk_BoosterFreqType;
> uint8_t Fclk_MinFreqStep; // Minimum delta between
> current and target frequeny in order for FW to change clock.
> uint16_t Fclk_MinActiveFreq; // MHz
> uint16_t Fclk_BoosterFreq; // MHz
> @@ -1780,19 +1780,19 @@ typedef struct {
>
> typedef struct {
> DpmActivityMonitorCoeffInt_t DpmActivityMonitorCoeffInt;
> - uint32_t MmHubPadding[8]; // SMU internal use
> + uint32_t MmHubPadding[8]; // SMU internal use
> } DpmActivityMonitorCoeffIntExternal_t;
>
> // Workload bits
> -#define WORKLOAD_PPLIB_DEFAULT_BIT 0
> -#define WORKLOAD_PPLIB_FULL_SCREEN_3D_BIT 1
> -#define WORKLOAD_PPLIB_POWER_SAVING_BIT 2
> -#define WORKLOAD_PPLIB_VIDEO_BIT 3
> -#define WORKLOAD_PPLIB_VR_BIT 4
> -#define WORKLOAD_PPLIB_COMPUTE_BIT 5
> -#define WORKLOAD_PPLIB_CUSTOM_BIT 6
> -#define WORKLOAD_PPLIB_W3D_BIT 7
> -#define WORKLOAD_PPLIB_COUNT 8
> +#define WORKLOAD_PPLIB_DEFAULT_BIT 0
> +#define WORKLOAD_PPLIB_FULL_SCREEN_3D_BIT 1
> +#define WORKLOAD_PPLIB_POWER_SAVING_BIT 2
> +#define WORKLOAD_PPLIB_VIDEO_BIT 3
> +#define WORKLOAD_PPLIB_VR_BIT 4
> +#define WORKLOAD_PPLIB_COMPUTE_BIT 5
> +#define WORKLOAD_PPLIB_CUSTOM_BIT 6
> +#define WORKLOAD_PPLIB_W3D_BIT 7
> +#define WORKLOAD_PPLIB_COUNT 8
>
>
> // These defines are used with the following messages:
> @@ -1824,8 +1824,8 @@ typedef struct {
>
> typedef struct {
> RlcPaceFlopsPerByteOverride_t RlcPaceFlopsPerByteOverride;
> -
> - uint32_t MmHubPadding[8]; // SMU internal use
> +
> + uint32_t MmHubPadding[8]; // SMU internal use
> } RlcPaceFlopsPerByteOverrideExternal_t;
>
> // These defines are used with the SMC_MSG_SetUclkFastSwitch message.
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] drm/amd/pm: Clean up errors in smu11_driver_if_sienna_cichlid.h
2023-07-24 8:34 ` sunran001
@ 2023-07-24 21:52 ` Alex Deucher
-1 siblings, 0 replies; 8+ messages in thread
From: Alex Deucher @ 2023-07-24 21:52 UTC (permalink / raw)
To: sunran001; +Cc: alexander.deucher, dri-devel, amd-gfx, linux-kernel
This doesn't apply cleanly.
Alex
On Mon, Jul 24, 2023 at 4:34 AM <sunran001@208suo.com> wrote:
>
> Fix the following errors reported by checkpatch:
>
> ERROR: trailing whitespace
>
> Signed-off-by: Ran Sun <sunran001@208suo.com>
> ---
> drivers/gpu/drm/amd/pm/swsmu/inc/pmfw_if/smu_v13_0_1_pmfw.h | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/pm/swsmu/inc/pmfw_if/smu_v13_0_1_pmfw.h
> b/drivers/gpu/drm/amd/pm/swsmu/inc/pmfw_if/smu_v13_0_1_pmfw.h
> index c5e26d619bf0..8ec588248aac 100644
> --- a/drivers/gpu/drm/amd/pm/swsmu/inc/pmfw_if/smu_v13_0_1_pmfw.h
> +++ b/drivers/gpu/drm/amd/pm/swsmu/inc/pmfw_if/smu_v13_0_1_pmfw.h
> @@ -30,7 +30,7 @@
>
> #define ENABLE_DEBUG_FEATURES
>
> -// Firmware features
> +// Firmware features
> // Feature Control Defines
> #define FEATURE_CCLK_DPM_BIT 0
> #define FEATURE_FAN_CONTROLLER_BIT 1
> @@ -92,7 +92,7 @@
> #define FEATURE_ZSTATES_ECO_BIT 57
> #define FEATURE_CC6_BIT 58
> #define FEATURE_DS_UMCCLK_BIT 59
> -#define FEATURE_DS_HSPCLK_BIT 60
> +#define FEATURE_DS_HSPCLK_BIT 60
> #define NUM_FEATURES 61
>
> typedef struct {
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] drm/amd/pm: Clean up errors in smu11_driver_if_sienna_cichlid.h
@ 2023-07-24 21:52 ` Alex Deucher
0 siblings, 0 replies; 8+ messages in thread
From: Alex Deucher @ 2023-07-24 21:52 UTC (permalink / raw)
To: sunran001; +Cc: alexander.deucher, amd-gfx, dri-devel, linux-kernel
This doesn't apply cleanly.
Alex
On Mon, Jul 24, 2023 at 4:34 AM <sunran001@208suo.com> wrote:
>
> Fix the following errors reported by checkpatch:
>
> ERROR: trailing whitespace
>
> Signed-off-by: Ran Sun <sunran001@208suo.com>
> ---
> drivers/gpu/drm/amd/pm/swsmu/inc/pmfw_if/smu_v13_0_1_pmfw.h | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/pm/swsmu/inc/pmfw_if/smu_v13_0_1_pmfw.h
> b/drivers/gpu/drm/amd/pm/swsmu/inc/pmfw_if/smu_v13_0_1_pmfw.h
> index c5e26d619bf0..8ec588248aac 100644
> --- a/drivers/gpu/drm/amd/pm/swsmu/inc/pmfw_if/smu_v13_0_1_pmfw.h
> +++ b/drivers/gpu/drm/amd/pm/swsmu/inc/pmfw_if/smu_v13_0_1_pmfw.h
> @@ -30,7 +30,7 @@
>
> #define ENABLE_DEBUG_FEATURES
>
> -// Firmware features
> +// Firmware features
> // Feature Control Defines
> #define FEATURE_CCLK_DPM_BIT 0
> #define FEATURE_FAN_CONTROLLER_BIT 1
> @@ -92,7 +92,7 @@
> #define FEATURE_ZSTATES_ECO_BIT 57
> #define FEATURE_CC6_BIT 58
> #define FEATURE_DS_UMCCLK_BIT 59
> -#define FEATURE_DS_HSPCLK_BIT 60
> +#define FEATURE_DS_HSPCLK_BIT 60
> #define NUM_FEATURES 61
>
> typedef struct {
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH] drm/amd/pm: Clean up errors in smu11_driver_if_sienna_cichlid.h
[not found] <20230724083344.9285-1-xujianghui@cdjrlc.com>
@ 2023-07-24 8:34 ` sunran001
0 siblings, 0 replies; 8+ messages in thread
From: sunran001 @ 2023-07-24 8:34 UTC (permalink / raw)
To: alexander.deucher; +Cc: dri-devel, amd-gfx, linux-kernel
Fix the following errors reported by checkpatch:
ERROR: trailing whitespace
Signed-off-by: Ran Sun <sunran001@208suo.com>
---
drivers/gpu/drm/amd/pm/swsmu/inc/pmfw_if/smu_v13_0_1_pmfw.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/amd/pm/swsmu/inc/pmfw_if/smu_v13_0_1_pmfw.h
b/drivers/gpu/drm/amd/pm/swsmu/inc/pmfw_if/smu_v13_0_1_pmfw.h
index c5e26d619bf0..8ec588248aac 100644
--- a/drivers/gpu/drm/amd/pm/swsmu/inc/pmfw_if/smu_v13_0_1_pmfw.h
+++ b/drivers/gpu/drm/amd/pm/swsmu/inc/pmfw_if/smu_v13_0_1_pmfw.h
@@ -30,7 +30,7 @@
#define ENABLE_DEBUG_FEATURES
-// Firmware features
+// Firmware features
// Feature Control Defines
#define FEATURE_CCLK_DPM_BIT 0
#define FEATURE_FAN_CONTROLLER_BIT 1
@@ -92,7 +92,7 @@
#define FEATURE_ZSTATES_ECO_BIT 57
#define FEATURE_CC6_BIT 58
#define FEATURE_DS_UMCCLK_BIT 59
-#define FEATURE_DS_HSPCLK_BIT 60
+#define FEATURE_DS_HSPCLK_BIT 60
#define NUM_FEATURES 61
typedef struct {
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH] drm/amd/pm: Clean up errors in smu11_driver_if_sienna_cichlid.h
@ 2023-07-24 8:34 ` sunran001
0 siblings, 0 replies; 8+ messages in thread
From: sunran001 @ 2023-07-24 8:34 UTC (permalink / raw)
To: alexander.deucher; +Cc: amd-gfx, dri-devel, linux-kernel
Fix the following errors reported by checkpatch:
ERROR: trailing whitespace
Signed-off-by: Ran Sun <sunran001@208suo.com>
---
drivers/gpu/drm/amd/pm/swsmu/inc/pmfw_if/smu_v13_0_1_pmfw.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/amd/pm/swsmu/inc/pmfw_if/smu_v13_0_1_pmfw.h
b/drivers/gpu/drm/amd/pm/swsmu/inc/pmfw_if/smu_v13_0_1_pmfw.h
index c5e26d619bf0..8ec588248aac 100644
--- a/drivers/gpu/drm/amd/pm/swsmu/inc/pmfw_if/smu_v13_0_1_pmfw.h
+++ b/drivers/gpu/drm/amd/pm/swsmu/inc/pmfw_if/smu_v13_0_1_pmfw.h
@@ -30,7 +30,7 @@
#define ENABLE_DEBUG_FEATURES
-// Firmware features
+// Firmware features
// Feature Control Defines
#define FEATURE_CCLK_DPM_BIT 0
#define FEATURE_FAN_CONTROLLER_BIT 1
@@ -92,7 +92,7 @@
#define FEATURE_ZSTATES_ECO_BIT 57
#define FEATURE_CC6_BIT 58
#define FEATURE_DS_UMCCLK_BIT 59
-#define FEATURE_DS_HSPCLK_BIT 60
+#define FEATURE_DS_HSPCLK_BIT 60
#define NUM_FEATURES 61
typedef struct {
^ permalink raw reply related [flat|nested] 8+ messages in thread
end of thread, other threads:[~2023-07-24 21:52 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <20230724082935.9199-1-xujianghui@cdjrlc.com>
2023-07-24 8:30 ` [PATCH] drm/amd/pm: Clean up errors in smu11_driver_if_sienna_cichlid.h sunran001
2023-07-24 8:30 ` sunran001
2023-07-24 21:51 ` Alex Deucher
2023-07-24 21:51 ` Alex Deucher
[not found] <20230724083344.9285-1-xujianghui@cdjrlc.com>
2023-07-24 8:34 ` sunran001
2023-07-24 8:34 ` sunran001
2023-07-24 21:52 ` Alex Deucher
2023-07-24 21:52 ` Alex Deucher
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.