mcu: Do not invert the direction of pullup pins by default

A pullup setting on an input pin (ie, '^') should enable the hardware
pullup resistor, but it should not invert the trigger level (ie, it
should remain trigger on high).  Those that need to change the trigger
level (ie, trigger on low) must now do that explicitly (ie, '^!').
This makes the code match what other firmwares do.

Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
This commit is contained in:
Kevin O'Connor 2016-11-08 09:29:38 -05:00
parent 3b5b895a10
commit 5e6127869a
3 changed files with 8 additions and 8 deletions

View File

@ -11,7 +11,7 @@ enable_pin: ar25
step_distance: .0225
max_velocity: 500
max_accel: 3000
endstop_pin: ^!ar0
endstop_pin: ^ar0
position_min: -0.25
position_endstop: 0
position_max: 200
@ -24,7 +24,7 @@ enable_pin: ar25
step_distance: .0225
max_velocity: 500
max_accel: 3000
endstop_pin: ^!ar1
endstop_pin: ^ar1
position_min: -0.25
position_endstop: 0
position_max: 200
@ -37,7 +37,7 @@ enable_pin: ar25
step_distance: .005
max_velocity: 250
max_accel: 30
endstop_pin: ^!ar2
endstop_pin: ^ar2
position_min: 0.1
position_endstop: 0.5
position_max: 200

View File

@ -10,7 +10,7 @@ enable_pin: !PA7
step_distance: .0225
max_velocity: 500
max_accel: 3000
endstop_pin: ^PB6
endstop_pin: ^!PB6
homing_speed: 50.0
homing_stepper_phases: 32
homing_endstop_accuracy: .200
@ -25,7 +25,7 @@ enable_pin: !PA6
step_distance: .0225
max_velocity: 500
max_accel: 3000
endstop_pin: ^PB5
endstop_pin: ^!PB5
homing_speed: 50.0
homing_stepper_phases: 32
homing_endstop_accuracy: .200
@ -40,7 +40,7 @@ enable_pin: !PA5
step_distance: .005
max_velocity: 250
max_accel: 30
endstop_pin: ^PB4
endstop_pin: ^!PB4
homing_speed: 4.0
homing_retract_dist: 2.0
homing_stepper_phases: 32

View File

@ -9,10 +9,10 @@ import serialhdl, pins, chelper
def parse_pin_extras(pin, can_pullup=False):
pullup = invert = 0
if can_pullup and pin.startswith('^'):
pullup = invert = 1
pullup = 1
pin = pin[1:].strip()
if pin.startswith('!'):
invert = invert ^ 1
invert = 1
pin = pin[1:].strip()
return pin, pullup, invert