lkml.org 
[lkml]   [2016]   [Oct]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH] ARM: sti: stih407-clocks: Identify critical clocks
From
Date
On 10/21/2016 11:08 AM, Peter Griffin wrote:
> Lots of platforms contain clocks which if turned off would prove fatal.
> The only way to recover is to restart the board(s). This driver takes
> references to clocks which are required to be always-on. The Common
> Clk Framework will then take references to them. This way they will
> not be turned off during the clk_disabled_unused() procedure.
>
> In this patch we are identifying clocks, which if gated would render
> the STiH407 development board unserviceable.
>
> Signed-off-by: Peter Griffin <peter.griffin@linaro.org>
> ---
> arch/arm/boot/dts/stih407-clock.dtsi | 10 ++++++++++
> 1 file changed, 10 insertions(+)
>
> diff --git a/arch/arm/boot/dts/stih407-clock.dtsi b/arch/arm/boot/dts/stih407-clock.dtsi
> index 13029c0..34c119a 100644
> --- a/arch/arm/boot/dts/stih407-clock.dtsi
> +++ b/arch/arm/boot/dts/stih407-clock.dtsi
> @@ -101,6 +101,7 @@
> clocks = <&clk_sysin>;
>
> clock-output-names = "clk-s-a0-pll-ofd-0";
> + clock-critical = <0>; /* clk-s-a0-pll-ofd-0 */
> };
>
> clk_s_a0_flexgen: clk-s-a0-flexgen {
> @@ -112,6 +113,7 @@
> <&clk_sysin>;
>
> clock-output-names = "clk-ic-lmi0";
> + clock-critical = <CLK_IC_LMI0>;
> };
> };
>
> @@ -126,6 +128,7 @@
> "clk-s-c0-fs0-ch1",
> "clk-s-c0-fs0-ch2",
> "clk-s-c0-fs0-ch3";
> + clock-critical = <0>; /* clk-s-c0-fs0-ch0 */
> };
>
> clk_s_c0: clockgen-c@09103000 {
> @@ -139,6 +142,7 @@
> clocks = <&clk_sysin>;
>
> clock-output-names = "clk-s-c0-pll0-odf-0";
> + clock-critical = <0>; /* clk-s-c0-pll0-odf-0 */
> };
>
> clk_s_c0_pll1: clk-s-c0-pll1 {
> @@ -194,6 +198,12 @@
> "clk-main-disp",
> "clk-aux-disp",
> "clk-compo-dvp";
> + clock-critical = <CLK_PROC_STFE>,
> + <CLK_ICN_CPU>,
> + <CLK_TX_ICN_DMU>,
> + <CLK_EXT2F_A9>,
> + <CLK_ICN_LMI>,
> + <CLK_ICN_SBC>;
> };
> };
>
>

Acked-by: Patrice Chotard <patrice.chotard@st.com>

And applied !

Effectively, the job was done for STiH410 but i forgot to apply the same for STiH407

Thanks Peter ;-)

\
 
 \ /
  Last update: 2016-10-21 14:18    [W:0.390 / U:0.032 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site