Hi Hoan, On Wed, Dec 28, 2016 at 2:40 AM, Hoan Nguyen An <na-hoan@xxxxxxxxxxx> wrote: > I have retested PM, recheck the PM test procedures. > I am using board H3 ES1.1, procedures involving i2cset is right. > > With the same test procedures and setting for this version and previous > version(v4.9-rc2), but the results are different. > > (Only U-boot versions are different for two version) > > I am using U-boot version(U-Boot 2015.04 (Sep 13 2016 - 11:32:36)). > > I attach the test log file of PM. Please have a look at the log. > > On 27/12/16 22:02, Geert Uytterhoeven wrote: >> On Tue, Dec 27, 2016 at 11:35 AM, Hoan Nguyen An <na-hoan@xxxxxxxxxxx> >>> My name is Hoan from Jinzai Solution. >>> We have tested the linux stable v4.9 for Gen3 Salvator-X (H3) >>> >>> So we would like to report the summary of the failure. >>> 1 PM: It is not working, can't return to the system. NG New issue >>> root@linaro-nano:~# echo mem > /sys/power/state >>> [ 221.280712] PM: Syncing filesystems ... done. >>> [ 221.286474] Freezing user space processes ... (elapsed 0.001 seconds) >>> done. >>> [ 221.295013] Freezing remaining freezable tasks ... (elapsed 0.001 >>> seconds) done. >>> [ 221.303691] Suspending console(s) (use no_console_suspend to debug) >> >> Are you using R-Car H3 ES1.0? >> AFAIK, tbe newer firmware supports system suspend/resume on H3 ES1.1 only, >> and only when following the right procedure involving i2cset. > > I only set the following command for test. > root@linaro-nano:~# echo enabled > > /sys/devices/platform/soc/e6e68000.serial/tty/ttySC1/power/wakeup > root@linaro-nano:~# echo enabled > > /sys/devices/platform/soc/e6e88000.serial/tty/ttySC0/power/wakeup Wake-up through any other wake-up source than SW23 is not yet supported on Salvator-X boards, as the PSCI firmware doesn't know about those wake-up sources, and powers down the whole SoC during suspend to RAM. It may start to work once kernel support for suspend-to-idle has been added. > root@linaro-nano:~# echo mem > /sys/power/state > > Could you please tell me more details for using i2cset?. The official (cumbersome) procedure goes like this: - Set to PMIC to backup mode via i2c-tools command # i2cset -f -y 7 0x30 0x20 0x0F - (Suspend) Change SW23 to OFF - Request System Suspend to RAM # echo mem > /sys/power/state - (Resume) Change SW23 to ON Note that this doesn't work with current upstream, as the upstream DTS doesn't have a device node for i2c7 yet... Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds