Re: [PATCH] PM / Domains: Validate cases of a non-bound driver in genpd governor

[Kevin Hilman <khilman@kernel.org>]

Ulf Hansson <ulf.hansson@linaro.org> writes:

> Recently genpd removed the requirement of a having a driver bound for its
> attached devices to allow genpd to power off. That change should also have
> removed a corresponding validation in the governor, let's correct that.
>
> Fixes: 298cd0f08801 ("PM / Domains: Remove dev->driver check for runtime")
> Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>

Acked-by: Kevin Hilman <khilman@linaro.org>

> ---
>  drivers/base/power/domain_governor.c | 3 ---
>  1 file changed, 3 deletions(-)
>
> diff --git a/drivers/base/power/domain_governor.c b/drivers/base/power/domain_governor.c
> index e60dd12..1e937ac 100644
> --- a/drivers/base/power/domain_governor.c
> +++ b/drivers/base/power/domain_governor.c
> @@ -160,9 +160,6 @@ static bool default_power_down_ok(struct dev_pm_domain *pd)
>  		struct gpd_timing_data *td;
>  		s64 constraint_ns;
>  
> -		if (!pdd->dev->driver)
> -			continue;
> -
>  		/*
>  		 * Check if the device is allowed to be off long enough for the
>  		 * domain to turn off and on (that's how much time it will