[PATCH v2 3/4] clk: samsung: clk-pll: Add support for pll_531x

[sunyeal.hong]

Hello Alim,

> -----Original Message-----
> From: Alim Akhtar <alim.akhtar at samsung.com>
> Sent: Wednesday, July 10, 2024 11:35 AM
> To: 'sunyeal.hong' <sunyeal.hong at samsung.com>; 'Krzysztof Kozlowski'
> <krzk at kernel.org>; 'Sylwester Nawrocki' <s.nawrocki at samsung.com>; 'Chanwoo
> Choi' <cw00.choi at samsung.com>; 'Michael Turquette'
> <mturquette at baylibre.com>; 'Stephen Boyd' <sboyd at kernel.org>; 'Rob
> Herring' <robh at kernel.org>; 'Conor Dooley' <conor+dt at kernel.org>
> Cc: linux-samsung-soc at vger.kernel.org; linux-clk at vger.kernel.org;
> devicetree at vger.kernel.org; linux-arm-kernel at lists.infradead.org; linux-
> kernel at vger.kernel.org
> Subject: RE: [PATCH v2 3/4] clk: samsung: clk-pll: Add support for
> pll_531x
> 
> Hello Sunyeal,
> 
> > -----Original Message-----
> > From: sunyeal.hong <sunyeal.hong at samsung.com>
> > Sent: Wednesday, July 10, 2024 7:50 AM
> > To: 'Alim Akhtar' <alim.akhtar at samsung.com>; 'Krzysztof Kozlowski'
> > <krzk at kernel.org>; 'Sylwester Nawrocki' <s.nawrocki at samsung.com>;
> > 'Chanwoo Choi' <cw00.choi at samsung.com>; 'Michael Turquette'
> > <mturquette at baylibre.com>; 'Stephen Boyd' <sboyd at kernel.org>; 'Rob
> > Herring' <robh at kernel.org>; 'Conor Dooley' <conor+dt at kernel.org>
> > Cc: linux-samsung-soc at vger.kernel.org; linux-clk at vger.kernel.org;
> > devicetree at vger.kernel.org; linux-arm-kernel at lists.infradead.org;
> > linux- kernel at vger.kernel.org
> > Subject: RE: [PATCH v2 3/4] clk: samsung: clk-pll: Add support for
> > pll_531x
> >
> > Hello Alim,
> >
> > > -----Original Message-----
> > > From: Alim Akhtar <alim.akhtar at samsung.com>
> > > Sent: Monday, July 8, 2024 8:58 PM
> > > To: 'Sunyeal Hong' <sunyeal.hong at samsung.com>; 'Krzysztof Kozlowski'
> > > <krzk at kernel.org>; 'Sylwester Nawrocki' <s.nawrocki at samsung.com>;
> > > 'Chanwoo Choi' <cw00.choi at samsung.com>; 'Michael Turquette'
> > > <mturquette at baylibre.com>; 'Stephen Boyd' <sboyd at kernel.org>; 'Rob
> > > Herring' <robh at kernel.org>; 'Conor Dooley' <conor+dt at kernel.org>
> > > Cc: linux-samsung-soc at vger.kernel.org; linux-clk at vger.kernel.org;
> > > devicetree at vger.kernel.org; linux-arm-kernel at lists.infradead.org;
> > > linux- kernel at vger.kernel.org
> > > Subject: RE: [PATCH v2 3/4] clk: samsung: clk-pll: Add support for
> > > pll_531x
> > >
> > > Hello Sunyeal,
> > >
> > > > -----Original Message-----
> > > > From: Sunyeal Hong <sunyeal.hong at samsung.com>
> > > > Sent: Monday, July 8, 2024 4:44 AM
> > > > To: Krzysztof Kozlowski <krzk at kernel.org>; Sylwester Nawrocki
> > > > <s.nawrocki at samsung.com>; Chanwoo Choi
> > <cw00.choi at samsung.com>; Alim
> > > > Akhtar <alim.akhtar at samsung.com>; Michael Turquette
> > > > <mturquette at baylibre.com>; Stephen Boyd <sboyd at kernel.org>; Rob
> > > > Herring <robh at kernel.org>; Conor Dooley <conor+dt at kernel.org>
> > > > Cc: linux-samsung-soc at vger.kernel.org; linux-clk at vger.kernel.org;
> > > > devicetree at vger.kernel.org; linux-arm-kernel at lists.infradead.org;
> > > > linux- kernel at vger.kernel.org; Sunyeal Hong
> > > > <sunyeal.hong at samsung.com>
> > > > Subject: [PATCH v2 3/4] clk: samsung: clk-pll: Add support for
> > > > pll_531x
> > > >
> > > > pll531x PLL is used in Exynos Auto v920 SoC for shared pll.
> > > > pll531x: Integer/fractional PLL with mid frequency FVCO (800 to
> > > > 3120
> > > > MHz)
> > > >
> > > > PLL531x
> > > > FOUT = (MDIV + F/2^32-F[31]) * FIN/(PDIV x 2^SDIV)
> > > >
> > > Any reason for not mentioning equation for integer PLL?
> > >
> > If the F value is 0, it operates as an integer PLL.
> Thanks for clarification, it is good to mention the same in the commit
> message.
> 
Okay. I will update comment for integer PLL description.
> [snip]
> > > > --
> > > > 2.45.2
> > >
> >
> 

Thanks,
Sunyeal Hong