Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1613030

Re: [PATCH v2 2/4] clk: rockchip: rk3228: make noc and some special clk as critical_clocks

From Maxime Ripard <maxime.ripard@free-electrons.com>
Newsgroups linux.kernel
Subject Re: [PATCH v2 2/4] clk: rockchip: rk3228: make noc and some special clk as critical_clocks
Date 2017-03-30 14:50 +0200
Message-ID <tqHGx-6sD-7@gated-at.bofh.it> (permalink)
References <tpWeC-62o-19@gated-at.bofh.it> <tpWoi-65Z-25@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


[Multipart message — attachments visible in raw view] - view raw

On Tue, Mar 28, 2017 at 06:06:52PM +0800, Elaine Zhang wrote:
> Signed-off-by: Elaine Zhang <zhangqing@rock-chips.com>
> ---
>  drivers/clk/rockchip/clk-rk3228.c | 30 +++++++++++++++++++++++++++++-
>  1 file changed, 29 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/clk/rockchip/clk-rk3228.c b/drivers/clk/rockchip/clk-rk3228.c
> index db6e5a9e6de6..4d3203f887e2 100644
> --- a/drivers/clk/rockchip/clk-rk3228.c
> +++ b/drivers/clk/rockchip/clk-rk3228.c
> @@ -445,7 +445,7 @@ enum rk3228_plls {
>  			RK2928_CLKGATE_CON(2), 12, GFLAGS,
>  			&rk3228_spdif_fracmux),
>  
> -	GATE(0, "jtag", "ext_jtag", 0,
> +	GATE(0, "jtag", "ext_jtag", CLK_IGNORE_UNUSED,

CLK_IGNORE_UNUSED only prevents a given clock from being gated at
late_initcall time, but will not prevent it from being gated later in
the life of the system, for example if a reparenting occurs, or if all
the clocks sharing the same clock tree become disabled.

If your clock really should never ever be gated in order for Linux to
operate properly, you should use CLK_IS_CRITICAL.

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH v2 0/4] make some special clk as critical_clocks Elaine Zhang <zhangqing@rock-chips.com> - 2017-03-28 12:10 +0200
  [PATCH v2 1/4] clk: rockchip: rk3036: make pclk_ddrupctl as critical_clock Elaine Zhang <zhangqing@rock-chips.com> - 2017-03-28 12:20 +0200
  [PATCH v2 2/4] clk: rockchip: rk3228: make noc and some special clk as critical_clocks Elaine Zhang <zhangqing@rock-chips.com> - 2017-03-28 12:20 +0200
    Re: [PATCH v2 2/4] clk: rockchip: rk3228: make noc and some special clk as critical_clocks Heiko Stuebner <heiko@sntech.de> - 2017-03-28 22:10 +0200
    Re: [PATCH v2 2/4] clk: rockchip: rk3228: make noc and some special  clk as critical_clocks Maxime Ripard <maxime.ripard@free-electrons.com> - 2017-03-30 14:50 +0200
      Re: [PATCH v2 2/4] clk: rockchip: rk3228: make noc and some special clk as critical_clocks Heiko Stuebner <heiko@sntech.de> - 2017-03-30 15:20 +0200
        Re: [PATCH v2 2/4] clk: rockchip: rk3228: make noc and some special  clk as critical_clocks Maxime Ripard <maxime.ripard@free-electrons.com> - 2017-03-30 15:20 +0200
  [PATCH v2 4/4] clk: rockchip: rk3368: make some special clk as critical_clocks Elaine Zhang <zhangqing@rock-chips.com> - 2017-03-28 12:20 +0200

csiph-web