diff options
author | YueHaibing <yuehaibing@huawei.com> | 2021-10-08 10:46:52 +0300 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2021-11-18 21:17:00 +0300 |
commit | 97eaf7af7fd1a4f6e882e812846d59c95085c296 (patch) | |
tree | 8931abe1667e211e759ddeff72156a59614e2aa3 /drivers/opp | |
parent | 5a67f827379ffef6760aecb8a3f8866d89e7502e (diff) | |
download | linux-97eaf7af7fd1a4f6e882e812846d59c95085c296.tar.xz |
opp: Fix return in _opp_add_static_v2()
[ Upstream commit 27ff8187f13ecfec8a26fb1928e906f46f326cc5 ]
Fix sparse warning:
drivers/opp/of.c:924 _opp_add_static_v2() warn: passing zero to 'ERR_PTR'
For duplicate OPPs 'ret' be set to zero.
Fixes: deac8703da5f ("PM / OPP: _of_add_opp_table_v2(): increment count only if OPP is added")
Signed-off-by: YueHaibing <yuehaibing@huawei.com>
Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
Diffstat (limited to 'drivers/opp')
-rw-r--r-- | drivers/opp/of.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/opp/of.c b/drivers/opp/of.c index 2a97c6535c4c..c32ae7497392 100644 --- a/drivers/opp/of.c +++ b/drivers/opp/of.c @@ -921,7 +921,7 @@ free_required_opps: free_opp: _opp_free(new_opp); - return ERR_PTR(ret); + return ret ? ERR_PTR(ret) : NULL; } /* Initializes OPP tables based on new bindings */ |