crypto: octeontx2 - uninitialized variable in kvf_limits_store()

If kstrtoint() fails then "lfs_num" is uninitialized and the warning
doesn't make any sense.  Just delete it.

Fixes: 8ec8015a3168 ("crypto: octeontx2 - add support to process the crypto request")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>

diff --git a/drivers/crypto/marvell/octeontx2/otx2_cptpf_main.c b/drivers/crypto/marvell/octeontx2/otx2_cptpf_main.c
index 146a55ac4b9b0ed7f4c29523769127eed2d508e8..be1ad55a208f622d537184813c35a41319c1528b 100644 (file)
--- a/drivers/crypto/marvell/octeontx2/otx2_cptpf_main.c
+++ b/drivers/crypto/marvell/octeontx2/otx2_cptpf_main.c
@@ -494,12 +494,11 @@ static ssize_t kvf_limits_store(struct device *dev,
 {
        struct otx2_cptpf_dev *cptpf = dev_get_drvdata(dev);
        int lfs_num;
+       int ret;
 
-       if (kstrtoint(buf, 0, &lfs_num)) {
-               dev_err(dev, "lfs count %d must be in range [1 - %d]\n",
-                       lfs_num, num_online_cpus());
-               return -EINVAL;
-       }
+       ret = kstrtoint(buf, 0, &lfs_num);
+       if (ret)
+               return ret;
        if (lfs_num < 1 || lfs_num > num_online_cpus()) {
                dev_err(dev, "lfs count %d must be in range [1 - %d]\n",
                        lfs_num, num_online_cpus());