Hi Geert,
Thanks for your work.
On 2019-08-16 14:39:19 +0200, Geert Uytterhoeven wrote:
> As of commit 980532a5dda319ee ("soc: renesas: rcar-sysc: Use
> GENPD_FLAG_ALWAYS_ON"), the local variable "gov" is assigned just once,
> so it can be eliminated.
>
> Signed-off-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx>
Reviewed-by: Niklas Söderlund <niklas.soderlund+renesas@xxxxxxxxxxxx>
> ---
> To be queued in renesas-devel for v5.4.
>
> drivers/soc/renesas/rcar-sysc.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/drivers/soc/renesas/rcar-sysc.c b/drivers/soc/renesas/rcar-sysc.c
> index 20fa9f34295c2879..40c1ddaa3b4fcae5 100644
> --- a/drivers/soc/renesas/rcar-sysc.c
> +++ b/drivers/soc/renesas/rcar-sysc.c
> @@ -212,7 +212,6 @@ static int __init rcar_sysc_pd_setup(struct rcar_sysc_pd *pd)
> {
> struct generic_pm_domain *genpd = &pd->genpd;
> const char *name = pd->genpd.name;
> - struct dev_power_governor *gov = &simple_qos_governor;
> int error;
>
> if (pd->flags & PD_CPU) {
> @@ -266,7 +265,7 @@ static int __init rcar_sysc_pd_setup(struct rcar_sysc_pd *pd)
> rcar_sysc_power(&pd->ch, true);
>
> finalize:
> - error = pm_genpd_init(genpd, gov, false);
> + error = pm_genpd_init(genpd, &simple_qos_governor, false);
> if (error)
> pr_err("Failed to init PM domain %s: %d\n", name, error);
>
> --
> 2.17.1
>
--
Regards,
Niklas Söderlund
