soc: esp32: virtual e-fuses support #90165
Open
+109
−1
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
wmrsouza
force-pushed
the
support/virtual_efuses
branch
from
23dce02 to
3d42ffc
Compare
|
The following west manifest projects have changed revision in this Pull Request:
✅ All manifest checks OK Note: This message is automatically posted and updated by the Manifest GitHub Action. |
github-actions
bot
added
manifest
manifest-hal_espressif
DNM (manifest)
wmrsouza
force-pushed
the
support/virtual_efuses
branch
2 times, most recently
from
3a929dd to
088d61d
Compare
wmrsouza
force-pushed
the
support/virtual_efuses
branch
from
088d61d to
4748fab
Compare
github-actions
bot
requested review from
LucasTambor,
marekmatej,
raffarost,
sylvioalves and
uLipe
wmrsouza
force-pushed
the
support/virtual_efuses
branch
3 times, most recently
from
b6a26fd to
ee685e3
Compare
github-actions
bot
removed
the
DNM (manifest)
esp32 hal revision update to enable use of virtual efuses Signed-off-by: Marcio Ribeiro <marcio.ribeiro@espressif.com>
Adds support for virtual e-fuses on esp32 socs Signed-off-by: Marcio Ribeiro <marcio.ribeiro@espressif.com>
wmrsouza
force-pushed
the
support/virtual_efuses
branch
from
ee685e3 to
df2cce3
Compare
|
LucasTambor
reviewed
May 30, 2025
| This mode is useful when need to keep changes after reboot | ||
| (testing secure_boot and flash_encryption). | ||
|
|
||
| config EFUSE_VIRTUAL_OFFSET |
There was a problem hiding this comment.
What about ESP32_ prefix?
sylvioalves
reviewed
May 30, 2025
| @@ -164,7 +164,7 @@ manifest: | |||
| groups: | |||
| - hal | |||
| - name: hal_espressif | |||
| revision: fa60a7aaa1f834c66413bbaa771794cf8aa82946 | |||
| revision: 3400dc972461b46e72691f676f70c7e3548ab3f9 | |||
There was a problem hiding this comment.
This needs update or removal.
marekmatej
requested changes
May 30, 2025
| @@ -26,6 +27,15 @@ void IRAM_ATTR __esp_platform_app_start(void) | |||
|
|
|||
| esp_flash_config(); | |||
|
|
|||
| #if CONFIG_ESP32_EFUSE_VIRTUAL | |||
There was a problem hiding this comment.
Can we please hide and unite the efuse init code in soc/common/include/esp_efuse.h using only a single call? That way, we don't bloat the soc.c each time we add a new feature.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Adds support for virtual e-fuses on esp32 socs