public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] clk: qcom: ipq5332: Drop set rate parent from gpll0 dependent clocks
@ 2023-08-30  6:35 Varadarajan Narayanan
  2023-08-30 23:37 ` Stephen Boyd
  0 siblings, 1 reply; 3+ messages in thread
From: Varadarajan Narayanan @ 2023-08-30  6:35 UTC (permalink / raw)
  To: agross, andersson, konrad.dybcio, mturquette, sboyd,
	linux-arm-msm, linux-clk, linux-kernel
  Cc: Varadarajan Narayanan

IPQ5332's GPLL0's nominal/turbo frequency is 800MHz.
This must not be scaled based on the requirement of
dependent clocks. Hence remove the CLK_SET_RATE_PARENT
flag.

Signed-off-by: Varadarajan Narayanan <quic_varada@quicinc.com>
---
 drivers/clk/qcom/gcc-ipq5332.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/drivers/clk/qcom/gcc-ipq5332.c b/drivers/clk/qcom/gcc-ipq5332.c
index b02026f..b836159 100644
--- a/drivers/clk/qcom/gcc-ipq5332.c
+++ b/drivers/clk/qcom/gcc-ipq5332.c
@@ -71,7 +71,6 @@ static struct clk_fixed_factor gpll0_div2 = {
 				&gpll0_main.clkr.hw },
 		.num_parents = 1,
 		.ops = &clk_fixed_factor_ops,
-		.flags = CLK_SET_RATE_PARENT,
 	},
 };
 
@@ -85,7 +84,6 @@ static struct clk_alpha_pll_postdiv gpll0 = {
 				&gpll0_main.clkr.hw },
 		.num_parents = 1,
 		.ops = &clk_alpha_pll_postdiv_ro_ops,
-		.flags = CLK_SET_RATE_PARENT,
 	},
 };
 
-- 
2.7.4


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] clk: qcom: ipq5332: Drop set rate parent from gpll0 dependent clocks
  2023-08-30  6:35 [PATCH] clk: qcom: ipq5332: Drop set rate parent from gpll0 dependent clocks Varadarajan Narayanan
@ 2023-08-30 23:37 ` Stephen Boyd
  2023-08-31  9:30   ` Varadarajan Narayanan
  0 siblings, 1 reply; 3+ messages in thread
From: Stephen Boyd @ 2023-08-30 23:37 UTC (permalink / raw)
  To: Varadarajan Narayanan, agross, andersson, konrad.dybcio,
	linux-arm-msm, linux-clk, linux-kernel, mturquette
  Cc: Varadarajan Narayanan

Quoting Varadarajan Narayanan (2023-08-29 23:35:10)
> IPQ5332's GPLL0's nominal/turbo frequency is 800MHz.
> This must not be scaled based on the requirement of
> dependent clocks. Hence remove the CLK_SET_RATE_PARENT
> flag.
> 
> Signed-off-by: Varadarajan Narayanan <quic_varada@quicinc.com>
> ---

Any Fixes tag?

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] clk: qcom: ipq5332: Drop set rate parent from gpll0 dependent clocks
  2023-08-30 23:37 ` Stephen Boyd
@ 2023-08-31  9:30   ` Varadarajan Narayanan
  0 siblings, 0 replies; 3+ messages in thread
From: Varadarajan Narayanan @ 2023-08-31  9:30 UTC (permalink / raw)
  To: Stephen Boyd
  Cc: agross, andersson, konrad.dybcio, linux-arm-msm, linux-clk,
	linux-kernel, mturquette

On Wed, Aug 30, 2023 at 04:37:40PM -0700, Stephen Boyd wrote:
> Quoting Varadarajan Narayanan (2023-08-29 23:35:10)
> > IPQ5332's GPLL0's nominal/turbo frequency is 800MHz.
> > This must not be scaled based on the requirement of
> > dependent clocks. Hence remove the CLK_SET_RATE_PARENT
> > flag.
> >
> > Signed-off-by: Varadarajan Narayanan <quic_varada@quicinc.com>
> > ---
>
> Any Fixes tag?

Have posted v2 with Fixes tag. Please take a look.

Thanks
Varada

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2023-08-31  9:30 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-08-30  6:35 [PATCH] clk: qcom: ipq5332: Drop set rate parent from gpll0 dependent clocks Varadarajan Narayanan
2023-08-30 23:37 ` Stephen Boyd
2023-08-31  9:30   ` Varadarajan Narayanan

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox