On 2016-11-17 16:57, Leo Yan wrote:
Hi Vikram,
On Thu, Nov 17, 2016 at 03:06:24PM -0800, Vikram Mulukutla wrote:
Hi Leo,
[...]
Thanks for doing this; just one minor comment on the second patch. If you're testing with this series, please test suspend/resume, I may have seen a problem when I did a backport to our own internal 3.18 tree in the resume path, I didn't have time to debug then. I'll be happy to help with any debugging.
Thanks for reminding the suspend/resume testing. I did this testing on Hikey + kernel 4.4 with below command, have not found issue after run 10 times. So if you think other thing I should note, please let me know.
echo +3 > /sys/class/rtc/rtc0/wakealarm; echo mem > /sys/power/state
Thanks, Leo Yan
Cool, thanks for testing that. As long as you don't see any new warnings about overflowing counters in the kernel log, things look good.
[...]
Thanks, Vikram