On Wed, Sep 28, 2011 at 08:29:45AM +0200, Sascha Hauer wrote:

On Tue, Sep 27, 2011 at 11:15:47PM +0800, Shawn Guo wrote:

quoted

quoted

I do not want to add mxc_cpu_type and cpu_is_mx6q() stuff, so I intend
to remove early_initcall there, and let imx31/35_soc_init() explicitly
calls mxc_init_l2x0().

I'm changing my mind (so quickly).  It turns out we will fall into a
number of other traps if we build imx6q with imx3 without mxc_cpu_type
set up for imx6q, arch_idle(), __imx_ioremap(), etc.  So for now, I
want to add mxc_cpu_type for imx6q to cooperate with all these traps.

I'm fine with reducing the usage of cpu_is_* macros up to the point when
we don't need them anymore. But as you already found out, it's better to
do this seperately.

I'm changing my mind again.  I decided to clean up these traps with a
patch series, which I'm sending right now.

-- 
Regards,
Shawn