From: Gregory CLEMENT <gregory.clement@free-electrons.com>
To: Ulf Hansson <ulf.hansson@linaro.org>,
Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>
Cc: "Rafael J. Wysocki" <rjw@rjwysocki.net>,
Kevin Hilman <khilman@kernel.org>,
linux-pm@vger.kernel.org, Len Brown <len.brown@intel.com>,
Pavel Machek <pavel@ucw.cz>,
Geert Uytterhoeven <geert+renesas@glider.be>,
Lina Iyer <lina.iyer@linaro.org>,
Krzysztof Kozlowski <k.kozlowski.k@gmail.com>,
Russell King <rmk+kernel@arm.linux.org.uk>,
Andrew Lunn <andrew@lunn.ch>
Subject: Re: [PATCH 1/4] soc: dove: Let genpd deal with disabling of unused PM domains
Date: Thu, 15 Oct 2015 09:57:55 +0200 [thread overview]
Message-ID: <87a8rk36nw.fsf@free-electrons.com> (raw)
In-Reply-To: <1444134450-17895-1-git-send-email-ulf.hansson@linaro.org> (Ulf Hansson's message of "Tue, 6 Oct 2015 14:27:30 +0200")
Hi Ulf and Sebastian,
On mar., oct. 06 2015, Ulf Hansson <ulf.hansson@linaro.org> wrote:
> Genpd tries from a late_initcall to disable unused PM domains, which makes
> it unnecessary to also manage this from any earlier phase when for example
> initializing a genpd.
I can't test it and I didn't follow this part.
Sebastian could you have a look on this patch?
Thanks,
Gregory
>
> Cc: Russell King <rmk+kernel@arm.linux.org.uk>
> Cc: Andrew Lunn <andrew@lunn.ch>
> Cc: Gregory CLEMENT <gregory.clement@free-electrons.com>
> Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
> ---
> drivers/soc/dove/pmu.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/drivers/soc/dove/pmu.c b/drivers/soc/dove/pmu.c
> index 052aecf..abd0879 100644
> --- a/drivers/soc/dove/pmu.c
> +++ b/drivers/soc/dove/pmu.c
> @@ -396,7 +396,6 @@ int __init dove_init_pmu(void)
>
> __pmu_domain_register(domain, np);
> }
> - pm_genpd_poweroff_unused();
>
> /* Loss of the interrupt controller is not a fatal error. */
> parent_irq = irq_of_parse_and_map(pmu->of_node, 0);
> --
> 1.9.1
>
--
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
next prev parent reply other threads:[~2015-10-15 7:58 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-10-06 12:27 [PATCH 1/4] soc: dove: Let genpd deal with disabling of unused PM domains Ulf Hansson
2015-10-15 7:57 ` Gregory CLEMENT [this message]
2015-10-15 7:59 ` Russell King - ARM Linux
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87a8rk36nw.fsf@free-electrons.com \
--to=gregory.clement@free-electrons.com \
--cc=andrew@lunn.ch \
--cc=geert+renesas@glider.be \
--cc=k.kozlowski.k@gmail.com \
--cc=khilman@kernel.org \
--cc=len.brown@intel.com \
--cc=lina.iyer@linaro.org \
--cc=linux-pm@vger.kernel.org \
--cc=pavel@ucw.cz \
--cc=rjw@rjwysocki.net \
--cc=rmk+kernel@arm.linux.org.uk \
--cc=sebastian.hesselbarth@gmail.com \
--cc=ulf.hansson@linaro.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.