On Wednesday 28 August 2013, Sebastian Hesselbarth wrote: > > > > This forces zynq to move some initialization our clock code relies on to > > init_irq(). Also, the current code already takes an approach of > > doing either common init or machine specific init. > > Soeren, > > you know that patch 16/16 takes care of zynq's clock init? > > It's your own patch you provided from the last RFC. Looking at it, it > moves zynq_sclr_init() to .init_irq and removes the call to > of_clk_init() from zynq_clock_init() which is called by > zynq_sclr_init(). > > Isn't that solving the above issues for mach-zynq? Please be careful with the patch ordering here. The patch series should be bisectable, i.e. no patch should ever knowingly break any of the platforms, with the fix getting added in a later patch. You should be able to do that by cleaning up all platforms to not rely on ordering first, then add this patch, and finally remove the other calls. Arnd -- To unsubscribe from this list: send the line "unsubscribe linux-tegra" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html