Skip to content

feat(board): Add ozobot circuit kit board.#12465

Merged
me-no-dev merged 2 commits into
espressif:masterfrom
ozobot:circuit_kit
Mar 25, 2026
Merged

feat(board): Add ozobot circuit kit board.#12465
me-no-dev merged 2 commits into
espressif:masterfrom
ozobot:circuit_kit

Conversation

@opilat

@opilat opilat commented Mar 20, 2026

Copy link
Copy Markdown
Contributor

Description of Change
Add board definition for upcoming board from us (ozobot circuit kit).

@opilat opilat requested a review from P-R-O-C-H-Y as a code owner March 20, 2026 08:02
@CLAassistant

CLAassistant commented Mar 20, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@github-actions

github-actions Bot commented Mar 20, 2026

Copy link
Copy Markdown
Contributor
Messages
📖 🎉 Good Job! All checks are passing!

👋 Hello opilat, 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.
- 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 bd8e796

@P-R-O-C-H-Y P-R-O-C-H-Y added the Type: 3rd party Boards Related to third-party/non-Espressif hardware boards. label Mar 20, 2026
@P-R-O-C-H-Y

Copy link
Copy Markdown
Member

Hi @opilat, I see the commit is from this account: @ondrapCZE
Please sign the CLA and also provide PR description. Thanks

@P-R-O-C-H-Y P-R-O-C-H-Y added the Status: Awaiting Response Awaiting a response from the author label Mar 20, 2026
@opilat

opilat commented Mar 20, 2026

Copy link
Copy Markdown
Contributor Author

@P-R-O-C-H-Y Thanks for quick response. I added short description and signed the CLA

@P-R-O-C-H-Y P-R-O-C-H-Y removed the Status: Awaiting Response Awaiting a response from the author label Mar 20, 2026

@P-R-O-C-H-Y P-R-O-C-H-Y left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@opilat PTAL

Comment thread boards.txt Outdated
ozobot_circuit_kit.build.flash_mode=dio
ozobot_circuit_kit.build.boot=qio
ozobot_circuit_kit.build.boot_freq=80m
ozobot_circuit_kit.build.defines=-DBOARD_HAS_PIN_REMAP {build.disable_pin_remap} -DBOARD_HAS_PSRAM '-DUSB_MANUFACTURER="OZOEDU"' '-DUSB_PRODUCT="Ozobot circuit kit"'

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

You can move the USB_MANUFACTURER and USB_PRODUCT define to the pins_arduino.h file as other manufacturers do under the VID/PID.

Comment thread variants/ozobot_circuit_kit/pins_arduino.h Outdated
@P-R-O-C-H-Y P-R-O-C-H-Y added the Status: Pending Merge Pull Request is ready to be merged label Mar 24, 2026
@me-no-dev

Copy link
Copy Markdown
Member

@opilat since maintainers are not allowed to make changes to your PR, please make the following changes manually: https://github.com/espressif/arduino-esp32/actions/runs/23486419396/job/68343054788?pr=12465

@opilat opilat force-pushed the circuit_kit branch 3 times, most recently from bde5dc6 to 1ea0b0b Compare March 24, 2026 13:33
@github-actions

Copy link
Copy Markdown
Contributor

⚠️ Pre-commit Hooks Failed

Some pre-commit hooks failed and require manual fixes. Please see the detailed error report below.

What to do:

  1. 📋 View the detailed error report to see which hooks failed
  2. 🔧 Fix the issues locally in your code
  3. 💾 Commit and push your changes
  4. 🔄 The hooks will run again automatically

Need help? Ask in the comments below.

@github-actions github-actions Bot added the Status: Pre-commit fixes required ⚠️ The pre-commit bot can't fix the issues automatically. Please fix the issues manually. label Mar 24, 2026
@github-actions github-actions Bot removed the Status: Pre-commit fixes required ⚠️ The pre-commit bot can't fix the issues automatically. Please fix the issues manually. label Mar 24, 2026
@me-no-dev me-no-dev merged commit c1f40e3 into espressif:master Mar 25, 2026
24 checks passed
Jason2866 added a commit to pioarduino/arduino-esp32 that referenced this pull request Apr 2, 2026
* Fixed USB-CDC menu entry for codecell C3, so default is enabled and other option is 'disabled' (espressif#12477)

* feat(board): Add ozobot circuit kit board (espressif#12465)

Co-authored-by: ondrap <pilat.ondrej@seznam.cz>

* IDF v5.5.4 (espressif#12493)

* fix(usb): Fix typo CFG_TUD_ENDOINT > CFG_TUD_ENDPOINT

* IDF v5.5.4 73550728

* ci: update dangerjs permissions to contents: read (espressif#12491)

* feat(spi): Set power channel on ESP32P4 (espressif#12079)

* feat(spi): Set power channel on ESP32P4

* moved changes from SPI.cpp to esp32-hal-spi.c

* Check all 10 of the pins on LDO4

* make setLDO function static

* ci(pre-commit): Apply automatic fixes

---------

Co-authored-by: Jan Procházka <90197375+P-R-O-C-H-Y@users.noreply.github.com>
Co-authored-by: Lucas Saavedra Vaz <32426024+lucasssvaz@users.noreply.github.com>
Co-authored-by: pre-commit-ci-lite[bot] <117423508+pre-commit-ci-lite[bot]@users.noreply.github.com>

* feat(USB): Add support for USB Audio (espressif#12466)

* feat(USB): Add support for USB Audio

* fix(build): Fix syntax errors

* fix(build) Fix CoPilot suggestions

* fix(build): Update libraries/USB/src/USBAudioCard.h

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* fix(build): Implement suggestions and CMakeLists.txt

* feat(uac): Add UAC1 descriptors

* fix(uac): Fix warnings treated as errors

* feat(uac): Add UAC2 descriptors

* feat(uac): Update example

* feat(uac): Add some comments to explain how the example works

* feat(uac): Add documentation to the USBAudioCard class

* fix(uac): Apply Copilot suggestions

* feat(uac): Clarify chip usage in the example

* feat(uac): Add volume handling for all channels

* feat(uac): Also handle channels mute

* ci(pre-commit): Apply automatic fixes

* fix(uac): Revert changes to descriptors by CI

* fix(ci): Add USBAudioCardDescriptors.h to pre-commit config

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: pre-commit-ci-lite[bot] <117423508+pre-commit-ci-lite[bot]@users.noreply.github.com>

* fix(bounty): Provide fixes for reported bugbounty (espressif#12486)

* build(deps): bump cryptography from 46.0.5 to 46.0.6 in /tests (espressif#12489)

Bumps [cryptography](https://github.com/pyca/cryptography) from 46.0.5 to 46.0.6.
- [Changelog](https://github.com/pyca/cryptography/blob/main/CHANGELOG.rst)
- [Commits](pyca/cryptography@46.0.5...46.0.6)

---
updated-dependencies:
- dependency-name: cryptography
  dependency-version: 46.0.6
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Mark Moore <mark@moore.dk>
Co-authored-by: opilat <ondrej.pilat@ozobot.com>
Co-authored-by: ondrap <pilat.ondrej@seznam.cz>
Co-authored-by: Me No Dev <me-no-dev@users.noreply.github.com>
Co-authored-by: Tomas Sebestik <tomas.sebestik@espressif.com>
Co-authored-by: lbernstone <lbernstone@gmail.com>
Co-authored-by: Jan Procházka <90197375+P-R-O-C-H-Y@users.noreply.github.com>
Co-authored-by: Lucas Saavedra Vaz <32426024+lucasssvaz@users.noreply.github.com>
Co-authored-by: pre-commit-ci-lite[bot] <117423508+pre-commit-ci-lite[bot]@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Jason2866 added a commit to pioarduino/arduino-esp32 that referenced this pull request Apr 2, 2026
* Update CMakeLists.txt

* Simplify requirements handling in CMakeLists.txt

* Add espressif/usb component for ESP32 targets

* Add cjson dependency to idf_component.yml

* Add requirement for espressif/cjson in YAML

* Add MQTT component version 1.0.0 to dependencies

* Cleanup RainMaker dependencies from idf_component.yml

Removed deprecated RainMaker components and their versions.

* Delete libraries/ESP_SR directory

* Delete libraries/Insights directory

* Delete libraries/RainMaker directory

* Delete libraries/TFLiteMicro directory

* Remove unused libraries from CMakeLists.txt

* Refactor USB handling to use new wrapper functions

* Refactor SPI clock source handling and cleanup

* Delete libraries/ESP_I2S directory

* Remove ESP_I2S from CMakeLists.txt

* Merge pull request #1 from pioarduino/mergev4

Mergev4

* Mergev4 (#2)

* Fixed USB-CDC menu entry for codecell C3, so default is enabled and other option is 'disabled' (espressif#12477)

* feat(board): Add ozobot circuit kit board (espressif#12465)

Co-authored-by: ondrap <pilat.ondrej@seznam.cz>

* IDF v5.5.4 (espressif#12493)

* fix(usb): Fix typo CFG_TUD_ENDOINT > CFG_TUD_ENDPOINT

* IDF v5.5.4 73550728

* ci: update dangerjs permissions to contents: read (espressif#12491)

* feat(spi): Set power channel on ESP32P4 (espressif#12079)

* feat(spi): Set power channel on ESP32P4

* moved changes from SPI.cpp to esp32-hal-spi.c

* Check all 10 of the pins on LDO4

* make setLDO function static

* ci(pre-commit): Apply automatic fixes

---------

Co-authored-by: Jan Procházka <90197375+P-R-O-C-H-Y@users.noreply.github.com>
Co-authored-by: Lucas Saavedra Vaz <32426024+lucasssvaz@users.noreply.github.com>
Co-authored-by: pre-commit-ci-lite[bot] <117423508+pre-commit-ci-lite[bot]@users.noreply.github.com>

* feat(USB): Add support for USB Audio (espressif#12466)

* feat(USB): Add support for USB Audio

* fix(build): Fix syntax errors

* fix(build) Fix CoPilot suggestions

* fix(build): Update libraries/USB/src/USBAudioCard.h

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* fix(build): Implement suggestions and CMakeLists.txt

* feat(uac): Add UAC1 descriptors

* fix(uac): Fix warnings treated as errors

* feat(uac): Add UAC2 descriptors

* feat(uac): Update example

* feat(uac): Add some comments to explain how the example works

* feat(uac): Add documentation to the USBAudioCard class

* fix(uac): Apply Copilot suggestions

* feat(uac): Clarify chip usage in the example

* feat(uac): Add volume handling for all channels

* feat(uac): Also handle channels mute

* ci(pre-commit): Apply automatic fixes

* fix(uac): Revert changes to descriptors by CI

* fix(ci): Add USBAudioCardDescriptors.h to pre-commit config

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: pre-commit-ci-lite[bot] <117423508+pre-commit-ci-lite[bot]@users.noreply.github.com>

* fix(bounty): Provide fixes for reported bugbounty (espressif#12486)

* build(deps): bump cryptography from 46.0.5 to 46.0.6 in /tests (espressif#12489)

Bumps [cryptography](https://github.com/pyca/cryptography) from 46.0.5 to 46.0.6.
- [Changelog](https://github.com/pyca/cryptography/blob/main/CHANGELOG.rst)
- [Commits](pyca/cryptography@46.0.5...46.0.6)

---
updated-dependencies:
- dependency-name: cryptography
  dependency-version: 46.0.6
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Mark Moore <mark@moore.dk>
Co-authored-by: opilat <ondrej.pilat@ozobot.com>
Co-authored-by: ondrap <pilat.ondrej@seznam.cz>
Co-authored-by: Me No Dev <me-no-dev@users.noreply.github.com>
Co-authored-by: Tomas Sebestik <tomas.sebestik@espressif.com>
Co-authored-by: lbernstone <lbernstone@gmail.com>
Co-authored-by: Jan Procházka <90197375+P-R-O-C-H-Y@users.noreply.github.com>
Co-authored-by: Lucas Saavedra Vaz <32426024+lucasssvaz@users.noreply.github.com>
Co-authored-by: pre-commit-ci-lite[bot] <117423508+pre-commit-ci-lite[bot]@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Mark Moore <mark@moore.dk>
Co-authored-by: opilat <ondrej.pilat@ozobot.com>
Co-authored-by: ondrap <pilat.ondrej@seznam.cz>
Co-authored-by: Me No Dev <me-no-dev@users.noreply.github.com>
Co-authored-by: Tomas Sebestik <tomas.sebestik@espressif.com>
Co-authored-by: lbernstone <lbernstone@gmail.com>
Co-authored-by: Jan Procházka <90197375+P-R-O-C-H-Y@users.noreply.github.com>
Co-authored-by: Lucas Saavedra Vaz <32426024+lucasssvaz@users.noreply.github.com>
Co-authored-by: pre-commit-ci-lite[bot] <117423508+pre-commit-ci-lite[bot]@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Status: Pending Merge Pull Request is ready to be merged Type: 3rd party Boards Related to third-party/non-Espressif hardware boards.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants