Skip to content
This repository was archived by the owner on Dec 29, 2023. It is now read-only.

Commit ab6514e

Browse files
Leonid KabanovLeonid Kabanov
Leonid Kabanov
authored and
Leonid Kabanov
committed
Fix bug with uninitialized structure
1 parent fa71ef4 commit ab6514e

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

SPI.cpp

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@
99
#include <driver/spi_master.h>
1010
#include <esp_log.h>
1111
#include "sdkconfig.h"
12+
#include "string.h"
1213

1314
//#define DEBUG 1
1415

@@ -48,6 +49,7 @@ void SPI::init(int mosiPin, int misoPin, int clkPin, int csPin) {
4849
ESP_LOGD(LOG_TAG, "init: mosi=%d, miso=%d, clk=%d, cs=%d", mosiPin, misoPin, clkPin, csPin);
4950

5051
spi_bus_config_t bus_config;
52+
memset(&bus_config, 0, sizeof(spi_bus_config_t));
5153
bus_config.sclk_io_num = clkPin; // CLK
5254
bus_config.mosi_io_num = mosiPin; // MOSI
5355
bus_config.miso_io_num = misoPin; // MISO
@@ -57,7 +59,6 @@ void SPI::init(int mosiPin, int misoPin, int clkPin, int csPin) {
5759
bus_config.flags = (SPICOMMON_BUSFLAG_SCLK | SPICOMMON_BUSFLAG_MOSI | SPICOMMON_BUSFLAG_MISO);
5860

5961
ESP_LOGI(LOG_TAG, "... Initializing bus; host=%d", m_host);
60-
6162
esp_err_t errRc = ::spi_bus_initialize(
6263
m_host,
6364
&bus_config,
@@ -70,6 +71,7 @@ void SPI::init(int mosiPin, int misoPin, int clkPin, int csPin) {
7071
}
7172

7273
spi_device_interface_config_t dev_config;
74+
memset(&dev_config, 0, sizeof(spi_device_interface_config_t));
7375
dev_config.address_bits = 0;
7476
dev_config.command_bits = 0;
7577
dev_config.dummy_bits = 0;
@@ -117,6 +119,7 @@ void SPI::transfer(uint8_t* txData, uint8_t* rxData, size_t dataLen) {
117119
}
118120
#endif
119121
spi_transaction_t trans_desc;
122+
memset(&trans_desc, 0, sizeof(spi_transaction_t));
120123
//trans_desc.address = 0;
121124
//trans_desc.command = 0;
122125
trans_desc.flags = 0;

0 commit comments

Comments
 (0)