[nrf fromlist] net: openthread: cache radio_api->get_capabilities

The capabilities returned by radio_api->get_capabilities do not change.
The result is cached in a variable to improve performance.

Upstream PR: zephyrproject-rtos/zephyr#79919

Signed-off-by: Andrzej Kuroś <[email protected]>

Author: ankuns

modules/openthread/platform/radio.c

@@ -104,8 +104,9 @@ static struct ieee802154_radio_api *radio_api;
 
 /* Get the default tx output power from Kconfig */
 static int8_t tx_power = CONFIG_OPENTHREAD_DEFAULT_TX_POWER;
-static uint16_t channel;
 static bool promiscuous;
+static uint16_t channel;
+static enum ieee802154_hw_caps radio_caps;
 
 static uint16_t energy_detection_time;
 static uint8_t energy_detection_channel;
@@ -368,8 +369,9 @@ void platformRadioInit(void)
 			   OT_WORKER_PRIORITY, NULL);
 	k_thread_name_set(&ot_work_q.thread, "ot_radio_workq");
 
-	if ((radio_api->get_capabilities(radio_dev) &
-	     IEEE802154_HW_TX_RX_ACK) != IEEE802154_HW_TX_RX_ACK) {
+	radio_caps = radio_api->get_capabilities(radio_dev);
+
+	if ((radio_caps & IEEE802154_HW_TX_RX_ACK) != IEEE802154_HW_TX_RX_ACK) {
 		LOG_ERR("Only radios with automatic ack handling "
 			"are currently supported");
 		k_panic();
@@ -415,7 +417,7 @@ void transmit_message(struct k_work *tx_job)
 					     sTransmitFrame.mInfo.mTxInfo.mIsSecurityProcessed);
 	net_pkt_set_ieee802154_mac_hdr_rdy(tx_pkt, sTransmitFrame.mInfo.mTxInfo.mIsHeaderUpdated);
 
-	if ((radio_api->get_capabilities(radio_dev) & IEEE802154_HW_TXTIME) &&
+	if ((radio_caps & IEEE802154_HW_TXTIME) &&
 	    (sTransmitFrame.mInfo.mTxInfo.mTxDelay != 0)) {
 #if defined(CONFIG_NET_PKT_TXTIME)
 		uint32_t tx_at = sTransmitFrame.mInfo.mTxInfo.mTxDelayBaseTime +
@@ -425,7 +427,7 @@ void transmit_message(struct k_work *tx_job)
 		tx_err =
 			radio_api->tx(radio_dev, IEEE802154_TX_MODE_TXTIME_CCA, tx_pkt, tx_payload);
 	} else if (sTransmitFrame.mInfo.mTxInfo.mCsmaCaEnabled) {
-		if (radio_api->get_capabilities(radio_dev) & IEEE802154_HW_CSMA) {
+		if (radio_caps & IEEE802154_HW_CSMA) {
 			tx_err = radio_api->tx(radio_dev, IEEE802154_TX_MODE_CSMA_CA, tx_pkt,
 					       tx_payload);
 		} else {
@@ -662,7 +664,7 @@ void platformRadioProcess(otInstance *aInstance)
 		reset_pending_event(PENDING_EVENT_TX_DONE);
 
 		if (sState == OT_RADIO_STATE_TRANSMIT ||
-		    radio_api->get_capabilities(radio_dev) & IEEE802154_HW_SLEEP_TO_TX) {
+		    (radio_caps & IEEE802154_HW_SLEEP_TO_TX)) {
 			sState = OT_RADIO_STATE_RECEIVE;
 			handle_tx_done(aInstance);
 		}
@@ -848,10 +850,6 @@ otError otPlatRadioTransmit(otInstance *aInstance, otRadioFrame *aPacket)
 
 	__ASSERT_NO_MSG(aPacket == &sTransmitFrame);
 
-	enum ieee802154_hw_caps radio_caps;
-
-	radio_caps = radio_api->get_capabilities(radio_dev);
-
 	if ((sState == OT_RADIO_STATE_RECEIVE) || (radio_caps & IEEE802154_HW_SLEEP_TO_TX)) {
 		if (run_tx_task(aInstance) == 0) {
 			error = OT_ERROR_NONE;
@@ -880,11 +878,8 @@ int8_t otPlatRadioGetRssi(otInstance *aInstance)
 	int8_t ret_rssi = INT8_MAX;
 	int error = 0;
 	const uint16_t detection_time = 1;
-	enum ieee802154_hw_caps radio_caps;
 	ARG_UNUSED(aInstance);
 
-	radio_caps = radio_api->get_capabilities(radio_dev);
-
 	if (!(radio_caps & IEEE802154_HW_ENERGY_SCAN)) {
 		/*
 		 * TODO: No API in Zephyr to get the RSSI
@@ -913,13 +908,10 @@ otRadioCaps otPlatRadioGetCaps(otInstance *aInstance)
 {
 	otRadioCaps caps = OT_RADIO_CAPS_NONE;
 
-	enum ieee802154_hw_caps radio_caps;
 	ARG_UNUSED(aInstance);
 	__ASSERT(radio_api,
 	    "platformRadioInit needs to be called prior to otPlatRadioGetCaps");
 
-	radio_caps = radio_api->get_capabilities(radio_dev);
-
 	if (radio_caps & IEEE802154_HW_ENERGY_SCAN) {
 		caps |= OT_RADIO_CAPS_ENERGY_SCAN;
 	}