Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add proper support for Wyze bulbs #10196

Closed
wants to merge 2 commits into from

Conversation

Quantum-cross
Copy link

@Quantum-cross Quantum-cross commented Dec 20, 2020

Description:

This adds support for WYZE bulbs. I've previously submitted a template for these bulbs in the past -- but they always had the issue of being very dimly lit even when "off".

It turns out that you need to put the CT pin into a high impedance state in order to fully power down the LEDs.

Checklist:

  • The pull request is done against the latest dev branch
  • Only relevant files were touched
  • Only one feature/fix was added per PR and the code change compiles without warnings
  • The code change is tested and works on Tasmota core ESP8266 V.2.7.4.9
  • The code change is tested and works on Tasmota core ESP32 V.1.0.5-rc4
  • I accept the CLA.

NOTE: The code change must pass CI tests. Your PR cannot be merged unless tests pass

@arendst
Copy link
Owner

arendst commented Dec 20, 2020

I see through what hoops you had to jump to get this simple fix inserted.

As I want to get rid of internal templates I will experiment with another means of selecting specific device code selection by implementing a specific GPIO.

Hold on.

arendst added a commit that referenced this pull request Dec 20, 2020
- Support for GPIO option selection
- Gpio ``Option_a1`` enabling PWM1 high impedance if powered off and no fading as used by Wyze bulbs (#10196)
@arendst
Copy link
Owner

arendst commented Dec 20, 2020

Now this is implemented and can be enabled by selecting any GPIO as Option_a1.

The current template:

Wyze WLPA19 A19 800lm        {"NAME":"Wyze Bulb","GPIO":[0,0,0,0,0,0,0,0,0,416,417,0,0,0],"FLAG":0,"BASE":48}

can be changed to:

Wyze WLPA19 A19 800lm        {"NAME":"Wyze Bulb","GPIO":[5728,0,0,0,0,0,0,0,0,416,417,0,0,0],"FLAG":0,"BASE":48}

to make it work.

Let me know if it works indeed.

@Quantum-cross
Copy link
Author

I have commented your commit to note what needs to be changed for it to work. It needs to be PWM2 that is changed to input.

@arendst
Copy link
Owner

arendst commented Dec 20, 2020

Thx! Will close this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants