i2c: qup: off by ones in qup_i2c_probe()
authorDan Carpenter <dan.carpenter@oracle.com>
Thu, 3 Apr 2014 07:22:54 +0000 (10:22 +0300)
committerWolfram Sang <wsa@the-dreams.de>
Thu, 3 Apr 2014 08:36:29 +0000 (10:36 +0200)
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>
drivers/i2c/busses/i2c-qup.c

index c9d5f78..ee40980 100644 (file)
@@ -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;