Skip to content

[Renesas RX72N] Fix SP_MATH and SP_MATH_ALL conflict in user_settings.h#10355

Open
miyazakh wants to merge 1 commit intowolfSSL:masterfrom
miyazakh:rx72n_fix_usersettings
Open

[Renesas RX72N] Fix SP_MATH and SP_MATH_ALL conflict in user_settings.h#10355
miyazakh wants to merge 1 commit intowolfSSL:masterfrom
miyazakh:rx72n_fix_usersettings

Conversation

@miyazakh
Copy link
Copy Markdown
Contributor

Description

Fix a conflict in user_settings.h where both SP_MATH and SP_MATH_ALL were defined.

Testing

Run wolfcrypt on the device

Checklist

  • added tests
  • updated/added doxygen
  • updated appropriate READMEs
  • Updated manual and documentation

@miyazakh miyazakh self-assigned this Apr 30, 2026
Copilot AI review requested due to automatic review settings April 30, 2026 06:44
@miyazakh miyazakh changed the title Fix SP_MATH and SP_MATH_ALL conflict [Renesas RX72N] Fix SP_MATH and SP_MATH_ALL conflict in user_settings.h Apr 30, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Resolves a configuration macro conflict in the Renesas RX72N EnvisionKit e2studio demo user_settings.h that triggers a build-time validation error when both WOLFSSL_SP_MATH and WOLFSSL_SP_MATH_ALL are defined.

Changes:

  • Remove #define WOLFSSL_SP_MATH while keeping #define WOLFSSL_SP_MATH_ALL enabled to avoid the mutual-exclusivity error.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@github-actions
Copy link
Copy Markdown

MemBrowse Memory Report

No memory changes detected for:

@miyazakh
Copy link
Copy Markdown
Contributor Author

retest this please

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