boards: stm32h7s78_dk: fix SRAM region definition

Define the SRAM region for Ethernet descriptor and buffer
using memory-region = <...> in the mac node. Also add
netif:eth to ext_flash yaml board file and fix CI checks.

Signed-off-by: Benjamin Klaric <benjamin.klaric01@gmail.com>

Author: bklaric1

boards/st/stm32h7s78_dk/doc/index.rst

@@ -204,10 +204,9 @@ USB port2 (FS) is supported.
 Ethernet
 --------
 
-In order to use Ethernet on STM32H7S78-DK, you need to set the ``JP6`` jumper 
+In order to use Ethernet on STM32H7S78-DK, you need to set the ``JP6`` jumper
 to PC1 position on the back side of the board.
 
-
 Programming and Debugging
 *************************
 

boards/st/stm32h7s78_dk/stm32h7s78_dk-common.dtsi

@@ -11,6 +11,11 @@
 #include "zephyr/dt-bindings/display/panel.h"
 #include <zephyr/dt-bindings/input/input-event-codes.h>
 
+/*
+ * WARNING:
+ * JP6 must be in PC1 position when using Ethernet.
+ */
+
 / {
 	chosen {
 		zephyr,console = &uart4;
@@ -211,15 +216,16 @@
 
 &mac {
 	pinctrl-0 = <&eth_rmii_ref_clk_pd7
-			&eth_rmii_crs_dv_pa7
-			&eth_rmii_rxd0_pc4
-			&eth_rmii_rxd1_pc5
-			&eth_rmii_tx_en_pg11
-			&eth_rmii_txd0_pb0
-			&eth_rmii_txd1_pb1>;
+		     &eth_rmii_crs_dv_pa7
+			 &eth_rmii_rxd0_pc4
+			 &eth_rmii_rxd1_pc5
+			 &eth_rmii_tx_en_pg11
+			 &eth_rmii_txd0_pb0
+			 &eth_rmii_txd1_pb1>;
 	pinctrl-names = "default";
 	phy-connection-type = "rmii";
 	phy-handle = <&eth_phy>;
+	memory-region = <&sram1>;
 	status = "okay";
 };
 

boards/st/stm32h7s78_dk/stm32h7s78_dk_stm32h7s7xx_ext_flash_app.yaml

@@ -17,4 +17,5 @@ supported:
   - octospi
   - usbd
   - memc
+  - netif:eth
 vendor: st

dts/bindings/ethernet/st,stm32h7-ethernet.yaml

@@ -12,3 +12,10 @@ description: |
 compatible: "st,stm32h7-ethernet"
 
 include: st,stm32-ethernet-common.yaml
+
+properties:
+  memory-region:
+    type: phandle
+    required: true
+    description: |
+      Reference to SRAM node used for Ethernet DMA buffer and descriptor.

soc/st/stm32/stm32h7rsx/mpu_regions.c

@@ -31,12 +31,7 @@ static const struct arm_mpu_region mpu_regions[] = {
 	MPU_REGION_ENTRY("ID", 0x08FFF800, REGION_FLASH_ATTR(REGION_2K)),
 
 #if DT_NODE_HAS_STATUS_OKAY(DT_NODELABEL(mac))
-
-#if DT_NODE_HAS_STATUS_OKAY(DT_NODELABEL(sram2))
-#define sram_eth_node	DT_NODELABEL(sram2)
-#else
-#define sram_eth_node	DT_NODELABEL(sram1)
-#endif
+#define sram_eth_node DT_PHANDLE(DT_NODELABEL(mac), memory_region)
 
 #if DT_NODE_HAS_STATUS_OKAY(sram_eth_node)
 	/* Region 5 - Ethernet DMA buffer RAM */

soc/st/stm32/stm32h7rsx/sections.ld

@@ -5,12 +5,7 @@
  */
 
 #if DT_NODE_HAS_STATUS_OKAY(DT_NODELABEL(mac))
-
-#if DT_NODE_HAS_STATUS_OKAY(DT_NODELABEL(sram2))
-#define sram_eth_node	DT_NODELABEL(sram2)
-#else
-#define sram_eth_node	DT_NODELABEL(sram1)
-#endif
+#define sram_eth_node DT_PHANDLE(DT_NODELABEL(mac), memory_region)
 
 #if DT_NODE_HAS_STATUS_OKAY(sram_eth_node)
 SECTION_DATA_PROLOGUE(eth_stm32,(NOLOAD),)