Commit 85c19925ea88c5999589e7bed03f89640a014f57

Authored by Dan Carpenter
Committed by Greg Kroah-Hartman
1 parent 7e7db3b653

drm/radeon/dpm/btc: off by one in btc_set_mc_special_registers()

commit 96d8df846f52a720c8ae1fadadfad7c9e733e336 upstream.

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>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

Showing 1 changed file with 3 additions and 3 deletions Side-by-side Diff

drivers/gpu/drm/radeon/btc_dpm.c
... ... @@ -1913,7 +1913,7 @@
1913 1913 }
1914 1914 j++;
1915 1915  
1916   - if (j > SMC_EVERGREEN_MC_REGISTER_ARRAY_SIZE)
  1916 + if (j >= SMC_EVERGREEN_MC_REGISTER_ARRAY_SIZE)
1917 1917 return -EINVAL;
1918 1918  
1919 1919 tmp = RREG32(MC_PMG_CMD_MRS);
... ... @@ -1928,7 +1928,7 @@
1928 1928 }
1929 1929 j++;
1930 1930  
1931   - if (j > SMC_EVERGREEN_MC_REGISTER_ARRAY_SIZE)
  1931 + if (j >= SMC_EVERGREEN_MC_REGISTER_ARRAY_SIZE)
1932 1932 return -EINVAL;
1933 1933 break;
1934 1934 case MC_SEQ_RESERVE_M >> 2:
... ... @@ -1942,7 +1942,7 @@
1942 1942 }
1943 1943 j++;
1944 1944  
1945   - if (j > SMC_EVERGREEN_MC_REGISTER_ARRAY_SIZE)
  1945 + if (j >= SMC_EVERGREEN_MC_REGISTER_ARRAY_SIZE)
1946 1946 return -EINVAL;
1947 1947 break;
1948 1948 default: