Skip to content

Conversation

@djiatsaf-st
Copy link
Contributor

This PR adds SPI peripherals to work with STM32C011/31xx boards.

- stm32cO11/31 share the same spi peripheral

- include stm32_dma header to be able to configure
spi with dma config macros (STM32_DMA_PERIPH_TX,...)
in dts

Signed-off-by: Fabrice DJIATSA <[email protected]>
@JarmouniA
Copy link
Contributor

@djiatsaf-st Can we actually test it? See #73601 (comment)

@djiatsaf-st
Copy link
Contributor Author

@djiatsaf-st Can we actually test it? See #73601 (comment)

Hello @JarmouniA ,

We still have a flash/ram overflow when testing on spi_loopback.

However, to check that everything is configured correctly on the C011/31 series, I created a sample based on the spi_loopback test by keeping only the necessary functions (spi_complete_loop, spi_complete_multiple) so as not to exceed the size of the ram and flash.

Soon we will add the C071 serie which has a larger ram and flash size

Copy link
Member

@erwango erwango left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's remove the board addition.
Since we won't be able to test nor build it this will remain dead code.

We'll ensure to enable test once C071 board will be available upstream.

@erwango erwango added this to the v4.1.0 milestone Nov 15, 2024
#include <zephyr/dt-bindings/adc/adc.h>
#include <zephyr/dt-bindings/clock/stm32c0_clock.h>
#include <zephyr/dt-bindings/gpio/gpio.h>
#include <zephyr/dt-bindings/dma/stm32_dma.h>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Required when the spi1 will have a DMA channel PHandle

@nashif nashif merged commit 94a6ed6 into zephyrproject-rtos:main Nov 19, 2024
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

platform: STM32 ST Micro STM32

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants