Re: [PATCH] PM / Domains: pdd->dev can't be NULL in genpd_dev_pm_qos_notifier()

On 16 May 2017 at 07:22, Viresh Kumar viresh.kumar@linaro.org wrote:

The pm_domain_data (pdd) pointer is set from genpd_alloc_dev_data() and pdd->dev is guaranteed to be valid. There is no need to check pdd and pdd->dev in rest of the code as pdd->dev will always be valid for a non NULL pdd pointer.

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

Acked-by: Ulf Hansson ulf.hansson@linaro.org

Kind regards Uffe

---

drivers/base/power/domain.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/base/power/domain.c b/drivers/base/power/domain.c index da49a8383dc3..71c95ad808d5 100644 --- a/drivers/base/power/domain.c +++ b/drivers/base/power/domain.c @@ -443,7 +443,7 @@ static int genpd_dev_pm_qos_notifier(struct notifier_block *nb,

            pdd = dev->power.subsys_data ?
                            dev->power.subsys_data->domain_data : NULL;

•           if (pdd && pdd->dev) {
  

•           if (pdd) {
                    to_gpd_data(pdd)->td.constraint_changed = true;
                    genpd = dev_to_genpd(dev);
            } else {
  

-- 2.13.0.303.g4ebf3021692d