Bug Z axis doesnt raise after after homeing

[Maker-Paul]

Did you test the latest bugfix-2.1.x code?

Yes, and the problem still exists.

Bug Description

#define Z_AFTER_HOMING 10 is not working in the latest Marlin Bugfix 2.1.x and #define XY_AFTER_HOMING { X_CENTER, Y_CENTER causes the nozzle to travel horizontally without lifting from the bed.

Temporary workaround
//#define Z_AFTER_HOMING 10 // (mm) Height to move to after homing (if Z was homed)
//#define XY_AFTER_HOMING { X_CENTER, Y_CENTER } // (mm) Move to an XY position after homing (and raising Z)

#define EVENT_GCODE_AFTER_HOMING "G0 Z10\nG0 X127 Y127" //"M300 P440 S200" // Commands to run after G28 (and move to XY_AFTER_HOMING)

Bug Timeline

No response

Expected behavior

No response

Actual behavior

No response

Steps to Reproduce

No response

Version of Marlin Firmware

Marlin bugfix 2.1.x

Printer model

Tronxy

Electronics

Stock Mainboard

LCD/Controller

BTT TFT35 E3 V3

Other add-ons

No response

Bed Leveling

ABL Bilinear mesh

Your Slicer

Prusa Slicer

Host Software

Other (explain below)

Don't forget to include

• A ZIP file containing your Configuration.h and Configuration_adv.h.

Additional information & file uploads

Configuration.zip

[ellensp]

I think I see what is going on.

In Motion.cpp is

  /**
   * Move Z to Z_POST_CLEARANCE,
   * The axis is allowed to move down.
   */
  void Motion::do_move_after_z_homing() {
    DEBUG_SECTION(mzah, "do_move_after_z_homing", DEBUGGING(LEVELING));
    #ifdef Z_POST_CLEARANCE
      do_z_clearance(
        Z_POST_CLEARANCE,
        ALL(HOMING_Z_WITH_PROBE, HAS_STOWABLE_PROBE) && TERN0(HAS_BED_PROBE, endstops.z_probe_enabled),
        true
      );
    #elif ENABLED(USE_PROBE_FOR_Z_HOMING)
      probe.move_z_after_probing();
    #endif
  }

  #ifndef Z_POST_CLEARANCE  // May be set by proui/dwin.h :-P
    #ifdef Z_AFTER_HOMING
      #define Z_POST_CLEARANCE Z_AFTER_HOMING
    #else
      #define Z_POST_CLEARANCE Z_CLEARANCE_FOR_HOMING
    #endif
  #endif

so Z_POST_CLEARANCE is being defined after the test to see if its defined

try and replace it with the following that just swaps the order

  #ifndef Z_POST_CLEARANCE  // May be set by proui/dwin.h :-P
    #ifdef Z_AFTER_HOMING
      #define Z_POST_CLEARANCE Z_AFTER_HOMING
    #else
      #define Z_POST_CLEARANCE Z_CLEARANCE_FOR_HOMING
    #endif
  #endif

  /**
   * Move Z to Z_POST_CLEARANCE,
   * The axis is allowed to move down.
   */
  void Motion::do_move_after_z_homing() {
    DEBUG_SECTION(mzah, "do_move_after_z_homing", DEBUGGING(LEVELING));
    #ifdef Z_POST_CLEARANCE
      do_z_clearance(
        Z_POST_CLEARANCE,
        ALL(HOMING_Z_WITH_PROBE, HAS_STOWABLE_PROBE) && TERN0(HAS_BED_PROBE, endstops.z_probe_enabled),
        true
      );
    #elif ENABLED(USE_PROBE_FOR_Z_HOMING)
      probe.move_z_after_probing();
    #endif
  }

[Maker-Paul]

Yep That Fixed it thanks

[ellensp]

Pr created