public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] clk: imx: pll14xx: Add 756 MHz entry for PLL1443x
@ 2026-02-17  9:41 Ian Ray
  2026-02-17  9:54 ` Marco Felsch
  0 siblings, 1 reply; 4+ messages in thread
From: Ian Ray @ 2026-02-17  9:41 UTC (permalink / raw)
  To: Abel Vesa, Peng Fan, Michael Turquette, Stephen Boyd, Frank Li,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam
  Cc: Ian Ray, linux-clk, imx, linux-arm-kernel, linux-kernel

The PLL1443x is used to implement VIDEO_PLL1 on i.MX8MP and can be used
for dual-channel LVDS displays.  Add new rate 756 MHz, which divides by
7 to provide 108 MHz LVDS pixel clock, and by 2 for 378 MHz LDB clock.

Signed-off-by: Ian Ray <ian.ray@gehealthcare.com>
---
 drivers/clk/imx/clk-pll14xx.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/clk/imx/clk-pll14xx.c b/drivers/clk/imx/clk-pll14xx.c
index 36d0e80b55b8..84279cf9f12b 100644
--- a/drivers/clk/imx/clk-pll14xx.c
+++ b/drivers/clk/imx/clk-pll14xx.c
@@ -63,6 +63,7 @@ static const struct imx_pll14xx_rate_table imx_pll1416x_tbl[] = {
 
 static const struct imx_pll14xx_rate_table imx_pll1443x_tbl[] = {
 	PLL_1443X_RATE(1039500000U, 173, 2, 1, 16384),
+	PLL_1443X_RATE(756000000U, 252, 2, 2, 0),
 	PLL_1443X_RATE(650000000U, 325, 3, 2, 0),
 	PLL_1443X_RATE(594000000U, 198, 2, 2, 0),
 	PLL_1443X_RATE(519750000U, 173, 2, 2, 16384),
-- 
2.49.0


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

* Re: [PATCH] clk: imx: pll14xx: Add 756 MHz entry for PLL1443x
  2026-02-17  9:41 [PATCH] clk: imx: pll14xx: Add 756 MHz entry for PLL1443x Ian Ray
@ 2026-02-17  9:54 ` Marco Felsch
  2026-02-17 10:23   ` Ian Ray
  0 siblings, 1 reply; 4+ messages in thread
From: Marco Felsch @ 2026-02-17  9:54 UTC (permalink / raw)
  To: Ian Ray
  Cc: Abel Vesa, Peng Fan, Michael Turquette, Stephen Boyd, Frank Li,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam, linux-clk,
	imx, linux-arm-kernel, linux-kernel

Hi Ian,

On 26-02-17, Ian Ray wrote:
> The PLL1443x is used to implement VIDEO_PLL1 on i.MX8MP and can be used
> for dual-channel LVDS displays.  Add new rate 756 MHz, which divides by
> 7 to provide 108 MHz LVDS pixel clock, and by 2 for 378 MHz LDB clock.

the on-demand rate calculation doe for the PLL1443X PLLs is not
sufficient for you?

Regards,
  Marco


> 
> Signed-off-by: Ian Ray <ian.ray@gehealthcare.com>
> ---
>  drivers/clk/imx/clk-pll14xx.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/clk/imx/clk-pll14xx.c b/drivers/clk/imx/clk-pll14xx.c
> index 36d0e80b55b8..84279cf9f12b 100644
> --- a/drivers/clk/imx/clk-pll14xx.c
> +++ b/drivers/clk/imx/clk-pll14xx.c
> @@ -63,6 +63,7 @@ static const struct imx_pll14xx_rate_table imx_pll1416x_tbl[] = {
>  
>  static const struct imx_pll14xx_rate_table imx_pll1443x_tbl[] = {
>  	PLL_1443X_RATE(1039500000U, 173, 2, 1, 16384),
> +	PLL_1443X_RATE(756000000U, 252, 2, 2, 0),
>  	PLL_1443X_RATE(650000000U, 325, 3, 2, 0),
>  	PLL_1443X_RATE(594000000U, 198, 2, 2, 0),
>  	PLL_1443X_RATE(519750000U, 173, 2, 2, 16384),
> -- 
> 2.49.0
> 
> 
> 

-- 
#gernperDu 
#CallMeByMyFirstName

Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | https://www.pengutronix.de/ |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-9    |

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

* Re: [PATCH] clk: imx: pll14xx: Add 756 MHz entry for PLL1443x
  2026-02-17  9:54 ` Marco Felsch
@ 2026-02-17 10:23   ` Ian Ray
  2026-02-17 11:12     ` Marco Felsch
  0 siblings, 1 reply; 4+ messages in thread
From: Ian Ray @ 2026-02-17 10:23 UTC (permalink / raw)
  To: Marco Felsch
  Cc: Abel Vesa, Peng Fan, Michael Turquette, Stephen Boyd, Frank Li,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam, linux-clk,
	imx, linux-arm-kernel, linux-kernel

On Tue, Feb 17, 2026 at 10:54:49AM +0100, Marco Felsch wrote:
> CAUTION: This email originated from outside of GE HealthCare. Only open links or attachments if you trust the sender. Report suspicious emails using Outlook’s “Report” button.
> 
> Hi Ian,
> 
> On 26-02-17, Ian Ray wrote:
> > The PLL1443x is used to implement VIDEO_PLL1 on i.MX8MP and can be used
> > for dual-channel LVDS displays.  Add new rate 756 MHz, which divides by
> > 7 to provide 108 MHz LVDS pixel clock, and by 2 for 378 MHz LDB clock.
> 
> the on-demand rate calculation doe for the PLL1443X PLLs is not
> sufficient for you?

Apologies! The on-demand rate calculation does indeed work. So please
ignore this patch.

(The patch was needed on kernel 6.12 IIRC, and I automatically re-based
it on 6.18 without re-testing. Once again, apologies for the noise.)

> 
> Regards,
>   Marco
> 
> 
> >
> > Signed-off-by: Ian Ray <ian.ray@gehealthcare.com>
> > ---
> >  drivers/clk/imx/clk-pll14xx.c | 1 +
> >  1 file changed, 1 insertion(+)
> >
> > diff --git a/drivers/clk/imx/clk-pll14xx.c b/drivers/clk/imx/clk-pll14xx.c
> > index 36d0e80b55b8..84279cf9f12b 100644
> > --- a/drivers/clk/imx/clk-pll14xx.c
> > +++ b/drivers/clk/imx/clk-pll14xx.c
> > @@ -63,6 +63,7 @@ static const struct imx_pll14xx_rate_table imx_pll1416x_tbl[] = {
> >
> >  static const struct imx_pll14xx_rate_table imx_pll1443x_tbl[] = {
> >       PLL_1443X_RATE(1039500000U, 173, 2, 1, 16384),
> > +     PLL_1443X_RATE(756000000U, 252, 2, 2, 0),
> >       PLL_1443X_RATE(650000000U, 325, 3, 2, 0),
> >       PLL_1443X_RATE(594000000U, 198, 2, 2, 0),
> >       PLL_1443X_RATE(519750000U, 173, 2, 2, 16384),
> > --
> > 2.49.0
> >
> >
> >
> 
> --
> #gernperDu
> #CallMeByMyFirstName
> 
> Pengutronix e.K.                           |                             |
> Steuerwalder Str. 21                       | https://www.pengutronix.de/ |
> 31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
> Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-9    |

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

* Re: [PATCH] clk: imx: pll14xx: Add 756 MHz entry for PLL1443x
  2026-02-17 10:23   ` Ian Ray
@ 2026-02-17 11:12     ` Marco Felsch
  0 siblings, 0 replies; 4+ messages in thread
From: Marco Felsch @ 2026-02-17 11:12 UTC (permalink / raw)
  To: Ian Ray
  Cc: Abel Vesa, Peng Fan, Michael Turquette, Stephen Boyd, Frank Li,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam, linux-clk,
	imx, linux-arm-kernel, linux-kernel

On 26-02-17, Ian Ray wrote:
> On Tue, Feb 17, 2026 at 10:54:49AM +0100, Marco Felsch wrote:
> > CAUTION: This email originated from outside of GE HealthCare. Only open links or attachments if you trust the sender. Report suspicious emails using Outlook’s “Report” button.
> > 
> > Hi Ian,
> > 
> > On 26-02-17, Ian Ray wrote:
> > > The PLL1443x is used to implement VIDEO_PLL1 on i.MX8MP and can be used
> > > for dual-channel LVDS displays.  Add new rate 756 MHz, which divides by
> > > 7 to provide 108 MHz LVDS pixel clock, and by 2 for 378 MHz LDB clock.
> > 
> > the on-demand rate calculation doe for the PLL1443X PLLs is not
> > sufficient for you?
> 
> Apologies! The on-demand rate calculation does indeed work. So please
> ignore this patch.
> 
> (The patch was needed on kernel 6.12 IIRC, and I automatically re-based
> it on 6.18 without re-testing. Once again, apologies for the noise.)

No worries, sometimes there are needs to very specific PLL setting due
to emc or so but these are rare cases and therefore I asked :)

Regards,
  Marco


> 
> > 
> > Regards,
> >   Marco
> > 
> > 
> > >
> > > Signed-off-by: Ian Ray <ian.ray@gehealthcare.com>
> > > ---
> > >  drivers/clk/imx/clk-pll14xx.c | 1 +
> > >  1 file changed, 1 insertion(+)
> > >
> > > diff --git a/drivers/clk/imx/clk-pll14xx.c b/drivers/clk/imx/clk-pll14xx.c
> > > index 36d0e80b55b8..84279cf9f12b 100644
> > > --- a/drivers/clk/imx/clk-pll14xx.c
> > > +++ b/drivers/clk/imx/clk-pll14xx.c
> > > @@ -63,6 +63,7 @@ static const struct imx_pll14xx_rate_table imx_pll1416x_tbl[] = {
> > >
> > >  static const struct imx_pll14xx_rate_table imx_pll1443x_tbl[] = {
> > >       PLL_1443X_RATE(1039500000U, 173, 2, 1, 16384),
> > > +     PLL_1443X_RATE(756000000U, 252, 2, 2, 0),
> > >       PLL_1443X_RATE(650000000U, 325, 3, 2, 0),
> > >       PLL_1443X_RATE(594000000U, 198, 2, 2, 0),
> > >       PLL_1443X_RATE(519750000U, 173, 2, 2, 16384),
> > > --
> > > 2.49.0
> > >
> > >
> > >
> > 
> > --
> > #gernperDu
> > #CallMeByMyFirstName
> > 
> > Pengutronix e.K.                           |                             |
> > Steuerwalder Str. 21                       | https://www.pengutronix.de/ |
> > 31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
> > Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-9    |
> 

-- 
#gernperDu 
#CallMeByMyFirstName

Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | https://www.pengutronix.de/ |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-9    |

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

end of thread, other threads:[~2026-02-17 11:12 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-02-17  9:41 [PATCH] clk: imx: pll14xx: Add 756 MHz entry for PLL1443x Ian Ray
2026-02-17  9:54 ` Marco Felsch
2026-02-17 10:23   ` Ian Ray
2026-02-17 11:12     ` Marco Felsch

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