From mboxrd@z Thu Jan  1 00:00:00 1970
From: plagnioj@jcrosoft.com (Jean-Christophe PLAGNIOL-VILLARD)
Date: Thu, 28 Apr 2011 13:19:17 +0200
Subject: [PATCH] clkdev: add support to lookup for early platform device
In-Reply-To: <20110428100719.GZ17290@n2100.arm.linux.org.uk>
References: <20110427150057.GH29103@game.jcrosoft.org>
	<20110428020910.GA13539@linux-sh.org>
	<20110428024559.GK29103@game.jcrosoft.org>
	<20110428030817.GC13539@linux-sh.org>
	<20110428031704.GL29103@game.jcrosoft.org>
	<20110428033808.GE13539@linux-sh.org>
	<20110428051203.GB3852@game.jcrosoft.org>
	<20110428083547.GM17290@n2100.arm.linux.org.uk>
	<20110428092613.GD3852@game.jcrosoft.org>
	<20110428100719.GZ17290@n2100.arm.linux.org.uk>
Message-ID: <20110428111917.GC31786@game.jcrosoft.org>
To: linux-arm-kernel@lists.infradead.org
List-Id: linux-arm-kernel.lists.infradead.org

On 11:07 Thu 28 Apr     , Russell King - ARM Linux wrote:
> On Thu, Apr 28, 2011 at 11:26:13AM +0200, Jean-Christophe PLAGNIOL-VILLARD wrote:
> > On 09:35 Thu 28 Apr     , Russell King - ARM Linux wrote:
> > > On Thu, Apr 28, 2011 at 07:12:03AM +0200, Jean-Christophe PLAGNIOL-VILLARD wrote:
> > > > How did you manage to get the clock for the earlytimer?
> > > 
> > > This is getting silly.  "Let's move all our devices to early" is insane.
> > > I can see someone then wanting an early-early-device because early has
> > > become just too late in the initialization.
> > > 
> > > Come on guys, keep it simple and stop trying to contort the interfaces
> > > you're given into doing things they're not designed to do.
> > I agress but I need to pass resources to the timer so early devices make
> > sense
> 
> Again, you shouldn't be setting up the timer at map_io time - the kernel
> just isn't ready for it.
> 
> map_io is meant just for setting up IO mappings.  You can then use the
> init_early callback to register tables, setting up sched_clock, getting
> the timer running for sched_clock, etc.  At this point, memblock must not
> be used, but bootmem is active, but normal page allocation is not.
> 
> The clocksource/clockevent initialization should happen after that when
> the system_timer->init function is called.
no the timer is init in the right place in system_timer->init

Best Regards,
J.