[RFC PATCH 3/3] serial: mxc: restore booting for imx8mn_bsh_smm_s2
Michael Nazzareno Trimarchi
michael at amarulasolutions.com
Sat May 17 22:52:43 CEST 2025
Hi
On Sat, May 17, 2025 at 10:50 PM Fabio Estevam <festevam at gmail.com> wrote:
>
> On Sat, May 17, 2025 at 5:46 PM Michael Nazzareno Trimarchi
> <michael at amarulasolutions.com> wrote:
>
> > > I was on master except this commit
> > >
> > > commit 128d997a8772cc174f38d529d8b25f90b3aa8ad8
> > > Author: Jonas Karlman <jonas at kwiboo.se>
> > > Date: Sat May 10 15:32:01 2025 +0000
> > >
> > > clk: Fix clk_set_parent() regression
> > >
> > > The commit ac30d90f3367 ("clk: Ensure the parent clocks are enabled
> > > while reparenting") add a call to clk_enable() for the parent clock.
> > >
> > > For clock drivers that do not implement the enable() ops, like most
> > > Rockchip clock drivers, this now cause the set_parent() ops to never
> > > be called when CLK_CCF=n (default for Rockchip).
>
> CLK_CCF=y in your case, so this does not help for you.
> >
> > BTW is really needed now?
> >
> > git grep init_uart_clk board/freescale/imx8mn_evk/
> > board/freescale/imx8mn_evk/spl.c: init_uart_clk(1);
>
> This can be removed. I tested without this line, and it boots fine.
Are you sending a patch? or I will send it
Michael
--
Michael Nazzareno Trimarchi
Co-Founder & Chief Executive Officer
M. +39 347 913 2170
michael at amarulasolutions.com
__________________________________
Amarula Solutions BV
Joop Geesinkweg 125, 1114 AB, Amsterdam, NL
T. +31 (0)85 111 9172
info at amarulasolutions.com
www.amarulasolutions.com
More information about the U-Boot
mailing list