[BUG] Error: "Please select no more than one LCD controller option." Wanhao D6

[Deneteus]

### Bug Description

PlatformIO Build fails due to multiple errors when compiling using the Wanhao D6 Example configuration files included with Marlin 2.0

### My Configurations

I used the stock example configuration files for the Wanhao D6 and they failed to compile.
\Marlin\config\examples\Wanhao\Duplicator 6\

Steps to Reproduce

1. Copy the Wanhao D6 example configuration files to the Marlin project folder.
2. Open the Marlin project in Atom and confirm the configuration.h file is the correct file.
3. Under Platform IO Home choose Inspect and make sure the Project Environment is megatmega2560
4. Choose PlatformIO on the tool bar and select Build.
5. Watch as the Sanity Check Error occurs and the build fails.

Expected behavior: Example Configuration should compile with no errors.

Actual behavior: Multiple sanity check errors occur related to multiple LCD controller options.

Additional Information

Just to make sure my config had not been corrupted from the previous version I wiped out my Marlin folder and redownloaded it.

The software updates and updates to libraries had already been done before this.
The printer is currently running an older version of Marlin 2.0 from earlier this year.

I am assuming its this line that's causing the issue.

[thisiskeithb]

Both ULTIPANEL and U8GLIB_SSD1306 are defined in that config. Comment out ULTIPANEL and see if your LCD still works after updating.

[Deneteus]

Both ULTIPANEL and U8GLIB_SSD1306 are defined in that config. Comment out ULTIPANEL and see if your LCD still works after updating.

Disabling Ultipanel causes this:

I disabled advanced_pause in the advanced config file and I was able to complete the build.

There was one error with the LiquidCrystal_SR1W.cpp and one with the DogM.cpp.

I was able to fix the Liquid Crystal and DogM error by changing the slash / in the line to another character. An entire schematic in that file is a bit much. LOL

[boelle]

Configurations, please
Please ZIP up your Configuration.h and Configuration_adv.h files (as
requested in the Issue template) and drop them into your next reply.
We'll check them over and see if anything is amiss.

[thisiskeithb]

Configurations, please
Please ZIP up your Configuration.h and Configuration_adv.h files (as
requested in the Issue template) and drop them into your next reply.
We'll check them over and see if anything is amiss.

They are using the stock D6 configs:

I used the stock example configuration files for the Wanhao D6 and they failed to compile.
\Marlin\config\examples\Wanhao\Duplicator 6

[boelle]

They are using the stock D6 configs:

did not read that, just saw a missing zip

[Deneteus]

I figured there would be no point in attaching the stock files.
Uploading Wanhao D6 Marlin 2.0 Stock Example Configs.zip…

[boelle]

as said i did not read the post just looked if something was attached

but yeah standard config not working is kind of embarresing,

[Deneteus]

I tested using the firmware as compiled after those two changes and it failed.

[shitcreek]

The problem is line 1991: // #define U8GLIB_SSD1306
I've made the correction. Thanks.

[thisiskeithb]

Does this work with only ULTIPANEL defined? I thought U8GLIB_SSD1306 was required since this printer has an OLED screen.

There are extra options underneath U8GLIB_SSD1306 in the D6 config compared to the stock config which makes me think it's still required:

Marlin/config/examples/Wanhao/Duplicator 6/Configuration.h

Lines 1991 to 1994 in f646499

	#define U8GLIB_SSD1306
	#define LCD_WIDTH 22
	#define LCD_HEIGHT 5
	#define LCD_RESET_PIN 5

Stock config:

Marlin/config/default/Configuration.h

Lines 1978 to 1982 in f646499

	//
	// SSD1306 OLED full graphics generic display
	//
	//#define U8GLIB_SSD1306

[shitcreek]

It compiled without issue. @Deneteus mind testing this out to confirm it works with U8GLIB_SSD1306 undefined?

[thisiskeithb]

Digging back into the origins of this config, it came with both those LCD types enabled from the start. I wonder if something changed in SanityCheck.h for this to start throwing errors or if this config ever worked in the first place.

[Deneteus]

@shitcreek I had not tested it with the SSD1306 OLED LCD undefined.
@thisiskeithb The motherboard for the D6 is an Ultimaker 2 clone. It is based on the ATMega2560 microcontroller.

[thisiskeithb]

If that doesn't work, it seems this Marlin port is popular and they have a different LCD defined:

//
// TinyBoy2 128x64 OLED / Encoder Panel
//
#define OLED_PANEL_TINYBOY2
#define LCD_RESET_PIN 5

[Deneteus]

If that doesn't work, it seems this Marlin port is popular and they have a different LCD defined:

//
// TinyBoy2 128x64 OLED / Encoder Panel
//
#define OLED_PANEL_TINYBOY2
#define LCD_RESET_PIN 5

I'll test that in a minute!

[thisiskeithb]

This config builds with a single LCD defined now (as expected), so we just need confirmation that it works. 😄

Here's the binary I built while testing with OLED_PANEL_TINYBOY2 enabled: Wanhao-D6-Marlin-2.0.x-f646499.hex.zip

[Deneteus]

Got it! Just gonna go flash it!

[Deneteus]

I am waiting on a netbook that I use to update firmware to finish restarting from a Windows Update. It had been going for almost 8hrs. LOL

[Deneteus]

@thisiskeithb Alright it flashed. LCD works. I just need to build it with the Tiny LCD settings and different settings for a Bondtech extruder.

[thisiskeithb]

Thanks for finding this bug 🙂

PR #16117 will fix the default D6 config.

Edit: OLED screens sure are nice.

[Deneteus]

Do you know if there is an issue related to this?

[Deneteus]

I got my printer updated with all of the changes I needed. Rob's config files had some tweaks to the default feed rates, adaptive step smoothing and power loss recovery.

I enabled S_CURVE_acceleration, Bondtech ESTEPS, Tuned A4988 Stepper Defaults, Raised the max temp since I upgraded to Microswiss All Metal.

Deneteus Marlin 2.0 D6 Configs.zip

[shitcreek]

unused variable warnings can be ignored. Ideally we should haveisHeatnested inside a condition.

[github-actions]

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.