From: rpurdie@rpsys.net (Richard Purdie)
To: linux-arm-kernel@lists.infradead.org
Subject: [RFC 0/5] restart cleanups
Date: Wed, 02 Nov 2011 13:49:07 +0000 [thread overview]
Message-ID: <1320241747.20107.41.camel@ted> (raw)
In-Reply-To: <20111101170225.GS19187@n2100.arm.linux.org.uk>
On Tue, 2011-11-01 at 17:02 +0000, Russell King - ARM Linux wrote:
> (Poodle doesn't have any MAINTAINERS entry, I think Richard Purdie last
> touched Poodle in a 'maintainery' way back in 2006.)
>
> poodle does this:
>
> static void poodle_restart(char mode, const char *cmd)
> {
> arm_machine_restart('h', cmd);
> }
>
> Given that the default value of 'mode' (in the absense of any reboot=
> command line argument to the kernel) is 'h', this seems to be a no-op.
> Below is the history; I suspect that the reason this first appeared was
> because of the need to fiddle with RCSR - that's now gone so I suspect
> the entire restart handler can be deleted.
>
> What I don't understand in that case is why 'mode' wasn't simply passed
> through in the first patch.
next prev parent reply other threads:[~2011-11-02 13:49 UTC|newest]
Thread overview: 32+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-11-01 16:07 [RFC 0/5] restart cleanups Russell King - ARM Linux
2011-11-01 16:08 ` [PATCH 1/5] ARM: restart: remove argument to setup_mm_for_reboot() Russell King - ARM Linux
2011-11-01 22:27 ` Ryan Mallon
2011-11-02 14:48 ` Russell King - ARM Linux
2011-11-01 16:08 ` [PATCH 2/5] ARM: restart: get rid of needless at91_arch_reset additional indirection Russell King - ARM Linux
2011-11-01 16:09 ` [PATCH 3/5] ARM: restart: remove local_irq_disable() from within arch_reset() Russell King - ARM Linux
2011-11-01 16:36 ` H Hartley Sweeten
2011-11-01 17:56 ` Russell King - ARM Linux
2011-11-01 16:09 ` [PATCH 4/5] ARM: restart: turn reboot setup code into a library function Russell King - ARM Linux
2011-11-01 16:09 ` [PATCH 5/5] ARM: restart: only perform setup for restart when soft-restarting Russell King - ARM Linux
2011-11-02 12:12 ` Will Deacon
2011-11-02 12:37 ` Russell King - ARM Linux
2011-11-02 13:11 ` Will Deacon
2011-11-01 16:38 ` [RFC 0/5] restart cleanups Russell King - ARM Linux
2011-11-03 9:12 ` Russell King - ARM Linux
2011-11-03 9:24 ` Kukjin Kim
2011-11-03 9:57 ` Russell King - ARM Linux
2011-11-01 17:02 ` Russell King - ARM Linux
2011-11-02 13:49 ` Richard Purdie [this message]
2011-11-02 14:49 ` Russell King - ARM Linux
2011-11-04 13:58 ` Andrea Adami
2011-11-04 14:01 ` Russell King - ARM Linux
2011-11-04 14:22 ` Andrea Adami
2011-11-01 19:23 ` Will Deacon
2011-11-01 19:41 ` Russell King - ARM Linux
2011-11-01 21:19 ` Will Deacon
2011-11-01 23:09 ` Russell King - ARM Linux
2011-11-02 21:33 ` Robert Jarzmik
2011-11-02 23:31 ` Russell King - ARM Linux
2011-11-02 13:35 ` Nicolas Pitre
[not found] <1676495226.4381981320219764753.JavaMail.root@zimbra20-e3.priv.proxad.net>
2011-11-02 7:46 ` Robert Jarzmik
2011-11-02 10:30 ` Russell King - ARM Linux
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=1320241747.20107.41.camel@ted \
--to=rpurdie@rpsys.net \
--cc=linux-arm-kernel@lists.infradead.org \
/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.