Commit 42df64b1 authored by Dan Carpenter's avatar Dan Carpenter Committed by Ben Dooks

i2c-stu300: off by one issue

If we don't find the correct rate, we want to end the loop with "i"
pointing to the last element in the array.
Signed-off-by: default avatarDan Carpenter <error27@gmail.com>
Signed-off-by: default avatarBen Dooks <ben-linux@fluff.org>
parent 28ad3321
...@@ -498,7 +498,7 @@ static int stu300_set_clk(struct stu300_dev *dev, unsigned long clkrate) ...@@ -498,7 +498,7 @@ static int stu300_set_clk(struct stu300_dev *dev, unsigned long clkrate)
int i = 0; int i = 0;
/* Locate the apropriate clock setting */ /* Locate the apropriate clock setting */
while (i < ARRAY_SIZE(stu300_clktable) && while (i < ARRAY_SIZE(stu300_clktable) - 1 &&
stu300_clktable[i].rate < clkrate) stu300_clktable[i].rate < clkrate)
i++; i++;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment