Take control over auto ack feature of 802.15.4 driver

[MeisterBob]

is there a way to get more control over the auto ack feature of the 802.15.4 driver in zephyr?

I would like to ack  frames without destination address only if nrf_802154_ack_data_pending_bit_should_be_set(). For now I added

    if (m_flags.frame_filtered &&
        m_flags.frame_parsed_result &&
        nrf_802154_frame_parser_ar_bit_is_set(&m_current_rx_frame_data) &&
        nrf_802154_pib_auto_ack_get())
    {
+        uint8_t dest_addr_type = nrf_802154_frame_parser_dst_addr_type_get(&m_current_rx_frame_data);
+        if ((dest_addr_type != DEST_ADDR_TYPE_NONE) ||
+            (dest_addr_type == DEST_ADDR_TYPE_NONE &&
+            nrf_802154_ack_data_pending_bit_should_be_set(&m_current_rx_frame_data)))
+        {
            nrf_802154_tx_work_buffer_reset(&m_default_frame_props);
            mp_ack = nrf_802154_ack_generator_create(&m_current_rx_frame_data);
+        }
    }

to nrf_802154_trx_receive_frame_bcmatched() but as this isn't default behavior of IEEE 802.15.4 I don't think a pull request would be merged. Would it be possible to get some kind of hook or weak function that I can overwrite in my application?

[MeisterBob]

That's what I said. But would it be possible to add a hook, weak function or something else to make this available? I would be willing to create a pull request  if you would point out the preferred way of implementation.

[Einar Thorsrud]

I understand. I was thinking about adding the hook as well as using a weak function as I am not sure it would make sense in general. I could be wrong though, and you are free to make a pull request, though (see this post). That way you can get feedback from the code owner / maintainers on the preferred method etc in the pull request.