boards: arduino: portenta_h7: Enable Ethernet and USB-HS Support.

rahul-arasikere · fabiobaltieri · commit d86334365f88 · 2024-06-11T11:38:41.000+01:00
Enable the mac node along with the necessary pinmuxing to enable the
ethernet and usb hs peripheral. I additionally defined the
`ethernet_phy_en` node the uses a fixed regulator to release
the PHY reset.

Additionally an usb_cdc_acm node was defined to enable serial over usb for
the console.

Signed-off-by: Rahul Arasikere &lt;arasikere.rahul@gmail.com&gt;
diff --git a/boards/arduino/portenta_h7/Kconfig.defconfig b/boards/arduino/portenta_h7/Kconfig.defconfig
@@ -0,0 +1,43 @@
+# Copyright 2024 Rahul Arasikere <arasikere.rahul@gmail.com>
+# SPDX-License-Identifier: Apache-2.0
+
+if BOARD_ARDUINO_PORTENTA_H7
+
+if NETWORKING
+
+config NET_L2_ETHERNET
+	default y
+
+endif # NETWORKING
+
+if USB_DEVICE_STACK
+
+config USB_DEVICE_PRODUCT
+	default "Arduino SA Portenta H7"
+
+config USB_DEVICE_PID
+	default 0x035b
+
+config USB_DEVICE_VID
+	default 0x2341
+
+config USB_DEVICE_INITIALIZE_AT_BOOT
+	default y
+
+if LOG
+
+# Logger cannot use itself to log
+choice USB_CDC_ACM_LOG_LEVEL_CHOICE
+	default USB_CDC_ACM_LOG_LEVEL_OFF
+endchoice
+
+# Set USB log level to error only
+choice USB_DEVICE_LOG_LEVEL_CHOICE
+	default USB_DEVICE_LOG_LEVEL_ERR
+endchoice
+
+endif # LOG
+
+endif # USB_DEVICE_STACK
+
+endif # BOARD_ARDUINO_PORTENTA_H7
diff --git a/boards/arduino/portenta_h7/arduino_portenta_h7-common.dtsi b/boards/arduino/portenta_h7/arduino_portenta_h7-common.dtsi
@@ -20,6 +20,12 @@
 		};
 	};
 
+	otghs_ulpi_phy: otghs_ulpis_phy {
+		compatible = "usb-ulpi-phy";
+		reset-gpios = < &gpioj 4 GPIO_ACTIVE_LOW >;
+		#phy-cells = <0>;
+	};
+
 	aliases {
 		led0 = &red_led;
 		led1 = &green_led;
@@ -115,12 +121,6 @@
 	status = "okay";
 };
 
-zephyr_udc0: &usbotg_fs {
-	pinctrl-0 = <&usb_otg_fs_dm_pa11 &usb_otg_fs_dp_pa12>;
-	pinctrl-names = "default";
-	status = "disabled";
-};
-
 &mailbox {
 	status = "okay";
 };
@@ -171,11 +171,11 @@ zephyr_udc0: &usbotg_fs {
 
 &quadspi {
 	pinctrl-0 = < &quadspi_bk1_io0_pd11
-			&quadspi_bk1_io1_pd12
-			&quadspi_bk1_io2_pf7
-			&quadspi_bk1_io3_pd13
-			&quadspi_bk1_ncs_pg6
-			&quadspi_clk_pf10 >;
+		      &quadspi_bk1_io1_pd12
+		      &quadspi_bk1_io2_pf7
+		      &quadspi_bk1_io3_pd13
+		      &quadspi_bk1_ncs_pg6
+		      &quadspi_clk_pf10 >;
 	pinctrl-names = "default";
 	status = "okay";
 
@@ -208,3 +208,42 @@ zephyr_udc0: &usbotg_fs {
 		};
 	};
 };
+
+&mac {
+	pinctrl-0 = < &eth_ref_clk_pa1
+		      &eth_mdio_pa2
+		      &eth_crs_dv_pa7
+		      &eth_mdc_pc1
+		      &eth_rxd0_pc4
+		      &eth_rxd1_pc5
+		      &eth_tx_en_pg11
+		      &eth_txd1_pg12
+		      &eth_txd0_pg13 >;
+	pinctrl-names = "default";
+};
+
+zephyr_udc0: &usbotg_hs {
+	pinctrl-0 = < &usb_otg_hs_ulpi_d0_pa3
+		      &usb_otg_hs_ulpi_ck_pa5
+		      &usb_otg_hs_ulpi_d1_pb0
+		      &usb_otg_hs_ulpi_d2_pb1
+		      &usb_otg_hs_ulpi_d7_pb5
+		      &usb_otg_hs_ulpi_d3_pb10
+		      &usb_otg_hs_ulpi_d4_pb11
+		      &usb_otg_hs_ulpi_d5_pb12
+		      &usb_otg_hs_ulpi_d6_pb13
+		      &usb_otg_hs_ulpi_stp_pc0
+		      &usb_otg_hs_ulpi_nxt_ph4
+		      &usb_otg_hs_ulpi_dir_pi11 >;
+	pinctrl-names = "default";
+	phys = < &otghs_ulpi_phy >;
+	maximum-speed = "high-speed";
+	/* Include the USB1ULPIEN | USB1OTGHSULPIEN clock enable bit */
+	clocks = <&rcc STM32_CLOCK_BUS_AHB1 0x6000000>,
+		 <&rcc STM32_SRC_HSI48 USB_SEL(3)>;
+	num-bidir-endpoints = < 4 >;
+	status = "okay";
+	cdc_acm_uart0: cdc_acm_uart0 {
+		compatible = "zephyr,cdc-acm-uart";
+	};
+};
diff --git a/boards/arduino/portenta_h7/arduino_portenta_h7_stm32h747xx_m7.dts b/boards/arduino/portenta_h7/arduino_portenta_h7_stm32h747xx_m7.dts
@@ -15,8 +15,9 @@
 
 	/* HW resources are split between CM7 and CM4 */
 	chosen {
-		zephyr,console = &usart1;
-		zephyr,shell-uart = &usart1;
+		zephyr,console = &cdc_acm_uart0;
+		zephyr,shell-uart = &cdc_acm_uart0;
+		zephyr,cdc-acm-uart0 = &cdc_acm_uart0;
 		zephyr,sram = &sram0;
 		zephyr,flash = &flash0;
 		zephyr,code-partition = &slot0_partition;
@@ -30,6 +31,14 @@
 		status = "okay";
 	};
 
+	ethernet_phy_en: ethernet_phy_en {
+		compatible = "regulator-fixed";
+		regulator-name = "ethernet-phy-reset-release";
+		enable-gpios = <&gpioj 15 GPIO_ACTIVE_HIGH>;
+		regulator-boot-on;
+		status = "okay";
+	};
+
 	sdram1: sdram@c0000000 {
 		compatible = "zephyr,memory-region", "mmio-sram";
 		device_type = "memory";
@@ -80,7 +89,16 @@
 	status = "okay";
 };
 
+/* Only one should be enabled */
 &usbotg_fs {
+	status = "disabled";
+};
+
+&usbotg_hs {
+	status = "okay";
+};
+
+&cdc_acm_uart0 {
 	status = "okay";
 };
 
diff --git a/boards/arduino/portenta_h7/arduino_portenta_h7_stm32h747xx_m7_defconfig b/boards/arduino/portenta_h7/arduino_portenta_h7_stm32h747xx_m7_defconfig
@@ -30,7 +30,11 @@ CONFIG_SERIAL=y
 # Enable console
 CONFIG_CONSOLE=y
 CONFIG_UART_CONSOLE=y
+CONFIG_UART_LINE_CTRL=y
 
 # Enable regulator
 CONFIG_REGULATOR=y
 CONFIG_REGULATOR_FIXED=y
+
+# Enable USB Stack
+CONFIG_USB_DEVICE_STACK=y
