Skip to content

Make modem peripheral work on esp32p4 with esp_wifi_remote.#572

Open
mdwn wants to merge 1 commit intoesp-rs:masterfrom
mdwn:esp32p4-modem-support
Open

Make modem peripheral work on esp32p4 with esp_wifi_remote.#572
mdwn wants to merge 1 commit intoesp-rs:masterfrom
mdwn:esp32p4-modem-support

Conversation

@mdwn
Copy link
Contributor

@mdwn mdwn commented Feb 18, 2026

Thank you for your contribution!

We appreciate the time and effort you've put into this pull request.
To help us review it efficiently, please ensure you've gone through the following checklist:

Submission Checklist 📝

  • I have updated existing examples or added new ones (if applicable).
  • I have used cargo fmt command to ensure that all changed code is formatted correctly.
  • I have used cargo clippy command to ensure that all changed code passes latest Clippy nightly lints.
  • My changes were added to the CHANGELOG.md in the proper section.

Pull Request Details 📖

Description

With esp_wifi_remote, the modem peripheral can work as expected on esp32p4. This peripheral is now gated around the presence of the esp_wifi_remote component for esp32p4.

Testing

Using a few other changes:

W (3766) rpc_utils: Event: SSID nerds
I (3787) osc_pedal: Starting WiFi...
I (3894) RPC_WRAP: ESP Event: wifi station started
I (3914) RPC_WRAP: ESP Event: wifi station started
I (4030) osc_pedal: WiFi started. Connecting...
I (4030) H_API: esp_wifi_remote_connect
I (6941) RPC_WRAP: ESP Event: Station mode: Connected
I (6941) esp_wifi_remote: esp_wifi_internal_reg_rxcb: sta: 0x40077b54
I (6969) osc_pedal: WiFi connected. Waiting for network interface...
I (7954) esp_netif_handlers: sta ip: 192.168.1.130, mask: 255.255.0.0, gw: 192.168.1.1
I (7982) osc_pedal: WiFi connected! IP: 192.168.1.130
I (7982) osc_pedal: WiFi test passed!

Related to:

With esp_wifi_remote, the modem peripheral can work as expected on
esp32p4. This peripheral is now gated around the presence of the
esp_wifi_remote component for esp32p4.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

Comments