From: Viresh Kumar <viresh.kumar@linaro.org>
Date: Thu, 27 Aug 2020 09:36:43 +0000 (+0530)
Subject: opp: Drop unnecessary check from dev_pm_opp_attach_genpd()
X-Git-Tag: v5.15~2698^2~4^2~7
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=475ac8ead803e8c3a8f492e3129b1a19d7e2efd7;p=platform%2Fkernel%2Flinux-starfive.git

opp: Drop unnecessary check from dev_pm_opp_attach_genpd()

Since commit c0ab9e0812da ("opp: Allocate genpd_virt_devs from
dev_pm_opp_attach_genpd()"), the allocation of the virtual devices is
moved to dev_pm_opp_attach_genpd() and this check isn't required anymore
as it will always fail. Drop it.

Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
---

diff --git a/drivers/opp/core.c b/drivers/opp/core.c
index 8c69a76..fa97a87 100644
--- a/drivers/opp/core.c
+++ b/drivers/opp/core.c
@@ -2019,12 +2019,6 @@ struct opp_table *dev_pm_opp_attach_genpd(struct device *dev,
 			goto err;
 		}
 
-		if (opp_table->genpd_virt_devs[index]) {
-			dev_err(dev, "Genpd virtual device already set %s\n",
-				*name);
-			goto err;
-		}
-
 		virt_dev = dev_pm_domain_attach_by_name(dev, *name);
 		if (IS_ERR(virt_dev)) {
 			ret = PTR_ERR(virt_dev);