| Message ID | 1408495538-27480-3-git-send-email-linux@roeck-us.net (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
Guenter, On Tue, Aug 19, 2014 at 5:45 PM, Guenter Roeck <linux@roeck-us.net> wrote: > machine_restart is supported on non-ARM platforms, and and ultimately calls > arm_pm_restart, so dont call arm_pm_restart directly but use the more > generic function. > > Cc: Russell King <linux@arm.linux.org.uk> Do you need to submit this to his patch tracker to get him to pick it up? How are you envisioning that this series land? It crosses a lot of boundaries so I guess will need a reasonable amount of coordination between maintainers... > Signed-off-by: Guenter Roeck <linux@roeck-us.net> > Acked-by: Catalin Marinas <catalin.marinas@arm.com> > Acked-by: Heiko Stuebner <heiko@sntech.de> > --- > v7: No change. > v6: No change. > v5: No change. > v4: No change. > v3: No change. > v2: Added patch. > > drivers/power/reset/restart-poweroff.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) Reviewed-by: Doug Anderson <dianders@chromium.org> Tested-by: Doug Anderson <dianders@chromium.org>
On Wed, Aug 20, 2014 at 09:10:31PM -0700, Doug Anderson wrote: > Guenter, > > On Tue, Aug 19, 2014 at 5:45 PM, Guenter Roeck <linux@roeck-us.net> wrote: > > machine_restart is supported on non-ARM platforms, and and ultimately calls > > arm_pm_restart, so dont call arm_pm_restart directly but use the more > > generic function. > > > > Cc: Russell King <linux@arm.linux.org.uk> > > Do you need to submit this to his patch tracker to get him to pick it > up? How are you envisioning that this series land? It crosses a lot > of boundaries so I guess will need a reasonable amount of coordination > between maintainers... > > If I get an Acked-by: from all maintainers, I could send a pull request to Linus directly. How do I send a patch to Russell's patch tracker ? I thought I copied all mailing lists suggested by get_maintainer.pl, but maybe I missed one. Thanks, Guenter
Guenter, On Wed, Aug 20, 2014 at 9:42 PM, Guenter Roeck <linux@roeck-us.net> wrote: > On Wed, Aug 20, 2014 at 09:10:31PM -0700, Doug Anderson wrote: >> Guenter, >> >> On Tue, Aug 19, 2014 at 5:45 PM, Guenter Roeck <linux@roeck-us.net> wrote: >> > machine_restart is supported on non-ARM platforms, and and ultimately calls >> > arm_pm_restart, so dont call arm_pm_restart directly but use the more >> > generic function. >> > >> > Cc: Russell King <linux@arm.linux.org.uk> >> >> Do you need to submit this to his patch tracker to get him to pick it >> up? How are you envisioning that this series land? It crosses a lot >> of boundaries so I guess will need a reasonable amount of coordination >> between maintainers... >> >> > If I get an Acked-by: from all maintainers, I could send a pull request > to Linus directly. How do I send a patch to Russell's patch tracker ? > I thought I copied all mailing lists suggested by get_maintainer.pl, > but maybe I missed one. Ah, OK. Perhaps it's best to ignore me, then. ;) FYI: Russell's Patch System is at <http://www.arm.linux.org.uk/developer/patches/> and is used for getting individual patches into branches maintained by Russel. If you're just going to send a pull request for the series then I don't think you need it.
On Thu, Aug 21, 2014 at 12:30:44PM -0700, Doug Anderson wrote: > Guenter, > > On Wed, Aug 20, 2014 at 9:42 PM, Guenter Roeck <linux@roeck-us.net> wrote: > > On Wed, Aug 20, 2014 at 09:10:31PM -0700, Doug Anderson wrote: > >> Guenter, > >> > >> On Tue, Aug 19, 2014 at 5:45 PM, Guenter Roeck <linux@roeck-us.net> wrote: > >> > machine_restart is supported on non-ARM platforms, and and ultimately calls > >> > arm_pm_restart, so dont call arm_pm_restart directly but use the more > >> > generic function. > >> > > >> > Cc: Russell King <linux@arm.linux.org.uk> > >> > >> Do you need to submit this to his patch tracker to get him to pick it > >> up? How are you envisioning that this series land? It crosses a lot > >> of boundaries so I guess will need a reasonable amount of coordination > >> between maintainers... > >> > >> > > If I get an Acked-by: from all maintainers, I could send a pull request > > to Linus directly. How do I send a patch to Russell's patch tracker ? > > I thought I copied all mailing lists suggested by get_maintainer.pl, > > but maybe I missed one. > > Ah, OK. Perhaps it's best to ignore me, then. ;) FYI: Russell's No, I appreciate the information. > Patch System is at <http://www.arm.linux.org.uk/developer/patches/> > and is used for getting individual patches into branches maintained by > Russel. If you're just going to send a pull request for the series > then I don't think you need it. > I copied both linux-arm-kernel and Russell himself, so hopefully he is aware of the series and can let me know if he wants me to make any further changes to it, if it is ready to go, or if I need to do anything else. Thanks, Guenter
Hi, On Tue, Aug 19, 2014 at 05:45:29PM -0700, Guenter Roeck wrote: > machine_restart is supported on non-ARM platforms, and and ultimately calls > arm_pm_restart, so dont call arm_pm_restart directly but use the more > generic function. > > Cc: Russell King <linux@arm.linux.org.uk> > Signed-off-by: Guenter Roeck <linux@roeck-us.net> > Acked-by: Catalin Marinas <catalin.marinas@arm.com> > Acked-by: Heiko Stuebner <heiko@sntech.de> Acked-by: Sebastian Reichel <sre@kernel.org> Do you want me to take the patch via battery-2.6.git? -- Sebastian
On 08/21/2014 01:39 PM, Sebastian Reichel wrote: > Hi, > > On Tue, Aug 19, 2014 at 05:45:29PM -0700, Guenter Roeck wrote: >> machine_restart is supported on non-ARM platforms, and and ultimately calls >> arm_pm_restart, so dont call arm_pm_restart directly but use the more >> generic function. >> >> Cc: Russell King <linux@arm.linux.org.uk> >> Signed-off-by: Guenter Roeck <linux@roeck-us.net> >> Acked-by: Catalin Marinas <catalin.marinas@arm.com> >> Acked-by: Heiko Stuebner <heiko@sntech.de> > > Acked-by: Sebastian Reichel <sre@kernel.org> > > Do you want me to take the patch via battery-2.6.git? > Hi Sebastian, thanks a lot for the Ack. I am not sure if it is better to submit all patches together or through individual maintainer trees. Let's wait for Andrew's suggestion how we should proceed. Thanks, Guenter
diff --git a/drivers/power/reset/restart-poweroff.c b/drivers/power/reset/restart-poweroff.c index 3e51f8d..edd707e 100644 --- a/drivers/power/reset/restart-poweroff.c +++ b/drivers/power/reset/restart-poweroff.c @@ -20,7 +20,8 @@ static void restart_poweroff_do_poweroff(void) { - arm_pm_restart(REBOOT_HARD, NULL); + reboot_mode = REBOOT_HARD; + machine_restart(NULL); } static int restart_poweroff_probe(struct platform_device *pdev)