diff options
| author | Kevin O'Connor <kevin@koconnor.net> | 2017-03-05 14:10:19 -0500 |
|---|---|---|
| committer | Kevin O'Connor <kevin@koconnor.net> | 2017-03-09 14:50:40 -0500 |
| commit | 8d6ecd9af87d3e0da312588bacb1c43332412107 (patch) | |
| tree | b00d8e8c2df7c2567a4ce045556f6c4d2295814b /src | |
| parent | 342a7096ea32e024ea6620bfa2b379ba3dceaddd (diff) | |
| download | kutter-8d6ecd9af87d3e0da312588bacb1c43332412107.tar.gz kutter-8d6ecd9af87d3e0da312588bacb1c43332412107.tar.xz kutter-8d6ecd9af87d3e0da312588bacb1c43332412107.zip | |
endstop: No need to store pin_value
The pin_value can be stored in the existing flags variable.
Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
Diffstat (limited to 'src')
| -rw-r--r-- | src/endstop.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/src/endstop.c b/src/endstop.c index ae41656a..3701f642 100644 --- a/src/endstop.c +++ b/src/endstop.c @@ -16,10 +16,10 @@ struct end_stop { uint32_t rest_time; struct stepper *stepper; struct gpio_in pin; - uint8_t pin_value, flags; + uint8_t flags; }; -enum { ESF_HOMING=1, ESF_REPORT=2 }; +enum { ESF_PIN_HIGH=1<<0, ESF_HOMING=1<<1, ESF_REPORT=1<<2 }; // Timer callback for an end stop static uint_fast8_t @@ -27,7 +27,8 @@ end_stop_event(struct timer *t) { struct end_stop *e = container_of(t, struct end_stop, time); uint8_t val = gpio_in_read(e->pin); - if (val != e->pin_value) { + if ((val ? ~e->flags : e->flags) & ESF_PIN_HIGH) { + // No match - reschedule for the next attempt e->time.waketime += e->rest_time; return SF_RESCHEDULE; } @@ -62,8 +63,7 @@ command_end_stop_home(uint32_t *args) e->flags = 0; return; } - e->pin_value = args[3]; - e->flags = ESF_HOMING; + e->flags = ESF_HOMING | (args[3] ? ESF_PIN_HIGH : 0); sched_timer(&e->time); } DECL_COMMAND(command_end_stop_home, |