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

[FR] RRF M150 support #2216

Closed
xPew opened this issue Oct 11, 2021 · 15 comments · Fixed by #2241
Closed

[FR] RRF M150 support #2216

xPew opened this issue Oct 11, 2021 · 15 comments · Fixed by #2241
Labels
enhancement New feature or request

Comments

@xPew
Copy link
Contributor

xPew commented Oct 11, 2021

Is your feature request related to a problem? Please describe.
Seems like the firmware as is, can't handle RBG LED settings on RRF.

Describe the solution you'd like
Please modify the current code to be compatible with RRF.

Additional context
None of the button's works for now (r, g, b, w, on, off or the colour mixer)
https://marlinfw.org/docs/gcode/M150.html
https://duet3d.dozuki.com/Wiki/M150
If you have a solution, I have configured NeoPixels to test with on RRF 3.3

@xPew xPew added the enhancement New feature or request label Oct 11, 2021
@xPew
Copy link
Contributor Author

xPew commented Nov 7, 2021

Any plan to implement this @bigtreetech , @digant73, @Msq001?
I can help with testing if needed.

@bigtreetech
Copy link
Owner

Any plan to implement this @bigtreetech , @digant73, @Msq001? I can help with testing if needed.

Hello, So adding an X2 parameter to M150 for RRF can solve this problem?

@xPew
Copy link
Contributor Author

xPew commented Nov 8, 2021

Any plan to implement this @bigtreetech , @digant73, @Msq001? I can help with testing if needed.

Hello, So adding an X2 parameter to M150 for RRF can solve this problem?

Sadly no. I configured it based on the reccomendation to the RRF port, but seems like there is a syntax difference or something. The standalone commands from terminal works, but the predefined buttons in the LED menu dont.

@bigtreetech
Copy link
Owner

Can you help test whether M150 X2 R255 U255 B255 W0 P255 I255 command can control the color of neopixel to white?

@xPew
Copy link
Contributor Author

xPew commented Nov 8, 2021

Can you help test whether M150 X2 R255 U255 B255 W0 P255 I255 command can control the color of neopixel to white?

I can test it when I get home to my printer, but as far as I remember these commands worked from terminal perfectly. In fact I was only able to use the terminal to control my lights, so these should work.
In RRF config.g I configured M150 X2 T300:800:1250:250, as mentioned here: https://teamgloomy.github.io/neopixels.html
So M150 R U B … should be enough from terminal.

@bigtreetech
Copy link
Owner

Hello, I just conducted a test. In fact, the TFT firmware can normally control the color of neopixel now. Is your TFT firmware the latest version

@digant73
Copy link
Contributor

@xPew as reported by bigtreetech, it seems you are using an old TFT fw version (you also mentioned that from the LED menu you changed only r, g, b, w). Since some months now, the TFT fw also supports Neopixel. The LED menu gives you the possibility to set R, G, B, W, P, I.

@xPew
Copy link
Contributor Author

xPew commented Nov 11, 2021

Thanks, I will check it at the weekend than again. I last tried it a few weeks back with the latest at the time, but the menu didn't worked, just manual command's.
@bigtreetech which RRF version did you run your test?

@digant73
Copy link
Contributor

digant73 commented Nov 12, 2021

@xPew
The LED menu currently uses the gcode "M150 Rr Uu Bb Ww Pp Ii"
Which is the working gcode you send from the terminal?

@xPew
Copy link
Contributor Author

xPew commented Nov 20, 2021

@digant73 @bigtreetech Sorry for the delay, had a busy week, but now I did the testing...

The following format is working: M150 (X2) Rr Uu Bb Pp where:
Rnnn Red component, 0-255
Unnn Green component, 0-255
Bnnn Blue component, 0-255
Pnnn Brightness, 0-255

When I send a command like M150 R255 U0 B0 P255 with the web interface, the led turns red as expected. But when I press the red led button in the menu nothing happens. Oddly the terminal on the TFT cant change the color... The other features of the TFT works, so it communicates with the mainboard/fw, also I am using the latest firmware. Other commands from the TFT's terminal works fine, like M122.

If you can change the code send by the TFT to this in case of RRF I can test it prior merging.

digant73 added a commit to digant73/BIGTREETECH-TouchScreenFirmware that referenced this issue Nov 20, 2021
@digant73
Copy link
Contributor

digant73 commented Nov 20, 2021

@xPew please test PR #2241 and let me know if it works.
I used M150 X2 Rr Uu Bb Pp

@xPew
Copy link
Contributor Author

xPew commented Nov 20, 2021

@digant73 It's working as I'd expect it to work. Huge thanks for the fix. Predef colors, on, off and custom colors works perfectly now.
Any chance that you have some hotfixes for #1979 or #2141? :)

@digant73
Copy link
Contributor

good it fixed the issue. For the requested FRs, I think there is already someone else that could implement them (I don't have RRF, so I could not test it at all)

@xPew
Copy link
Contributor Author

xPew commented Nov 21, 2021

@digant73 #2141 is just a general UI change to let to use the grid view along with the list mode. Isn’t that possible?

Copy link

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked and limited conversation to collaborators Mar 28, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
3 participants