Commit 0ea275df authored by Dan Carpenter's avatar Dan Carpenter Committed by Herbert Xu

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: 8ec8015a ("crypto: octeontx2 - add support to process the crypto request")
Signed-off-by: default avatarDan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: default avatarHerbert Xu <herbert@gondor.apana.org.au>
parent 5876b0cb
...@@ -494,12 +494,11 @@ static ssize_t kvf_limits_store(struct device *dev, ...@@ -494,12 +494,11 @@ static ssize_t kvf_limits_store(struct device *dev,
{ {
struct otx2_cptpf_dev *cptpf = dev_get_drvdata(dev); struct otx2_cptpf_dev *cptpf = dev_get_drvdata(dev);
int lfs_num; int lfs_num;
int ret;
if (kstrtoint(buf, 0, &lfs_num)) { ret = kstrtoint(buf, 0, &lfs_num);
dev_err(dev, "lfs count %d must be in range [1 - %d]\n", if (ret)
lfs_num, num_online_cpus()); return ret;
return -EINVAL;
}
if (lfs_num < 1 || lfs_num > num_online_cpus()) { if (lfs_num < 1 || lfs_num > num_online_cpus()) {
dev_err(dev, "lfs count %d must be in range [1 - %d]\n", dev_err(dev, "lfs count %d must be in range [1 - %d]\n",
lfs_num, num_online_cpus()); lfs_num, num_online_cpus());
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment