CODEWITHSUNDEEP
c++embeddedreal-timemicrocontrollerspi
Professor Jimatura
Professor Jimatura

Reputation: 79

How to exploit double buffering for reading digital inputs state?

I have following situation. I have a microcontroller which communicates with two external I/O expander chips via one SPI peripheral. Each of the chips has eight digital inputs and is equiped with the latch input which ensures that both bytes of the digital inputs can be sampled at one instant in time. To communicate the state of both the bytes into my microcontroller I need to do two SPI transactions. At the same time I need to ensure that the software in my microcontroller will work with consistent state of both the bytes.

My first idea how to solve this problem was to use sort of double buffer. Below is a pseudocode describing my idea.

uint8_t di_array_01[2] = {0};
uint8_t di_array_02[2] = {0};

uint8_t *ready_data = di_array_01;
uint8_t *shadow_data = di_array_02;
uint8_t *temp;

if(chip_0_data_received) {
 *shadow_data = di_state_chip_0;
 chip_0_data_received = false;
} else if(chip_1_data_received) {
 *(shadow_data + 1) = di_state_chip_1;
 temp = ready_data;
 ready_data = shadow_data;
 shadow_data = temp;
 chip_1_data_received = false;
}

The higher software layer will always work with the content of the array pointed by the ready_data pointer. My intention is that setting of the boolean flags chip_0_data_received (chip_1_data_received) will be done in the "end of transaction" interrupt and the code below will be invoked from the background loop along with code for starting of the SPI transaction.

Does anybody see any potential problem which I have omitted?

Upvotes: 3

Views: 130

Answers (1)

Tom V
Tom V

Reputation: 5510

If your data is only 16 bits in total you can read and write it atomically.

uint16_t combined_data;

// in reading function
if (chip_0_data_received && chip_1_data_received)
{
  combined_data = (((uint16_t)di_state_chip_1 << 8) | di_state_chip_0);
}

// in using function
uint16_t get_combined_data = combined_data;
uint8_t data_chip_1 = ((get_combined_data >> 8) & 0xFF);
uint8_t data_chip_0 = ((get_combined_data >> 0) & 0xFF);

Upvotes: 2

Related Questions