From 1f6620f87006dc02c608466cd990778aaadf386a Mon Sep 17 00:00:00 2001 From: Viresh Kumar Date: Wed, 14 Oct 2020 09:56:28 +0530 Subject: opp: Don't always remove static OPPs in _of_add_opp_table_v1() The patch missed returning 0 early in case of success and hence the static OPPs got removed by mistake. Fix it. Fixes: 90d46d71cce2 ("opp: Handle multiple calls for same OPP table in _of_add_opp_table_v1()") Reported-by: Aisheng Dong Signed-off-by: Viresh Kumar Tested-by: Dong Aisheng --- drivers/opp/of.c | 2 ++ 1 file changed, 2 insertions(+) (limited to 'drivers/opp') diff --git a/drivers/opp/of.c b/drivers/opp/of.c index 874b58756220..9faeb83e4b32 100644 --- a/drivers/opp/of.c +++ b/drivers/opp/of.c @@ -944,6 +944,8 @@ static int _of_add_opp_table_v1(struct device *dev, struct opp_table *opp_table) nr -= 2; } + return 0; + remove_static_opp: _opp_remove_all_static(opp_table); -- cgit v1.2.3