From 70068985a7dc8f9beb0091f2a53b2fbdf359523c Mon Sep 17 00:00:00 2001 From: Kevin O'Connor Date: Tue, 15 May 2018 17:04:30 -0400 Subject: stepper: Introduce and use gpio_out_toggle_noirq() The gpio_out_toggle() function in the sam3x8e and stm32f1 code was only valid if it was called with irqs disabled. Commits 018c5daa and 9c52ad43 enabled the lcd code which called gpio_out_toggle() with irqs enabled. This could cause corruption of the gpio state. Introduce a gpio_out_toggle_noirq() function that will only be invoked with irqs disabled, and fix gpio_out_toggle() on sam3x8e and stm32f1 so that it safe to call even if irqs are enabled. Signed-off-by: Kevin O'Connor --- src/simulator/gpio.c | 2 ++ 1 file changed, 2 insertions(+) (limited to 'src/simulator/gpio.c') diff --git a/src/simulator/gpio.c b/src/simulator/gpio.c index 08359679..327c05ce 100644 --- a/src/simulator/gpio.c +++ b/src/simulator/gpio.c @@ -9,6 +9,8 @@ struct gpio_out gpio_out_setup(uint8_t pin, uint8_t val) { return (struct gpio_out){.pin=pin}; } +void gpio_out_toggle_noirq(struct gpio_out g) { +} void gpio_out_toggle(struct gpio_out g) { } void gpio_out_write(struct gpio_out g, uint8_t val) { -- cgit v1.2.3-70-g09d2