Hi, On Wed, 2009-11-11 at 11:33 +0100, Pavel Machek wrote: > 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 <wuzhangjin@xxxxxxxxx> > > Comments are slighlty "interesting", but otherwise it looks ok. > > > + /* stop all perf counters */ > > + stop_perf_counters(); > > This is not exactly useful comment, right? > Will remove it later ;) the same to the following two. > > + /* mach specific suspend */ > > + mach_suspend(); > ... > > + /* mach specific resume */ > > + mach_resume(); > > > It is probably ok, but you may want to avoid them in future. > > ACK. > Pavel > Thanks! Wu Zhangjin
