Skip to content

arch/risc-v: Support the ethernet driver on ESP32-P4#18827

Merged
xiaoxiang781216 merged 4 commits into
apache:masterfrom
tmedicci:feature/esp32p4_eth
May 5, 2026
Merged

arch/risc-v: Support the ethernet driver on ESP32-P4#18827
xiaoxiang781216 merged 4 commits into
apache:masterfrom
tmedicci:feature/esp32p4_eth

Conversation

@tmedicci
Copy link
Copy Markdown
Contributor

@tmedicci tmedicci commented Apr 30, 2026

Summary

Enables the ethernet driver support on ESP32-P4 based on a common-source implementation. This required a set of changes regarding the event handler for the Wi-Fi driver (which is now shared with the ethernet driver).

Impact

Impact on user: Yes. Users can now use Ethernet on ESP32-P4.

Impact on build: No.

Impact on hardware: Yes. Mostly ESP32-P4, but all the supported SoCs from Espressif required small changes and optimization.

Impact on documentation: Yes. Documentation for the Ethernet network was properly added to ESP32-P4 chip its Functio EV board.

Impact on security: No.

Impact on compatibility: No.

Testing

esp32p4-function-ev-board:ethernet defconfig can be used to test Ethernet on ESP32-P4. iperf testing is also possible. Please note that for v1.5.2 of the board, it's easier to flash using the USB-Serial/JTAG port (usually /dev/ttyACM) and use an externally connected USB-to-serial converter attached to UART0.

Building

make -j distclean
./tools/configure.sh -S esp32p4-function-ev-board:ethernet
make flash ESPTOOL_PORT=/dev/ttyACM0 -s -j$(nproc)
picocom -b 115200 /dev/ttyUSB1

Running

Checking eth0 with DHCP client enabled:

Attach the ethernet cable to the board and check the interface status and its IP address (please ensure that the cable is connected to a network with a DHCP server enabled):

nsh> ifconfig
eth0	Link encap:Ethernet HWaddr 30:ed:a0:ec:f1:60 at RUNNING mtu 1500
	inet addr:10.0.10.50 DRaddr:10.0.10.1 Mask:255.255.255.0

iperf testing:

Both UDP and TCP modes can be tested.

UDP

On the host PC, run the iperf server:

iperf -u -s -p 11111 -i 1

On the device, run the iperf client:

nsh> iperf -u -c <host_pc_ip_address> -p 11111 -i 1 -t 5 &
TCP

On the host PC, run the iperf server:

iperf -s -p 11111 -i 1

On the device, run the iperf client:

nsh> iperf -c <host_pc_ip_address> -p 11111 -i 1 -t 5 &

Results

For ifconfig, the network's status should be RUNNING and it should acquire a valid IP address.

iperf's results are the following. Both are closed to the port's bandwidth limit of 100Mbits/sec.

UDP

nsh>      IP: 10.0.10.50

 mode=udp-client sip=10.0.10.50:5001,dip=10.0.10.81:11111, interval=1, time=5

           Interval         Transfer         Bandwidth

   0.00-   1.01 sec   11992384 Bytes   94.99 Mbits/sec
   1.01-   2.02 sec   12082176 Bytes   95.70 Mbits/sec
   2.02-   3.03 sec   12083648 Bytes   95.71 Mbits/sec
   3.03-   4.04 sec   12082176 Bytes   95.70 Mbits/sec
   4.04-   5.05 sec   12082176 Bytes   95.70 Mbits/sec
   0.00-   5.05 sec   60322560 Bytes   95.56 Mbits/sec
iperf exit

TCP

nsh>      IP: 10.0.10.50

 mode=tcp-client sip=10.0.10.50:5001,dip=10.0.10.81:11111, interval=1, time=5

           Interval         Transfer         Bandwidth

   0.00-   1.01 sec   10256384 Bytes   81.24 Mbits/sec
   1.01-   2.02 sec   10338304 Bytes   81.89 Mbits/sec
   2.02-   3.03 sec   10338304 Bytes   81.89 Mbits/sec
   3.03-   4.04 sec   10338304 Bytes   81.89 Mbits/sec
   4.04-   5.05 sec   10338304 Bytes   81.89 Mbits/sec
   0.00-   5.05 sec   51609600 Bytes   81.76 Mbits/sec
iperf exit

@github-actions github-actions Bot added Area: Documentation Improvements or additions to documentation Arch: risc-v Issues related to the RISC-V (32-bit or 64-bit) architecture Arch: xtensa Issues related to the Xtensa architecture Size: XL The size of the change in this PR is very large. Consider breaking down the PR into smaller pieces. Board: risc-v labels Apr 30, 2026
@acassis acassis requested review from cederom and linguini1 April 30, 2026 19:07
cederom
cederom previously approved these changes Apr 30, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@cederom cederom left a comment

Choose a reason for hiding this comment

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

Big Thank You @tmedicci :-)

linguini1
linguini1 previously approved these changes Apr 30, 2026
View reviewed changes
simbit18
simbit18 previously approved these changes May 1, 2026
View reviewed changes
xiaoxiang781216
xiaoxiang781216 reviewed May 1, 2026
View reviewed changes
Comment thread arch/risc-v/src/common/espressif/esp_emac.c Outdated
jerpelea
jerpelea previously approved these changes May 4, 2026
View reviewed changes
@tmedicci tmedicci dismissed stale reviews from jerpelea, simbit18, linguini1, and cederom via 47fd70e May 4, 2026 15:27
@tmedicci tmedicci force-pushed the feature/esp32p4_eth branch from 09cdbe1 to 47fd70e Compare May 4, 2026 15:27
@tmedicci tmedicci force-pushed the feature/esp32p4_eth branch from 47fd70e to b844164 Compare May 4, 2026 16:56
cederom
cederom previously approved these changes May 4, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@cederom cederom left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Thank you @tmedicci :-)

I understand this is initial implementation and will change in future, also will change because esp-sdk will change.

I agree with @xiaoxiang781216 to be as aligned with existing NuttX architecture as possible :-)

I do not have this board to test sorry.

@tmedicci tmedicci force-pushed the feature/esp32p4_eth branch from b844164 to 43414c1 Compare May 4, 2026 20:28
@lupyuen
Copy link
Copy Markdown
Member

lupyuen commented May 5, 2026

Please Rebase your PR with the Master Branch, we just fixed weact-stm32h750 thanks :-)

tmedicci added 4 commits May 5, 2026 08:44
@tmedicci
arch/risc-v: Support the ethernet driver on ESP32-P4
a186ab4 
This commit enables the ethernet driver support on ESP32-P4 based
on a common-source implementation. This required a set of changes
regargind the event handler for the Wi-Fi driver (which is now
shared with the ethernet driver).

Signed-off-by: Tiago Medicci Serrano <tiago.medicci@espressif.com>
@tmedicci
arch/xtensa: Update common-source base for ESP32[|S2|S3]
1ed98a9 
After recent changes on the event handler for the RISC-V-based
Espressif SoCs, the same set of changes were ported back to xtensa
devices.

Signed-off-by: Tiago Medicci Serrano <tiago.medicci@espressif.com>
@tmedicci
boards/risc-v: Enable ethernet on ESP32-P4-Function-EV-Board
61d0c5d 
This commit simply enables the ethernet support for the ESP32-P4's
Function EV board.

Signed-off-by: Tiago Medicci Serrano <tiago.medicci@espressif.com>
@tmedicci
Documentation: Describe ESP32-P4 Ethernet support
5193394 
This commit describes ethernet support on ESP32-P4 SoC and its
ESP32-P4-Function-EV-Board v1.5.2.

Signed-off-by: Tiago Medicci Serrano <tiago.medicci@espressif.com>
@tmedicci tmedicci force-pushed the feature/esp32p4_eth branch from 43414c1 to 5193394 Compare May 5, 2026 11:50
@tmedicci tmedicci requested review from cederom and simbit18 May 5, 2026 17:26
@xiaoxiang781216 xiaoxiang781216 merged commit 20c38d0 into apache:master May 5, 2026
42 checks passed
@xiaoxiang781216
Copy link
Copy Markdown
Contributor

xiaoxiang781216 commented May 9, 2026

@tmedicci this patch generate ci error, could you fix it:
https://github.com/apache/nuttx/actions/runs/25543549196/job/74974737920?pr=18856

====================================================================================
Configuration/Tool: esp32p4-function-ev-board/ethernet
2026-05-08 09:57:02
------------------------------------------------------------------------------------
  Cleaning...
  Configuring...
  Building NuttX...
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed

  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0

100 12395    0 12395    0     0  26408      0 --:--:-- --:--:-- --:--:-- 26408
100  496k    0  496k    0     0   794k      0 --:--:-- --:--:-- --:--:-- 3123k
  [1/1] Normalize esp32p4-function-ev-board/ethernet
55d54
< CONFIG_NSH_ARCHINIT=y
Saving the new configuration file
HEAD detached at pull/18856/merge
Changes not staged for commit:
  (use "git add <file>..." to update what will be committed)
  (use "git restore <file>..." to discard changes in working directory)
	modified:   boards/risc-v/esp32p4/esp32p4-function-ev-board/configs/ethernet/defconfig

@tmedicci
Copy link
Copy Markdown
Contributor Author

tmedicci commented May 12, 2026

@tmedicci this patch generate ci error, could you fix it: https://github.com/apache/nuttx/actions/runs/25543549196/job/74974737920?pr=18856

====================================================================================
Configuration/Tool: esp32p4-function-ev-board/ethernet
2026-05-08 09:57:02
------------------------------------------------------------------------------------
  Cleaning...
  Configuring...
  Building NuttX...
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed

  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0

100 12395    0 12395    0     0  26408      0 --:--:-- --:--:-- --:--:-- 26408
100  496k    0  496k    0     0   794k      0 --:--:-- --:--:-- --:--:-- 3123k
  [1/1] Normalize esp32p4-function-ev-board/ethernet
55d54
< CONFIG_NSH_ARCHINIT=y
Saving the new configuration file
HEAD detached at pull/18856/merge
Changes not staged for commit:
  (use "git add <file>..." to update what will be committed)
  (use "git restore <file>..." to discard changes in working directory)
	modified:   boards/risc-v/esp32p4/esp32p4-function-ev-board/configs/ethernet/defconfig

I'll check it. Probably just pending normalizing.

@lupyuen
Copy link
Copy Markdown
Member

lupyuen commented May 12, 2026

@tmedicci This has been fixed thanks :-)

@cederom
Copy link
Copy Markdown
Contributor

cederom commented May 12, 2026

Just got the https://www.waveshare.com/esp32-p4-nano.htm it has the ETH port will play in a moment :-)

@cederom
Copy link
Copy Markdown
Contributor

cederom commented May 13, 2026

No luck in first contact, #18873, help @tmedicci :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jerpelea jerpelea jerpelea approved these changes

@xiaoxiang781216 xiaoxiang781216 xiaoxiang781216 approved these changes

@fdcavalcanti fdcavalcanti Awaiting requested review from fdcavalcanti fdcavalcanti is a code owner

@linguini1 linguini1 Awaiting requested review from linguini1

@eren-terzioglu eren-terzioglu Awaiting requested review from eren-terzioglu

@cederom cederom Awaiting requested review from cederom

@simbit18 simbit18 Awaiting requested review from simbit18

Assignees

No one assigned

Labels

Arch: risc-v Issues related to the RISC-V (32-bit or 64-bit) architecture Arch: xtensa Issues related to the Xtensa architecture Area: Documentation Improvements or additions to documentation Board: risc-v Size: XL The size of the change in this PR is very large. Consider breaking down the PR into smaller pieces.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

8 participants

@tmedicci @lupyuen @xiaoxiang781216 @cederom @jerpelea @linguini1 @eren-terzioglu @simbit18