diff options
| author | Kartik Mohta <kartikmohta@gmail.com> | 2012-05-03 00:19:08 +0200 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2012-05-14 18:39:02 +0200 |
| commit | 5c37598142621ca8aadfd115d8e5f51c5337f8d5 (patch) | |
| tree | 0f67d5ca8189ae4ae12da0af42c9ca2166f6379b | |
| parent | [media] mt9m032: use the available subdev pointer, don't re-calculate it (diff) | |
| download | linux-5c37598142621ca8aadfd115d8e5f51c5337f8d5.tar.xz linux-5c37598142621ca8aadfd115d8e5f51c5337f8d5.zip | |
[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>
| -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) 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, |