rusefi
/
custom-board-bundle-sample-fork-of-rusefi
mirror of https://github.com/rusefi/custom-board-bundle-sample-fork-of-rusefi.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

refactoring - reducing GPIO complexity

This commit is contained in:
rusefi 2017-04-21 13:53:47 -04:00
parent 928ab4e6f7
commit 8e8f2581d8
2 changed files with 27 additions and 27 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

27
firmware/controllers/system/efiGpio.h
View File

@ -12,8 +12,6 @@
#include "io_pins.h" #include "io_pins.h"
#include "main.h" #include "main.h"
#define INITIAL_PIN_STATE -1
void initPrimaryPins(void); void initPrimaryPins(void);
void initOutputPins(void); void initOutputPins(void);
@ -23,31 +21,6 @@ void turnAllPinsOff(void);
#define turnAllPinsOff() {} #define turnAllPinsOff() {}
#endif /* EFI_GPIO */ #endif /* EFI_GPIO */
/**
* @brief Single output pin reference and state
*/
class OutputPin {
public:
OutputPin();
bool isInitialized();
void setValue(int logicValue);
void setDefaultPinState(pin_output_mode_e *defaultState);
bool getLogicValue();
void unregister();
#if EFI_PROD_CODE || defined(__DOXYGEN__)
ioportid_t port;
uint8_t pin;
#endif /* EFI_PROD_CODE */
int8_t currentLogicValue;
// 4 byte pointer is a bit of a memory waste here
pin_output_mode_e *modePtr;
/**
* we track current pin status so that we do not touch the actual hardware if we want to write new pin bit
* which is same as current pin value. This maybe helps in case of status leds, but maybe it's a total over-engineering
*/
};
class NamedOutputPin : public OutputPin { class NamedOutputPin : public OutputPin {
public: public:
NamedOutputPin(); NamedOutputPin();

27
firmware/hw_layer/io_pins.h
View File

@ -12,6 +12,7 @@
#include "rusefi_enums.h" #include "rusefi_enums.h"
#include "main.h" #include "main.h"
#define INITIAL_PIN_STATE -1
#define GPIO_NULL NULL #define GPIO_NULL NULL
// mode >= 0 is always true since that's an unsigned // mode >= 0 is always true since that's an unsigned
@ -51,5 +52,31 @@
// LED_HUGE_19, // LED_HUGE_19,
// LED_HUGE_20, // LED_HUGE_20,
#ifdef __cplusplus
/**
* @brief Single output pin reference and state
*/
class OutputPin {
public:
OutputPin();
bool isInitialized();
void setValue(int logicValue);
void setDefaultPinState(pin_output_mode_e *defaultState);
bool getLogicValue();
void unregister();
#if EFI_PROD_CODE || defined(__DOXYGEN__)
ioportid_t port;
uint8_t pin;
#endif /* EFI_PROD_CODE */
int8_t currentLogicValue;
// 4 byte pointer is a bit of a memory waste here
pin_output_mode_e *modePtr;
/**
* we track current pin status so that we do not touch the actual hardware if we want to write new pin bit
* which is same as current pin value. This maybe helps in case of status leds, but maybe it's a total over-engineering
*/
};
#endif /* __cplusplus */
#endif /* IO_PINS_H_ */ #endif /* IO_PINS_H_ */