On 09/09/2014 07:56 AM, Mugunthan V N wrote: > On Tuesday 09 September 2014 05:45 PM, Nishanth Menon wrote: >> On Tue, Sep 9, 2014 at 6:29 AM, Mugunthan V N <mugunthanvnm@xxxxxx> wrote: >>> >>>> Sorry this is now blocked inside TI. could you explain the testing done >>>> for sleep state? did you attempt sleep mode before testing this? >>>> >>> >>> I have not tested sleep mode, just tested boot and ping test. >> >> I must NAK then for the reason that this is a patch that is just build >> tested and is not tested for the feature it claims to introduce. >> >> > > Is suspend/resume supported in mainline, i was not able to test it. > > root@dra7xx-evm:~# echo mem > /sys/power/state > -sh: echo: write error: Invalid argument > > I was not able to use rtc wake also as the rtc device is not created. Right.. that is why I was curious how you posted a patch that supposedly supports sleep pin configurations in a kernel that does not support suspend-resume? Hopefully we will have it supported in 3.18 kernel[1] - still need out of tree wakeup timer patch[2] for testing though.. If you have at least tested on relevant branch which has the feature expected for 3.18, I might still be OK with the patch. [1]http://marc.info/?l=linux-kernel&m=141019565513129&w=2 and http://marc.info/?l=linux-kernel&m=141019566913133&w=2 [2] details in http://marc.info/?l=linux-kernel&m=140871637106555&w=2 -- Regards, Nishanth Menon -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html