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

Add support for MKS SGEN_L V2 motherboard #18822

Merged
merged 7 commits into from
Sep 1, 2020
Merged

Add support for MKS SGEN_L V2 motherboard #18822

merged 7 commits into from
Sep 1, 2020

Conversation

makerbase-mks
Copy link
Contributor

@makerbase-mks makerbase-mks commented Jul 28, 2020

Requirements

  • Add support for MKS SGEN_L V2 motherboard

Description

MKS has release SGEN_L V2 motherboard
V2.0 comparison V1.0

  • The CPU is LPC1769 120mhz
  • Support 2 way PWM FANs
  • The motherboard has 4KB EEPROM(AT24C32)
  • Reserve more IO for users

Related Issues

@anlupat
Copy link

anlupat commented Aug 11, 2020

@makerbase-mks please add extra comments on the pins file. About which connector to use for the fan(s). It is unclear to use the HE1 for the hotend fan when only using a single extruder. (if you only want to run the fan when the hotend is hot)

Function wise it is working great on my board in an Ender 3 with all the fancy things enabled.

@makerbase-mks
Copy link
Contributor Author

@anlupat
HE1 for hotend FAN when extruder = 1?

@anlupat
Copy link

anlupat commented Aug 13, 2020

@anlupat
HE1 for hotend FAN when extruder = 1?

I now have it setup like this, to me it is more clear which terminal or connector to use:

#if HOTENDS == 1
#ifndef FAN_PIN
#define FAN_PIN P1_04 // Fan1 Connector - Part Cooling Fan E0
#endif
#ifndef FAN1_PIN
#define FAN1_PIN P2_06 // HE1 Terminal - Hotend Fan (can be used with EXTRUDER_AUTO_FAN_TEMPERATURE (eq. #define E0_AUTO_FAN_PIN FAN1_PIN))
#endif
#else
#ifndef HEATER_1_PIN
#define HEATER_1_PIN P2_06 // HE1 Terminal - Extruder 1
#endif
#ifndef FAN_PIN
#define FAN_PIN P1_04 // Fan1 Connector - Part Cooling Fan E0
#endif
#ifndef FAN1_PIN
#define FAN1_PIN P2_04 // Fan2 Connector - Part Cooling Fan E1
#endif
#endif

@thisiskeithb
Copy link
Member

please add extra comments on the pins file.

This isn't done for other pins files when outputs are remapped since it's clear the remapping is happening right in the pins file, so I'm not sure we should start with this one.

If you're unsure of a pin's function, I'd recommend checking the manufacturer's pins diagram. Makerbase (and other manufacturers) host them in their respective github repos.

@anlupat
Copy link

anlupat commented Aug 13, 2020

then i would suggest to not use the HE1 terminal at all for a fan. Since nowhere in the documentation it is mentioned to use as a fan output.

@rhapsodyv
Copy link
Member

Users are beginning to receive their mks gen l 2 board, and started asking for marlin support.

Anything holding this PR that I can help?

#define E1_SERIAL_TX_PIN P1_17
#define E1_SERIAL_RX_PIN P1_17

#define Z2_SERIAL_TX_PIN P1_17
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are the Z2 serial pins actually needed?

I assume that when E1 is used as a Z2, it will automatically use the E1 serial pins as well.

@ils15
Copy link
Contributor

ils15 commented Aug 22, 2020

Hi
PR still holding?
why?

@Kokorone
Copy link

Is there a reason this PR isnt integrated yet that needs to be changed? I've been using these pins with no issue for a while now

@thisiskeithb
Copy link
Member

Is there a reason this PR isnt integrated yet that needs to be changed? I've been using these pins with no issue for a while now

There are other issues & PRs maintainers need to attend to.

@makerbase-mks
Copy link
Contributor Author

I think that support for MKS SGEN_L V2 motherboard can be added first, and new functions can be added and improved later. Now more and more users need it urgently to use the latest firmware.

@sjasonsmith
Copy link
Contributor

@thinkyhead as long as this isn't breaking the build, I think you should go ahead and merge it.
Even if there are some issues, it will let people start using the board and finding them.

@thinkyhead thinkyhead merged commit f375d9b into MarlinFirmware:bugfix-2.0.x Sep 1, 2020
thinkyhead pushed a commit to thinkyhead/Marlin that referenced this pull request Sep 2, 2020
@Doyle4
Copy link

Doyle4 commented Sep 12, 2020

Downloaded Marlin 2.0.6.1, downloaded 2.0.X bugfix, changed the files with a guide someone passed:

Download and replace this files: - It's files from Makerbase PR in to Marlin.
https://raw.githubusercontent.com/mks-viva/Marlin/4bdaec14dbb68006ef9067307143d45ea0080996/Marlin/src/pins/pins.h
https://raw.githubusercontent.com/mks-viva/Marlin/4bdaec14dbb68006ef9067307143d45ea0080996/Marlin/src/core/boards.h
https://raw.githubusercontent.com/mks-viva/Marlin/4bdaec14dbb68006ef9067307143d45ea0080996/Marlin/src/pins/lpc1769/pins_MKS_SGEN_L_V2.h
Path for replace you can see in URLs:

Marlin/src/pins/pins.h
Marlin/src/core/boards.h
Marlin/src/pins/lpc1769/pins_MKS_SGEN_L_V2.h

Im still getting failed..

Have to ask, how come a company has released a board thats not compatible to program?

@Kokorone
Copy link

I have no problems building and using the sgen l v2 on several printers right now with the current bugfix

@Doyle4
Copy link

Doyle4 commented Sep 12, 2020

Are you using the Bugfix 2.0.X to compile or swapping files over too 2.0.6.1?
Also, on a forum, some people are saying bugfix 2.0.X is 2.0.6 and not 2.0.6.1, is this correct? if its 2.0.6.1 i'll compile with bugfix instead as 2.0.6.1 does not work, nor with changing files.

@Kokorone
Copy link

Bugfix 2.0.x is always much more recent than any of the numbered releases. That’s the point of it being bugfix

@Doyle4
Copy link

Doyle4 commented Sep 12, 2020

Thought so, thank you.

Just tried and successful, although I had to turn off Filament runout sensor as it failed and can't find location to allocate a pin for it.

vgadreau pushed a commit to vgadreau/Marlin that referenced this pull request Dec 9, 2020
kageurufu pushed a commit to CR30-Users/Marlin-CR30 that referenced this pull request Apr 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.