media: atomisp: Fix error code in ov5693_probe()
authorDan Carpenter <dan.carpenter@oracle.com>
Tue, 14 Jul 2020 10:55:29 +0000 (12:55 +0200)
committerMauro Carvalho Chehab <mchehab+huawei@kernel.org>
Sun, 19 Jul 2020 06:08:32 +0000 (08:08 +0200)
If gmin_camera_platform_data() returns NULL then we should return a
negative error instead of success.

Fixes: 90ebe55ab886 ("media: staging: atomisp: Add driver prefix to Kconfig option and module names")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
drivers/staging/media/atomisp/i2c/ov5693/atomisp-ov5693.c

index 97ab10b..e698b63 100644 (file)
@@ -1899,7 +1899,7 @@ static int ov5693_probe(struct i2c_client *client)
 {
        struct ov5693_device *dev;
        int i2c;
-       int ret = 0;
+       int ret;
        void *pdata;
        unsigned int i;
 
@@ -1929,8 +1929,10 @@ static int ov5693_probe(struct i2c_client *client)
        pdata = gmin_camera_platform_data(&dev->sd,
                                          ATOMISP_INPUT_FORMAT_RAW_10,
                                          atomisp_bayer_order_bggr);
-       if (!pdata)
+       if (!pdata) {
+               ret = -EINVAL;
                goto out_free;
+       }
 
        ret = ov5693_s_config(&dev->sd, client->irq, pdata);
        if (ret)