* [PATCH v5 1/2] clk: rockchip: add pclk_pd_pmu to the list of rk3288 critical clocks
[not found] <1439287924-6988-1-git-send-email-hl@rock-chips.com>
@ 2015-08-11 22:52 ` Doug Anderson
[not found] ` <1439287924-6988-2-git-send-email-hl@rock-chips.com>
` (2 subsequent siblings)
3 siblings, 0 replies; 8+ messages in thread
From: Doug Anderson @ 2015-08-11 22:52 UTC (permalink / raw)
To: linux-arm-kernel
Hi,
On Tue, Aug 11, 2015 at 3:12 AM, Lin Huang <hl@rock-chips.com> wrote:
> From: Heiko Stuebner <heiko@sntech.de>
>
> 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.
>
> Signed-off-by: Heiko Stuebner <heiko@sntech.de>
> Signed-off-by: Lin Huang <hl@rock-chips.com>
> ---
> Changes in v5:
> - Add From: Heiko Stuebner <heiko@sntech.de>
> - Add Signed-off-by: Heiko Stuebner <heiko@sntech.de>
>
> drivers/clk/rockchip/clk-rk3288.c | 1 +
> 1 file changed, 1 insertion(+)
You forgot to carry forward my reviewed by:
Reviewed-by: Douglas Anderson <dianders@chromium.org>
^ permalink raw reply [flat|nested] 8+ messages in thread[parent not found: <1439287924-6988-2-git-send-email-hl@rock-chips.com>]
* [PATCH v5 2/2] pinctrl: rockchip: only enable gpio clock when it setting
[not found] ` <1439287924-6988-2-git-send-email-hl@rock-chips.com>
@ 2015-08-11 22:56 ` Doug Anderson
2015-08-13 12:24 ` Linus Walleij
1 sibling, 0 replies; 8+ messages in thread
From: Doug Anderson @ 2015-08-11 22:56 UTC (permalink / raw)
To: linux-arm-kernel
Hi,
On Tue, Aug 11, 2015 at 3:12 AM, Lin Huang <hl@rock-chips.com> wrote:
> gpio can keep state even the clock disable, for save power
> consumption, only enable gpio clock when it setting
>
> Signed-off-by: Lin Huang <hl@rock-chips.com>
> Reviewed-by: Heiko Stuebner <heiko@sntech.de>
> ---
> Changes in v5:
> -rebase patch
>
> drivers/pinctrl/pinctrl-rockchip.c | 55 +++++++++++++++++++++++++++++++++++---
> 1 file changed, 51 insertions(+), 4 deletions(-)
Reviewed-by: Douglas Anderson <dianders@chromium.org>
Also in the ChromeOS 3.14 tree (where this patch has been backported):
Tested-by: Douglas Anderson <dianders@chromium.org>
^ permalink raw reply [flat|nested] 8+ messages in thread* [PATCH v5 2/2] pinctrl: rockchip: only enable gpio clock when it setting
[not found] ` <1439287924-6988-2-git-send-email-hl@rock-chips.com>
2015-08-11 22:56 ` [PATCH v5 2/2] pinctrl: rockchip: only enable gpio clock when it setting Doug Anderson
@ 2015-08-13 12:24 ` Linus Walleij
2015-08-13 12:37 ` Heiko Stübner
1 sibling, 1 reply; 8+ messages in thread
From: Linus Walleij @ 2015-08-13 12:24 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Aug 11, 2015 at 12:12 PM, Lin Huang <hl@rock-chips.com> wrote:
> gpio can keep state even the clock disable, for save power
> consumption, only enable gpio clock when it setting
>
> Signed-off-by: Lin Huang <hl@rock-chips.com>
> Reviewed-by: Heiko Stuebner <heiko@sntech.de>
> ---
> Changes in v5:
> -rebase patch
Patch applied with Doug's test tags etc.
Yours,
Linus Walleij
^ permalink raw reply [flat|nested] 8+ messages in thread* [PATCH v5 2/2] pinctrl: rockchip: only enable gpio clock when it setting
2015-08-13 12:24 ` Linus Walleij
@ 2015-08-13 12:37 ` Heiko Stübner
2015-08-13 14:11 ` Linus Walleij
0 siblings, 1 reply; 8+ messages in thread
From: Heiko Stübner @ 2015-08-13 12:37 UTC (permalink / raw)
To: linux-arm-kernel
Hi Linus,
Am Donnerstag, 13. August 2015, 14:24:28 schrieb Linus Walleij:
> On Tue, Aug 11, 2015 at 12:12 PM, Lin Huang <hl@rock-chips.com> wrote:
> > gpio can keep state even the clock disable, for save power
> > consumption, only enable gpio clock when it setting
> >
> > Signed-off-by: Lin Huang <hl@rock-chips.com>
> > Reviewed-by: Heiko Stuebner <heiko@sntech.de>
> > ---
> > Changes in v5:
> > -rebase patch
>
> Patch applied with Doug's test tags etc.
did you also take the first patch?
Aka bot should probably stay together (hence Stephen's Ack for the clock
change), as the dynamically controlling the gpio clocks exposes the
pclk_pd_pmu.
Heiko
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH v5 2/2] pinctrl: rockchip: only enable gpio clock when it setting
2015-08-13 12:37 ` Heiko Stübner
@ 2015-08-13 14:11 ` Linus Walleij
2015-08-13 14:50 ` Heiko Stübner
0 siblings, 1 reply; 8+ messages in thread
From: Linus Walleij @ 2015-08-13 14:11 UTC (permalink / raw)
To: linux-arm-kernel
On Thu, Aug 13, 2015 at 2:37 PM, Heiko St?bner <heiko@sntech.de> wrote:
>> Patch applied with Doug's test tags etc.
>
> did you also take the first patch?
>
> Aka bot should probably stay together (hence Stephen's Ack for the clock
> change), as the dynamically controlling the gpio clocks exposes the
> pclk_pd_pmu.
OK wasn't obvious to me, patch 1/2 applied to the pinctrl tree now.
Yours,
Linus Walleij
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH v5 2/2] pinctrl: rockchip: only enable gpio clock when it setting
2015-08-13 14:11 ` Linus Walleij
@ 2015-08-13 14:50 ` Heiko Stübner
0 siblings, 0 replies; 8+ messages in thread
From: Heiko Stübner @ 2015-08-13 14:50 UTC (permalink / raw)
To: linux-arm-kernel
Am Donnerstag, 13. August 2015, 16:11:38 schrieb Linus Walleij:
> On Thu, Aug 13, 2015 at 2:37 PM, Heiko St?bner <heiko@sntech.de> wrote:
> >> Patch applied with Doug's test tags etc.
> >
> > did you also take the first patch?
> >
> > Aka bot should probably stay together (hence Stephen's Ack for the clock
> > change), as the dynamically controlling the gpio clocks exposes the
> > pclk_pd_pmu.
>
> OK wasn't obvious to me, patch 1/2 applied to the pinctrl tree now.
Thanks for picking patch 1/2 too.
Both me and Lin seem to have forgotten to mention this in this new revision
(sorry about that), as it was discussed in a previous revision only.
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH v5 1/2] clk: rockchip: add pclk_pd_pmu to the list of rk3288 critical clocks
[not found] <1439287924-6988-1-git-send-email-hl@rock-chips.com>
2015-08-11 22:52 ` [PATCH v5 1/2] clk: rockchip: add pclk_pd_pmu to the list of rk3288 critical clocks Doug Anderson
[not found] ` <1439287924-6988-2-git-send-email-hl@rock-chips.com>
@ 2015-08-11 23:01 ` Stephen Boyd
2015-08-13 14:10 ` Linus Walleij
3 siblings, 0 replies; 8+ messages in thread
From: Stephen Boyd @ 2015-08-11 23:01 UTC (permalink / raw)
To: linux-arm-kernel
On 08/11/2015 03:12 AM, Lin Huang wrote:
> From: Heiko Stuebner <heiko@sntech.de>
>
> 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.
>
> Signed-off-by: Heiko Stuebner <heiko@sntech.de>
> Signed-off-by: Lin Huang <hl@rock-chips.com>
> ---
>
Acked-by: Stephen Boyd <sboyd@codeaurora.org>
--
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project
^ permalink raw reply [flat|nested] 8+ messages in thread* [PATCH v5 1/2] clk: rockchip: add pclk_pd_pmu to the list of rk3288 critical clocks
[not found] <1439287924-6988-1-git-send-email-hl@rock-chips.com>
` (2 preceding siblings ...)
2015-08-11 23:01 ` [PATCH v5 1/2] clk: rockchip: add pclk_pd_pmu to the list of rk3288 critical clocks Stephen Boyd
@ 2015-08-13 14:10 ` Linus Walleij
3 siblings, 0 replies; 8+ messages in thread
From: Linus Walleij @ 2015-08-13 14:10 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Aug 11, 2015 at 12:12 PM, Lin Huang <hl@rock-chips.com> wrote:
> From: Heiko Stuebner <heiko@sntech.de>
>
> 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.
>
> Signed-off-by: Heiko Stuebner <heiko@sntech.de>
> Signed-off-by: Lin Huang <hl@rock-chips.com>
> ---
> Changes in v5:
> - Add From: Heiko Stuebner <heiko@sntech.de>
> - Add Signed-off-by: Heiko Stuebner <heiko@sntech.de>
Patch applied to the pinctrl tree with Doug's reviwe tag and
Stephen B's ACK.
Yours,
Linus Walleij
^ permalink raw reply [flat|nested] 8+ messages in thread