From 69b12f31b893ebb9cb450999a9328ea2a4210caf Mon Sep 17 00:00:00 2001 From: rusEfi Date: Sun, 9 Dec 2018 16:46:53 -0500 Subject: [PATCH] docs --- firmware/hw_layer/digital_input_hw.cpp | 2 ++ firmware/hw_layer/joystick.cpp | 4 ++++ 2 files changed, 6 insertions(+) diff --git a/firmware/hw_layer/digital_input_hw.cpp b/firmware/hw_layer/digital_input_hw.cpp index 483cffe481..b54d156c7c 100644 --- a/firmware/hw_layer/digital_input_hw.cpp +++ b/firmware/hw_layer/digital_input_hw.cpp @@ -16,6 +16,8 @@ * See https://stackoverflow.com/questions/43440599/stm32-multi-channel-input-capture-overcapturing-on-all-channels-interrupts-not * where they seem to be capturing something on multiple channels maybe not PWM mode of ICU is the key difference? * + * rus084 is reminding that EXTI could be enough for our needs + * See joystick.cpp * * @date Jun 23, 2013 * @author Andrey Belomutskiy, (c) 2012-2018 diff --git a/firmware/hw_layer/joystick.cpp b/firmware/hw_layer/joystick.cpp index 54da1a9663..85a5d737fb 100644 --- a/firmware/hw_layer/joystick.cpp +++ b/firmware/hw_layer/joystick.cpp @@ -4,6 +4,10 @@ * See lcd_controller.cpp for more information * See void onJoystick(joystick_button_e button) * + * TODO: separate EXTI layer from joystick logic + * TODO: spell out the limitations of EXTI, can we use multiple pins with same index? + * See also comments in digital_input_hw.cpp + * * @date Jan 2, 2015 * @author Andrey Belomutskiy, (c) 2012-2018 */