diff options
author | Laurent Pinchart <laurent.pinchart@ideasonboard.com> | 2014-01-01 12:40:35 -0500 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@osg.samsung.com> | 2015-04-02 20:30:26 -0400 |
commit | 2d01209fd2d96cb9cd07b5c353efc47b525d71dc (patch) | |
tree | 806bb7e1b3685e638c372678ef6539c66e7e0b20 /drivers/media/i2c/mt9v032.c | |
parent | 7c3be9f812be6c2466ecb705200fc13cc34caa9c (diff) |
[media] v4l: mt9v032: Consider control initialization errors as fatal
The device requires control to be properly operated, they're not
optional.
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
Diffstat (limited to 'drivers/media/i2c/mt9v032.c')
-rw-r--r-- | drivers/media/i2c/mt9v032.c | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/drivers/media/i2c/mt9v032.c b/drivers/media/i2c/mt9v032.c index 3793d317772b..6d64e23b1f94 100644 --- a/drivers/media/i2c/mt9v032.c +++ b/drivers/media/i2c/mt9v032.c | |||
@@ -961,9 +961,12 @@ static int mt9v032_probe(struct i2c_client *client, | |||
961 | 961 | ||
962 | mt9v032->subdev.ctrl_handler = &mt9v032->ctrls; | 962 | mt9v032->subdev.ctrl_handler = &mt9v032->ctrls; |
963 | 963 | ||
964 | if (mt9v032->ctrls.error) | 964 | if (mt9v032->ctrls.error) { |
965 | printk(KERN_INFO "%s: control initialization error %d\n", | 965 | dev_err(&client->dev, "control initialization error %d\n", |
966 | __func__, mt9v032->ctrls.error); | 966 | mt9v032->ctrls.error); |
967 | ret = mt9v032->ctrls.error; | ||
968 | goto err; | ||
969 | } | ||
967 | 970 | ||
968 | mt9v032->crop.left = MT9V032_COLUMN_START_DEF; | 971 | mt9v032->crop.left = MT9V032_COLUMN_START_DEF; |
969 | mt9v032->crop.top = MT9V032_ROW_START_DEF; | 972 | mt9v032->crop.top = MT9V032_ROW_START_DEF; |