Arduino ESP32 v3.3 based on ESP-IDF v5.5 (master) #10884

me-no-dev · 2025-01-21T15:15:42Z

No description provided.

github-actions · 2025-01-21T15:16:18Z

	Warnings
⚠️	Some issues found for the commit messages in this PR: the commit message `"IDF master (#10887)"`: summary looks empty type/action looks empty the commit message `"IDF master (#10981)"`: summary looks empty type/action looks empty the commit message `"IDF master (#10999)"`: summary looks empty type/action looks empty the commit message `"IDF master (#11150)"`: summary looks empty type/action looks empty the commit message `"IDF master (#11204)"`: summary looks empty type/action looks empty the commit message `"IDF master (#11289)"`: summary looks empty type/action looks empty the commit message `"IDF master (#11342)"`: summary looks empty type/action looks empty the commit message `"IDF master 0461e2ff (#11018)"`: summary looks empty type/action looks empty the commit message `"IDF master 7cf5dacd (#11358)"`: summary looks empty type/action looks empty the commit message `"IDF release/v5.5 (#11369)"`: summary looks empty type/action looks empty the commit message `"IDF release/v5.5 (#11504)"`: summary looks empty type/action looks empty the commit message `"IDF release/v5.5 adb3f2a5 (#11543)"`: summary looks empty type/action looks empty the commit message `"IDF release/v5.5 cf8dad07 (#11601)"`: summary looks empty type/action looks empty the commit message `"Update version for wifi remote (#11344)"`: summary looks empty type/action looks empty the commit message `"add c5 bootloader location to pioarduino script (#10889)"`: summary looks empty type/action looks empty the commit message `"c5 flash base address is 0x2000 (#11037)"`: summary looks empty type/action looks empty the commit message `"feat(NimBLE): Add support for NimBLE (#11424)"`: scope/component should be lowercase without whitespace, allowed special characters are `_` `/` `.` `,` `` `-` `.` the commit message `"feat(matter): removing wifi requirement for H2 and C5 (#11581)"`: body's lines must not be longer than 100 characters the commit message `"fix C5 compile (#11255)"`: summary* looks empty type/action looks empty Please fix these commit messages - here are some basic tips: follow Conventional Commits style correct format of commit message should be: `<type/action>(<scope/component>): <summary>`, for example `fix(esp32): Fixed startup timeout issue` allowed types are: `change,ci,docs,feat,fix,refactor,remove,revert,test` sufficiently descriptive message summary should be between 10 to 72 characters and start with upper case letter avoid Jira references in commit messages (unavailable/irrelevant for our customers) `TIP:` Install pre-commit hooks and run this check when committing (uses the Conventional Precommit Linter).
⚠️	The Pull Request description is empty. Please provide a detailed description.

	Messages
📖	This PR seems to be quite large (total lines of code: 14965), you might consider splitting it into smaller PRs

👋 Hello me-no-dev, we appreciate your contribution to this project!

📘 Please review the project's Contributions Guide for key guidelines on code, documentation, testing, and more.

🖊️ Please also make sure you have read and signed the Contributor License Agreement for this project.

Click to see more instructions ...

This automated output is generated by the PR linter DangerJS, which checks if your Pull Request meets the project's requirements and helps you fix potential issues.

DangerJS is triggered with each push event to a Pull Request and modify the contents of this comment.

Please consider the following:
- Danger mainly focuses on the PR structure and formatting and can't understand the meaning behind your code or changes.
- Danger is not a substitute for human code reviews; it's still important to request a code review from your colleagues.
- Resolve all warnings (⚠️ ) before requesting a review from human reviewers - they will appreciate it.
- Addressing info messages (📖) is strongly recommended; they're less critical but valuable.
- To manually retry these Danger checks, please navigate to the Actions tab and re-run last Danger workflow.

Review and merge process you can expect ...

We do welcome contributions in the form of bug reports, feature requests and pull requests.

1. An internal issue has been created for the PR, we assign it to the relevant engineer.
2. They review the PR and either approve it or ask you for changes or clarifications.
3. Once the GitHub PR is approved we do the final review, collect approvals from core owners and make sure all the automated tests are passing.
- At this point we may do some adjustments to the proposed change, or extend it by adding tests or documentation.
4. If the change is approved and passes the tests it is merged into the default branch.

Generated by 🚫 dangerJS against a69c71f

github-actions · 2025-01-21T15:50:30Z

Test Results

76 files 76 suites 13m 9s ⏱️
38 tests 38 ✅ 0 💤 0 ❌
241 runs 241 ✅ 0 💤 0 ❌

Results for commit a69c71f.

♻️ This comment has been updated with latest results.

cores/esp32/Esp.cpp

* add c5 bootloader location to pioarduino script * ci(pre-commit): Apply automatic fixes --------- Co-authored-by: pre-commit-ci-lite[bot] <117423508+pre-commit-ci-lite[bot]@users.noreply.github.com>

* feat(ci): Run sketches on ESP32-C5 * IDF master 1160a86b * fix(zigbee): Remove RCP mode from ESP32-C5

* IDF master 1160a86b * fix(ci): Do not compile RainMaker examples on ESP32

* IDF master c71d74e2 * IDF master 81e8b752 * IDF master 877057db

* c5 flash base address is 0x2000 * ci(pre-commit): Apply automatic fixes --------- Co-authored-by: pre-commit-ci-lite[bot] <117423508+pre-commit-ci-lite[bot]@users.noreply.github.com>

* feat(wifi): Add support for 2.4GHz and 5GHz band switching * ci(pre-commit): Apply automatic fixes --------- Co-authored-by: pre-commit-ci-lite[bot] <117423508+pre-commit-ci-lite[bot]@users.noreply.github.com>

* IDF master ee77c489 * IDF master ee77c489 * IDF master 50be9735 * IDF master 23c73cdc * IDF master a45d713b

IDF release/v5.5 adb3f2a5

* feat(matter): enables BLE Matter commissioning with NimBLE * fix(matter): commentary typo and formatting * fix(matter): commentary typo and formatting * fix(matter): removes forcing second network clustter * fix(matter): adds matter source code to CMakeLists.txt * ci(pre-commit): Apply automatic fixes --------- Co-authored-by: pre-commit-ci-lite[bot] <117423508+pre-commit-ci-lite[bot]@users.noreply.github.com>

This PR removes WiFi provisioning support from CI and examples (shifting to Thread/BLE provisioning), updates the CI configs for all Matter examples to drop the WiFi requirement, and adds new API keywords and a fresh “LambdaSingleCallbackManyEPs” example. - Deleted the WiFiProvWithinMatter example (its .ino and ci.json) since BLE is now used for provisioning. - Stripped "CONFIG_SOC_WIFI_SUPPORTED=y" from the CI JSON of existing examples to test Thread-only builds. - Updated keywords.txt with new Matter API identifiers and introduced a new “LambdaSingleCallbackManyEPs” example with CI and source

IDF release/v5.5 cf8dad07

pedrominatel · 2025-07-17T09:23:32Z

docs/conf_common.py

SuGlider · 2025-07-17T21:44:44Z

CMakeLists.txt

@@ -6,7 +6,7 @@
 #   idf.py build

 set(min_supported_idf_version "5.3.0")


Should it be 5.5.0 because of the C5 addition?

Suggested change

set(min_supported_idf_version "5.3.0")

set(min_supported_idf_version "5.5.0")

It may be better testing building some empty Arduno as IDF Component sketch using IDF 5.4|5.3

me-no-dev added the 🚫 System PR Pull requests for internal use or lib updates label Jan 21, 2025

me-no-dev force-pushed the release/v3.2.x branch from 5372b36 to f0cf3b1 Compare February 18, 2025 08:12

me-no-dev force-pushed the release/v3.3.x branch from 7afa967 to 592d06f Compare February 18, 2025 08:15

Base automatically changed from release/v3.2.x to master February 18, 2025 09:58

me-no-dev changed the title ~~Preparation for IDF v5.5 and ESP32-C5~~ Arduino ESP32 v3.3 based on ESP-IDF v5.5 (master) Feb 18, 2025

Jason2866 reviewed Mar 4, 2025

View reviewed changes

cores/esp32/Esp.cpp Outdated Show resolved Hide resolved

me-no-dev and others added 16 commits March 20, 2025 02:09

feat(idf): Add initial support for IDF v5.5 and ESP32-C5

88cda9f

fix(examples): Add changes required to some examples

bf90cbd

fix(c5): Update bootloader location

af5abd5

fix(report): Add missing chip names

0894d7d

fix(c5): Update debug log in setCpuFrequencyMhz

100ed8e

add c5 bootloader location to pioarduino script (#10889)

d9d3bf4

* add c5 bootloader location to pioarduino script * ci(pre-commit): Apply automatic fixes --------- Co-authored-by: pre-commit-ci-lite[bot] <117423508+pre-commit-ci-lite[bot]@users.noreply.github.com>

fix(ci): Fix/stop some examples for C5

8af81cd

fix(c5): Update PIN_RGB_LED in pins_arduino.h

9fbcb34

fix(psram): Add support for ESP32-C5 PSRAM

f45cd7b

fix(board): Update ESP32-C5 Dev Kit Pinout

75de09e

IDF master (#10887)

6283c15

* feat(ci): Run sketches on ESP32-C5 * IDF master 1160a86b * fix(zigbee): Remove RCP mode from ESP32-C5

IDF master (#10981)

042015e

* IDF master 1160a86b * fix(ci): Do not compile RainMaker examples on ESP32

IDF master (#10999)

e30e3c3

* IDF master c71d74e2 * IDF master 81e8b752 * IDF master 877057db

IDF master 0461e2ff (#11018)

3fe2fe5

c5 flash base address is 0x2000 (#11037)

d66eeb7

* c5 flash base address is 0x2000 * ci(pre-commit): Apply automatic fixes --------- Co-authored-by: pre-commit-ci-lite[bot] <117423508+pre-commit-ci-lite[bot]@users.noreply.github.com>

feat(wifi): Add support for 2.4GHz and 5GHz band switching (#11045)

7c1ac1a

* feat(wifi): Add support for 2.4GHz and 5GHz band switching * ci(pre-commit): Apply automatic fixes --------- Co-authored-by: pre-commit-ci-lite[bot] <117423508+pre-commit-ci-lite[bot]@users.noreply.github.com>

me-no-dev force-pushed the release/v3.3.x branch from a4ad009 to 7c1ac1a Compare March 20, 2025 00:10

me-no-dev and others added 5 commits March 27, 2025 13:39

Merge branch 'master' into release/v3.3.x

d81c195

Merge branch 'master' into release/v3.3.x

dac2dd9

IDF master (#11150)

a61961d

* IDF master ee77c489 * IDF master ee77c489 * IDF master 50be9735 * IDF master 23c73cdc * IDF master a45d713b

Merge branch 'master' into release/v3.3.x

998bf80

Merge branch 'master' into release/v3.3.x

ae2ae8d

me-no-dev and others added 13 commits June 24, 2025 17:26

Merge branch 'master' into release/v3.3.x

5871a80

Merge branch 'master' into release/v3.3.x

ff3cc81

Merge branch 'master' into release/v3.3.x

70696df

IDF release/v5.5 adb3f2a5 (#11543)

d4e5c5f

IDF release/v5.5 adb3f2a5

Merge branch 'master' into release/v3.3.x

f84fab5

fix(matter): Fix MatterSmartButon.ino when CHIPOBLE is on

1426927

change(esptool): Upgrade esptool to release v5.0.0 (#11563)

ccc0a69

Merge branch 'master' into release/v3.3.x

df3db3c

Merge branch 'master' into release/v3.3.x

fbf3c11

IDF release/v5.5 cf8dad07 (#11601)

ce7ef9c

IDF release/v5.5 cf8dad07

Merge branch 'master' into release/v3.3.x

3f32903

me-no-dev marked this pull request as ready for review July 16, 2025 12:11

me-no-dev requested review from lucasssvaz, P-R-O-C-H-Y, SuGlider and a team as code owners July 16, 2025 12:11

me-no-dev added the Status: Pending Merge Pull Request is ready to be merged label Jul 16, 2025

pre-commit-ci-lite bot and others added 3 commits July 16, 2025 12:12

ci(pre-commit): Apply automatic fixes

ac05f18

Merge branch 'master' into release/v3.3.x

5ccd952

feat(core): Update core version to 3.3.0

a69c71f

me-no-dev requested a review from pedrominatel as a code owner July 16, 2025 16:56

me-no-dev removed the Status: Pending Merge Pull Request is ready to be merged label Jul 16, 2025

pedrominatel reviewed Jul 17, 2025

View reviewed changes

docs/conf_common.py

Copy link

Member

pedrominatel Jul 17, 2025

Choose a reason for hiding this comment

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

LGTM.

lucasssvaz approved these changes Jul 17, 2025

View reviewed changes

pedrominatel approved these changes Jul 17, 2025

View reviewed changes

P-R-O-C-H-Y approved these changes Jul 17, 2025

View reviewed changes

SuGlider approved these changes Jul 17, 2025

View reviewed changes

SuGlider reviewed Jul 17, 2025

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Arduino ESP32 v3.3 based on ESP-IDF v5.5 (master) #10884

Arduino ESP32 v3.3 based on ESP-IDF v5.5 (master) #10884

me-no-dev commented Jan 21, 2025

Uh oh!

github-actions bot commented Jan 21, 2025 •

edited

Loading

Uh oh!

github-actions bot commented Jan 21, 2025 •

edited

Loading

Uh oh!

Uh oh!

pedrominatel Jul 17, 2025

Uh oh!

SuGlider Jul 17, 2025 •

edited

Loading

Uh oh!

SuGlider Jul 17, 2025

Uh oh!

Uh oh!

		@@ -6,7 +6,7 @@
		# idf.py build

		set(min_supported_idf_version "5.3.0")

	set(min_supported_idf_version "5.3.0")
	set(min_supported_idf_version "5.5.0")

Arduino ESP32 v3.3 based on ESP-IDF v5.5 (master) #10884

Are you sure you want to change the base?

Arduino ESP32 v3.3 based on ESP-IDF v5.5 (master) #10884

Conversation

me-no-dev commented Jan 21, 2025

Uh oh!

github-actions bot commented Jan 21, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

github-actions bot commented Jan 21, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Test Results

Uh oh!

Uh oh!

pedrominatel Jul 17, 2025

Choose a reason for hiding this comment

Uh oh!

SuGlider Jul 17, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

SuGlider Jul 17, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

github-actions bot commented Jan 21, 2025 •

edited

Loading

github-actions bot commented Jan 21, 2025 •

edited

Loading

SuGlider Jul 17, 2025 •

edited

Loading