MarlinFirmware / Configurations

Configurations for Marlin Firmware
https://marlinfw.org
GNU General Public License v3.0
1.98k stars 3.33k forks source link

skr mini v3 Define DIAG_JUMPERS_REMOVED 2.1.2 stable #1015

Closed akash1769 closed 4 months ago

akash1769 commented 4 months ago

using ender 5 pro/CrealityV1 example config. message says

Marlin\src\inc\Warnings.cpp:731:6: warning: #warning "Motherboard DIAG jumpers must be removed when SENSORLESS_HOMING is disabled. (Define DIAG_JUMPERS_REMOVED to suppress this warning.)" [-Wcpp] 731 | #warning "Motherboard DIAG jumpers must be removed when SENSORLESS_HOMING is disabled. (Define DIAG_JUMPERS_REMOVED to suppress this warning.)"

There is no sensorless homing defined in this example configuration.h and just adding

Define DIAG_JUMPERS_REMOVED

under //#define SENSORLESS_PROBING (dunno if there's a better place) returns

In file included from c:\users\akash\desktop\marlin 2.12\marlin\src\inc\MarlinConfigPre.h:39, from c:\users\akash\desktop\marlin 2.12\marlin\src\inc\marlinconfig.h:28, from buildroot/share/PlatformIO/scripts/common-dependencies.h:29: c:\users\akash\desktop\marlin 2.12\marlin\configuration.h:1417:2: error: invalid preprocessing directive #Define 1417 | #Define DIAG_JUMPERS_REMOVED | ^~ Error: Failed to parse Marlin features. See previous error messages. ========================================================== [FAILED] Took 2.11 seconds ==========================================================

Environment Status Duration


STM32G0B1RE_btt FAILED 00:00:02.105

Any suggestions? Google turned up nothing definitive

thisiskeithb commented 4 months ago

You need to format it as:

#define DIAG_JUMPERS_REMOVED

This Issue Queue is for Marlin bug reports and development-related issues, and we prefer not to handle user-support questions here. (As noted on this page.) For best results getting help with configuration and troubleshooting, please use the following resources:

akash1769 commented 4 months ago

Is it just the capital D?

akash1769 commented 4 months ago

Apparently... thanks for your help... I feel silly... I should know better