Skip to content

Commit 678f3f9

Browse files
committed
TARGET_NRF: corrected spi_init() to properly handle re-initialization on the same pins + minor editorial corrections.
1 parent 7fc73e4 commit 678f3f9

File tree

1 file changed

+21
-7
lines changed

1 file changed

+21
-7
lines changed

targets/TARGET_NORDIC/TARGET_NRF5/spi_api.c

Lines changed: 21 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -90,22 +90,22 @@ void SPI0_TWI0_IRQHandler(void);
9090
void SPI1_TWI1_IRQHandler(void);
9191
void SPIM2_SPIS2_SPI2_IRQHandler(void);
9292

93-
static const peripheral_handler_desc_t spi_hanlder_desc[SPI_COUNT] = {
93+
static const peripheral_handler_desc_t spi_handler_desc[SPI_COUNT] = {
9494
#if SPI0_ENABLED
9595
{
96-
SPIS0_IRQ,
96+
SPI0_IRQ,
9797
(uint32_t) SPI0_TWI0_IRQHandler
9898
},
9999
#endif
100100
#if SPI1_ENABLED
101101
{
102-
SPIS1_IRQ,
102+
SPI1_IRQ,
103103
(uint32_t) SPI1_TWI1_IRQHandler
104104
},
105105
#endif
106106
#if SPI2_ENABLED
107107
{
108-
SPIS2_IRQ,
108+
SPI2_IRQ,
109109
(uint32_t) SPIM2_SPIS2_SPI2_IRQHandler
110110
},
111111
#endif
@@ -254,12 +254,24 @@ void spi_init(spi_t *obj,
254254
PinName mosi, PinName miso, PinName sclk, PinName ssel)
255255
{
256256
int i;
257+
for (i = 0; i < SPI_COUNT; ++i) {
258+
spi_info_t *p_spi_info = &m_spi_info[i];
259+
if (p_spi_info->initialized &&
260+
p_spi_info->mosi_pin == (uint8_t)mosi &&
261+
p_spi_info->miso_pin == (uint8_t)miso &&
262+
p_spi_info->sck_pin == (uint8_t)sclk &&
263+
p_spi_info->ss_pin == (uint8_t)ssel) {
264+
// Reuse the already allocated SPI instance (instead of allocating
265+
// a new one), if it appears to be initialized with exactly the same
266+
// pin assignments.
267+
SPI_IDX(obj) = i;
268+
return;
269+
}
270+
}
271+
257272
for (i = 0; i < SPI_COUNT; ++i) {
258273
spi_info_t *p_spi_info = &m_spi_info[i];
259274
if (!p_spi_info->initialized) {
260-
261-
NVIC_SetVector(spi_hanlder_desc[i].IRQn, spi_hanlder_desc[i].vector);
262-
263275
p_spi_info->sck_pin = (uint8_t)sclk;
264276
p_spi_info->mosi_pin = (mosi != NC) ?
265277
(uint8_t)mosi : NRF_DRV_SPI_PIN_NOT_USED;
@@ -270,6 +282,8 @@ void spi_init(spi_t *obj,
270282
p_spi_info->spi_mode = (uint8_t)NRF_DRV_SPI_MODE_0;
271283
p_spi_info->frequency = NRF_DRV_SPI_FREQ_1M;
272284

285+
NVIC_SetVector(spi_handler_desc[i].IRQn, spi_handler_desc[i].vector);
286+
273287
// By default each SPI instance is initialized to work as a master.
274288
// Should the slave mode be used, the instance will be reconfigured
275289
// appropriately in 'spi_format'.

0 commit comments

Comments
 (0)