summary refs log tree commit diff
path: root/drivers
diff options
context:
space:
mode:
authorKartik Mohta <kartikmohta@gmail.com>2012-05-02 19:19:08 -0300
committerMauro Carvalho Chehab <mchehab@redhat.com>2012-05-14 13:39:02 -0300
commit5c37598142621ca8aadfd115d8e5f51c5337f8d5 (patch)
tree0f67d5ca8189ae4ae12da0af42c9ca2166f6379b /drivers
parentb22b9f3200fcde1817e5ccad9b8aaf794fb46119 (diff)
downloadlinux-5c37598142621ca8aadfd115d8e5f51c5337f8d5.tar.gz
[media] mt9v032: Correct the logic for the auto-exposure setting
The driver uses the ctrl value passed in as a bool to determine whether
to enable auto-exposure, but the auto-exposure setting is defined as an
enum where AUTO has a value of 0 and MANUAL has a value of 1. This leads
to a reversed logic where if you send in AUTO, it actually sets manual
exposure and vice-versa.

Signed-off-by: Kartik Mohta <kartikmohta@gmail.com>
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
Diffstat (limited to 'drivers')
-rw-r--r--drivers/media/video/mt9v032.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/media/video/mt9v032.c b/drivers/media/video/mt9v032.c
index 75e253a343c5..4ba4884c016e 100644
--- a/drivers/media/video/mt9v032.c
+++ b/drivers/media/video/mt9v032.c
@@ -481,7 +481,7 @@ static int mt9v032_s_ctrl(struct v4l2_ctrl *ctrl)
 
 	case V4L2_CID_EXPOSURE_AUTO:
 		return mt9v032_update_aec_agc(mt9v032, MT9V032_AEC_ENABLE,
-					      ctrl->val);
+					      !ctrl->val);
 
 	case V4L2_CID_EXPOSURE:
 		return mt9v032_write(client, MT9V032_TOTAL_SHUTTER_WIDTH,