From 34531895a014545bdd46a0a32f552718f4b0ad99 Mon Sep 17 00:00:00 2001
From: Michael Auchter <a@phire.org>
Date: Sun, 27 Sep 2015 16:16:58 -0500
Subject: [PATCH] limitlessled: Use LedControllerPool

This change is in preparation for adapting this component to support multiple
LimitlessLED bridges. Ultimately LedControllerPool helps to maintain the
mandatory 100ms pauses across multiple controllers so messages are reliably
received.
---
 .../components/light/limitlessled.py          | 22 +++++++++++--------
 requirements_all.txt                          |  2 +-
 2 files changed, 14 insertions(+), 10 deletions(-)

diff --git a/homeassistant/components/light/limitlessled.py b/homeassistant/components/light/limitlessled.py
index 631fcbed64e..082a36faa4f 100644
--- a/homeassistant/components/light/limitlessled.py
+++ b/homeassistant/components/light/limitlessled.py
@@ -33,32 +33,34 @@ from homeassistant.components.light import (Light, ATTR_BRIGHTNESS,
 from homeassistant.util.color import color_RGB_to_xy
 
 _LOGGER = logging.getLogger(__name__)
-REQUIREMENTS = ['ledcontroller==1.0.7']
+REQUIREMENTS = ['ledcontroller==1.1.0']
 
 
 def setup_platform(hass, config, add_devices_callback, discovery_info=None):
     """ Gets the LimitlessLED lights. """
     import ledcontroller
 
-    led = ledcontroller.LedController(config['host'])
+    pool = ledcontroller.LedControllerPool([config['host']])
 
     lights = []
     for i in range(1, 5):
         if 'group_%d_name' % (i) in config:
-            lights.append(LimitlessLED(led, i, config['group_%d_name' % (i)]))
+            lights.append(LimitlessLED(pool, 0, i, config['group_%d_name' % (i)]))
 
     add_devices_callback(lights)
 
 
+# pylint: disable=too-many-instance-attributes
 class LimitlessLED(Light):
     """ Represents a LimitlessLED light """
 
-    def __init__(self, led, group, name):
-        self.led = led
+    def __init__(self, pool, controller_id, group, name):
+        self.pool = pool
+        self.controller_id = controller_id
         self.group = group
 
         # LimitlessLEDs don't report state, we have track it ourselves.
-        self.led.off(self.group)
+        self.pool.execute(self.controller_id, "off", self.group)
 
         self._name = name or DEVICE_DEFAULT_NAME
         self._state = False
@@ -132,12 +134,14 @@ class LimitlessLED(Light):
         if ATTR_XY_COLOR in kwargs:
             self._xy_color = kwargs[ATTR_XY_COLOR]
 
-        self.led.set_color(self._xy_to_led_color(self._xy_color), self.group)
-        self.led.set_brightness(self._brightness / 255.0, self.group)
+        self.pool.execute(self.controller_id, "set_color",
+                          self._xy_to_led_color(self._xy_color), self.group)
+        self.pool.execute(self.controller_id, "set_brightness",
+                          self._brightness / 255.0, self.group)
         self.update_ha_state()
 
     def turn_off(self, **kwargs):
         """ Turn the device off. """
         self._state = False
-        self.led.off(self.group)
+        self.pool.execute(self.controller_id, "off", self.group)
         self.update_ha_state()
diff --git a/requirements_all.txt b/requirements_all.txt
index 4610100b161..5ff29e9d8a5 100644
--- a/requirements_all.txt
+++ b/requirements_all.txt
@@ -14,7 +14,7 @@ astral==0.8.1
 phue==0.8
 
 # Limitlessled/Easybulb/Milight library (lights.limitlessled)
-ledcontroller==1.0.7
+ledcontroller==1.1.0
 
 # Chromecast bindings (media_player.cast)
 pychromecast==0.6.12