Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Intoduce support for ESP32P4 #1438

Open
wants to merge 4 commits into
base: release-0.6
Choose a base branch
from

Conversation

UncleGrumpy
Copy link
Collaborator

Adds device info for erlang:system_info(esp32_chip_info) for the ESP32P4, as well as building release images. This does not include any network support yet.

These changes are made under both the "Apache 2.0" and the "GNU Lesser General
Public License 2.1 or later" license terms (dual license).

SPDX-License-Identifier: Apache-2.0 OR LGPL-2.1-or-later

Copy link
Collaborator

@bettio bettio left a comment

Choose a reason for hiding this comment

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

So, we already have CI, tests and everything else for ESP32-P4 in release-0.6.
So I think it is worth to apply this on release-0.6 as a last step of what we did.
In that case let's not remove support for 4.x, we'll do a cleanup PR in main branch.

The ESP32-H2 was first introduced in ESP-IDF v5.1 but the macro for v5.1 builds excludes the H2,
leaving it included for builds with previous versions (for example 4.3) where support is not
included.

Signed-off-by: Winford <winford@object.stream>
Adds missing `esp32_chip_info` key to ESP32 specific keys in the function documentation for
`erlang:system_info/1`.

Signed-off-by: Winford <winford@object.stream>
Adds the ESP32P4 chip to the `get_model` function used to retrieve esp32 chip info by
`erlang:system_info/1`.

Cleans up the macros used to include various esp32 models by ESP-IDF version.

Signed-off-by: Winford <winford@object.stream>
Introduces builds for ESP32P4 devices.

Signed-off-by: Winford <winford@object.stream>
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.

2 participants