]> git.proxmox.com Git - mirror_ubuntu-kernels.git/commitdiff
media: vicodec: Fix error codes in probe function
authorDan Carpenter <dan.carpenter@oracle.com>
Tue, 7 Apr 2020 09:32:37 +0000 (11:32 +0200)
committerMauro Carvalho Chehab <mchehab+huawei@kernel.org>
Tue, 21 Apr 2020 11:31:26 +0000 (13:31 +0200)
If these functions fail then we return success, but we should instead
preserve negative error code and return that.

Fixes: fde649b418d1 ("media: vicodec: Register another node for stateless decoder")
Fixes: c022a4a95722 ("media: vicodec: add struct for encoder/decoder instance")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
drivers/media/test-drivers/vicodec/vicodec-core.c

index 30ced1c2138751dbfdf67295f942fb8d94efc490..e879290727ef449ca03e10940cd16b3feec42914 100644 (file)
@@ -2114,16 +2114,19 @@ static int vicodec_probe(struct platform_device *pdev)
 
        platform_set_drvdata(pdev, dev);
 
-       if (register_instance(dev, &dev->stateful_enc,
-                             "stateful-encoder", true))
+       ret = register_instance(dev, &dev->stateful_enc, "stateful-encoder",
+                               true);
+       if (ret)
                goto unreg_dev;
 
-       if (register_instance(dev, &dev->stateful_dec,
-                             "stateful-decoder", false))
+       ret = register_instance(dev, &dev->stateful_dec, "stateful-decoder",
+                               false);
+       if (ret)
                goto unreg_sf_enc;
 
-       if (register_instance(dev, &dev->stateless_dec,
-                             "stateless-decoder", false))
+       ret = register_instance(dev, &dev->stateless_dec, "stateless-decoder",
+                               false);
+       if (ret)
                goto unreg_sf_dec;
 
 #ifdef CONFIG_MEDIA_CONTROLLER