commit 2316afa8468050c96699b71acf5084fbee3a762f
author Brendan Jackman <brendan.jackman@arm.com> Tue Mar 28 14:56:33 2017 +0100
committer Brendan Jackman <bjackman@users.noreply.github.com> Tue Mar 28 14:59:46 2017 +0100
tree d1a63f4cb4d30d870d9ea8158efc93a7acef08bb
parent 7f1121d8db4d7ac857ecaab457574e21bbc2f14d

energy_model: Don't allow idle_states=None in PowerDomain class

This will cause an error in EnergyModel._guess_idle_states. Should use []
instead.

libs/utils/energy_model.py

diff --git a/libs/utils/energy_model.py b/libs/utils/energy_model.py
index 287fed7..7517dd2 100644
--- a/libs/utils/energy_model.py
+++ b/libs/utils/energy_model.py
@@ -202,6 +202,8 @@
     :type cpus: tuple(int)
     """
     def __init__(self, idle_states, cpu=None, children=None):
+        if idle_states is None:
+            raise ValueError('idle_states cannot be None (but may be empty)')
         super(PowerDomain, self).__init__(cpu, children)
         self.idle_states = idle_states