From b54dc19b81d7586110a5ff7b0c4a5e04bc9f2bc6 Mon Sep 17 00:00:00 2001 From: Valentin Schneider Date: Wed, 26 Jul 2017 11:39:49 +0100 Subject: [PATCH] devlib/module/thermal: Fix thermal zone disabling Calling thermal.disable_all_zones() would raise an exception. I've changed a few things: * use self.zones.itervalues() in disable_all_zones to fix that exception * renamed zone.set_mode() to zone.set_enabled() --- devlib/module/thermal.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/devlib/module/thermal.py b/devlib/module/thermal.py index 4fa8e15..fa13fbb 100644 --- a/devlib/module/thermal.py +++ b/devlib/module/thermal.py @@ -61,8 +61,8 @@ class ThermalZone(object): value = self.target.read_value(self.target.path.join(self.path, 'mode')) return value == 'enabled' - def set_mode(self, enable): - value = 'enabled' if enable else 'disabled' + def set_enabled(self, enabled=True): + value = 'enabled' if enabled else 'disabled' self.target.write_value(self.target.path.join(self.path, 'mode'), value) def get_temperature(self): @@ -100,5 +100,5 @@ class ThermalModule(Module): def disable_all_zones(self): """Disables all the thermal zones in the target""" - for zone in self.zones: - zone.set_mode('disabled') + for zone in self.zones.itervalues(): + zone.set_enabled(False)