From: "Heiko Stübner" <heiko@sntech.de> To: hl <hl@rock-chips.com> Cc: dianders@chromium.org, linux-rockchip@lists.infradead.org, mturquette@baylibre.com, sboyd@codeaurora.org, linux-clk@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v4 1/2] clk: rockchip: add pclk_pd_pmu to the list of rk3288 critical clocks Date: Fri, 07 Aug 2015 13:36:38 +0200 [thread overview] Message-ID: <2226681.uyCeUEdNHm@diego> (raw) In-Reply-To: <55C47D59.8080403@rock-chips.com> Am Freitag, 7. August 2015, 17:41:45 schrieb hl: > Hi Heiko, > > > pclk_pd_pmu needs to keep running and with the upcoming gpio clock > > handling this is not always the case anymore. So add it to the list > > of critical clocks for now. > > > > From: Heiko Stuebner <heiko@sntech.de> > > Signed-off-by: Lin Huang <hl@rock-chips.com> > > is it the right way? if it still can not meet the requirement, > please let me know. nearly, but not quite ;-) You can take a look at my minnie patch [0] for inspiration. Also please make sure to include linus.walleij@linaro.org also in this first mail. This clock patch is a prequisite for the pinctrl change so we would like Linus Walleij to merge both into his pinctrl tree. Ideally both times in the "To" and not the "Cc" recipients, as To vs. Cc is often used to distinguish whom you want to apply the patch. Heiko [0] http://lists.infradead.org/pipermail/linux-rockchip/2015-August/003608.html > > > --- > > Changes in v4: > > - Add From: Heiko Stuebner <heiko@sntech.de> > > > > drivers/clk/rockchip/clk-rk3288.c | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/drivers/clk/rockchip/clk-rk3288.c > > b/drivers/clk/rockchip/clk-rk3288.c index 0df5bae..9040878 100644 > > --- a/drivers/clk/rockchip/clk-rk3288.c > > +++ b/drivers/clk/rockchip/clk-rk3288.c > > @@ -780,6 +780,7 @@ static const char *const rk3288_critical_clocks[] > > __initconst = {> > > "aclk_cpu", > > "aclk_peri", > > "hclk_peri", > > > > + "pclk_pd_pmu", > > > > }; > > > > #ifdef CONFIG_PM_SLEEP
WARNING: multiple messages have this Message-ID (diff)
From: heiko@sntech.de (Heiko Stübner) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH v4 1/2] clk: rockchip: add pclk_pd_pmu to the list of rk3288 critical clocks Date: Fri, 07 Aug 2015 13:36:38 +0200 [thread overview] Message-ID: <2226681.uyCeUEdNHm@diego> (raw) In-Reply-To: <55C47D59.8080403@rock-chips.com> Am Freitag, 7. August 2015, 17:41:45 schrieb hl: > Hi Heiko, > > > pclk_pd_pmu needs to keep running and with the upcoming gpio clock > > handling this is not always the case anymore. So add it to the list > > of critical clocks for now. > > > > From: Heiko Stuebner <heiko@sntech.de> > > Signed-off-by: Lin Huang <hl@rock-chips.com> > > is it the right way? if it still can not meet the requirement, > please let me know. nearly, but not quite ;-) You can take a look at my minnie patch [0] for inspiration. Also please make sure to include linus.walleij at linaro.org also in this first mail. This clock patch is a prequisite for the pinctrl change so we would like Linus Walleij to merge both into his pinctrl tree. Ideally both times in the "To" and not the "Cc" recipients, as To vs. Cc is often used to distinguish whom you want to apply the patch. Heiko [0] http://lists.infradead.org/pipermail/linux-rockchip/2015-August/003608.html > > > --- > > Changes in v4: > > - Add From: Heiko Stuebner <heiko@sntech.de> > > > > drivers/clk/rockchip/clk-rk3288.c | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/drivers/clk/rockchip/clk-rk3288.c > > b/drivers/clk/rockchip/clk-rk3288.c index 0df5bae..9040878 100644 > > --- a/drivers/clk/rockchip/clk-rk3288.c > > +++ b/drivers/clk/rockchip/clk-rk3288.c > > @@ -780,6 +780,7 @@ static const char *const rk3288_critical_clocks[] > > __initconst = {> > > "aclk_cpu", > > "aclk_peri", > > "hclk_peri", > > > > + "pclk_pd_pmu", > > > > }; > > > > #ifdef CONFIG_PM_SLEEP
next prev parent reply other threads:[~2015-08-07 11:36 UTC|newest] Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top 2015-08-07 9:31 [PATCH v4 1/2] clk: rockchip: add pclk_pd_pmu to the list of rk3288 critical clocks Lin Huang 2015-08-07 9:31 ` Lin Huang 2015-08-07 9:31 ` [PATCH v4 2/2] pinctrl: rockchip: only enable gpio clock when it setting Lin Huang 2015-08-07 9:31 ` Lin Huang 2015-08-07 9:41 ` [PATCH v4 1/2] clk: rockchip: add pclk_pd_pmu to the list of rk3288 critical clocks hl 2015-08-07 9:41 ` hl 2015-08-07 9:41 ` hl 2015-08-07 9:41 ` hl 2015-08-07 11:36 ` Heiko Stübner [this message] 2015-08-07 11:36 ` Heiko Stübner
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=2226681.uyCeUEdNHm@diego \ --to=heiko@sntech.de \ --cc=dianders@chromium.org \ --cc=hl@rock-chips.com \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-clk@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-rockchip@lists.infradead.org \ --cc=mturquette@baylibre.com \ --cc=sboyd@codeaurora.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: linkBe 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.