Sanity-check inverting and pullups for Z probe and Z endstop
`Z_MIN_ENDSTOP_INVERTING` must match `Z_MIN_PROBE_ENDSTOP_INVERTING` when using `Z_MIN_PROBE_USES_Z_MIN_ENDSTOP_PIN`. This is easy to overlook.
This commit is contained in:
		
							parent
							
								
									a9f64537ff
								
							
						
					
					
						commit
						da6a094424
					
				| @ -459,7 +459,7 @@ | ||||
| #define X_MAX_ENDSTOP_INVERTING false // set to true to invert the logic of the endstop.
 | ||||
| #define Y_MAX_ENDSTOP_INVERTING false // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MAX_ENDSTOP_INVERTING false // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MIN_PROBE_ENDSTOP_INVERTING false // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MIN_PROBE_ENDSTOP_INVERTING false // set to true to invert the logic of the probe.
 | ||||
| 
 | ||||
| // Enable this feature if all enabled endstop pins are interrupt-capable.
 | ||||
| // This will remove the need to poll the interrupt pins, saving many CPU cycles.
 | ||||
|  | ||||
| @ -454,6 +454,8 @@ | ||||
|       #error "Z_MIN_PROBE_USES_Z_MIN_ENDSTOP_PIN requires USE_ZMIN_PLUG to be enabled." | ||||
|     #elif !HAS_Z_MIN | ||||
|       #error "Z_MIN_PROBE_USES_Z_MIN_ENDSTOP_PIN requires the Z_MIN_PIN to be defined." | ||||
|     #elif ENABLED(Z_MIN_PROBE_ENDSTOP_INVERTING) != ENABLED(Z_MIN_ENDSTOP_INVERTING) | ||||
|       #error "Z_MIN_PROBE_USES_Z_MIN_ENDSTOP_PIN requires Z_MIN_ENDSTOP_INVERTING to match Z_MIN_PROBE_ENDSTOP_INVERTING." | ||||
|     #endif | ||||
|   #elif ENABLED(Z_MIN_PROBE_ENDSTOP) | ||||
|     #if !HAS_Z_MIN_PROBE_PIN | ||||
|  | ||||
| @ -459,7 +459,7 @@ | ||||
| #define X_MAX_ENDSTOP_INVERTING false // set to true to invert the logic of the endstop.
 | ||||
| #define Y_MAX_ENDSTOP_INVERTING false // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MAX_ENDSTOP_INVERTING false // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MIN_PROBE_ENDSTOP_INVERTING false // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MIN_PROBE_ENDSTOP_INVERTING true // set to true to invert the logic of the probe.
 | ||||
| 
 | ||||
| // Enable this feature if all enabled endstop pins are interrupt-capable.
 | ||||
| // This will remove the need to poll the interrupt pins, saving many CPU cycles.
 | ||||
|  | ||||
| @ -441,7 +441,7 @@ | ||||
| #define X_MAX_ENDSTOP_INVERTING true  // set to true to invert the logic of the endstop.
 | ||||
| #define Y_MAX_ENDSTOP_INVERTING true  // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MAX_ENDSTOP_INVERTING true  // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MIN_PROBE_ENDSTOP_INVERTING false // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MIN_PROBE_ENDSTOP_INVERTING false // set to true to invert the logic of the probe.
 | ||||
| 
 | ||||
| // Enable this feature if all enabled endstop pins are interrupt-capable.
 | ||||
| // This will remove the need to poll the interrupt pins, saving many CPU cycles.
 | ||||
|  | ||||
| @ -441,7 +441,7 @@ | ||||
| #define X_MAX_ENDSTOP_INVERTING true  // set to true to invert the logic of the endstop.
 | ||||
| #define Y_MAX_ENDSTOP_INVERTING true  // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MAX_ENDSTOP_INVERTING true  // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MIN_PROBE_ENDSTOP_INVERTING false // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MIN_PROBE_ENDSTOP_INVERTING false // set to true to invert the logic of the probe.
 | ||||
| 
 | ||||
| // Enable this feature if all enabled endstop pins are interrupt-capable.
 | ||||
| // This will remove the need to poll the interrupt pins, saving many CPU cycles.
 | ||||
|  | ||||
| @ -451,7 +451,7 @@ | ||||
| #define X_MAX_ENDSTOP_INVERTING true  // set to true to invert the logic of the endstop.
 | ||||
| #define Y_MAX_ENDSTOP_INVERTING true  // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MAX_ENDSTOP_INVERTING true  // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MIN_PROBE_ENDSTOP_INVERTING true  // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MIN_PROBE_ENDSTOP_INVERTING true  // set to true to invert the logic of the probe.
 | ||||
| 
 | ||||
| // Enable this feature if all enabled endstop pins are interrupt-capable.
 | ||||
| // This will remove the need to poll the interrupt pins, saving many CPU cycles.
 | ||||
|  | ||||
| @ -453,7 +453,7 @@ | ||||
| #define X_MAX_ENDSTOP_INVERTING true  // set to true to invert the logic of the endstop.
 | ||||
| #define Y_MAX_ENDSTOP_INVERTING true  // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MAX_ENDSTOP_INVERTING true  // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MIN_PROBE_ENDSTOP_INVERTING false // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MIN_PROBE_ENDSTOP_INVERTING false // set to true to invert the logic of the probe.
 | ||||
| 
 | ||||
| // Enable this feature if all enabled endstop pins are interrupt-capable.
 | ||||
| // This will remove the need to poll the interrupt pins, saving many CPU cycles.
 | ||||
|  | ||||
| @ -489,7 +489,7 @@ | ||||
| #define X_MAX_ENDSTOP_INVERTING false // set to true to invert the logic of the endstop.
 | ||||
| #define Y_MAX_ENDSTOP_INVERTING false // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MAX_ENDSTOP_INVERTING false // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MIN_PROBE_ENDSTOP_INVERTING false // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MIN_PROBE_ENDSTOP_INVERTING false // set to true to invert the logic of the probe.
 | ||||
| 
 | ||||
| // Enable this feature if all enabled endstop pins are interrupt-capable.
 | ||||
| // This will remove the need to poll the interrupt pins, saving many CPU cycles.
 | ||||
|  | ||||
| @ -459,7 +459,7 @@ | ||||
| #define X_MAX_ENDSTOP_INVERTING true  // set to true to invert the logic of the endstop.
 | ||||
| #define Y_MAX_ENDSTOP_INVERTING true  // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MAX_ENDSTOP_INVERTING true  // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MIN_PROBE_ENDSTOP_INVERTING true  // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MIN_PROBE_ENDSTOP_INVERTING true  // set to true to invert the logic of the probe.
 | ||||
| 
 | ||||
| // Enable this feature if all enabled endstop pins are interrupt-capable.
 | ||||
| // This will remove the need to poll the interrupt pins, saving many CPU cycles.
 | ||||
|  | ||||
| @ -459,7 +459,7 @@ | ||||
| #define X_MAX_ENDSTOP_INVERTING true  // set to true to invert the logic of the endstop.
 | ||||
| #define Y_MAX_ENDSTOP_INVERTING true  // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MAX_ENDSTOP_INVERTING true  // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MIN_PROBE_ENDSTOP_INVERTING true  // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MIN_PROBE_ENDSTOP_INVERTING true  // set to true to invert the logic of the probe.
 | ||||
| 
 | ||||
| // Enable this feature if all enabled endstop pins are interrupt-capable.
 | ||||
| // This will remove the need to poll the interrupt pins, saving many CPU cycles.
 | ||||
|  | ||||
| @ -459,7 +459,7 @@ | ||||
| #define X_MAX_ENDSTOP_INVERTING true  // set to true to invert the logic of the endstop.
 | ||||
| #define Y_MAX_ENDSTOP_INVERTING true  // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MAX_ENDSTOP_INVERTING true  // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MIN_PROBE_ENDSTOP_INVERTING false // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MIN_PROBE_ENDSTOP_INVERTING true // set to true to invert the logic of the probe.
 | ||||
| 
 | ||||
| // Enable this feature if all enabled endstop pins are interrupt-capable.
 | ||||
| // This will remove the need to poll the interrupt pins, saving many CPU cycles.
 | ||||
|  | ||||
| @ -456,7 +456,7 @@ | ||||
| #define X_MAX_ENDSTOP_INVERTING true  // set to true to invert the logic of the endstop.
 | ||||
| #define Y_MAX_ENDSTOP_INVERTING true  // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MAX_ENDSTOP_INVERTING true  // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MIN_PROBE_ENDSTOP_INVERTING false // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MIN_PROBE_ENDSTOP_INVERTING true // set to true to invert the logic of the probe.
 | ||||
| 
 | ||||
| // Enable this feature if all enabled endstop pins are interrupt-capable.
 | ||||
| // This will remove the need to poll the interrupt pins, saving many CPU cycles.
 | ||||
|  | ||||
| @ -474,7 +474,7 @@ | ||||
| #define X_MAX_ENDSTOP_INVERTING true  // set to true to invert the logic of the endstop.
 | ||||
| #define Y_MAX_ENDSTOP_INVERTING true  // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MAX_ENDSTOP_INVERTING true  // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MIN_PROBE_ENDSTOP_INVERTING false // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MIN_PROBE_ENDSTOP_INVERTING true // set to true to invert the logic of the probe.
 | ||||
| 
 | ||||
| // Enable this feature if all enabled endstop pins are interrupt-capable.
 | ||||
| // This will remove the need to poll the interrupt pins, saving many CPU cycles.
 | ||||
|  | ||||
| @ -480,7 +480,7 @@ | ||||
| #define X_MAX_ENDSTOP_INVERTING true  // set to true to invert the logic of the endstop.
 | ||||
| #define Y_MAX_ENDSTOP_INVERTING true  // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MAX_ENDSTOP_INVERTING true  // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MIN_PROBE_ENDSTOP_INVERTING false // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MIN_PROBE_ENDSTOP_INVERTING true // set to true to invert the logic of the probe.
 | ||||
| 
 | ||||
| // Enable this feature if all enabled endstop pins are interrupt-capable.
 | ||||
| // This will remove the need to poll the interrupt pins, saving many CPU cycles.
 | ||||
|  | ||||
| @ -451,7 +451,7 @@ | ||||
| #define X_MAX_ENDSTOP_INVERTING true  // set to true to invert the logic of the endstop.
 | ||||
| #define Y_MAX_ENDSTOP_INVERTING true  // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MAX_ENDSTOP_INVERTING true  // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MIN_PROBE_ENDSTOP_INVERTING true  // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MIN_PROBE_ENDSTOP_INVERTING true  // set to true to invert the logic of the probe.
 | ||||
| 
 | ||||
| // Enable this feature if all enabled endstop pins are interrupt-capable.
 | ||||
| // This will remove the need to poll the interrupt pins, saving many CPU cycles.
 | ||||
|  | ||||
| @ -459,7 +459,7 @@ | ||||
| #define X_MAX_ENDSTOP_INVERTING false // set to true to invert the logic of the endstop.
 | ||||
| #define Y_MAX_ENDSTOP_INVERTING false // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MAX_ENDSTOP_INVERTING false // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MIN_PROBE_ENDSTOP_INVERTING false // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MIN_PROBE_ENDSTOP_INVERTING false // set to true to invert the logic of the probe.
 | ||||
| 
 | ||||
| // Enable this feature if all enabled endstop pins are interrupt-capable.
 | ||||
| // This will remove the need to poll the interrupt pins, saving many CPU cycles.
 | ||||
|  | ||||
| @ -506,7 +506,7 @@ | ||||
| #define X_MAX_ENDSTOP_INVERTING false // set to true to invert the logic of the endstop.
 | ||||
| #define Y_MAX_ENDSTOP_INVERTING false // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MAX_ENDSTOP_INVERTING false // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MIN_PROBE_ENDSTOP_INVERTING true  // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MIN_PROBE_ENDSTOP_INVERTING false  // set to true to invert the logic of the probe.
 | ||||
| 
 | ||||
| // Enable this feature if all enabled endstop pins are interrupt-capable.
 | ||||
| // This will remove the need to poll the interrupt pins, saving many CPU cycles.
 | ||||
|  | ||||
| @ -506,7 +506,7 @@ | ||||
| #define X_MAX_ENDSTOP_INVERTING true  // set to true to invert the logic of the endstop.
 | ||||
| #define Y_MAX_ENDSTOP_INVERTING true  // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MAX_ENDSTOP_INVERTING true  // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MIN_PROBE_ENDSTOP_INVERTING true  // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MIN_PROBE_ENDSTOP_INVERTING true  // set to true to invert the logic of the probe.
 | ||||
| 
 | ||||
| // Enable this feature if all enabled endstop pins are interrupt-capable.
 | ||||
| // This will remove the need to poll the interrupt pins, saving many CPU cycles.
 | ||||
|  | ||||
| @ -506,7 +506,7 @@ | ||||
| #define X_MAX_ENDSTOP_INVERTING false // set to true to invert the logic of the endstop.
 | ||||
| #define Y_MAX_ENDSTOP_INVERTING false // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MAX_ENDSTOP_INVERTING false // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MIN_PROBE_ENDSTOP_INVERTING false // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MIN_PROBE_ENDSTOP_INVERTING false // set to true to invert the logic of the probe.
 | ||||
| 
 | ||||
| // Enable this feature if all enabled endstop pins are interrupt-capable.
 | ||||
| // This will remove the need to poll the interrupt pins, saving many CPU cycles.
 | ||||
|  | ||||
| @ -493,7 +493,7 @@ | ||||
| #define X_MAX_ENDSTOP_INVERTING false // set to true to invert the logic of the endstop.
 | ||||
| #define Y_MAX_ENDSTOP_INVERTING false // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MAX_ENDSTOP_INVERTING false // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MIN_PROBE_ENDSTOP_INVERTING false // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MIN_PROBE_ENDSTOP_INVERTING false // set to true to invert the logic of the probe.
 | ||||
| 
 | ||||
| // Enable this feature if all enabled endstop pins are interrupt-capable.
 | ||||
| // This will remove the need to poll the interrupt pins, saving many CPU cycles.
 | ||||
|  | ||||
| @ -504,7 +504,7 @@ | ||||
| #define X_MAX_ENDSTOP_INVERTING false // set to true to invert the logic of the endstop.
 | ||||
| #define Y_MAX_ENDSTOP_INVERTING false // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MAX_ENDSTOP_INVERTING false // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MIN_PROBE_ENDSTOP_INVERTING false // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MIN_PROBE_ENDSTOP_INVERTING false // set to true to invert the logic of the probe.
 | ||||
| 
 | ||||
| // Enable this feature if all enabled endstop pins are interrupt-capable.
 | ||||
| // This will remove the need to poll the interrupt pins, saving many CPU cycles.
 | ||||
|  | ||||
| @ -462,7 +462,7 @@ | ||||
| #define X_MAX_ENDSTOP_INVERTING true  // set to true to invert the logic of the endstop.
 | ||||
| #define Y_MAX_ENDSTOP_INVERTING true  // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MAX_ENDSTOP_INVERTING true  // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MIN_PROBE_ENDSTOP_INVERTING false // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MIN_PROBE_ENDSTOP_INVERTING false // set to true to invert the logic of the probe.
 | ||||
| 
 | ||||
| // Enable this feature if all enabled endstop pins are interrupt-capable.
 | ||||
| // This will remove the need to poll the interrupt pins, saving many CPU cycles.
 | ||||
|  | ||||
| @ -449,7 +449,7 @@ | ||||
| #define X_MAX_ENDSTOP_INVERTING true  // set to true to invert the logic of the endstop.
 | ||||
| #define Y_MAX_ENDSTOP_INVERTING true  // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MAX_ENDSTOP_INVERTING true  // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MIN_PROBE_ENDSTOP_INVERTING true  // set to true to invert the logic of the endstop.
 | ||||
| #define Z_MIN_PROBE_ENDSTOP_INVERTING true  // set to true to invert the logic of the probe.
 | ||||
| 
 | ||||
| // Enable this feature if all enabled endstop pins are interrupt-capable.
 | ||||
| // This will remove the need to poll the interrupt pins, saving many CPU cycles.
 | ||||
|  | ||||
		Loading…
	
	
			
			x
			
			
		
	
		Reference in New Issue
	
	Block a user