Lutron. Bugfix callback registration.

[pvizeli]

Description:

• Simplify stuff
• Move callback subscription to EVENT_HOMEASSISTANT_START

Related issue (if applicable): fixes #6739

CC #7019

[mention-bot]

@pvizeli, thanks for your PR! By analyzing the history of the files in this pull request, we identified @thecynic and @fabaff to be potential reviewers.

[balloob]

Is this async friendly?

[pvizeli]

Yes it add only a callback into dict

[pvizeli]

Like the other lutron component.

[thecynic]

Sorry for the lack of updates folks. Been busy at work and now on vacation. I'll try to get back to this when I'm back if you don't take care of it beforehand. The pylutron library itself is not guaranteed to be async safe.

…

On Apr 11, 2017 11:57 AM, "Paulus Schoutsen" ***@***.***> wrote: ***@***.**** commented on this pull request. ------------------------------ In homeassistant/components/lutron.py <#7042 (comment)> : > """Initialize the device.""" self._lutron_device = lutron_device self._controller = controller self._area_name = area_name - self.hass = hass - self.object_id = '{} {}'.format(area_name, lutron_device.name) - + @asyncio.coroutine + def async_add_to_hass(self): + """Register callbacks.""" self._controller.subscribe(self._lutron_device, self._update_callback) Is this async friendly? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#7042 (review)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABcEMVV4OwJYC0Sp992hIsM__nKWelDaks5ru6MBgaJpZM4M53FJ> .