[PATCH 1/1] clk: imx6q: fix uart earlycon unwork

* [PATCH 1/1] clk: imx6q: fix uart earlycon unwork
@ 2021-07-02  8:54 Dong Aisheng
  2021-07-19 10:39 ` Abel Vesa
  2021-08-06  1:12 ` Stephen Boyd
  0 siblings, 2 replies; 5+ messages in thread
From: Dong Aisheng @ 2021-07-02  8:54 UTC (permalink / raw)
  To: linux-clk
  Cc: linux-arm-kernel, sboyd, dongas86, shawnguo, kernel, abel.vesa,
	aford173, Dong Aisheng, stable

The earlycon depends on the bootloader setup UART clocks being retained.
There're actually two uart clocks (ipg, per) on MX6QDL,
but the 'Fixes' commit change to register only one which means
another clock may be disabled during booting phase
and result in the earlycon unwork.

Cc: stable@vger.kernel.org # v5.10+
Fixes: 379c9a24cc23 ("clk: imx: Fix reparenting of UARTs not associated with stdout")
Signed-off-by: Dong Aisheng <aisheng.dong@nxp.com>
---
 drivers/clk/imx/clk-imx6q.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/clk/imx/clk-imx6q.c b/drivers/clk/imx/clk-imx6q.c
index 496900de0b0b..de36f58d551c 100644
--- a/drivers/clk/imx/clk-imx6q.c
+++ b/drivers/clk/imx/clk-imx6q.c
@@ -974,6 +974,6 @@ static void __init imx6q_clocks_init(struct device_node *ccm_node)
 			       hws[IMX6QDL_CLK_PLL3_USB_OTG]->clk);
 	}
 
-	imx_register_uart_clocks(1);
+	imx_register_uart_clocks(2);
 }
 CLK_OF_DECLARE(imx6q, "fsl,imx6q-ccm", imx6q_clocks_init);
-- 
2.25.1


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