From: Kevin Hilman <khilman@kernel.org>
To: Chris Zhong <zyw@rock-chips.com>
Cc: heiko@sntech.de, dianders@chromium.org, mturquette@linaro.org,
Ian Campbell <ijc+devicetree@hellion.org.uk>,
Russell King <linux@arm.linux.org.uk>,
Rob Herring <robh+dt@kernel.org>, Pawel Moll <pawel.moll@arm.com>,
Mark Rutland <mark.rutland@arm.com>,
Linus Walleij <linus.walleij@linaro.org>,
linux-rockchip@lists.infradead.org, linux-gpio@vger.kernel.org,
linux-kernel@vger.kernel.org, Kumar Gala <galak@codeaurora.org>,
devicetree@vger.kernel.org, Tony Xie <xxx@rock-chips.com>,
linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH v6 0/7] This is the 1st version of suspend for RK3288.
Date: Wed, 29 Oct 2014 13:06:05 -0700 [thread overview]
Message-ID: <7hppda7iaa.fsf@deeprootsystems.com> (raw)
In-Reply-To: <1414583525-17395-1-git-send-email-zyw@rock-chips.com> (Chris Zhong's message of "Wed, 29 Oct 2014 19:51:58 +0800")
Hi Chris,
Chris Zhong <zyw@rock-chips.com> writes:
> RK3288 can shut down the cpu, gpu and other device controllers in suspend,
> and it will pull the GLOBAL_PWROFF pin to high in the final stage of the
> process of suspend, pull the pin to low again when resume.
I tried to test this on top of linux-next (next-20141029) and it doesn't
wake up from serial port activity.
Can you describe how to test this, as well as describe dependencies on
other out-of-tree patches, including pointers to where they've been
posted.
Also, please describe how you tested this and on which hardware
platforms. It's a big help to reviewers to know how it's been tested,
and for anyone with similar hardware to know what else it's been tested
on.
Thanks,
Kevin
next prev parent reply other threads:[~2014-10-29 20:06 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-10-29 11:51 [PATCH v6 0/7] This is the 1st version of suspend for RK3288 Chris Zhong
2014-10-29 11:52 ` [PATCH v6 5/7] ARM: rockchip: Add pmu-sram binding Chris Zhong
2014-10-29 11:52 ` [PATCH v6 6/7] ARM: dts: add RK3288 suspend support Chris Zhong
2014-10-29 11:52 ` [PATCH v6 7/7] ARM: dts: add suspend voltage setting for RK808 Chris Zhong
[not found] ` <1414583525-17395-8-git-send-email-zyw-TNX95d0MmH7DzftRWevZcw@public.gmane.org>
2014-10-29 15:40 ` Doug Anderson
2014-10-29 16:29 ` Javier Martinez Canillas
[not found] ` <545115DF.9070008-ZGY8ohtN/8pPYcu2f3hruQ@public.gmane.org>
2014-10-29 16:51 ` Doug Anderson
2014-10-30 0:32 ` Javier Martinez Canillas
2014-10-29 20:06 ` Kevin Hilman [this message]
2014-10-29 21:50 ` [PATCH v6 0/7] This is the 1st version of suspend for RK3288 Heiko Stübner
2014-10-29 22:26 ` Doug Anderson
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=7hppda7iaa.fsf@deeprootsystems.com \
--to=khilman@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=dianders@chromium.org \
--cc=galak@codeaurora.org \
--cc=heiko@sntech.de \
--cc=ijc+devicetree@hellion.org.uk \
--cc=linus.walleij@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-gpio@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-rockchip@lists.infradead.org \
--cc=linux@arm.linux.org.uk \
--cc=mark.rutland@arm.com \
--cc=mturquette@linaro.org \
--cc=pawel.moll@arm.com \
--cc=robh+dt@kernel.org \
--cc=xxx@rock-chips.com \
--cc=zyw@rock-chips.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).