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

Current surge after Power up with M80 #78

Closed
tphan26363 opened this issue Apr 19, 2020 · 1 comment
Closed

Current surge after Power up with M80 #78

tphan26363 opened this issue Apr 19, 2020 · 1 comment

Comments

@tphan26363
Copy link

tphan26363 commented Apr 19, 2020

The M80 is use to power up . There are 2 thing consider:
-The default value of PSU_POWERUP_DELAY =100 is too fast for some power supply to settle. The result is the TMC driver is not initialize properly and could cause damage to TMC driver( in my case it was flying like a bullet to the end stop)
-The power.cpp need to add an additional delay after initialize the driver. This is because some of the board is very fast causing the motor to start to move immediately after initialize and create a high surge current to the motor creating a knocking sound. Allow some time for the current to settle before start moving the stepper. It then can be quiet and smooth

void Power::power_on() {
  lastPowerOn = millis();
  if (!powersupply_on) {
    PSU_PIN_ON();

    #if HAS_TRINAMIC_CONFIG
      delay(PSU_POWERUP_DELAY); // Wait for power to settle
      restore_stepper_drivers();
       delay(PSU_POWERUP_DELAY); //add a delay to make sure no surge current if it start to move instanly
    #endif
  }
@thinkyhead
Copy link
Member

Thank you for pointing this out. I will have it patched in bugfix-2.0.x shortly with an increased delay of 250ms and the same delay after resetting the stepper drivers.

thinkyhead added a commit to MarlinFirmware/Marlin that referenced this issue Apr 20, 2020
jmp0x0000 pushed a commit to jmp0x0000/Marlin that referenced this issue Aug 7, 2020
njibhu pushed a commit to njibhu/Marlin that referenced this issue Aug 24, 2020
HairingX pushed a commit to HairingX/Marlin that referenced this issue Jun 16, 2021
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

2 participants