From: Kevin Hilman <khilman@deeprootsystems.com>
To: "Gopinath, Thara" <thara@ti.com>
Cc: "linux-omap@vger.kernel.org" <linux-omap@vger.kernel.org>
Subject: Re: [PATCH] OMAP: pm.c correct the initcall for an early init.
Date: Mon, 13 Dec 2010 17:06:36 -0800 [thread overview]
Message-ID: <87zks92meb.fsf@deeprootsystems.com> (raw)
In-Reply-To: <5A47E75E594F054BAF48C5E4FC4B92AB03736F4913@dbde02.ent.ti.com> (Thara Gopinath's message of "Fri, 3 Dec 2010 14:17:10 +0530")
"Gopinath, Thara" <thara@ti.com> writes:
>>>-----Original Message-----
>>>From: Kevin Hilman [mailto:khilman@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 <thara@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.
OK, please describe this in more detail the changelog.
Thanks,
Kevin
next prev parent reply other threads:[~2010-12-14 1:06 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-12-01 11:01 [PATCH] OMAP: pm.c correct the initcall for an early init Thara Gopinath
2010-12-02 13:33 ` Kevin Hilman
2010-12-03 8:47 ` Gopinath, Thara
2010-12-14 1:06 ` Kevin Hilman [this message]
2010-12-15 15:38 ` Gopinath, Thara
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87zks92meb.fsf@deeprootsystems.com \
--to=khilman@deeprootsystems.com \
--cc=linux-omap@vger.kernel.org \
--cc=thara@ti.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.