diff options
author | Dan Carpenter <error27@gmail.com> | 2010-04-07 11:22:58 +0200 |
---|---|---|
committer | Ben Dooks <ben-linux@fluff.org> | 2010-04-20 02:17:13 +0200 |
commit | 42df64b1f83fa9b786067a0b5a3aca0fbb2d8db8 (patch) | |
tree | a0692e8bb1951a87849f03a9b04a853a8732ff17 /drivers/i2c/busses/i2c-stu300.c | |
parent | i2c-pnx: Add stop conditions for end of transfer (diff) | |
download | linux-42df64b1f83fa9b786067a0b5a3aca0fbb2d8db8.tar.xz linux-42df64b1f83fa9b786067a0b5a3aca0fbb2d8db8.zip |
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: Dan Carpenter <error27@gmail.com>
Signed-off-by: Ben Dooks <ben-linux@fluff.org>
Diffstat (limited to 'drivers/i2c/busses/i2c-stu300.c')
-rw-r--r-- | drivers/i2c/busses/i2c-stu300.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/i2c/busses/i2c-stu300.c b/drivers/i2c/busses/i2c-stu300.c index 1f5b38be73bc..495be451d326 100644 --- a/drivers/i2c/busses/i2c-stu300.c +++ b/drivers/i2c/busses/i2c-stu300.c @@ -498,7 +498,7 @@ static int stu300_set_clk(struct stu300_dev *dev, unsigned long clkrate) int i = 0; /* Locate the apropriate clock setting */ - while (i < ARRAY_SIZE(stu300_clktable) && + while (i < ARRAY_SIZE(stu300_clktable) - 1 && stu300_clktable[i].rate < clkrate) i++; |