Re: [PATCH] clk: imx7d: Move clks_init_on before any clock operations

From: Fabio Estevam
Date: Wed Mar 28 2018 - 08:15:02 EST


Hi Abel,

On Tue, Mar 27, 2018 at 11:23 AM, Abel Vesa <abel.vesa@xxxxxxx> wrote:
> From: Dong Aisheng <b29396@xxxxxxxxxxxxx>
>
> For init on clocks we should move it at the first place in imx7d_clocks_init()
> before any clock operations, else the clock operation may fail in case the clock
> is still not on.

The patch and the commit log does not match.

You are not moving the code block to the "first place" inside
imx7d_clocks_init(), so this is confusing.

Also, why does this need to be done only in the imx7d clock driver?