Hi Tony,
On Mon, 29 Sep 2014, Tony Lindgren wrote:
- Paul Walmsley paul@pwsan.com [140929 10:14]:
PM: chip off except CORE via suspend: Pass ( 1/ 1): 3730beaglexm
PM: chip off except CORE via dynamic idle: Pass ( 1/ 1): 3730beaglexm
Most likely the two tests above should now also work also for 37xxevm, 3530es3beagle and 3530es31beagle in case you want to enable that.
I think this might be an issue of my own inability to communicate clearly what's being tested here and why :-)
That "chip off except CORE" PM validation test is only needed due to the 3630/37xx ES1.0 errata that forces us to disable CORE off-mode on that revision:
---------- echo 1 > /debug/pm_debug/enable_off_mode [ 60.351837] omap3_pm_off_mode_enable: Core OFF disabled due to errata i583 ----------
(from http://www.pwsan.com/omap/testlogs/test_v3.17-rc6/20140921213558/pm/3730beag...)
If we were to enable that test for the other four OMAP3 boards, they would probably all show up as failing, since they are all able to enter core-off mode successfully :-)
Maybe you have some ideas for how this can be communicated better in the test results? One thing that would probably help is if the board was renamed to "3730es10beaglexm" or something like that. Also maybe mentioning the fact that the test is only needed due to errata, like:
----------- PM: chip off (except CORE, due to errata) via dynamic idle: Pass ( 1/ 1): 3730es10beaglexm -----------
At some point, will probably also generate an HTML version of the summary, where the test name and board names can link to some explanatory information about what is being tested, and why.
Anyway, any thoughts are welcome.
PM: chip off via suspend: Pass ( 4/ 4): 37xxevm, 3530es3beagle, 3530es31beagle, 3730es12beaglexm
PM: chip off via dynamic idle: Pass ( 4/ 4): 37xxevm, 3530es3beagle, 3530es31beagle, 3730es12beaglexm
Since looks like these are working finally :)
Thanks largely to your hard work! Hooray!
- Paul