diff mbox series

[2/2] clk: samsung: Fixes PLL locktime for PLL142XX used on FSD platfom

Message ID 20240930111859.22264-3-v.pavani@samsung.com (mailing list archive)
State New
Headers show
Series [1/2] clk: samsung: Use samsung CCF common function | expand

Commit Message

Varada Pavani Sept. 30, 2024, 11:18 a.m. UTC
Add PLL locktime for PLL142XX controller.

Fixes: 4f346005aaed ("clk: samsung: fsd: Add initial clock support")
Cc: stable@vger.kernel.org
Signed-off-by: Varada Pavani <v.pavani@samsung.com>
---
 drivers/clk/samsung/clk-pll.c | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

Comments

Krzysztof Kozlowski Sept. 30, 2024, 12:35 p.m. UTC | #1
On 30/09/2024 13:18, Varada Pavani wrote:
> Add PLL locktime for PLL142XX controller.
> 

So you send the same? Or something new? Please provide proper changelog
and mark patches as v2/v3 or RESEND.

But anyway this cannot be RESEND as I explicitly asked for fixing.

<form letter>
This is a friendly reminder during the review process.

It seems my or other reviewer's previous comments were not fully
addressed. Maybe the feedback got lost between the quotes, maybe you
just forgot to apply it. Please go back to the previous discussion and
either implement all requested changes or keep discussing them.

Thank you.
</form letter>

Best regards,
Krzysztof
Varada Pavani Oct. 1, 2024, 1:34 p.m. UTC | #2
> -----Original Message-----
> From: Krzysztof Kozlowski [mailto:krzk@kernel.org]
> Sent: 30 September 2024 18:05
> To: Varada Pavani <v.pavani@samsung.com>; s.nawrocki@samsung.com;
> cw00.choi@samsung.com; alim.akhtar@samsung.com;
> mturquette@baylibre.com; sboyd@kernel.org; linux-samsung-
> soc@vger.kernel.org; linux-clk@vger.kernel.org; linux-arm-
> kernel@lists.infradead.org; linux-kernel@vger.kernel.org
> Cc: aswani.reddy@samsung.com; pankaj.dubey@samsung.com;
> gost.dev@samsung.com; stable@vger.kernel.org
> Subject: Re: [PATCH 2/2] clk: samsung: Fixes PLL locktime for PLL142XX used
> on FSD platfom
> 
> On 30/09/2024 13:18, Varada Pavani wrote:
> > Add PLL locktime for PLL142XX controller.
> >
> 
> So you send the same? Or something new? Please provide proper changelog
> and mark patches as v2/v3 or RESEND.
> 
> But anyway this cannot be RESEND as I explicitly asked for fixing.
> 
> <form letter>
> This is a friendly reminder during the review process.
> 
> It seems my or other reviewer's previous comments were not fully
> addressed. Maybe the feedback got lost between the quotes, maybe you
> just forgot to apply it. Please go back to the previous discussion and either
> implement all requested changes or keep discussing them.
> 
> Thank you.
> </form letter>
> 
> Best regards,
> Krzysztof

Sorry, have sent the same patch again. As I need to post one patch alone which got
missed due to internal technical issue. Next time onwards will make sure to mark the
patches as V2/V3 or RESEND accordingly.

Regards,
Varada Pavani
diff mbox series

Patch

diff --git a/drivers/clk/samsung/clk-pll.c b/drivers/clk/samsung/clk-pll.c
index 4be879ab917e..d4c5ae20de4f 100644
--- a/drivers/clk/samsung/clk-pll.c
+++ b/drivers/clk/samsung/clk-pll.c
@@ -206,6 +206,7 @@  static const struct clk_ops samsung_pll3000_clk_ops = {
  */
 /* Maximum lock time can be 270 * PDIV cycles */
 #define PLL35XX_LOCK_FACTOR	(270)
+#define PLL142XX_LOCK_FACTOR	(150)
 
 #define PLL35XX_MDIV_MASK       (0x3FF)
 #define PLL35XX_PDIV_MASK       (0x3F)
@@ -272,7 +273,11 @@  static int samsung_pll35xx_set_rate(struct clk_hw *hw, unsigned long drate,
 	}
 
 	/* Set PLL lock time. */
-	writel_relaxed(rate->pdiv * PLL35XX_LOCK_FACTOR,
+	if (pll->type == pll_142xx)
+		writel_relaxed(rate->pdiv * PLL142XX_LOCK_FACTOR,
+			pll->lock_reg);
+	else
+		writel_relaxed(rate->pdiv * PLL35XX_LOCK_FACTOR,
 			pll->lock_reg);
 
 	/* Change PLL PMS values */