Skip to content

Conversation

tore-espressif
Copy link
Collaborator

@tore-espressif tore-espressif commented Oct 15, 2025

This will enable ESP32-H4 builds on esp-idf/master.

@peter-marcisovsky
Copy link
Collaborator

@tore-espressif FYI, if you are wandering why did build of UVC example failed after updating idf-build-apps, explanation here

@tore-espressif tore-espressif force-pushed the ci/enable_h4_build branch 3 times, most recently from b529c66 to bea3ce4 Compare October 15, 2025 19:54
@tore-espressif
Copy link
Collaborator Author

H4 builds are not correctly generated. Although it works locally. Marking as draft for now

@tore-espressif tore-espressif marked this pull request as draft October 16, 2025 06:39
Comment on lines 52 to 53
idf-build-apps find --config-file ${CONFIG_PATH} --manifest-file ${MANIFEST_PATH}
idf-build-apps build --config-file ${CONFIG_PATH} --manifest-file ${MANIFEST_PATH}
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
idf-build-apps build -vv --config-file ${CONFIG_PATH} --manifest-file ${MANIFEST_PATH}

the find command here is unneeded, and try debug mode to see why it happens.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@hfudev Thanks for taking a look. I use the find command to catch bugs exactly as this one. Without the find command I would not have noticed that the H4 target is missing.

I will try with -vv thanks

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

H4 was added to the list, but no app was selected https://github.com/espressif/esp-usb/actions/runs/18557342763/job/52897863877?pr=296#step:4:2274

On latest esp-idf/master H4 has already defined SOC_USB_OTG_SUPPORTED 1. Could is idf-build-apps using the latest soc_caps?

This will enable ESP32-H4 builds on esp-idf/master.
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