diff mbox series

[v2,1/1] clk: imx6ul: retain early UART clocks during kernel init

Message ID 20230421115517.1940990-1-alexander.stein@ew.tq-group.com (mailing list archive)
State Awaiting Upstream, archived
Headers show
Series [v2,1/1] clk: imx6ul: retain early UART clocks during kernel init | expand

Commit Message

Alexander Stein April 21, 2023, 11:55 a.m. UTC
Make sure to keep UART clocks enabled during kernel init if
earlyprintk or earlycon are active.

Signed-off-by: Alexander Stein <alexander.stein@ew.tq-group.com>
---
Changes in v2:
* Rebased to next-20230420

Apparently i.MX6UL was missed in commit 0822f933735c ("clk: imx6: retain
early UART clocks during kernel init"). But as commit 379c9a24cc23 ("clk:
imx: Fix reparenting of UARTs not associated with stdout") changes the
calling signature it's not warranting a Fixes for the old commit.

 drivers/clk/imx/clk-imx6ul.c | 2 ++
 1 file changed, 2 insertions(+)

Comments

Peng Fan (OSS) April 24, 2023, 2:52 a.m. UTC | #1
On 4/21/2023 7:55 PM, Alexander Stein wrote:
> Make sure to keep UART clocks enabled during kernel init if
> earlyprintk or earlycon are active.
> 
> Signed-off-by: Alexander Stein <alexander.stein@ew.tq-group.com>

Reviewed-by: Peng Fan <peng.fan@nxp.com>

> ---
> Changes in v2:
> * Rebased to next-20230420
> 
> Apparently i.MX6UL was missed in commit 0822f933735c ("clk: imx6: retain
> early UART clocks during kernel init"). But as commit 379c9a24cc23 ("clk:
> imx: Fix reparenting of UARTs not associated with stdout") changes the
> calling signature it's not warranting a Fixes for the old commit.
> 
>   drivers/clk/imx/clk-imx6ul.c | 2 ++
>   1 file changed, 2 insertions(+)
> 
> diff --git a/drivers/clk/imx/clk-imx6ul.c b/drivers/clk/imx/clk-imx6ul.c
> index e3696a88b5a3..f9394e94f69d 100644
> --- a/drivers/clk/imx/clk-imx6ul.c
> +++ b/drivers/clk/imx/clk-imx6ul.c
> @@ -544,6 +544,8 @@ static void __init imx6ul_clocks_init(struct device_node *ccm_node)
>   
>   	clk_set_parent(hws[IMX6UL_CLK_ENET1_REF_SEL]->clk, hws[IMX6UL_CLK_ENET_REF]->clk);
>   	clk_set_parent(hws[IMX6UL_CLK_ENET2_REF_SEL]->clk, hws[IMX6UL_CLK_ENET2_REF]->clk);
> +
> +	imx_register_uart_clocks();
>   }
>   
>   CLK_OF_DECLARE(imx6ul, "fsl,imx6ul-ccm", imx6ul_clocks_init);
Abel Vesa May 18, 2023, 10:43 a.m. UTC | #2
On Fri, 21 Apr 2023 13:55:17 +0200, Alexander Stein wrote:
> Make sure to keep UART clocks enabled during kernel init if
> earlyprintk or earlycon are active.
> 
> 

Applied, thanks!

[1/1] clk: imx6ul: retain early UART clocks during kernel init
      commit: 912d7af473f163ccdeb02aaabc3534177936b86c

Best regards,
diff mbox series

Patch

diff --git a/drivers/clk/imx/clk-imx6ul.c b/drivers/clk/imx/clk-imx6ul.c
index e3696a88b5a3..f9394e94f69d 100644
--- a/drivers/clk/imx/clk-imx6ul.c
+++ b/drivers/clk/imx/clk-imx6ul.c
@@ -544,6 +544,8 @@  static void __init imx6ul_clocks_init(struct device_node *ccm_node)
 
 	clk_set_parent(hws[IMX6UL_CLK_ENET1_REF_SEL]->clk, hws[IMX6UL_CLK_ENET_REF]->clk);
 	clk_set_parent(hws[IMX6UL_CLK_ENET2_REF_SEL]->clk, hws[IMX6UL_CLK_ENET2_REF]->clk);
+
+	imx_register_uart_clocks();
 }
 
 CLK_OF_DECLARE(imx6ul, "fsl,imx6ul-ccm", imx6ul_clocks_init);