[PATCH v10 0/4] This suspend patch is only support cut off the power of cpu and some external
zyw at rock-chips.com
Mon Dec 1 17:18:04 PST 2014
On 12/02/2014 06:08 AM, Doug Anderson wrote:
> On Mon, Dec 1, 2014 at 11:51 AM, Kevin Hilman <khilman at kernel.org> wrote:
>> Chris Zhong <zyw at rock-chips.com> writes:
>>> devices, since we still lack power_domain driver, so the other power rail
>>> of rk3288 need keep power on.
>>> I have tested it on rk3288-evb board, atop next-20141112. goto suspend by type
>>> "echo mem > /sys/power/state", vdd_cpu is about 0mv by measuring, so it can be
>>> determined in sleep mode, then press power button to wakeup it.
>> I tested this on top of today's linux-next (next-20141201) and it
>> suspends, but doesn't wake up from any of the button presses. What
>> wakeup sources are configured for the rk3288-evb-rk808?
> Just to close the loop (I talked with Kevin over IM about this, too):
> I have a huge description of how I tested this as part of my patch at
> <https://patchwork.kernel.org/patch/5414941/>. Chris: I think Kevin
> has asked you several times to include information like this in your
> cover letter. Please, please, please can you try to remember to do
I wrote it to v8 cover letter, but I found that they are all landed on
linux-next now, so I remove they from v9, v10.
[v5,1/6] ARM: rockchip: convert to regmap and use pmu syscon if available
[v5,2/6] ARM: rockchip: add option to access the pmu via a phandle in smp_operations
[v5,3/6] ARM: dts: rockchip: add pmu references to cpus nodes
[v5,4/6] ARM: rockchip: add basic smp support for rk3288
[v5,5/6] ARM: dts: rockchip: add intmem node for rk3288 smp support
[v5,6/6] ARM: dts: rockchip: add reset for CPU nodes
[v3] usb: dwc2: add bus suspend/resume for dwc2
But I missed this 3 patches, since I thought they just for fix timer
disorder when I use coreboot.
Doug said they are related to SMP and related to suspend/resume.
1. https://patchwork.kernel.org/patch/5051881/ - clocksource:
arch_timer: Allow the device tree to specify uninitialized timer
2. https://patchwork.kernel.org/patch/5363671/ - clocksource:
arch_timer: Fix code to use physical timers when requested
3. https://patchwork.kernel.org/patch/5382141/ - ARM: dts: rk3288: add
I'm not sure whether we need these 2 patches.
4. Revert (b77d439 ARM: dts: rockchip: temporarily disable smp on
5. https://patchwork.kernel.org/patch/5325111/ - usb: dwc2: resume
root hub when device detect with suspend state
> For those that don't want to click on my link, I'll include the
> relevant bits here:
> Total patches atop that version of Linux were:
> 1. https://patchwork.kernel.org/patch/5051881/ - clocksource:
> arch_timer: Allow the device tree to specify uninitialized timer
> 2. https://patchwork.kernel.org/patch/5363671/ - clocksource:
> arch_timer: Fix code to use physical timers when requested
> 3. https://patchwork.kernel.org/patch/5382141/ - ARM: dts: rk3288: add
> 4. Revert (b77d439 ARM: dts: rockchip: temporarily disable smp on
> 5. https://patchwork.kernel.org/patch/5325111/ - usb: dwc2: resume
> root hub when device detect with suspend state
> 6. https://patchwork.kernel.org/patch/5410611/ - ARM: rockchip: add
> suspend and resume for RK3288
> 7. https://patchwork.kernel.org/patch/5410621/ - ARM: rockchip: Add
> pmu-sram binding
> 8. https://patchwork.kernel.org/patch/5410631/ - ARM: dts: add RK3288
> suspend support
> 9. https://patchwork.kernel.org/patch/5410641/ - ARM: dts: rockchip:
> add suspend settings for rk3288-evb-rk808
> It looks like my pinctrl patches might be dropped due to cross
> dependency problems, so tomorrow's linux-next will probably also need
> (https://patchwork.kernel.org/patch/5344551/ - pinctrl: rockchip:
> Handle wakeup pins).
> I've also got a local hack to the Rockchip "pm.c" to replace the usage
> of "PMU_ARMINT_WAKEUP_EN" with 0x0e. There seems to be some sort of
> ARM Interrupt waking us up all the time right when we go to sleep and
> the above will hack it so that only GPIOs + SDMMC Card Detect can wake
> us up. Someone should track down what's going on there, but for now
> I've used the hack to prove that the basic code actually works.
More information about the Linux-rockchip