>>-----Original Message----- >>From: Kevin Hilman [mailto:khil...@deeprootsystems.com] >>Sent: Thursday, December 02, 2010 7:03 PM >>To: Gopinath, Thara >>Cc: linux-omap@vger.kernel.org >>Subject: Re: [PATCH] OMAP: pm.c correct the initcall for an early init. >> >>Thara Gopinath <th...@ti.com> writes: >> >>> omap2_common_pm_init is the API where generic system devices like >>> mpu, l3 etc get initialized. This has to happen really early on >>> during the boot and not at a later time. This is especially important >>> with the new opp changes as these devices need to be built before the >>> opp tables init happen. Today both are device initcalls and it works >>> just because of the order of compilation >> >>Why postcore? there are several other inicalls earlier than >>device_initcall.
Because the init in omap_device is a core_initcall. With respect to opp layer, making this anything above device_initcall will work. But then tomorrow some other module needs these generic devices in their init, we will again have to bump up the init priority of this fn. It is a good thing to do this early on in the boot cycle rather than later. >> >>Also, does this actually work? Is the driver core initialized at >>postcore_initcall time such that omap_devices w/platform_device >>creation actually works? Yes by post_initcall the omap_device initializations would have happened. In fact these initializations happen as core_initcall. >> >>Kevin >> >>> Signed-off-by: Thara Gopinath <th...@ti.com> >>> --- >>> arch/arm/mach-omap2/pm.c | 2 +- >>> 1 files changed, 1 insertions(+), 1 deletions(-) >>> >>> diff --git a/arch/arm/mach-omap2/pm.c b/arch/arm/mach-omap2/pm.c >>> index 59ca03b..6ec2ee1 100644 >>> --- a/arch/arm/mach-omap2/pm.c >>> +++ b/arch/arm/mach-omap2/pm.c >>> @@ -143,5 +143,5 @@ static int __init omap2_common_pm_init(void) >>> >>> return 0; >>> } >>> -device_initcall(omap2_common_pm_init); >>> +postcore_initcall(omap2_common_pm_init); -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
