media:i2c: fix coverity defects

Fixed out of bound access
Fixed wrong shift value to compute gain

Coverity ID 29587
Coverity ID 36185

Bug 200192387

Change-Id: I60999628c38192c41ce7e75099b65643d7476f2c
Signed-off-by: Wenjia Zhou <wenjiaz@nvidia.com>
Reviewed-on: http://git-master/r/1307363
Reviewed-by: Automatic_Commit_Validation_User
Reviewed-by: Shubham Chandra <shubhamc@nvidia.com>
Reviewed-by: Frank Chen <frankc@nvidia.com>
Reviewed-by: svccoveritychecker <svccoveritychecker@nvidia.com>
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bhanu Murthy V <bmurthyv@nvidia.com>

1 files changed, 2 insertions, 2 deletions

diff --git a/drivers/media/i2c/ov23850.c b/drivers/media/i2c/ov23850.c
index 141287d8e..bad408019 100644
--- a/drivers/media/i2c/ov23850.c
+++ b/drivers/media/i2c/ov23850.c
@@ -1,7 +1,7 @@
 /*
  * ov23850.c - ov23850 sensor driver
  *
- * Copyright (c) 2013-2016, NVIDIA CORPORATION.  All rights reserved.
+ * Copyright (c) 2013-2017, NVIDIA CORPORATION.  All rights reserved.
  *
  * This program is free software; you can redistribute it and/or modify it
  * under the terms and conditions of the GNU General Public License,
@@ -252,7 +252,7 @@ static inline void ov23850_get_gain_reg(ov23850_reg *regs,
                                 u16 gain)
 {
         regs->addr = OV23850_GAIN_ADDR_MSB;
-        regs->val = (gain >> 16) & 0x07;
+        regs->val = (gain >> 8) & 0x07;
         (regs + 1)->addr = OV23850_GAIN_ADDR_LSB;
         (regs + 1)->val = (gain) & 0xff;
 }