diff options
| author | Dan Carpenter <[email protected]> | 2019-03-21 09:27:07 +0300 |
|---|---|---|
| committer | Alex Deucher <[email protected]> | 2019-03-21 14:05:26 -0500 |
| commit | 06b61d4a3ccaba6c502f1ff8646ba3579e5755d3 (patch) | |
| tree | aa9e92974da1d15c95e33f2dd1321a7b88ba808c | |
| parent | 4e8738a206d28d681c1c5195e4d085d5ffcb00fc (diff) | |
drm/amd/powerplay: delete some dead code
The "size" variable is unsigned. We never pass invalid sizes to this
function and we already used it as an array offset earlier so it's
too late to check here.
Reviewed-by: Huang Rui <[email protected]>
Signed-off-by: Dan Carpenter <[email protected]>
Signed-off-by: Alex Deucher <[email protected]>
| -rw-r--r-- | drivers/gpu/drm/amd/powerplay/smu_v11_0.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/drivers/gpu/drm/amd/powerplay/smu_v11_0.c b/drivers/gpu/drm/amd/powerplay/smu_v11_0.c index 026bebe6fb45..9d75911ce674 100644 --- a/drivers/gpu/drm/amd/powerplay/smu_v11_0.c +++ b/drivers/gpu/drm/amd/powerplay/smu_v11_0.c @@ -1668,9 +1668,6 @@ static int smu_v11_0_set_power_profile_mode(struct smu_context *smu, long *input } if (smu->power_profile_mode == PP_SMC_POWER_PROFILE_CUSTOM) { - if (size < 0) - return -EINVAL; - ret = smu_v11_0_get_activity_monitor_coeff(smu, (uint8_t *)(&activity_monitor), WORKLOAD_PPLIB_CUSTOM_BIT); |