From: Santosh Shilimkar <santosh.shilimkar@ti.com>
To: Tony Lindgren <tony@atomide.com>
Cc: linux-omap@vger.kernel.org, Paul Walmsley <paul@pwsan.com>,
Kevin Hilman <khilman@ti.com>, Benoit Cousson <b-cousson@ti.com>
Subject: RE: [PATCH] omap: Move omap_hwmod_late_init() to mdesc->timer->init() code
Date: Wed, 23 Feb 2011 11:12:47 +0530 [thread overview]
Message-ID: <eeb4d4bc7d16b99b52d278491e7a5cdb@mail.gmail.com> (raw)
In-Reply-To: <20110222164902.GJ15225@atomide.com>
> -----Original Message-----
> From: linux-omap-owner@vger.kernel.org [mailto:linux-omap-
> owner@vger.kernel.org] On Behalf Of Tony Lindgren
> Sent: Tuesday, February 22, 2011 10:19 PM
> To: Santosh Shilimkar
> Cc: linux-omap@vger.kernel.org; Paul Walmsley; Kevin Hilman; Benoit
> Cousson
> Subject: Re: [PATCH] omap: Move omap_hwmod_late_init() to mdesc-
> >timer->init() code
>
> * Santosh Shilimkar <santosh.shilimkar@ti.com> [110222 06:35]:
> > To adhere to recent early_init changes, commit '44dc0' made
> > omap_hwmod_late_init() to core_initcall to avoid ioremap()
> failures.
> >
> > Later 'e7c7d7' removed _mpu_rt_va population
> omap_hwmod_late_init()
> >
> > So now if we move the omap_hwmod_late_init() to mdesc->timer-
> >init(),
> > timer1 should work with hwmod instead of any special hwmod
> settings.
>
> There should be no need to have all hwmods initialized just to get
> the system timer running. The system timer should be initialized
> with minimal code needed.
>
Sure
> Sure it can use hwmod functions as long as it makes sense, but
> we certainly don't want the whole hwmod framework initialized
> that early.
>
Something like hwmod_early_init() would solve the issue of early
devices. At least this is better than hacking early hwmod
drivers.
Looks like that's what we decided to go head with
looking at other thread.
Regards,
Santosh
next prev parent reply other threads:[~2011-02-23 5:42 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-02-22 14:37 [PATCH] omap: Move omap_hwmod_late_init() to mdesc->timer->init() code Santosh Shilimkar
2011-02-22 16:49 ` Tony Lindgren
2011-02-23 5:42 ` Santosh Shilimkar [this message]
2011-02-23 7:18 ` Paul Walmsley
2011-02-23 14:20 ` Cousson, Benoit
2011-02-23 14:29 ` Santosh Shilimkar
2011-02-24 8:49 ` Paul Walmsley
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=eeb4d4bc7d16b99b52d278491e7a5cdb@mail.gmail.com \
--to=santosh.shilimkar@ti.com \
--cc=b-cousson@ti.com \
--cc=khilman@ti.com \
--cc=linux-omap@vger.kernel.org \
--cc=paul@pwsan.com \
--cc=tony@atomide.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox