Skip to content

Bring FPGA SPI tests to feature-hal-spec-spi #10816

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 9 commits into from
Jul 8, 2019
Merged

Bring FPGA SPI tests to feature-hal-spec-spi #10816

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
0f611a8
K64F SPI driver: add limitation for supported clock modes
mprse Jun 10, 2019
3ac2593
Bring FPGA SPI tests into SPI feature branch
mprse Jun 6, 2019
7333acf
Add documentation for FPGA-Test-Shield SPI tests (new HAL API)
mprse Jun 12, 2019
b92b497
Add header files for fpga spi tests and map test cases to defined beh…
mprse Jul 3, 2019
372d848
Disable nanostack-border-router example for NUCLEO_F429ZI
mprse Jul 5, 2019
bd8142e
Restrict tests to Arduino FF only and remove unused symbol
mprse Jul 5, 2019
13fc600
Disable nanostack-border-router example for K66F
mprse Jul 8, 2019
b9218c8
TARGET_TMPM4G9/spi_api.c: Add missing #endif
mprse Jul 8, 2019
b06b57c
EP_AGORA: Remove SPIF component.
mprse Jul 8, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
38 changes: 19 additions & 19 deletions TESTS/mbed_hal/spi/main.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -176,7 +176,7 @@ static void async_handler(spi_t *obj, void *ctx, spi_async_event_t *event)

/* Test that spi_get_module() returns the SPIName - unique identifier to the peripheral associated
* to this SPI channel. */
void test_get_module()
void spi_test_get_module()
{
SPIName spi_name;

Expand All @@ -193,7 +193,7 @@ void test_get_module()

/* Test that spi_get_capabilities() fills the given spi_capabilities_t instance with capabilities
* of the specified SPI peripheral. */
void test_get_capabilities()
void spi_test_get_capabilities()
{
spi_capabilities_t capabilities;

Expand Down Expand Up @@ -229,7 +229,7 @@ void test_get_capabilities()

/* Test that spi_init() successfully initializes the pins and spi_free() can successfully
* reset the pins to their default state. */
void test_init_free()
void spi_test_init_free()
{
spi_t spi_obj = { 0 };
spi_capabilities_t capabilities = { 0 };
Expand Down Expand Up @@ -295,7 +295,7 @@ void test_init_free()
}

/* Test that spi_format() sets/updates the transmission format of the SPI peripheral. */
void test_set_format()
void spi_test_set_format()
{
spi_t spi_obj = { 0 };
spi_capabilities_t capabilities = { 0 };
Expand Down Expand Up @@ -343,8 +343,8 @@ void test_set_format()
TEST_ASSERT_TRUE(true);
}

/* Test that test_set_frequency() sets the frequency used during the SPI transfer. */
void test_set_frequency()
/* Test that spi_test_set_frequency() sets the frequency used during the SPI transfer. */
void spi_test_set_frequency()
{
spi_t spi_obj = { 0 };
uint32_t real_freq;
Expand Down Expand Up @@ -372,7 +372,7 @@ void test_set_frequency()
/* Test that spi_transfer() can successfully perform transfer in master mode
* (TX/RX buffers are defined and have the same sizes) and returns the number of
* symbols clocked on the bus during this transfer. */
void test_transfer_master()
void spi_test_transfer_master()
{
spi_t spi_obj = { 0 };
spi_capabilities_t capabilities = { 0 };
Expand Down Expand Up @@ -420,7 +420,7 @@ void test_transfer_master()
/* Test that spi_transfer() can successfully perform transfer in master mode
* (TX/RX buffers are undefined or have different sizes) and returns the number of
* symbols clocked on the bus during this transfer. */
void test_transfer_master_fill_sym()
void spi_test_transfer_master_fill_sym()
{
spi_t spi_obj = { 0 };
spi_capabilities_t capabilities = { 0 };
Expand Down Expand Up @@ -457,7 +457,7 @@ void test_transfer_master_fill_sym()
#ifdef DEVICE_SPI_ASYNCH
/* Test that spi_transfer_async() can successfully perform asynchronous transfer
* in master mode. */
void test_transfer_master_async()
void spi_test_transfer_master_async()
{
spi_t spi_obj = {0};
spi_capabilities_t capabilities = {0};
Expand Down Expand Up @@ -521,7 +521,7 @@ void test_transfer_master_async()
}

/* Test that spi_transfer_async_abort() can successfully abort an on-going async transfer. */
void test_transfer_master_async_abort()
void spi_test_transfer_master_async_abort()
{
spi_t spi_obj = {0};
spi_capabilities_t capabilities = {0};
Expand Down Expand Up @@ -615,16 +615,16 @@ utest::v1::status_t greentea_failure_handler_abort(const Case *const source, con
}

Case cases[] = {
Case("SPI - get module name test", test_get_module, greentea_failure_handler),
Case("SPI - get capabilities test", test_get_capabilities, greentea_failure_handler),
Case("SPI - init, free test", test_init_free, greentea_failure_handler),
Case("SPI - set format test", test_set_format, greentea_failure_handler),
Case("SPI - set frequency test", test_set_frequency, greentea_failure_handler),
Case("SPI - master transfer test", test_transfer_master, greentea_failure_handler),
Case("SPI - fill symbol test", test_transfer_master_fill_sym, greentea_failure_handler),
Case("SPI - get module name test", spi_test_get_module, greentea_failure_handler),
Case("SPI - get capabilities test", spi_test_get_capabilities, greentea_failure_handler),
Case("SPI - init, free test", spi_test_init_free, greentea_failure_handler),
Case("SPI - set format test", spi_test_set_format, greentea_failure_handler),
Case("SPI - set frequency test", spi_test_set_frequency, greentea_failure_handler),
Case("SPI - master transfer test", spi_test_transfer_master, greentea_failure_handler),
Case("SPI - fill symbol test", spi_test_transfer_master_fill_sym, greentea_failure_handler),
#ifdef DEVICE_SPI_ASYNCH
Case("SPI - master transfer async test", test_transfer_master_async, greentea_failure_handler),
Case("SPI - master transfer async abort test", test_transfer_master_async_abort, greentea_failure_handler),
Case("SPI - master transfer async test", spi_test_transfer_master_async, greentea_failure_handler),
Case("SPI - master transfer async abort test", spi_test_transfer_master_async_abort, greentea_failure_handler),
#endif
};

Expand Down
22 changes: 11 additions & 11 deletions TESTS/mbed_hal/spi/spi_api_tests.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ extern "C" {
*
* @note `mosi` or `miso` pin can be undefined (but not both). `mclk` must be defined.
*/
void test_get_module();
void spi_test_get_module();

/** Test that spi_get_capabilities() fills the given spi_capabilities_t instance with capabilities
* of the specified SPI peripheral.
Expand All @@ -52,7 +52,7 @@ void test_get_module();
* support_slave_mode capability.
*
*/
void test_get_capabilities();
void spi_test_get_capabilities();

/** Test that spi_init() successfully initializes the pins and spi_free() can successfully
* reset the pins to their default state.
Expand All @@ -63,7 +63,7 @@ void test_get_capabilities();
* Then both operations are successfully performed.
*
*/
void test_init_free();
void spi_test_init_free();

/** Test that spi_format() sets/updates the transmission format of the SPI peripheral.
*
Expand All @@ -72,19 +72,19 @@ void test_init_free();
* Then function is executed successfully.
*
*/
void test_set_format();
void spi_test_set_format();

/** Test that test_set_frequency() sets the frequency used during the SPI transfer.
/** Test that spi_test_set_frequency() sets the frequency used during the SPI transfer.
*
* Given is platform with SPI support.
* When test_set_frequency() is called and valid frequency is specified.
* When spi_test_set_frequency() is called and valid frequency is specified.
* Then function is executed successfully and actual frequency which will be used during the transfer is returned.
*
* @note Frequency can be only set by SPI peripheral operating in master mode.
* @note Frequency must be in range specified by the capabilities of the SPI peripheral.
*
*/
void test_set_frequency();
void spi_test_set_frequency();

/** Test that spi_transfer() can successfully perform transfer in master mode
* (TX/RX buffers are defined and have the same sizes) and returns the number of
Expand All @@ -96,7 +96,7 @@ void test_set_frequency();
* Then function is executed successfully and returns number of symbols clocked on the bus during this transfer.
*
*/
void test_transfer_master();
void spi_test_transfer_master();

/** Test that spi_transfer() can successfully perform transfer in master mode
* (TX/RX buffers are undefined or have different sizes) and returns the number of
Expand All @@ -108,7 +108,7 @@ void test_transfer_master();
* Then function is executed successfully and returns number of symbols clocked on the bus during this transfer.
*
*/
void test_transfer_master_fill_sym();
void spi_test_transfer_master_fill_sym();

/** Test that spi_transfer_async() can successfully perform asynchronous transfer
* in master mode.
Expand All @@ -119,7 +119,7 @@ void test_transfer_master_fill_sym();
* when the transfer completes (passing the operation status).
*
*/
void test_transfer_master_async();
void spi_test_transfer_master_async();

/** Test that spi_transfer_async_abort() can successfully abort an on-going async transfer.
*
Expand All @@ -130,7 +130,7 @@ void test_transfer_master_async();
* the status which indicates that the transfer has been aborted.
*
*/
void test_transfer_master_async_abort();
void spi_test_transfer_master_async_abort();

/**@}*/

Expand Down
Loading