Hi Abel, On Tue, Mar 27, 2018 at 11:23 AM, Abel Vesa <[email protected]> wrote: > From: Dong Aisheng <[email protected]> > > 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?

