Skip to content

Feat/jc esp32p4 m3 dev#53

Open
dvucinozd wants to merge 16 commits into
espressif:masterfrom
dvucinozd:feat/jc-esp32p4-m3-dev
Open

Feat/jc esp32p4 m3 dev#53
dvucinozd wants to merge 16 commits into
espressif:masterfrom
dvucinozd:feat/jc-esp32p4-m3-dev

Conversation

@dvucinozd

@dvucinozd dvucinozd commented May 7, 2026

Copy link
Copy Markdown

Description

This PR adds and hardens support for the Guiton JC-ESP32P4-M3-DEV board in esp-claw using the esp_board_manager workflow.

What changed

  • Added/updated board definition files under:
    • application/edge_agent/boards/guiton/jc_esp32p4_m3_dev/
    • board_info.yaml
    • board_peripherals.yaml
    • board_devices.yaml
    • setup_device.c
    • sdkconfig.defaults.board
  • Added optional OV5647 low-light tuning profile:
    • ov5647_low_light.json
  • Updated board README to be upstream-ready (no local machine paths, reproducible validation command).
  • Integrated and validated the board through Board Manager generation flow:
    • idf.py bmgr -c ./boards -b jc_esp32p4_m3_dev
  • Stabilized runtime bring-up:
    • Disabled fs_spiffs device bring-up for this board profile (SD card FAT is primary storage).
    • Time-sync startup behavior was improved to wait for real STA connectivity before SNTP sync.
  • Camera path improvements:
    • Updated OV5647 defaults and JPEG quality settings for better captured image quality.

Motivation / context

This board is a new hardware target and needed complete board metadata + stable runtime behavior before upstreaming.
The goal is reliable boot, Wi-Fi onboarding, camera availability, and clean board-manager generation with minimal local workarounds.

Related

Testing

Tested on physical JC-ESP32P4-M3-DEV hardware via COM3.

Validation performed

  1. Board config generation:

    • idf.py bmgr -c ./boards -b jc_esp32p4_m3_dev
    • Result: peripherals/devices validated successfully.
  2. Build:

    • idf.py build
    • Result: success.
  3. Flash:

    • idf.py -p COM3 flash
    • Result: success.
  4. Boot/runtime log checks:

    • Verified successful boot without reboot loop.
    • Verified STA obtains IP and app reaches prompt.
    • Verified Telegram polling path continues normally.
    • Verified camera initializes as /dev/video0.
    • Verified fs_spiffs mount failure logs are removed after disabling SPIFFS device init on this board profile.
    • Display remains intentionally headless-safe when LCD path is unavailable (init_skip behavior preserved).

Environment:

  • ESP-IDF v5.5.4
  • Windows host
  • Real hardware (not emulated)

Checklist

  • 🚨 This PR does not introduce breaking changes.
  • All CI checks (GH Actions) pass.
  • Documentation is updated as needed.
  • Tests are updated or added as necessary.
  • Code is well-commented, especially in complex areas.
  • Git history is clean — commits are squashed to the minimum necessary.

@CLAassistant

CLAassistant commented May 8, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@laride

laride commented May 9, 2026

Copy link
Copy Markdown
Collaborator

Thanks for the PR!

I noticed that this PR includes both support for a new development board and several core logic changes. To make the review process easier and keep the scope clearer, could you please split them into separate PRs?

@dvucinozd

dvucinozd commented May 9, 2026 via email

Copy link
Copy Markdown
Author

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.

3 participants