On Wed 2009-11-11 14:57:05, Wu Zhangjin wrote:
> (Add CC to Rafael J. Wysocki, Len Brown and Pavel Machek)
> This patch add basic suspend support for loongson2f family machines,
> loongson2f have a specific feature: when we set it's frequency to ZERO,
> it will go into a wait mode, and then can be waked up by the external
> interrupt. so, if we setup suitable interrupts before putting it into
> wait mode, we will be able wake it up whenever we want via sending the
> relative interrupts to it.
> These interrupts are board-specific, Yeeloong2F use the keyboard
> interrupt and SCI interrupt, but LingLoong and Fuloong2F use the
> interrupts connected to the processors directly. and BTW: some old
> LingLoong and FuLoong2F have no such interrupts connected, so, there is
> no way to wake them up from suspend mode. and therefore, please do not
> enable the kernel support for them.
> The board-specific support will be added in the coming patches.
> Signed-off-by: Wu Zhangjin <firstname.lastname@example.org>
Comments are slighlty "interesting", but otherwise it looks ok.
> + /* stop all perf counters */
> + stop_perf_counters();
This is not exactly useful comment, right?
> + /* mach specific suspend */
> + mach_suspend();
> + /* mach specific resume */
> + mach_resume();
It is probably ok, but you may want to avoid them in future.