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

SM2135: Support 15W bulbs using higher current #10634

Closed
arendst opened this issue Jan 20, 2021 · 9 comments
Closed

SM2135: Support 15W bulbs using higher current #10634

arendst opened this issue Jan 20, 2021 · 9 comments
Labels
enhancement Type - Enhancement that will be worked on fixed Result - The work on the issue has ended

Comments

@arendst
Copy link
Owner

arendst commented Jan 20, 2021

I have 3 models of RGBCCT light bulbs (5W, 13W and 15W) that use SM2135 LED driver.

Found out that I had to modify the current setting for these light bulbs in order to output the correct brigtness.

It would be good if the current setting for component SM2135 can be altered using command instead of compiling custom firmwares for each type, with the lowest current as the default.

Originally posted by @VaThInK in #10570

@close-issue-app

This comment has been minimized.

@close-issue-app close-issue-app bot added the template missing/incomplete Action - Template Missing or incomplete (issue will be closed) label Jan 20, 2021
@arendst arendst added the enhancement Type - Enhancement that will be worked on label Jan 20, 2021
arendst added a commit that referenced this issue Jan 20, 2021
Add support for SM2135 current selection using GPIO ``SM2135 DAT`` index (#10634)
@arendst arendst added fixed Result - The work on the issue has ended and removed template missing/incomplete Action - Template Missing or incomplete (issue will be closed) labels Jan 20, 2021
@arendst
Copy link
Owner Author

arendst commented Jan 20, 2021

The fix makes limited SM2135 current selection possible by using an index to GPIO SM2135 DAT like:

  • 1 = GRB 20 mA / CW 15 mA = Original LSC E14 bulb
  • 2 = BGR 20 mA / CW 15 mA
  • 3 = GRB 20 mA / CW 30 mA
  • 4 = BGR 20 mA / CW 30 mA
  • 5 = GRB 45 mA / CW 60 mA
  • 6 = BGR 45 mA / CW 60 mA

Be careful NOT TO SELECT higher currents on devices which do not support this based on their power supply.

@asozio
Copy link

asozio commented Aug 1, 2022

It would appear the bulb I am working on (Arlec Smart 10W 830lm RGBCCT Bulb GLD320HA) is configured as a GBR rather than any of the options above. Is it possible to update the SM2135 options to include this configuration as well?

@barbudor
Copy link
Contributor

barbudor commented Aug 1, 2022

I don't think it is going to be possible to update the list of index without a breaking change
However it may be possible to implement something based on SetOption37
Something like
If SO37 == 0 (default)
Use mapping and current from GPIO Index
Else
Use current from GPIO Index but mapping from SO37

@arendst
Does it makes sense ?
Would you like me to try a PR ?

@asozio
Copy link

asozio commented Sep 3, 2022

@barbudor Given that I only had one of these bulbs and it would require some modifications that use a SetOption I just rewired the connections from the G/B on the SM2135 to be switched (Pins 1/8). This way I can use the standard configuration of tasmota by @arendst above.

@barbudor
Copy link
Contributor

barbudor commented Sep 3, 2022

Great hack @asozio!

@asozio
Copy link

asozio commented Sep 3, 2022

Turns out I had to do another, so tested that one with the above SetOption37 suggestion. Looks like GLD130HA and potentially GLD320HA both would work with SetOption37 7 (as detailed in https://tasmota.github.io/docs/SetOption37/

@barbudor
Copy link
Contributor

barbudor commented Sep 3, 2022

Are these also based on SM2135 ? I was under the impression that the SM2135 was not using SetOption37
The template for GLD320HA is incomplete and I see none for the GLD130HA

@asozio
Copy link

asozio commented Sep 3, 2022

I thought I had updated the GLD320HA page, but must have not committed it.
The config I was using for the GLD320HA was:
{"NAME":"DMP-L1 Light","GPIO":[0,0,0,0,4067,0,0,0,0,4032,0,0,0,0],"FLAG":0,"BASE":18}

For the GLD132HA (Edison Screw version of the GLD130HA I believe) I am using:
{"NAME":"GLD130HA","GPIO":[0,0,0,0,4069,0,0,0,0,4032,0,0,0,0],"FLAG":0,"BASE":18} and "SetOption37 6"
although I am have not confirmed the LED current so would suggest lowering the SM2135 option from 6.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Type - Enhancement that will be worked on fixed Result - The work on the issue has ended
Projects
None yet
Development

No branches or pull requests

3 participants