Made alignment typedefs more robust for IAR #410

Merged

LoveKarlsson merged 3 commits from iar-robust-alignment into development

2026-04-29 10:30:45 -05:00

LoveKarlsson commented

2025-08-06 03:26:14 -05:00

(Migrated from github.com)

Description

The Zephyr project unfortunately defines a macro __packed which overrides the IAR keyword, and that causes the typedefs in core/alignment.h not be unaligned. This is also being backported to 3.6 as that version is what Zephyr is currently using.

PR checklist

changelog : provided
Mbed TLS development PR not required because: code is in crypto
Mbed TLS 4.1 PR not required because: code is in crypto
TF-PSA-Crypto development PR provided https://github.com/Mbed-TLS/TF-PSA-Crypto/pull/410
TF-PSA-Crypto 1.1 PR provided https://github.com/Mbed-TLS/TF-PSA-Crypto/pull/779
framework PR not required
3.6 PR provided https://github.com/Mbed-TLS/mbedtls/pull/10202
tests not required because: integration fix, up to the integrator to validate

## Description The Zephyr project unfortunately defines a macro __packed which overrides the IAR keyword, and that causes the typedefs in core/alignment.h not be unaligned. This is also being backported to 3.6 as that version is what Zephyr is currently using. ## PR checklist - [x] **changelog** : provided - [x] **Mbed TLS development PR** not required because: code is in crypto - [x] **Mbed TLS 4.1 PR** not required because: code is in crypto - [x] **TF-PSA-Crypto development PR** provided https://github.com/Mbed-TLS/TF-PSA-Crypto/pull/410 - [x] **TF-PSA-Crypto 1.1 PR** provided https://github.com/Mbed-TLS/TF-PSA-Crypto/pull/779 - [x] **framework PR** not required - [x] **3.6 PR** provided https://github.com/Mbed-TLS/mbedtls/pull/10202 - **tests** not required because: integration fix, up to the integrator to validate

valeriosetti commented

2026-04-14 09:24:28 -05:00

(Migrated from github.com)

@davidhorstmann-arm I just added this to the Community project because this has recently been re-raised again in Zephyr scope. I also started the CI on it.
I don't know if I can review this though because I don't have an IAR compiler that I can use to verify the solution.

@davidhorstmann-arm I just added this to the `Community` project because this has recently been re-raised again in [Zephyr scope](https://github.com/zephyrproject-rtos/TF-PSA-Crypto/pull/7). I also started the CI on it. I don't know if I can review this though because I don't have an IAR compiler that I can use to verify the solution.

gilles-peskine-arm (Migrated from github.com) reviewed

2026-04-15 05:08:07 -05:00

gilles-peskine-arm (Migrated from github.com) left a comment

No description provided.

LGTM I've checked locally that this doesn't break the build with IAR 9.40 for arm, in a bare-metal build. I haven't tried running tests or building in Zephyr.

valeriosetti (Migrated from github.com) reviewed

2026-04-24 09:48:00 -05:00

valeriosetti (Migrated from github.com) left a comment

No description provided.

CI failures seems related to an outdated `framework` submodule. Can you please rebase the PR on top of `development`? I will re-approve again once it's updated

valeriosetti commented

2026-04-24 09:49:17 -05:00

(Migrated from github.com)