Re: [PATCH V4] clk: imx: Fix reparenting of UARTs not associated with stdout

2021-03-24 Thread Adam Ford
On Mon, Mar 22, 2021 at 4:42 PM Abel Vesa wrote: > > On 21-03-13 06:28:17, Adam Ford wrote: > > Most if not all i.MX SoC's call a function which enables all UARTS. > > This is a problem for users who need to re-parent the clock source, > > because any attempt to change the parent results in an bus

Re: [PATCH V4] clk: imx: Fix reparenting of UARTs not associated with stdout

2021-03-22 Thread Abel Vesa
On 21-03-13 06:28:17, Adam Ford wrote: > Most if not all i.MX SoC's call a function which enables all UARTS. > This is a problem for users who need to re-parent the clock source, > because any attempt to change the parent results in an busy error > due to the fact that the clocks have been enabled

Re: [PATCH V4] clk: imx: Fix reparenting of UARTs not associated with stdout

2021-03-22 Thread Abel Vesa
On 21-03-20 18:00:25, Adam Ford wrote: > On Sun, Mar 14, 2021 at 4:40 AM Ahmad Fatoum wrote: > > > > On 13.03.21 16:16, Ahmad Fatoum wrote: > > >> +/* i.MX boards use device trees now. For build tests without > > >> CONFIG_OF, do nothing */ > > >> +#ifdef CONFIG_OF > > >> if (imx_keep_uart_

Re: [PATCH V4] clk: imx: Fix reparenting of UARTs not associated with stdout

2021-03-20 Thread Adam Ford
On Sun, Mar 14, 2021 at 4:40 AM Ahmad Fatoum wrote: > > On 13.03.21 16:16, Ahmad Fatoum wrote: > >> +/* i.MX boards use device trees now. For build tests without CONFIG_OF, > >> do nothing */ > >> +#ifdef CONFIG_OF > >> if (imx_keep_uart_clocks) { > >> int i; > >> > >> -

Re: [PATCH V4] clk: imx: Fix reparenting of UARTs not associated with stdout

2021-03-14 Thread Ahmad Fatoum
On 13.03.21 16:16, Ahmad Fatoum wrote: >> +/* i.MX boards use device trees now. For build tests without CONFIG_OF, do >> nothing */ >> +#ifdef CONFIG_OF >> if (imx_keep_uart_clocks) { >> int i; >> >> -imx_uart_clocks = clks; >> -for (i = 0; imx_uart_clo

Re: [PATCH V4] clk: imx: Fix reparenting of UARTs not associated with stdout

2021-03-13 Thread Ahmad Fatoum
Hi, On 13.03.21 13:28, Adam Ford wrote: > Most if not all i.MX SoC's call a function which enables all UARTS. > This is a problem for users who need to re-parent the clock source, > because any attempt to change the parent results in an busy error > due to the fact that the clocks have been enable

[PATCH V4] clk: imx: Fix reparenting of UARTs not associated with stdout

2021-03-13 Thread Adam Ford
Most if not all i.MX SoC's call a function which enables all UARTS. This is a problem for users who need to re-parent the clock source, because any attempt to change the parent results in an busy error due to the fact that the clocks have been enabled already. clk: failed to reparent uart1 to sy