nrf_802154: rev 0f7ea29d2ae899d52c3dd0ce2355dc4d4176b2b8

This commit updates revision of the nrf_802154 component.

Signed-off-by: Rafal Kuznia <[email protected]>

Author: e-rk

nrf_802154/common/include/nrf_802154.h

@@ -92,6 +92,14 @@ extern "C" {
 #define NRF_802154_MAX_PENDING_NOTIFICATIONS \
     (NRF_802154_RX_BUFFERS + NRF_802154_MAX_DISREGARDABLE_NOTIFICATIONS + 4 + 1)
 
+/**
+ * @brief Temporary macro to detect new API signatures.
+ *
+ * The macro is used during transition to new API declarations, where the TX functions return
+ * an error code instead of a boolean value.
+ */
+#define NRF_802154_TX_FUNCTIONS_RETURN_ERROR_CODE
+
 /**
  * @brief Initializes the 802.15.4 driver.
  *
@@ -548,11 +556,14 @@ bool nrf_802154_receive_at_scheduled_cancel(uint32_t id);
  *                         @c cca          | @c true
  *                         @c tx_timestamp_encode | @c false
  *
- * @retval  true   The transmission procedure was scheduled.
- * @retval  false  The driver could not schedule the transmission procedure.
+ * @retval NRF_802154_TX_ERROR_NONE  The TX request was successful.
+ *                                   Transmit success or failure will be indicated by the callout.
+ *
+ * @returns Error that prevented the frame from being transmitted.
+ *          No callout will be called.
  */
-bool nrf_802154_transmit_raw(uint8_t                              * p_data,
-                             const nrf_802154_transmit_metadata_t * p_metadata);
+nrf_802154_tx_error_t nrf_802154_transmit_raw(uint8_t                              * p_data,
+                                              const nrf_802154_transmit_metadata_t * p_metadata);
 
 /**
  * @brief Requests transmission at the specified time.
@@ -627,12 +638,16 @@ bool nrf_802154_transmit_raw(uint8_t                              * p_data,
  *                         @c extra_cca_attempts | @c 0
  *                         @c tx_timestamp_encode | @c 0
  *
- * @retval  true   The transmission procedure was scheduled.
- * @retval  false  The driver could not schedule the transmission procedure.
+ * @retval NRF_802154_TX_ERROR_NONE  The TX request was successful.
+ *                                   Transmit success or failure will be indicated by the callout.
+ *
+ * @returns Error that prevented the frame from being transmitted.
+ *          No callout will be called.
  */
-bool nrf_802154_transmit_raw_at(uint8_t                                 * p_data,
-                                uint64_t                                  tx_time,
-                                const nrf_802154_transmit_at_metadata_t * p_metadata);
+nrf_802154_tx_error_t nrf_802154_transmit_raw_at(
+    uint8_t                                 * p_data,
+    uint64_t                                  tx_time,
+    const nrf_802154_transmit_at_metadata_t * p_metadata);
 
 /**
  * @brief Cancels a delayed transmission scheduled by a call to @ref nrf_802154_transmit_raw_at.
@@ -1085,11 +1100,15 @@ void nrf_802154_cca_cfg_get(nrf_802154_cca_cfg_t * p_cca_cfg);
  *                         @c frame_props  | @ref NRF_802154_TRANSMITTED_FRAME_PROPS_DEFAULT_INIT
  *                         @c tx_timestamp_encode | false
  *
- * @retval  true   The chain of CSMA-CA and transmission procedure was scheduled.
- * @retval  false  The driver could not schedule the procedure chain.
+ * @retval NRF_802154_TX_ERROR_NONE  The TX request was successful.
+ *                                   Transmit success or failure will be indicated by the callout.
+ *
+ * @returns Error that prevented the frame from being transmitted.
+ *          No callout will be called.
  */
-bool nrf_802154_transmit_csma_ca_raw(uint8_t                                      * p_data,
-                                     const nrf_802154_transmit_csma_ca_metadata_t * p_metadata);
+nrf_802154_tx_error_t nrf_802154_transmit_csma_ca_raw(
+    uint8_t                                      * p_data,
+    const nrf_802154_transmit_csma_ca_metadata_t * p_metadata);
 
 /**
  * @brief Sets the minimum value of the backoff exponent (BE) in the CSMA-CA algorithm.

nrf_802154/common/include/nrf_802154_compiler.h

@@ -0,0 +1,103 @@
+/*
+ * Copyright (c) 2025, Nordic Semiconductor ASA
+ * All rights reserved.
+ *
+ * SPDX-License-Identifier: BSD-3-Clause
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions are met:
+ *
+ * 1. Redistributions of source code must retain the above copyright notice, this
+ *    list of conditions and the following disclaimer.
+ *
+ * 2. Redistributions in binary form must reproduce the above copyright
+ *    notice, this list of conditions and the following disclaimer in the
+ *    documentation and/or other materials provided with the distribution.
+ *
+ * 3. Neither the name of Nordic Semiconductor ASA nor the names of its
+ *    contributors may be used to endorse or promote products derived from this
+ *    software without specific prior written permission.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
+ * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
+ * IMPLIED WARRANTIES OF MERCHANTABILITY, AND FITNESS FOR A PARTICULAR PURPOSE
+ * ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE
+ * LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
+ * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
+ * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
+ * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
+ * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
+ * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
+ * POSSIBILITY OF SUCH DAMAGE.
+ *
+ */
+
+#ifndef NRF_802154_COMPILER_H__
+#define NRF_802154_COMPILER_H__
+
+#ifdef __cplusplus
+extern "C" {
+#endif
+
+/**
+ * @defgroup nrf_802154_config_compiler Compiler compatibility macros
+ * @{
+ * @ingroup nrf_802154
+ * @brief Compiler compatibility macros for the 802.15.4 radio driver.
+ */
+
+/**
+ * @def SWITCH_CASE_FALLTHROUGH
+ *
+ * Portable fallthrough annotation for switch case statements.
+ * 
+ * This macro provides a portable way to annotate intentional fallthrough
+ * between switch case labels across different C standards and compilers:
+ * - C23: Uses [[fallthrough]] attribute
+ * - GCC 7+/Clang: Uses __attribute__((fallthrough))
+ * - Older compilers: Falls back to comment-based annotation
+ * 
+ * Usage:
+ * @code
+ * switch (value) {
+ *     case 1:
+ *         do_something();
+ *         SWITCH_CASE_FALLTHROUGH;
+ *     case 2:
+ *         do_something_else();
+ *         break;
+ * }
+ * @endcode
+ */
+#if defined(__STDC_VERSION__) && (__STDC_VERSION__ >= 202311L)
+    /* C23 and later */
+    #define SWITCH_CASE_FALLTHROUGH [[fallthrough]]
+#elif defined(__has_attribute)
+    /* Clang and GCC with __has_attribute support */
+    #if __has_attribute(fallthrough)
+        #define SWITCH_CASE_FALLTHROUGH __attribute__((fallthrough))
+    #else
+        #define SWITCH_CASE_FALLTHROUGH /* fallthrough */
+    #endif
+#elif defined(__GNUC__) && (__GNUC__ >= 7)
+    /* GCC 7+ */
+    #define SWITCH_CASE_FALLTHROUGH __attribute__((fallthrough))
+#elif defined(__clang__)
+    /* Clang (older versions) */
+    #if defined(__clang_major__) && (__clang_major__ >= 10)
+        #define SWITCH_CASE_FALLTHROUGH __attribute__((fallthrough))
+    #else
+        #define SWITCH_CASE_FALLTHROUGH /* fallthrough */
+    #endif
+#else
+    /* Fallback for other compilers */
+    #define SWITCH_CASE_FALLTHROUGH /* fallthrough */
+#endif
+
+/** @} */
+
+#ifdef __cplusplus
+}
+#endif
+
+#endif // NRF_802154_COMPILER_H__

nrf_802154/common/include/nrf_802154_types.h

@@ -92,6 +92,7 @@ typedef uint8_t nrf_802154_tx_error_t;
 #define NRF_802154_TX_ERROR_KEY_ID_INVALID           0x08 // !< Transmission did not start due to invalid key ID in frame's security header.
 #define NRF_802154_TX_ERROR_FRAME_COUNTER_ERROR      0x09 // !< Transmission did not start due a frame counter error.
 #define NRF_802154_TX_ERROR_TIMESTAMP_ENCODING_ERROR 0x0A // !< Timestamp could not been encoded in the transmission process.
+#define NRF_802154_TX_ERROR_INVALID_REQUEST          0x0B // !< The frame or transmit metadata is invalid.
 
 /**
  * @brief Possible errors during the frame reception.
@@ -493,13 +494,6 @@ typedef struct
     } data;                  // !< Result values that are valid only for successful operations.
 } nrf_802154_transmit_done_metadata_t;
 
-/**
- * @brief Function pointer used for notifying about transmission failure.
- */
-typedef void (* nrf_802154_transmit_failed_notification_t)(
-    nrf_802154_tx_error_t                       error,
-    const nrf_802154_transmit_done_metadata_t * p_meta);
-
 /**
  * @brief Structure that holds results of energy detection procedure.
  */

nrf_802154/doc/CHANGELOG.rst

@@ -22,6 +22,9 @@ Notable changes
   This feature improves the reception performance of Enhanced ACK (EnhACK) frames, for which the PHR field value is not known in advance. (KRKNWK-19125)
 * When the proprietary SL library is used, the radio driver no longer requests the HFCLK directly through the :c:func:`nrf_802154_clock_hfclk_start` API.
   Clock requests are now handled internally through MPSL. (KRKNWK-20716)
+* The experimental IFS feature has been removed. (KRKNWK-20788)
+* The return value of :c:func:`nrf_802154_transmit_raw`, :c:func:`nrf_802154_transmit_csma_ca_raw`, and :c:func:`nrf_802154_transmit_raw_at` was updated.
+  These API functions now return ``nrf_802154_tx_error_t`` instead of ``bool``. (KRKNWK-18536)
 
 Added
 =====

nrf_802154/driver/CMakeLists.txt

@@ -74,7 +74,6 @@ target_sources(nrf-802154-driver
     src/mac_features/nrf_802154_filter.c
     src/mac_features/nrf_802154_frame_parser.c
     src/mac_features/nrf_802154_ie_writer.c
-    src/mac_features/nrf_802154_ifs.c
     src/mac_features/nrf_802154_imm_tx.c
     src/mac_features/nrf_802154_security_pib_ram.c
     src/mac_features/nrf_802154_security_writer.c

nrf_802154/driver/src/mac_features/ack_generator/nrf_802154_enh_ack_generator.c

@@ -41,6 +41,7 @@
 #include "nrf_802154_enh_ack_generator.h"
 
 #include "nrf_802154_assert.h"
+#include "nrf_802154_compiler.h"
 #include <string.h>
 
 #include "mac_features/nrf_802154_frame_parser.h"
@@ -624,7 +625,7 @@ uint8_t * nrf_802154_enh_ack_generator_create(
         #endif
 
             ack_state_set(ACK_STATE_PROCESSING);
-        // Fallthrough
+            SWITCH_CASE_FALLTHROUGH;
 
         case ACK_STATE_PROCESSING:
         {

nrf_802154/driver/src/mac_features/nrf_802154_csma_ca.c

@@ -350,43 +350,48 @@ static bool channel_busy(void)
     return result;
 }
 
-bool nrf_802154_csma_ca_start(const nrf_802154_frame_t                     * p_frame,
-                              const nrf_802154_transmit_csma_ca_metadata_t * p_metadata)
+nrf_802154_tx_error_t nrf_802154_csma_ca_start(
+    const nrf_802154_frame_t                     * p_frame,
+    const nrf_802154_transmit_csma_ca_metadata_t * p_metadata)
 {
     nrf_802154_log_function_enter(NRF_802154_LOG_VERBOSITY_LOW);
 
 #if (NRF_802154_FRAME_TIMESTAMP_ENABLED)
     uint64_t ts = nrf_802154_sl_timer_current_time_get();
 
-    nrf_802154_stat_timestamp_write(last_csmaca_start_timestamp, ts);
+    nrf_802154_stat_timestamp_write_last_csmaca_start_timestamp(ts);
 #endif
 
-    bool result = csma_ca_state_set(CSMA_CA_STATE_IDLE, CSMA_CA_STATE_BACKOFF);
-
-    NRF_802154_ASSERT(result);
-    (void)result;
+    nrf_802154_tx_error_t error = NRF_802154_TX_ERROR_TIMESLOT_DENIED;
 
-    uint8_t channel =
-        p_metadata->tx_channel.use_metadata_value ? p_metadata->tx_channel.channel :
-        nrf_802154_pib_channel_get();
-
-    m_frame      = *p_frame;
-    m_data_props = p_metadata->frame_props;
-    m_nb         = 0;
-    m_be         = nrf_802154_pib_csmaca_min_be_get();
-    m_tx_channel = channel;
-#if NRF_802154_TX_TIMESTAMP_PROVIDER_ENABLED
-    m_tx_timestamp_encode = p_metadata->tx_timestamp_encode;
-#endif
-    (void)nrf_802154_tx_power_convert_metadata_to_tx_power_split(channel,
-                                                                 p_metadata->tx_power,
-                                                                 &m_tx_power);
+    bool result = csma_ca_state_set(CSMA_CA_STATE_IDLE, CSMA_CA_STATE_BACKOFF);
 
-    random_backoff_start();
+    if (result)
+    {
+        uint8_t channel =
+            p_metadata->tx_channel.use_metadata_value ? p_metadata->tx_channel.channel :
+            nrf_802154_pib_channel_get();
+
+        m_frame      = *p_frame;
+        m_data_props = p_metadata->frame_props;
+        m_nb         = 0;
+        m_be         = nrf_802154_pib_csmaca_min_be_get();
+        m_tx_channel = channel;
+    #if NRF_802154_TX_TIMESTAMP_PROVIDER_ENABLED
+        m_tx_timestamp_encode = p_metadata->tx_timestamp_encode;
+    #endif
+        (void)nrf_802154_tx_power_convert_metadata_to_tx_power_split(channel,
+                                                                     p_metadata->tx_power,
+                                                                     &m_tx_power);
+
+        random_backoff_start();
+
+        error = NRF_802154_TX_ERROR_NONE;
+    }
 
     nrf_802154_log_function_exit(NRF_802154_LOG_VERBOSITY_LOW);
 
-    return true;
+    return error;
 }
 
 static bool csma_ca_can_abort(nrf_802154_term_t              term_lvl,

nrf_802154/driver/src/mac_features/nrf_802154_csma_ca.h

@@ -63,8 +63,9 @@
  *
  * @retval true   The function always returns true for compatibility reasons
  */
-bool nrf_802154_csma_ca_start(const nrf_802154_frame_t                     * p_frame,
-                              const nrf_802154_transmit_csma_ca_metadata_t * p_metadata);
+nrf_802154_tx_error_t nrf_802154_csma_ca_start(
+    const nrf_802154_frame_t                     * p_frame,
+    const nrf_802154_transmit_csma_ca_metadata_t * p_metadata);
 
 /**
  *@}

nrf_802154/driver/src/mac_features/nrf_802154_delayed_trx.c

@@ -845,12 +845,14 @@ void nrf_802154_delayed_trx_deinit(void)
     }
 }
 
-bool nrf_802154_delayed_trx_transmit(const nrf_802154_frame_t                * p_frame,
-                                     uint64_t                                  tx_time,
-                                     const nrf_802154_transmit_at_metadata_t * p_metadata)
+nrf_802154_tx_error_t nrf_802154_delayed_trx_transmit(
+    const nrf_802154_frame_t                * p_frame,
+    uint64_t                                  tx_time,
+    const nrf_802154_transmit_at_metadata_t * p_metadata)
 {
-    dly_op_data_t * p_dly_tx_data = available_dly_tx_slot_get();
-    bool            result        = false;
+    dly_op_data_t       * p_dly_tx_data = available_dly_tx_slot_get();
+    nrf_802154_tx_error_t error         = NRF_802154_TX_ERROR_TIMESLOT_DENIED;
+    bool                  result        = false;
 
     if (p_dly_tx_data != NULL)
     {
@@ -894,7 +896,12 @@ bool nrf_802154_delayed_trx_transmit(const nrf_802154_frame_t                * p
         result = dly_op_request(&dly_ts_param, p_dly_tx_data);
     }
 
-    return result;
+    if (result)
+    {
+        error = NRF_802154_TX_ERROR_NONE;
+    }
+
+    return error;
 }
 
 bool nrf_802154_delayed_trx_receive(uint64_t rx_time,

nrf_802154/driver/src/mac_features/nrf_802154_delayed_trx.h

@@ -78,9 +78,10 @@ void nrf_802154_delayed_trx_deinit(void);
  * @param[in]  p_metadata   Pointer to metadata structure. Contains detailed properties of data
  *                          to transmit and additional parameters for the procedure.
  */
-bool nrf_802154_delayed_trx_transmit(const nrf_802154_frame_t                * p_frame,
-                                     uint64_t                                  tx_time,
-                                     const nrf_802154_transmit_at_metadata_t * p_metadata);
+nrf_802154_tx_error_t nrf_802154_delayed_trx_transmit(
+    const nrf_802154_frame_t                * p_frame,
+    uint64_t                                  tx_time,
+    const nrf_802154_transmit_at_metadata_t * p_metadata);
 
 /**
  * @brief Cancels a transmission scheduled by a call to @ref nrf_802154_delayed_trx_transmit.