diff options
author | Kartik Mohta <kartikmohta@gmail.com> | 2012-05-02 18:19:08 -0400 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2012-05-14 12:39:02 -0400 |
commit | 5c37598142621ca8aadfd115d8e5f51c5337f8d5 (patch) | |
tree | 0f67d5ca8189ae4ae12da0af42c9ca2166f6379b /drivers/media/video/mt9v032.c | |
parent | b22b9f3200fcde1817e5ccad9b8aaf794fb46119 (diff) |
[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/media/video/mt9v032.c')
-rw-r--r-- | drivers/media/video/mt9v032.c | 2 |
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) | |||
481 | 481 | ||
482 | case V4L2_CID_EXPOSURE_AUTO: | 482 | case V4L2_CID_EXPOSURE_AUTO: |
483 | return mt9v032_update_aec_agc(mt9v032, MT9V032_AEC_ENABLE, | 483 | return mt9v032_update_aec_agc(mt9v032, MT9V032_AEC_ENABLE, |
484 | ctrl->val); | 484 | !ctrl->val); |
485 | 485 | ||
486 | case V4L2_CID_EXPOSURE: | 486 | case V4L2_CID_EXPOSURE: |
487 | return mt9v032_write(client, MT9V032_TOTAL_SHUTTER_WIDTH, | 487 | return mt9v032_write(client, MT9V032_TOTAL_SHUTTER_WIDTH, |