diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2013-09-27 16:18:39 -0400 |
|---|---|---|
| committer | Alex Deucher <alexander.deucher@amd.com> | 2013-10-09 17:13:47 -0400 |
| commit | 96d8df846f52a720c8ae1fadadfad7c9e733e336 (patch) | |
| tree | ee0998a802288fbc72a3aaa9f05172fd560608dd /drivers | |
| parent | 89cd67b326fa95872cc2b4524cd807128db6071d (diff) | |
drm/radeon/dpm/btc: off by one in btc_set_mc_special_registers()
It should be ">=" instead of ">" here. The table->mc_reg_address[]
array has SMC_EVERGREEN_MC_REGISTER_ARRAY_SIZE (16) elements.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Cc: stable@vger.kernel.org
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/gpu/drm/radeon/btc_dpm.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/gpu/drm/radeon/btc_dpm.c b/drivers/gpu/drm/radeon/btc_dpm.c index b162e98a2953..9b6950d9b3c0 100644 --- a/drivers/gpu/drm/radeon/btc_dpm.c +++ b/drivers/gpu/drm/radeon/btc_dpm.c | |||
| @@ -1930,7 +1930,7 @@ static int btc_set_mc_special_registers(struct radeon_device *rdev, | |||
| 1930 | } | 1930 | } |
| 1931 | j++; | 1931 | j++; |
| 1932 | 1932 | ||
| 1933 | if (j > SMC_EVERGREEN_MC_REGISTER_ARRAY_SIZE) | 1933 | if (j >= SMC_EVERGREEN_MC_REGISTER_ARRAY_SIZE) |
| 1934 | return -EINVAL; | 1934 | return -EINVAL; |
| 1935 | 1935 | ||
| 1936 | tmp = RREG32(MC_PMG_CMD_MRS); | 1936 | tmp = RREG32(MC_PMG_CMD_MRS); |
| @@ -1945,7 +1945,7 @@ static int btc_set_mc_special_registers(struct radeon_device *rdev, | |||
| 1945 | } | 1945 | } |
| 1946 | j++; | 1946 | j++; |
| 1947 | 1947 | ||
| 1948 | if (j > SMC_EVERGREEN_MC_REGISTER_ARRAY_SIZE) | 1948 | if (j >= SMC_EVERGREEN_MC_REGISTER_ARRAY_SIZE) |
| 1949 | return -EINVAL; | 1949 | return -EINVAL; |
| 1950 | break; | 1950 | break; |
| 1951 | case MC_SEQ_RESERVE_M >> 2: | 1951 | case MC_SEQ_RESERVE_M >> 2: |
| @@ -1959,7 +1959,7 @@ static int btc_set_mc_special_registers(struct radeon_device *rdev, | |||
| 1959 | } | 1959 | } |
| 1960 | j++; | 1960 | j++; |
| 1961 | 1961 | ||
| 1962 | if (j > SMC_EVERGREEN_MC_REGISTER_ARRAY_SIZE) | 1962 | if (j >= SMC_EVERGREEN_MC_REGISTER_ARRAY_SIZE) |
| 1963 | return -EINVAL; | 1963 | return -EINVAL; |
| 1964 | break; | 1964 | break; |
| 1965 | default: | 1965 | default: |