diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2014-04-03 11:22:54 +0400 |
---|---|---|
committer | Wolfram Sang <wsa@the-dreams.de> | 2014-04-03 12:36:29 +0400 |
commit | 199c1df28d41ec057d92aaa6bd8bf9ebd5686c57 (patch) | |
tree | f88a27b7a9bad2cd080d1f7a42af69056438da23 /drivers/i2c/busses | |
parent | 3428f018e6101fe2b999a39850755e8c70a72c75 (diff) | |
download | linux-199c1df28d41ec057d92aaa6bd8bf9ebd5686c57.tar.xz |
i2c: qup: off by ones in qup_i2c_probe()
These should ">= ARRAY_SIZE()" instead of "> ARRAY_SIZE()".
Fixes: 10c5a8425968 ('i2c: qup: New bus driver for the Qualcomm QUP I2C controller')
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Wolfram Sang <wsa@the-dreams.de>
Diffstat (limited to 'drivers/i2c/busses')
-rw-r--r-- | drivers/i2c/busses/i2c-qup.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/i2c/busses/i2c-qup.c b/drivers/i2c/busses/i2c-qup.c index c9d5f788e36b..ee409803fd94 100644 --- a/drivers/i2c/busses/i2c-qup.c +++ b/drivers/i2c/busses/i2c-qup.c @@ -633,12 +633,12 @@ static int qup_i2c_probe(struct platform_device *pdev) * associated with each byte written/received */ size = QUP_OUTPUT_BLOCK_SIZE(io_mode); - if (size > ARRAY_SIZE(blk_sizes)) + if (size >= ARRAY_SIZE(blk_sizes)) return -EIO; qup->out_blk_sz = blk_sizes[size] / 2; size = QUP_INPUT_BLOCK_SIZE(io_mode); - if (size > ARRAY_SIZE(blk_sizes)) + if (size >= ARRAY_SIZE(blk_sizes)) return -EIO; qup->in_blk_sz = blk_sizes[size] / 2; |