Yes you shared the table of 14 inputs. That comes out to over 16,000+ combinations. That's why you need to show only the inputs you're concerned with and the outputs it controls. Maybe some inputs are simple and don't need to be part of the data structure?
You can still use the code as is. The only thing you need to do is..
- Change pb0, pb1 and pb6 as inputs only. Don't use EXTI for these pins.
- You need a polling routine to read pb0, pb1 and pb6.
- The other inputs can still use interrupts.
So if you want to use all 14 inputs, you'll have a data structure like this
typedef union
{
struct
{
uint16_t data[2]; // byte 0 is current state, byte 1 is last state
}Byte;
struct
{
uint8_t pb6:1;
uint8_t pc13:1;
uint8_t pc14:1;
uint8_t pc15:1;
uint8_t pa0:1;
uint8_t pa1:1;
uint8_t pa2:1;
uint8_t pa3:1;
uint8_t pa4:1;
uint8_t pa5:1;
uint8_t pa6:1;
uint8_t pa7:1;
uint8_t pb0:1;
uint8_t pb1:1;
uint8_t :2;
}Status;
}InputStatus_t;
In the PollingInit you add a timer callback for 1ms to poll for those 3 GPIO's
void PollingInit(void)
{
HAL_SYSCFG_StrobeDBattpinsConfig(SYSCFG_UCPD1_STROBE);
PB12_On(); // init to 1
PA10_Off(); // init to 0
TimerCallbackRegisterOnly(&timerCallback, PA10_Off);
TimerCallbackRegisterOnly(&timerCallback, InputPoll);
TimerCallbackTimerStart(&timerCallback, InputPoll, 1, TIMER_REPEAT);
}
You just need to add a couple of functions and also a global variable inputReadDone
bool inputReadDone = false;
void InputPoll(void)
{
InputRead(&inputs);
}
void InputRead(InputStatus_t *input)
{
input->Status.pb0 = HAL_GPIO_ReadPin(PB0_GPIO_Port, PB0_Pin);
input->Status.pb1 = HAL_GPIO_ReadPin(PB1_GPIO_Port, PB1_Pin);
input->Status.pb6 = HAL_GPIO_ReadPin(PB6_GPIO_Port, PB6_Pin);
inputReadDone = true;
}
So when you call GPIO_Check function, it checks inputReadDone to be sure you've polled for the 3 GPIO's first before comparing input->Byte.data
void GPIO_Check(InputStatus_t *input)
{
if(inputReadDone)
{
inputReadDone = false;
if(input->Byte.data[1] != input->Byte.data[0]) // check for change, else do nothing.
{
input->Byte.data[1] = input->Byte.data[0]; // copy current state to last state
switch(input->Byte.data[0]) // index of truth table
{
case 0:
PB12_Off(); // pin is low
PA10_On(); // pin is high
TimerCallbackTimerStart(&timerCallback, PA10_Off, 500, TIMER_NO_REPEAT); // start timer to turn off PA10
break;
case 1: // all other case, PA12 is On (high state)
case 2:
case 3:
case 4:
case 5:
case 6:
case 7:
default:
PB12_On(); // pin is high
break;
}
}
}
}
Now that you're polling for the inputs, there can be a chance for an input to change state before you call GPIO_Check.
Tips and Tricks with TimerCallback https://www.youtube.com/@eebykarl
If you find my solution useful, please click the Accept as Solution so others see the solution.
