michaelh/zephyr
1
0
Fork 0
Code Releases Activity

emul: Add stub release impl. for emulated SPI driver

Browse source 
`release` is a mandatory method in the `struct spi_driver_api` API but
is not implemented in the SPI emulator. This can cause a test calling
`spi_release()` to segfault. Add a stub implementation that just returns
zero.

Signed-off-by: Tristan Honscheid <honscheid@google.com>
This commit is contained in:
Tristan Honscheid 2024-01-08 12:20:32 -07:00 • committed by Carles Cufí
commit 0198b7d5f7
1 changed files with 13 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

13
drivers/spi/spi_emul.c
View file

@ -81,6 +81,18 @@ static int spi_emul_io(const struct device *dev, const struct spi_config *config
return api->io(emul->target, config, tx_bufs, rx_bufs);
}
/**
* @brief This is a no-op stub of the SPI API's `release` method to protect drivers under test
* from hitting a segmentation fault when using SPI_LOCK_ON plus spi_release()
*/
static int spi_emul_release(const struct device *dev, const struct spi_config *config)
{
ARG_UNUSED(dev);
ARG_UNUSED(config);
return 0;
}
/**
* Set up a new emulator and add it to the list
*
@ -111,6 +123,7 @@ int spi_emul_register(const struct device *dev, struct spi_emul *emul)
static const struct spi_driver_api spi_emul_api = {
.transceive = spi_emul_io,
.release = spi_emul_release,
};
#define EMUL_LINK_AND_COMMA(node_id) \