search

MarlinFirmware / Marlin

Marlin is an optimized firmware for RepRap 3D printers based on the Arduino platform. Many commercial 3D printers come with Marlin installed. Check with your vendor if you need source code for your specific machine.
https://marlinfw.org
GNU General Public License v3.0
16.28k stars 19.24k forks source link

[FR] set dual baud rate for creality v4 boards with DWIN #21150

Closed spacecabbie closed 3 years ago

spacecabbie commented 3 years ago

Description

I use this guide:(https://www.reddit.com/r/ender3/comments/ihg2k7/dwin_display_on_ender3_v2_does_not_work_with_a/gbc1f0u?utm_source=share&utm_medium=web2x&context=3) to enable 25000 baud rate between Octoprint and printer

Assuming this is ok and supported (it working fine for me) could this be made part of standard config for Ender 3 v2 (4.2.2 and 4.2.7 boards)

Feature Workflow

basically adding: (or simulair) `#if ENABLED(DWIN_LCD)

define LCD_SERIAL_PORT 1

define BAUDRATE 115200

endif`

Additional Information

Of course the above workaround works i am to wonder why its not in the config perhaps a limitation (only 1 baud rate can be set)? Since I prefer to keep everything in the config files and recommended I do not like this solution and prefer an "official" one.

IF this is a know workaround but is not included due to adverse effects or possible problems is it possible to include a comment line in either the config file or MarlinCore.cpp since it seems harmless and i am sure quite some people are applying it.

CRCinAU commented 3 years ago

Why not just install Meatpack and use that?

spacecabbie commented 3 years ago

Meatpack

This is the first time I heard about it and sure looks like a cool plugin still (from a quick glance) having the baud rate at 25000 would make that even quicker (unless there is some sort of max).

But, there are so many roads to rome, reason to request it is to check if this is a good way to go also meatball might be a good one. In the end if no changes need to happen that at least (i hope) a small info text can be addedd to the recommended way to go. (l love the excelend documentation of marlin.)

ellensp commented 3 years ago

In bugfix (and thus future releases) the LCD baud rate is already separated.

ie

#if HAS_DGUS_LCD
  #define LCD_SERIAL_PORT 3
  #define LCD_BAUDRATE 115200
github-actions[bot] commented 3 years ago

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.