Skip to content

arch/xtensa/esp32s3: fix build when mbedTLS is enabled in apps#11738

Closed
pkarashchenko wants to merge 1 commit into
apache:masterfrom
pkarashchenko:bugfix/esp32s3-crypto-fix
Closed

arch/xtensa/esp32s3: fix build when mbedTLS is enabled in apps#11738
pkarashchenko wants to merge 1 commit into
apache:masterfrom
pkarashchenko:bugfix/esp32s3-crypto-fix

Conversation

@pkarashchenko

@pkarashchenko pkarashchenko commented Feb 21, 2024

Copy link
Copy Markdown
Contributor

Summary

@tmedicci could you please integrate espressif/esp-hal-3rdparty#2 in esp-hal-3rdparty?

Impact

Testing

Signed-off-by: Petro Karashchenko <petro.karashchenko@gmail.com>
@xiaoxiang781216

Copy link
Copy Markdown
Contributor

Let's ignore the ci temp break which is fixed by #11740.

@xiaoxiang781216

Copy link
Copy Markdown
Contributor

@pkarashchenko do you want to wait @tmedicci merge your change to esp-hal-3rdparty?

@pkarashchenko

Copy link
Copy Markdown
Contributor Author

@xiaoxiang781216 I created this patch for visibility mostly. It fixes only a part of my use case and the change in esp 3p repo is mandatory

@tmedicci

Copy link
Copy Markdown
Contributor

Summary

@tmedicci could you please integrate espressif/esp-hal-3rdparty#2 in esp-hal-3rdparty?

I'm checking this and testing it with our internal CI.

tmedicci added a commit that referenced this pull request Feb 29, 2024
@pkarashchenko pkarashchenko deleted the bugfix/esp32s3-crypto-fix branch February 29, 2024 19:08
pkarashchenko pushed a commit that referenced this pull request Feb 29, 2024
halyssonJr pushed a commit to halyssonJr/nuttx that referenced this pull request Apr 10, 2024
anchao pushed a commit to anchao/nuttx that referenced this pull request Sep 6, 2024
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.

4 participants