From 99e12cbf4ad1c48133f70625f1d42ff8025f588b Mon Sep 17 00:00:00 2001 From: Alain Volmat Date: Wed, 7 May 2025 21:19:02 +0200 Subject: [PATCH] dts: st: h7: move dma property of dcmi in stm32h7.dtsi Usage of dma is mandatory for the dcmi and this property is tightly coupled with the soc itself since the configuration of the dma depends on the source/destination, and the request line is also fixed for an ip. Instead of having to always have the dma property part of the board or shield dts/overlay, add the dma property into the dcmi node of the stm32h7.dtsi. Signed-off-by: Alain Volmat --- .../nicla_vision/arduino_nicla_vision_stm32h747xx_m7.dts | 4 ---- .../st_b_cams_omv_mb1683/boards/stm32h7b3i_dk.overlay | 4 ---- .../weact_ov2640_cam_module/boards/mini_stm32h743.overlay | 6 ------ dts/arm/st/h7/stm32h7.dtsi | 3 +++ 4 files changed, 3 insertions(+), 14 deletions(-) diff --git a/boards/arduino/nicla_vision/arduino_nicla_vision_stm32h747xx_m7.dts b/boards/arduino/nicla_vision/arduino_nicla_vision_stm32h747xx_m7.dts index df2ece15f0e..e9f39b173ae 100644 --- a/boards/arduino/nicla_vision/arduino_nicla_vision_stm32h747xx_m7.dts +++ b/boards/arduino/nicla_vision/arduino_nicla_vision_stm32h747xx_m7.dts @@ -240,10 +240,6 @@ zephyr_udc0: &usbotg_hs { pinctrl-names = "default"; status = "okay"; - dmas = <&dma1 0 38 (STM32_DMA_PERIPH_TO_MEMORY | STM32_DMA_PERIPH_NO_INC | - STM32_DMA_MEM_INC | STM32_DMA_PERIPH_8BITS | STM32_DMA_MEM_32BITS | - STM32_DMA_PRIORITY_HIGH) STM32_DMA_FIFO_1_4>; - port { dcmi_ep_in: endpoint { remote-endpoint-label = "gc2145_ep_out"; diff --git a/boards/shields/st_b_cams_omv_mb1683/boards/stm32h7b3i_dk.overlay b/boards/shields/st_b_cams_omv_mb1683/boards/stm32h7b3i_dk.overlay index c6e629d2342..19b9aaa6eaf 100644 --- a/boards/shields/st_b_cams_omv_mb1683/boards/stm32h7b3i_dk.overlay +++ b/boards/shields/st_b_cams_omv_mb1683/boards/stm32h7b3i_dk.overlay @@ -17,10 +17,6 @@ &dcmi_d0_pc6 &dcmi_d1_pc7 &dcmi_d2_pg10 &dcmi_d3_pc9 &dcmi_d4_pc11 &dcmi_d5_pd3 &dcmi_d6_pb8 &dcmi_d7_pb9>; pinctrl-names = "default"; - - dmas = <&dma1 0 75 (STM32_DMA_PERIPH_TO_MEMORY | STM32_DMA_PERIPH_NO_INC | - STM32_DMA_MEM_INC | STM32_DMA_PERIPH_8BITS | STM32_DMA_MEM_32BITS | - STM32_DMA_PRIORITY_HIGH) STM32_DMA_FIFO_1_4>; }; &dma1 { diff --git a/boards/shields/weact_ov2640_cam_module/boards/mini_stm32h743.overlay b/boards/shields/weact_ov2640_cam_module/boards/mini_stm32h743.overlay index e9923727607..80e3dbc9620 100644 --- a/boards/shields/weact_ov2640_cam_module/boards/mini_stm32h743.overlay +++ b/boards/shields/weact_ov2640_cam_module/boards/mini_stm32h743.overlay @@ -44,12 +44,6 @@ }; }; -&zephyr_camera_dvp { - dmas = <&dma1 0 75 (STM32_DMA_PERIPH_TO_MEMORY | STM32_DMA_PERIPH_NO_INC | - STM32_DMA_MEM_INC | STM32_DMA_PERIPH_8BITS | STM32_DMA_MEM_32BITS | - STM32_DMA_PRIORITY_HIGH) STM32_DMA_FIFO_1_4>; -}; - &dma1 { status = "okay"; }; diff --git a/dts/arm/st/h7/stm32h7.dtsi b/dts/arm/st/h7/stm32h7.dtsi index 409d0bb1edf..635f859b322 100644 --- a/dts/arm/st/h7/stm32h7.dtsi +++ b/dts/arm/st/h7/stm32h7.dtsi @@ -1086,6 +1086,9 @@ interrupts = <78 0>; interrupt-names = "dcmi"; clocks = <&rcc STM32_CLOCK(AHB2, 0U)>; + dmas = <&dma1 0 75 (STM32_DMA_PERIPH_TO_MEMORY | STM32_DMA_PERIPH_NO_INC | + STM32_DMA_MEM_INC | STM32_DMA_PERIPH_8BITS | STM32_DMA_MEM_32BITS | + STM32_DMA_PRIORITY_HIGH) STM32_DMA_FIFO_1_4>; status = "disabled"; }; };