From: anarsoul@gmail.com (Vasily Khoruzhick)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 5/8] ARM: PXA: Z2: Add poweroff function
Date: Tue, 15 Mar 2011 17:57:08 +0200 [thread overview]
Message-ID: <201103151757.13374.anarsoul@gmail.com> (raw)
In-Reply-To: <AANLkTi=FggbKjaLMU9rWG+cG=Jq18tDu8KHHy4KZLJe9@mail.gmail.com>
On Tuesday 15 March 2011 17:49:26 Eric Miao wrote:
> 2011/3/13 Marek Vasut <marek.vasut@gmail.com>:
> > On Saturday 12 March 2011 21:34:01 Vasily Khoruzhick wrote:
> >> On Saturday 12 March 2011 15:16:19 Marek Vasut wrote:
> >> > Besides, this is nonsense, you're doing a hack and you're trying to
> >> > justify it. You cannot powerdown the device, period. You should rather
> >> > patch the bootloader to implement this para-powerdown behaviour than
> >> > bloat kernel with this workaround.
> >>
> >> And, btw, there's same code in mainline kernel for zeus machine.
> >
> > Doesn't mean it's correct. CCing maintainer. Eric, can you also please
> > comment on it ? I really don't like the idea of abusing poweroff and
> > making people misunderstand.
>
> I'm actually quite OK with the idea of using deep sleep as a way
> for shutdown. As some of the device has no way for a "correct"
> shutdown without pulling out the battery. A fake one is at least
> better than none, provided it works as expected.
So should I send updated version of this patch? (Need to clear PSPR before
going to deepsleep)
Regards
Vasily
next prev parent reply other threads:[~2011-03-15 15:57 UTC|newest]
Thread overview: 60+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-03-11 10:23 [PATCH 1/8] ARM: PXA: Z2: Use only power button as wake source Vasily Khoruzhick
2011-03-11 10:23 ` [PATCH 2/8] ARM: PXA: Z2: Enable OHCI on Zipit Z2 Vasily Khoruzhick
2011-03-11 14:56 ` [PATCH v2 " Vasily Khoruzhick
2011-03-11 21:42 ` Marek Vasut
2011-03-11 21:49 ` Vasily Khoruzhick
2011-03-11 21:50 ` Marek Vasut
2011-03-11 21:55 ` Vasily Khoruzhick
2011-03-11 22:13 ` Marek Vasut
2011-03-11 22:34 ` Vasily Khoruzhick
2011-03-12 0:07 ` Marek Vasut
2011-03-12 11:32 ` Vasily Khoruzhick
2011-03-12 13:28 ` Marek Vasut
2011-03-13 10:18 ` Pavel Machek
2011-03-13 12:29 ` Marek Vasut
2011-03-13 17:10 ` Pavel Machek
2011-03-13 18:34 ` Marek Vasut
2011-03-13 22:15 ` Vasily Khoruzhick
2011-03-11 21:41 ` [PATCH " Marek Vasut
2011-03-11 10:23 ` [PATCH 3/8] ARM: PXA: Z2: Fix battery pdata Vasily Khoruzhick
2011-03-11 21:39 ` Marek Vasut
2011-03-11 10:23 ` [PATCH 4/8] ARM: PXA: Z2: Add mtd partition with logo Vasily Khoruzhick
2011-03-11 21:39 ` Marek Vasut
2011-03-11 22:31 ` Vasily Khoruzhick
2011-03-12 0:05 ` Marek Vasut
2011-03-15 14:39 ` Pavel Machek
2011-03-16 14:11 ` Marek Vasut
2011-03-16 16:51 ` Pavel Machek
2011-03-11 10:23 ` [PATCH 5/8] ARM: PXA: Z2: Add poweroff function Vasily Khoruzhick
2011-03-11 21:39 ` Marek Vasut
2011-03-11 21:53 ` Vasily Khoruzhick
2011-03-11 22:08 ` Marek Vasut
2011-03-11 22:12 ` Vasily Khoruzhick
2011-03-11 22:16 ` Marek Vasut
2011-03-11 22:30 ` Vasily Khoruzhick
2011-03-12 0:05 ` Marek Vasut
2011-03-12 2:29 ` Eric Miao
2011-03-12 8:06 ` Vasily Khoruzhick
2011-03-12 11:06 ` Marek Vasut
2011-03-12 11:30 ` Vasily Khoruzhick
2011-03-12 13:16 ` Marek Vasut
2011-03-12 20:33 ` Vasily Khoruzhick
2011-03-12 20:35 ` Marek Vasut
2011-03-12 20:34 ` Vasily Khoruzhick
2011-03-12 22:29 ` Marek Vasut
2011-03-15 15:49 ` Eric Miao
2011-03-15 15:57 ` Vasily Khoruzhick [this message]
2011-03-15 16:42 ` Eric Miao
2011-03-15 20:11 ` [PATCH v2] " Vasily Khoruzhick
2011-07-05 6:58 ` Eric Miao
2011-03-15 14:41 ` [PATCH 5/8] " Pavel Machek
2011-03-11 10:23 ` [PATCH 6/8] ARM: PXA: Z2: Fix libertas init/tear down sequences Vasily Khoruzhick
2011-03-11 21:37 ` Marek Vasut
2011-03-11 21:41 ` Vasily Khoruzhick
2011-03-11 21:51 ` Marek Vasut
2011-03-11 10:23 ` [PATCH 7/8] ARM: PXA: Z2: Keep kbd backlight disabled by default Vasily Khoruzhick
2011-03-11 21:36 ` Marek Vasut
2011-03-11 10:23 ` [PATCH 8/8] ARM: PXA: Z2: Add default triggers for LEDs Vasily Khoruzhick
2011-03-11 21:36 ` Marek Vasut
2011-03-11 21:43 ` [PATCH 1/8] ARM: PXA: Z2: Use only power button as wake source Marek Vasut
2011-03-13 10:15 ` Pavel Machek
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=201103151757.13374.anarsoul@gmail.com \
--to=anarsoul@gmail.com \
--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 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).