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

[BUG] Example import-2.0.x/config/examples/delta/Tevo Little Monster wrong board defined #354

Closed
lemmingDev opened this issue Sep 24, 2020 · 15 comments

Comments

@lemmingDev
Copy link

Bug Description

The the example configuration for Tevo Little Monster, the MOTHERBOARD is defined as BOARD_MKS_GEN_L

The motherboard that comes with all Tevo Little Monsters is the MKS SBase, so the definition should be BOARD_MKS_SBASE

My Configurations

Configs.zip

Steps to Reproduce

  1. Use the provided Tevo Little Monster Configuration.h and Configuration_adv.h files

Expected behavior: [What you expect to happen]
#define MOTHERBOARD BOARD_MKS_SBASE

Actual behavior: [What actually happens]
#define MOTHERBOARD BOARD_MKS_GEN_L

Additional Information

There could be other changes needed as well,

@thisiskeithb
Copy link
Member

Looks like they've been broken since day one: MarlinFirmware/Marlin#12370

What else in the config should be updated once you change MOTHERBOARD to BOARD_MKS_SBASE?

@lemmingDev
Copy link
Author

Hi

That was the first thing I could find. After changing that, It compiled in PlatformIO and successfully installed on the board using the firmware.bin on microsd, and it added the FIRMWARE.CUR file, which is the indication that it uploaded.

After that however, I was unable to connect to the board using the supplied MKS TFT touchscreen.

I'll take another stab at it in the next few days after I've printed the stuff I need (as Smoothieware is currently working fine - but would like Marlin) and report back

@sjasonsmith
Copy link
Contributor

sjasonsmith commented Sep 25, 2020

I was surprised that we would have the wrong board, especially since it is in the direction of a downgrade to an 8-bit board.

This Tevo site seems to agree that they come with an SGEN SBASE:
https://help.tevo.cn/downloads/firmware/

@lemmingDev
Copy link
Author

lemmingDev commented Sep 25, 2020

From the site above you listed, it indeed says that the TEVO Little Monster comes with the SBASE (Not SGEN)

image

I guess it was someone wanting to run Marlin 1.X on the printer, and that only supported 8bit boards...

Anyway - If no one else has time to have a crack a it, I'll try getting the latest to Marlin to work on it in a few days

@sjasonsmith
Copy link
Contributor

You are right. I read one name and typed the other.

@sjasonsmith
Copy link
Contributor

@lemmingDev I attempted to update this for you, but I am not comfortable with the changes I have to make, without having the printer to test it.

Do you have a working configuration for a stock Little Monster?

The details that I am unsure about:

  • Microsteps? 32 or 64? (And the steps/mm derived from this)
  • Axis and endstop inversion?
  • BLTouch configuration

@lemmingDev
Copy link
Author

https://raw.githubusercontent.com/Homers3D/Tevo-Little-Monster/master/Control%20Board/config

I'm pretty sure all the info can be gleaned from there

Haven't had a chance to have a crack at it myself yet...

@boelle
Copy link

boelle commented Nov 22, 2020

@lemmingDev had a chance to have a crack at it?

@lemmingDev
Copy link
Author

I've got 6 weeks off coming up in 3 weeks.

Will take a look at it then...

@rhapsodyv
Copy link
Member

Is it a marlin bug or a configuration error in the examples repository? Show we move this to the Configurations repo?

@thisiskeithb
Copy link
Member

It's a configuration issue, but there may need to be two configs since Tevo updates the boards in their printers over time.

For example: The Tevo Tornado used to ship with an MKS Base, then MKS Gen-L, and now they are shipping with an MKS SGen-L.

@sjasonsmith sjasonsmith transferred this issue from MarlinFirmware/Marlin Dec 2, 2020
@thinkyhead
Copy link
Member

There are now V1 and V2 Tornado configs. Shouldn't be hard to add a V3 there and two more versions for the little monster too.

@lemmingDev
Copy link
Author

@lemmingDev I attempted to update this for you, but I am not comfortable with the changes I have to make, without having the printer to test it.

Do you have a working configuration for a stock Little Monster?

The details that I am unsure about:

  • Microsteps? 32 or 64? (And the steps/mm derived from this)
  • Axis and endstop inversion?
  • BLTouch configuration

Hi

I have successfully got the latest Marlin working on a stock TEVO Little Monster will all features working fine.

I had to edit a couple of pins from the MKS SBASE pins file as the thermistors were wrong for the way this was wired...

All required files included in the attached zip. Would be fantastic if this could be added to the official release

Marlin Files.zip

@thinkyhead
Copy link
Member

Unfortunately the file download link is broken.

@lemmingDev
Copy link
Author

Are you sure?
I was just able to download the zip file above and extract the 3 files from inside...
In any case, here they are again
Marlin.Files (1).zip

@lemmingDev lemmingDev reopened this Jan 26, 2022
thinkyhead added a commit that referenced this issue Feb 16, 2022
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

No branches or pull requests

6 participants