drm/amd: Update `update_pcie_parameters` functions to use uint8_t arguments
[ Upstream commit 7752ccf85b929a22e658ec145283e8f31232f4bb ] The matching values for `pcie_gen_cap` and `pcie_width_cap` when fetched from powerplay tables are 1 byte, so narrow the arguments to match to ensure min() and max() comparisons without casts. Signed-off-by: Mario Limonciello <mario.limonciello@amd.com> Acked-by: Christian König <christian.koenig@amd.com> Signed-off-by: Alex Deucher <alexander.deucher@amd.com> Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
parent
b39f1303b9
commit
82c4cf2c05
|
@ -1232,7 +1232,7 @@ static int smu_smc_hw_setup(struct smu_context *smu)
|
|||
{
|
||||
struct smu_feature *feature = &smu->smu_feature;
|
||||
struct amdgpu_device *adev = smu->adev;
|
||||
uint32_t pcie_gen = 0, pcie_width = 0;
|
||||
uint8_t pcie_gen = 0, pcie_width = 0;
|
||||
uint64_t features_supported;
|
||||
int ret = 0;
|
||||
|
||||
|
|
|
@ -844,7 +844,7 @@ struct pptable_funcs {
|
|||
* &pcie_gen_cap: Maximum allowed PCIe generation.
|
||||
* &pcie_width_cap: Maximum allowed PCIe width.
|
||||
*/
|
||||
int (*update_pcie_parameters)(struct smu_context *smu, uint32_t pcie_gen_cap, uint32_t pcie_width_cap);
|
||||
int (*update_pcie_parameters)(struct smu_context *smu, uint8_t pcie_gen_cap, uint8_t pcie_width_cap);
|
||||
|
||||
/**
|
||||
* @i2c_init: Initialize i2c.
|
||||
|
|
|
@ -296,8 +296,8 @@ int smu_v13_0_get_pptable_from_firmware(struct smu_context *smu,
|
|||
uint32_t pptable_id);
|
||||
|
||||
int smu_v13_0_update_pcie_parameters(struct smu_context *smu,
|
||||
uint32_t pcie_gen_cap,
|
||||
uint32_t pcie_width_cap);
|
||||
uint8_t pcie_gen_cap,
|
||||
uint8_t pcie_width_cap);
|
||||
|
||||
#endif
|
||||
#endif
|
||||
|
|
|
@ -2376,8 +2376,8 @@ static int navi10_get_power_limit(struct smu_context *smu,
|
|||
}
|
||||
|
||||
static int navi10_update_pcie_parameters(struct smu_context *smu,
|
||||
uint32_t pcie_gen_cap,
|
||||
uint32_t pcie_width_cap)
|
||||
uint8_t pcie_gen_cap,
|
||||
uint8_t pcie_width_cap)
|
||||
{
|
||||
struct smu_11_0_dpm_context *dpm_context = smu->smu_dpm.dpm_context;
|
||||
PPTable_t *pptable = smu->smu_table.driver_pptable;
|
||||
|
|
|
@ -2085,14 +2085,14 @@ static int sienna_cichlid_display_disable_memory_clock_switch(struct smu_context
|
|||
#define MAX(a, b) ((a) > (b) ? (a) : (b))
|
||||
|
||||
static int sienna_cichlid_update_pcie_parameters(struct smu_context *smu,
|
||||
uint32_t pcie_gen_cap,
|
||||
uint32_t pcie_width_cap)
|
||||
uint8_t pcie_gen_cap,
|
||||
uint8_t pcie_width_cap)
|
||||
{
|
||||
struct smu_11_0_dpm_context *dpm_context = smu->smu_dpm.dpm_context;
|
||||
struct smu_11_0_pcie_table *pcie_table = &dpm_context->dpm_tables.pcie_table;
|
||||
uint8_t *table_member1, *table_member2;
|
||||
uint32_t min_gen_speed, max_gen_speed;
|
||||
uint32_t min_lane_width, max_lane_width;
|
||||
uint8_t min_gen_speed, max_gen_speed;
|
||||
uint8_t min_lane_width, max_lane_width;
|
||||
uint32_t smu_pcie_arg;
|
||||
int ret, i;
|
||||
|
||||
|
|
|
@ -2420,8 +2420,8 @@ int smu_v13_0_mode1_reset(struct smu_context *smu)
|
|||
}
|
||||
|
||||
int smu_v13_0_update_pcie_parameters(struct smu_context *smu,
|
||||
uint32_t pcie_gen_cap,
|
||||
uint32_t pcie_width_cap)
|
||||
uint8_t pcie_gen_cap,
|
||||
uint8_t pcie_width_cap)
|
||||
{
|
||||
struct smu_13_0_dpm_context *dpm_context = smu->smu_dpm.dpm_context;
|
||||
struct smu_13_0_pcie_table *pcie_table =
|
||||
|
|
Loading…
Reference in New Issue