[PATCH 1/2] clk: clk-imx8qxp: Add LPUART IPG entries

[Hiago De Franco]

On 09.03.2024 01:15, Peng Fan wrote:
> > Subject: [PATCH 1/2] clk: clk-imx8qxp: Add LPUART IPG entries
> > 
> > Since commit cc7df0b9e8bc ("serial: lpuart: Enable IPG clock") the colibri-
> > imx8qxp board no longer boots.
> > 
> > The reason is that the imx8qxp clock driver does not handle the LPUART IPG
> > clocks inside get_rate(), set_rate() and enable() functions.
> > 
> > Fix the boot regression by adding the LPUART IPG entries.
> > 
> > Fixes: cc7df0b9e8bc ("serial: lpuart: Enable IPG clock")
> > Reported-by: Marcel Ziswiler <marcel.ziswiler at toradex.com>
> > Signed-off-by: Fabio Estevam <festevam at gmail.com>
> 
> Reviewed-by: Peng Fan <peng.fan at nxp.com>

Tested-by: Hiago De Franco <hiago.franco at toradex.com> # Toradex Colibri iMX8X

> > ---
> >  drivers/clk/imx/clk-imx8qxp.c | 13 ++++++++++++-
> >  1 file changed, 12 insertions(+), 1 deletion(-)
> > 
> > diff --git a/drivers/clk/imx/clk-imx8qxp.c b/drivers/clk/imx/clk-imx8qxp.c
> > index 8bf7e325481..d900d4cd528 100644
> > --- a/drivers/clk/imx/clk-imx8qxp.c
> > +++ b/drivers/clk/imx/clk-imx8qxp.c
> > @@ -88,20 +88,23 @@ ulong imx8_clk_get_rate(struct clk *clk)
> >  		resource = SC_R_SDHC_1;
> >  		pm_clk = SC_PM_CLK_PER;
> >  		break;
> > -	case IMX8QXP_UART0_IPG_CLK:
> >  	case IMX8QXP_UART0_CLK:
> > +	case IMX8QXP_UART0_IPG_CLK:
> >  		resource = SC_R_UART_0;
> >  		pm_clk = SC_PM_CLK_PER;
> >  		break;
> >  	case IMX8QXP_UART1_CLK:
> > +	case IMX8QXP_UART1_IPG_CLK:
> >  		resource = SC_R_UART_1;
> >  		pm_clk = SC_PM_CLK_PER;
> >  		break;
> >  	case IMX8QXP_UART2_CLK:
> > +	case IMX8QXP_UART2_IPG_CLK:
> >  		resource = SC_R_UART_2;
> >  		pm_clk = SC_PM_CLK_PER;
> >  		break;
> >  	case IMX8QXP_UART3_CLK:
> > +	case IMX8QXP_UART3_IPG_CLK:
> >  		resource = SC_R_UART_3;
> >  		pm_clk = SC_PM_CLK_PER;
> >  		break;
> > @@ -170,18 +173,22 @@ ulong imx8_clk_set_rate(struct clk *clk, unsigned
> > long rate)
> >  		pm_clk = SC_PM_CLK_PER;
> >  		break;
> >  	case IMX8QXP_UART0_CLK:
> > +	case IMX8QXP_UART0_IPG_CLK:
> >  		resource = SC_R_UART_0;
> >  		pm_clk = SC_PM_CLK_PER;
> >  		break;
> >  	case IMX8QXP_UART1_CLK:
> > +	case IMX8QXP_UART1_IPG_CLK:
> >  		resource = SC_R_UART_1;
> >  		pm_clk = SC_PM_CLK_PER;
> >  		break;
> >  	case IMX8QXP_UART2_CLK:
> > +	case IMX8QXP_UART2_IPG_CLK:
> >  		resource = SC_R_UART_2;
> >  		pm_clk = SC_PM_CLK_PER;
> >  		break;
> >  	case IMX8QXP_UART3_CLK:
> > +	case IMX8QXP_UART3_IPG_CLK:
> >  		resource = SC_R_UART_3;
> >  		pm_clk = SC_PM_CLK_PER;
> >  		break;
> > @@ -263,18 +270,22 @@ int __imx8_clk_enable(struct clk *clk, bool enable)
> >  		pm_clk = SC_PM_CLK_PER;
> >  		break;
> >  	case IMX8QXP_UART0_CLK:
> > +	case IMX8QXP_UART0_IPG_CLK:
> >  		resource = SC_R_UART_0;
> >  		pm_clk = SC_PM_CLK_PER;
> >  		break;
> >  	case IMX8QXP_UART1_CLK:
> > +	case IMX8QXP_UART1_IPG_CLK:
> >  		resource = SC_R_UART_1;
> >  		pm_clk = SC_PM_CLK_PER;
> >  		break;
> >  	case IMX8QXP_UART2_CLK:
> > +	case IMX8QXP_UART2_IPG_CLK:
> >  		resource = SC_R_UART_2;
> >  		pm_clk = SC_PM_CLK_PER;
> >  		break;
> >  	case IMX8QXP_UART3_CLK:
> > +	case IMX8QXP_UART3_IPG_CLK:
> >  		resource = SC_R_UART_3;
> >  		pm_clk = SC_PM_CLK_PER;
> >  		break;
> > --
> > 2.34.1
>