Skip to content

Conversation

@esev
Copy link
Contributor

@esev esev commented Apr 18, 2021

Configure the TM1814 constant current at 22.5 mA as this seems to be a common value for several different RGBW LED chips.

I had originally intended this PR to use the strip.milliampsPerLed value, but was unable to find a way to make that work with the different voltages used for the TM1814 strip types (see comment below).

Related to: #1519

@esev esev marked this pull request as draft April 18, 2021 22:06
@esev
Copy link
Contributor Author

esev commented Apr 18, 2021

I'm thinking this may not be the right way to handle this setting. It looks like the milliampsPerLed value is intended to be the number of milliamps consumed by the entire LED module, not one single LED within the module. For the TM1814 this is likely supposed to be around (4*20mA)=80mA.

However this value doesn't play nicely with the "power units" calculations in WS2812FX::show. Those calculations seem to be based on a 5V RGB (3 LED) module that uses 53mA. Since the TM1814 modules are typically 12v/24v, these calculations won't match correctly if 80mA is used.

It might just be best to hard-code the constant current setting for the TM1814 modules at 20mA, since that seems to be at the low end of the common current value supported in the 5040 RGBW LEDs.

@mlangner84 WDYT? I think this is similar to what you had suggested in #1519

I'm going to mark this PR as a 'Draft' for now.

@makerglenn
Copy link

makerglenn commented Apr 19, 2021

Thanks for your efforts with the TM1814. I just found this thread and (in the background of my life) been trying to get the Costco Atomi String lights to work. I'm just mentioning this because this string of TM1814 LEDs are run at 5v not 12v or 24v. Hopefully you will take that into consideration with your calculations/development efforts. Hoping solid TM1814 support shows up in WLED soon. Thx again... I should mention for fwiw that the atomi outdoor "strip" of bulbs is 18 per strand.

@esev
Copy link
Contributor Author

esev commented Apr 19, 2021

@makerglenn do you know which LEDs are in the Atomi strip?

@davepl are these the same LEDs that you have? If so, would you be able to capture the constant current values used for these?

@esev
Copy link
Contributor Author

esev commented Apr 19, 2021

@davepl are these the same LEDs that you have? If so, would you be able to capture the constant current values used for these?

Never mind. Found it. Seems like this is the same 22.5mA.
https://www.youtube.com/watch?v=Z0krTGssXFA&t=20m5s

@makerglenn 22.0 (or 22.5) should likely work for these too.

@esev esev changed the title Configure TM1814 max current Use 22.5 for the TM1814 max current Apr 19, 2021
@esev esev marked this pull request as ready for review April 19, 2021 02:05
@makerglenn
Copy link

Yes those are the same LEDs as in the Dave's Garage video. I've cut off their proprietary controller and trying to drive it with WLED. I was trying to get the TM1814 string to work several weeks ago but to no avail - so I took it off my workbench for a while since I don't have a scope (s/w guy). I guess its time to dust them off and give your mods a try. Thx for the current info...

@esev
Copy link
Contributor Author

esev commented Apr 20, 2021

@makerglenn I think there is still going to be an issue with the refresh rates. I have a TM1814B which doesn't have a built-in pattern and doesn't require a higher refresh rate.

As an aside, I also ordered some of those Atomi lights. My plan is to add a level shifter on GPI02 of the internal wifi controller and bypass the HC89 chip, going directly from the level shifter to the green DO wire like shown here with a Pi. That's assuming I can flash WLED onto the internal wifi controller through the UART pins (Dave suggested it may have a pin layout similar to the ESP8266-12).

@makerglenn
Copy link

Yes the Atomi string lights use the non-B variant TM1814 that requires the refresh rate per your previous conversation. I successfully flashed the TYWE3S with the WLED ESP866_1M version. The 4 basic serial connections (Tx, Tx, 3.3v, Gnd) are nicely available and labeled on the edge of the Atomi PCB board. Easy to solder programming wires to then to the TYWE3S pins (ESP-like chip). I only had to ground out GPIO0 at the chip pin on powerup to get the TYWE3S into program mode. No issues programming the chip.... but that avenue does not work (as I found out later at the end of Dave's Garage video where he found out that there's a SC92 uP chip that actually drives the TM1814s! The TYWES3 only tells the SC92 what pre-programmed light sequence to play! I decided to breadboard WLED and try and get the string working without the Atomi logic in the way - then build my own controller (I have 6 strands of lights) or wait until the new QuinLED board comes out.

@esev
Copy link
Contributor Author

esev commented Apr 20, 2021

Is GPIO2 (TXD1) on the TYWE3S connected to the SC92, or GPIO1 (TXD0)? My plan was going to be to use GPIO2 and connect to the outgoing green wire (DO) through a level shifter.

Original: TYWE3S (GPIO1) -> SC92 -> On-board TM1814 -> Green (DO)
Planned: TYWE3S (GPIO2) -> Level Shifter -> Green (DO)

I guess I'll see more when I actually get mine and crack open the case.

@esev
Copy link
Contributor Author

esev commented Apr 20, 2021

@Aircoookie sorry for the side-discussions on this PR. I think the PR itself should be good to merge (if it's okay with you). 22.5 is the safer default value here as that appears to be what the native controller uses for several different TM1814 LED strips.

@Aircoookie Aircoookie merged commit eb99271 into wled:master Apr 20, 2021
@Aircoookie
Copy link
Member

Thank you :)
No worries about discussion, after all it is 100% related.

@coderabbitai coderabbitai bot mentioned this pull request Jan 3, 2026
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.

3 participants