ARMmbed
/
mbed-os
mirror of https://github.com/ARMmbed/mbed-os.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

QSPI: Move destructor into source file 

This allows the entire QSPI class to be mocked/faked for unit testing
purpose, without dependencies from the real implementation such as
`qspi_free()` from the HAL.
pull/14989/head
Lingkai Dong 2021-08-13 15:02:15 +01:00
parent 0cb62c4944
commit e04a16fd9f
2 changed files with 8 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
drivers/include/drivers/QSPI.h
View File

@ -115,12 +115,7 @@ public:
QSPI(const qspi_pinmap_t &pinmap, int mode = 0); QSPI(const qspi_pinmap_t &pinmap, int mode = 0);
QSPI(const qspi_pinmap_t &&, int = 0) = delete; // prevent passing of temporary objects QSPI(const qspi_pinmap_t &&, int = 0) = delete; // prevent passing of temporary objects
virtual ~QSPI() virtual ~QSPI();
{
lock();
qspi_free(&_qspi);
unlock();
}
/** Configure the data transmission format /** Configure the data transmission format
* *

7
drivers/source/QSPI.cpp
View File

@ -92,6 +92,13 @@ QSPI::QSPI(const qspi_pinmap_t &pinmap, int mode) : _qspi()
MBED_ASSERT(success); MBED_ASSERT(success);
} }
QSPI::~QSPI()
{
lock();
qspi_free(&_qspi);
unlock();
}
qspi_status_t QSPI::configure_format(qspi_bus_width_t inst_width, qspi_bus_width_t address_width, qspi_address_size_t address_size, qspi_bus_width_t alt_width, qspi_alt_size_t alt_size, qspi_bus_width_t data_width, int dummy_cycles) qspi_status_t QSPI::configure_format(qspi_bus_width_t inst_width, qspi_bus_width_t address_width, qspi_address_size_t address_size, qspi_bus_width_t alt_width, qspi_alt_size_t alt_size, qspi_bus_width_t data_width, int dummy_cycles)
{ {
// Check that alt_size/alt_width are a valid combination // Check that alt_size/alt_width are a valid combination