syssi / esphome-yeelight-ceiling-light

ESPHome custom firmware for some Yeelight Ceiling Lights
Apache License 2.0
150 stars 22 forks source link

Add Yeelight YLXD50YL (ceiling20) support #20

Closed ExPeacer closed 3 years ago

syssi commented 3 years ago

Is the nightlight supported or should it be removed from the configuration? You could add a sensor to monitor the ADC for completeness. If you are super keen you could try to identify the GPIOs of the i2c bus. The eeprom should be discoverable here.

syssi commented 3 years ago

Could you add this configuration to the ci.yaml? :-)

ExPeacer commented 3 years ago

Is the nightlight supported or should it be removed from the configuration? You could add a sensor to monitor the ADC for completeness. If you are super keen you could try to identify the GPIOs of the i2c bus. The eeprom should be discoverable here.

NightLight supported via GPIO23. Tried to add ADC, but the ADC pin simply not connected anywhere in my PCB. Tried to add all ADC-capable pin from ESP32 to config to test it, but got back non-relevant values. Anyway, I don't know any use case where an ADC value would be helpful in a lamp, so I just let it go. Same for i2c, this device not using that. I think if the lamp can do the same thing like the original FW, we are good. I will tinker a little with BLE, but with Single Core ESP32s maybe its a little overkill. There is some problem with the color control - if I want to change the the color temperature in ceiling light, it's turning off the lamp, and turning on the ambient light, with a 'yeelight_light_ceiling20 ceiling light' - This light does not support color mode Color temperature! message in log. Will try to figure it out.

ExPeacer commented 3 years ago

There is some problem with the color control - if I want to change the the color temperature in ceiling light, it's turning off the lamp, and turning on the ambient light, with a 'yeelight_light_ceiling20 ceiling light' - This light does not support color mode Color temperature! message in log. Will try to figure it out.

I just need to update HA... https://github.com/home-assistant/core/pull/54595

Everything works fine now!

syssi commented 3 years ago

Thanks for your support! The PR is ready to go. Please feel free to improve your configuration at another PR!