Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 2f8cdd95 authored by Alexander Aring's avatar Alexander Aring Committed by Marcel Holtmann
Browse files

at86rf230: remove unnecessary tx state change 



All supported transceivers can do a valid state change from TRX_OFF to
AACK_ON. This patch removes the state change chain from TRX_OFF ->
TX_ON -> AACK_ON instead we doing a directly state change from TRX_OFF
to AACK_ON.

Signed-off-by: default avatarAlexander Aring <alex.aring@gmail.com>
Signed-off-by: default avatarMarcel Holtmann <marcel@holtmann.org>
parent 3b951ca7

drivers/net/ieee802154/at86rf230.c

+3 −13
Original line number Diff line number Diff line
@@ -780,16 +780,6 @@ at86rf230_tx_on(void *context) 
				     at86rf230_tx_complete, true);

}



static void

at86rf230_tx_trac_error(void *context)

{

	struct at86rf230_state_change *ctx = context;

	struct at86rf230_local *lp = ctx->lp;



	at86rf230_async_state_change(lp, ctx, STATE_TX_ON,

				     at86rf230_tx_on, true);

}



static void

at86rf230_tx_trac_check(void *context)

{
@@ -799,12 +789,12 @@ at86rf230_tx_trac_check(void *context) 
	const u8 trac = (buf[1] & 0xe0) >> 5;



	/* If trac status is different than zero we need to do a state change

	 * to STATE_FORCE_TRX_OFF then STATE_TX_ON to recover the transceiver

	 * state to TX_ON.

	 * to STATE_FORCE_TRX_OFF then STATE_RX_AACK_ON to recover the

	 * transceiver.

	 */

	if (trac)

		at86rf230_async_state_change(lp, ctx, STATE_FORCE_TRX_OFF,

					     at86rf230_tx_trac_error, true);

					     at86rf230_tx_on, true);

	else

		at86rf230_tx_on(context);

}