From: Tony Lindgren <tony@atomide.com>
To: Paul Walmsley <paul@pwsan.com>
Cc: linux-omap@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-arm-kernel@lists.infradead.org,
"Aaro Koskinen" <aaro.koskinen@nokia.com>,
"Rajendra Nayak" <rnayak@ti.com>,
"Benoît Cousson" <b-cousson@ti.com>,
"Michael Buesch" <mb@bu3sch.de>
Subject: Re: [PATCH] OMAP: hwmod: add kernel cmdline flag to avoid resetting IP blocks during init
Date: Tue, 5 Jul 2011 23:41:16 -0700 [thread overview]
Message-ID: <20110706064116.GH5783@atomide.com> (raw)
In-Reply-To: <alpine.DEB.2.00.1107051936250.4217@utopia.booyaka.com>
* Paul Walmsley <paul@pwsan.com> [110705 18:37]:
> cc'ing Aaro
>
> On Mon, 4 Jul 2011, Tony Lindgren wrote:
>
> > Also related is the gptimer12 on secure omaps as reported by Aaro.
>
> This problem is actually worse. In this case it's not just preventing
> reset; we must prevent the kernel from using the device at all. My
> understanding is also that it's the X-Loader or bootloader -- or more
> broadly, secure mode code -- that actually chooses which devices are
> protected in this way. In other words, we can't make any automatic
> decisions that are based on knowing that a chip is a high-security chip --
> it will need to be defined by the board file :-(
>
> For this case, we probably need some board file function to tell the hwmod
> code to disregard a device completely, to tell the kernel to pretend that
> the device does not exist.
Yeah that should be done in a generic way. Some shared devices can also
be used by a coprocessor like the DSP instead.
So basically we want to tell the following from the board file or board
specific .dts file:
- Device is being used on the board and can be reset and configured.
This is the usual case.
- Device is being used on the board but can't be reset. This is the
case for booting Linux from other operating systems initially where
you want to keep the LCD on for debug console.
- Device is not being used on the board but can be reset for decent PM.
This is often needed in cases where the bootloader just enables all the
clocks.
- Device is reserved by secure mode or a coprocessor. In this case
the device can't be reset.
So I guess that makes the flags noreset, disabled and unavailable?
For the devices with no special flags we would then reset the device.
Regards,
Tony
next prev parent reply other threads:[~2011-07-06 6:41 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20110703010204.19046.85186.stgit@dusk>
2011-07-03 1:02 ` [PATCH] OMAP: hwmod: add kernel cmdline flag to avoid resetting IP blocks during init Paul Walmsley
2011-07-04 8:55 ` Tony Lindgren
2011-07-06 1:42 ` Paul Walmsley
2011-07-06 2:00 ` Paul Walmsley
2011-07-06 6:43 ` Tony Lindgren
2011-07-06 21:01 ` Paul Walmsley
2011-07-06 6:41 ` Tony Lindgren [this message]
2011-07-06 20:57 ` Paul Walmsley
2011-07-07 5:27 ` Tony Lindgren
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=20110706064116.GH5783@atomide.com \
--to=tony@atomide.com \
--cc=aaro.koskinen@nokia.com \
--cc=b-cousson@ti.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-omap@vger.kernel.org \
--cc=mb@bu3sch.de \
--cc=paul@pwsan.com \
--cc=rnayak@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).