adafruit-mirror/zephyr
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 9

drivers: spi: silabs: eusart: Always enable peripheral

Browse source 
The EUSART peripheral must always be enabled in configure(), as
its enable state gets cleared when the state machine is unretained
in deep sleep. The rest of the config registers are retained,
so they can continue to not be repainted on every configure() if
the context is the same as last time.

Remove unnecessary duplicated enable at the end of configure(),
the SPIInit HAL function already performs enable.

Signed-off-by: Aksel Skauge Mellbye <aksel.mellbye@silabs.com>
This commit is contained in:
Aksel Skauge Mellbye 2025-01-21 14:52:51 +01:00 committed by Benjamin Cabé
parent 12240a9058
commit 14a00f259f
1 changed files with 5 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
drivers/spi/spi_silabs_eusart.c
View file

@ -62,7 +62,11 @@ static int spi_silabs_eusart_configure(const struct device *dev, const struct sp
spi_frequency /= 2;
if (spi_context_configured(&data->ctx, config)) {
/* Already configured. No need to do it again. */
/* Already configured. No need to do it again, but must re-enable in case
* TXEN/RXEN were cleared due to deep sleep.
*/
EUSART_Enable(eusart_config->base, eusartEnable);
return 0;
}
@ -145,9 +149,6 @@ static int spi_silabs_eusart_configure(const struct device *dev, const struct sp
data->ctx.config = config;
/* Enable the peripheral */
eusart_config->base->CMD = (uint32_t)eusartEnable;
return 0;
}