From e08ff3fab9965084dbecfa11650c3c7b5585c3f1 Mon Sep 17 00:00:00 2001 From: Abtin Keshavarzian Date: Fri, 10 Jul 2026 06:57:16 -0700 Subject: [PATCH] [mac-frame] unconditionally include crypto/aes_ccm.hpp (#13327) Previously, `#include "crypto/aes_ccm.hpp"` in `mac_frame.cpp` was guarded under preprocessor conditionals (`OPENTHREAD_FTD || OPENTHREAD_MTD || ...`) because the crypto headers directly pulled in third-party crypto library headers (such as `mbedtls`), which caused build issues on RCP targets. Now that `aes_ccm.hpp` only includes platform crypto definitions and OpenThread internal headers, it is safe to include across all build configurations. This commit removes the preprocessor guard to simplify the include list in `mac_frame.cpp`. --- src/core/mac/mac_frame.cpp | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/src/core/mac/mac_frame.cpp b/src/core/mac/mac_frame.cpp index 8776d6fc0..d07a585c2 100644 --- a/src/core/mac/mac_frame.cpp +++ b/src/core/mac/mac_frame.cpp @@ -40,11 +40,8 @@ #include "common/frame_builder.hpp" #include "common/log.hpp" #include "common/num_utils.hpp" -#include "radio/trel_link.hpp" -#if OPENTHREAD_FTD || OPENTHREAD_MTD || OPENTHREAD_CONFIG_MAC_SOFTWARE_TX_SECURITY_ENABLE || \ - (OPENTHREAD_CONFIG_MAC_HEADER_IE_SUPPORT && OPENTHREAD_CONFIG_MAC_SOFTWARE_RETX_SECURITY_ENABLE) #include "crypto/aes_ccm.hpp" -#endif +#include "radio/trel_link.hpp" namespace ot { namespace Mac {