From: Kevin Hilman <khilman@ti.com>
To: "Menon, Nishanth" <nm@ti.com>
Cc: Jean Pihet <jean.pihet@newoldbits.com>,
linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
balbi@ti.com, Jean Pihet <j-pihet@ti.com>
Subject: Re: [PATCH 04/14] ARM: OMAP3+: SmartReflex: Add a shutdown hook
Date: Wed, 29 Feb 2012 10:07:08 -0800 [thread overview]
Message-ID: <878vjlfr8z.fsf@ti.com> (raw)
In-Reply-To: <CAOMWX4dbufbEJpr0nTmwTHR1xz9diMBbO=khKxvMZLWrKesJRw@mail.gmail.com> (Nishanth Menon's message of "Tue, 28 Feb 2012 16:59:14 -0600")
"Menon, Nishanth" <nm@ti.com> writes:
> On Tue, Feb 28, 2012 at 16:40, Kevin Hilman <khilman@ti.com> wrote:
>> Jean Pihet <jean.pihet@newoldbits.com> writes:
>>
>>> From: Nishanth Menon <nm@ti.com>
>>>
>>> SmartReflex fix for erratum ID i724.
>>>
>>> Since OMAP's VP and PRM modules do not get reset by warm reset,
>>> we should ensure that proper shutdown procedure is followed prior
>>> to allowing the kernel to reboot back up.
>>
>> prior to reboot? how?
>>
>> This patch only ensures a proper shutdown if the kernel goes down
>> cleanly.
>>
>> How is this handled on reboot?
> There is no known solution for watchdog resets(part of the errata WA
> limitation) :( - the best we could do was to make shutdown path safe -
> this includes s/w (userspace triggered) reboots.
OK, then the changelog needs an update to reflect that.
Kevin
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
WARNING: multiple messages have this Message-ID (diff)
From: khilman@ti.com (Kevin Hilman)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 04/14] ARM: OMAP3+: SmartReflex: Add a shutdown hook
Date: Wed, 29 Feb 2012 10:07:08 -0800 [thread overview]
Message-ID: <878vjlfr8z.fsf@ti.com> (raw)
In-Reply-To: <CAOMWX4dbufbEJpr0nTmwTHR1xz9diMBbO=khKxvMZLWrKesJRw@mail.gmail.com> (Nishanth Menon's message of "Tue, 28 Feb 2012 16:59:14 -0600")
"Menon, Nishanth" <nm@ti.com> writes:
> On Tue, Feb 28, 2012 at 16:40, Kevin Hilman <khilman@ti.com> wrote:
>> Jean Pihet <jean.pihet@newoldbits.com> writes:
>>
>>> From: Nishanth Menon <nm@ti.com>
>>>
>>> SmartReflex fix for erratum ID i724.
>>>
>>> Since OMAP's VP and PRM modules do not get reset by warm reset,
>>> we should ensure that proper shutdown procedure is followed prior
>>> to allowing the kernel to reboot back up.
>>
>> prior to reboot? ?how?
>>
>> This patch only ensures a proper shutdown if the kernel goes down
>> cleanly.
>>
>> How is this handled on reboot?
> There is no known solution for watchdog resets(part of the errata WA
> limitation) :( - the best we could do was to make shutdown path safe -
> this includes s/w (userspace triggered) reboots.
OK, then the changelog needs an update to reflect that.
Kevin
next prev parent reply other threads:[~2012-02-29 18:07 UTC|newest]
Thread overview: 44+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-02-08 17:43 [PATCH v4 00/14] ARM: OMAP3+: SmartReflex: bugfixes and clean-ups Jean Pihet
2012-02-08 17:43 ` Jean Pihet
2012-02-08 17:43 ` [PATCH 01/14] ARM: OMAP3+: SmartReflex: Layer Cleanup [V4] Jean Pihet
2012-02-08 17:43 ` Jean Pihet
2012-02-28 22:37 ` Kevin Hilman
2012-02-28 22:37 ` Kevin Hilman
2012-02-08 17:43 ` [PATCH 02/14] ARM: OMAP3+: SmartReflex: fix err interrupt disable sequence Jean Pihet
2012-02-08 17:43 ` Jean Pihet
2012-02-08 17:43 ` [PATCH 03/14] ARM: OMAP3+: SmartReflex Class3: disable errorgen before disable VP Jean Pihet
2012-02-08 17:43 ` Jean Pihet
2012-02-08 17:43 ` [PATCH 04/14] ARM: OMAP3+: SmartReflex: Add a shutdown hook Jean Pihet
2012-02-08 17:43 ` Jean Pihet
2012-02-28 22:40 ` Kevin Hilman
2012-02-28 22:40 ` Kevin Hilman
2012-02-28 22:59 ` Menon, Nishanth
2012-02-28 22:59 ` Menon, Nishanth
2012-02-29 18:07 ` Kevin Hilman [this message]
2012-02-29 18:07 ` Kevin Hilman
2012-02-29 23:33 ` Jean Pihet
2012-02-29 23:33 ` Jean Pihet
2012-02-08 17:43 ` [PATCH 05/14] ARM: OMAP3+: SmartReflex: use __devexit_p annotation Jean Pihet
2012-02-08 17:43 ` Jean Pihet
2012-02-08 17:43 ` [PATCH 06/14] ARM: OMAP3+: SmartReflex: Fix status masking in ERRCONFIG register Jean Pihet
2012-02-08 17:43 ` Jean Pihet
2012-02-08 17:43 ` [PATCH 07/14] ARM: OMAP3+: SmartReflex: clear ERRCONFIG_VPBOUNDINTST only on a need Jean Pihet
2012-02-08 17:43 ` Jean Pihet
2012-02-08 17:43 ` [PATCH 08/14] ARM: OMAP3+: hwmod: add SmartReflex IRQs Jean Pihet
2012-02-08 17:43 ` Jean Pihet
2012-02-28 22:47 ` Kevin Hilman
2012-02-28 22:47 ` Kevin Hilman
2012-02-08 17:43 ` [PATCH 09/14] ARM: OMAP3+: SmartReflex: add missing platform_set_drvdata() Jean Pihet
2012-02-08 17:43 ` Jean Pihet
2012-02-08 17:43 ` [PATCH 10/14] ARM: OMAP3+: SmartReflex: move late_initcall() closer to its argument Jean Pihet
2012-02-08 17:43 ` Jean Pihet
2012-02-08 17:43 ` [PATCH 11/14] ARM: OMAP3+: SmartReflex: misc cleanups Jean Pihet
2012-02-08 17:43 ` Jean Pihet
2012-02-08 17:43 ` [PATCH 12/14] ARM: OMAP3+: SmartReflex: micro-optimization for sanity check Jean Pihet
2012-02-08 17:43 ` Jean Pihet
2012-02-08 17:43 ` [PATCH 13/14] ARM: OMAP3+: SmartReflex: fix the use of debugfs_create_* API Jean Pihet
2012-02-08 17:43 ` Jean Pihet
2012-02-08 17:43 ` [PATCH 14/14] ARM: OMAP3+: SmartReflex: fix error handling Jean Pihet
2012-02-08 17:43 ` Jean Pihet
2012-02-28 22:50 ` [PATCH v4 00/14] ARM: OMAP3+: SmartReflex: bugfixes and clean-ups Kevin Hilman
2012-02-28 22:50 ` Kevin Hilman
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=878vjlfr8z.fsf@ti.com \
--to=khilman@ti.com \
--cc=balbi@ti.com \
--cc=j-pihet@ti.com \
--cc=jean.pihet@newoldbits.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-omap@vger.kernel.org \
--cc=nm@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.