[BUG] Safe homing movement

[igna09]

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

Yes, and the problem still exists.

Bug Description

Hello, I am updating from an old 2.1.x version, I have safe homing enabled, i set XY_PROBE_FEEDRATE to 2400 mm/min and HOMING_FEEDRATE_MM_M { (2400 ), (2400 ), (2400 ) }
After homing X and Y axis I expected to move safe home at 2400 but it seems to move way faster

Bug Timeline

No response

Expected behavior

No response

Actual behavior

No response

Steps to Reproduce

No response

Version of Marlin Firmware

bugfix-2.1.x

Printer model

No response

Electronics

No response

LCD/Controller

No response

Other add-ons

No response

Bed Leveling

None

Your Slicer

None

Host Software

None

Don't forget to include

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

Additional information & file uploads

Marlin.zip

[ellensp]

You have

// Edit homing feedrates with M210 and MarlinUI menu items
#define EDITABLE_HOMING_FEEDRATE

So HOMING_FEEDRATE_MM_M is stored in eeprom

So the value in Configuration could be out of date

Use M210 to see the current value

[igna09]

M210 gives me M210 X240 Y240 Z240

[igna09]

i tried also to disable M210 but it is not working anyway, only way to reduce that speed is to reduce max feedrate

[classicrocker883]

XY_PROBE_FEEDRATE @ 2400 is fairly slow compared to default (133*60)~=8000

I thought it had to do with the following:

planner.h

#if HAS_Y_AXIS
  #define PLANNER_XY_FEEDRATE_MM_S _MIN(planner.settings.max_feedrate_mm_s[X_AXIS], planner.settings.max_feedrate_mm_s[Y_AXIS])
#elif HAS_X_AXIS
  #define PLANNER_XY_FEEDRATE_MM_S planner.settings.max_feedrate_mm_s[X_AXIS]
#else
  #define PLANNER_XY_FEEDRATE_MM_S 60.0f
#endif

motion.h

// Determine XY_PROBE_FEEDRATE_MM_S - The feedrate used between Probe Points
#if ABL_USES_GRID
  // ABL LINEAR and BILINEAR use 'G29 S' value, or MMM_TO_MMS(XY_PROBE_FEEDRATE)
  extern feedRate_t xy_probe_feedrate_mm_s;
  #define XY_PROBE_FEEDRATE_MM_S xy_probe_feedrate_mm_s
#elif defined(XY_PROBE_FEEDRATE)
  #define XY_PROBE_FEEDRATE_MM_S MMM_TO_MMS(XY_PROBE_FEEDRATE)
#else
  #define XY_PROBE_FEEDRATE_MM_S PLANNER_XY_FEEDRATE_MM_S
#endif 

Conditionals-5-post.h

#define XY_PROBE_FEEDRATE ((homing_feedrate_mm_m.x + homing_feedrate_mm_m.y) / 2)

eventually you can end up with a very fast feed rate with DEFAULT_MAX_FEEDRATE since it is used in planner.settings.max_feedrate_mm_s

• feedRate_t _DMF[] PROGMEM = DEFAULT_MAX_FEEDRATE;

(as _DMF in settings.cpp)

so - in your case homing (XY_PROBE_FEEDRATE_MM_S) is 400mm/s because it = PLANNER_XY_FEEDRATE_MM_S

[classicrocker883]

I do have a PR which should fix this issue #27411 if you want to try it out