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

Commit 74de4c80 authored by Alexander Aring's avatar Alexander Aring Committed by Marcel Holtmann
Browse files

at86rf230: refactor receive handling



This patch refactor the receive handling into one function.

Signed-off-by: default avatarAlexander Aring <alex.aring@gmail.com>
Signed-off-by: default avatarMarcel Holtmann <marcel@holtmann.org>
parent ef5428a1
Loading
Loading
Loading
Loading
+14 −20
Original line number Original line Diff line number Diff line
@@ -781,13 +781,23 @@ at86rf230_tx_trac_status(void *context)
}
}


static void
static void
at86rf230_rx(struct at86rf230_local *lp,
at86rf230_rx_read_frame_complete(void *context)
	     const u8 *data, const u8 len, const u8 lqi)
{
{
	struct sk_buff *skb;
	struct at86rf230_state_change *ctx = context;
	struct at86rf230_local *lp = ctx->lp;
	u8 rx_local_buf[AT86RF2XX_MAX_BUF];
	u8 rx_local_buf[AT86RF2XX_MAX_BUF];
	const u8 *buf = lp->irq.buf;
	struct sk_buff *skb;
	u8 len, lqi;


	memcpy(rx_local_buf, data, len);
	len = buf[1];
	if (!ieee802154_is_valid_psdu_len(len)) {
		dev_vdbg(&lp->spi->dev, "corrupted frame received\n");
		len = IEEE802154_MTU;
	}
	lqi = buf[2 + len];

	memcpy(rx_local_buf, buf + 2, len);
	enable_irq(lp->spi->irq);
	enable_irq(lp->spi->irq);


	skb = dev_alloc_skb(IEEE802154_MTU);
	skb = dev_alloc_skb(IEEE802154_MTU);
@@ -800,22 +810,6 @@ at86rf230_rx(struct at86rf230_local *lp,
	ieee802154_rx_irqsafe(lp->hw, skb, lqi);
	ieee802154_rx_irqsafe(lp->hw, skb, lqi);
}
}


static void
at86rf230_rx_read_frame_complete(void *context)
{
	struct at86rf230_state_change *ctx = context;
	struct at86rf230_local *lp = ctx->lp;
	const u8 *buf = lp->irq.buf;
	u8 len = buf[1];

	if (!ieee802154_is_valid_psdu_len(len)) {
		dev_vdbg(&lp->spi->dev, "corrupted frame received\n");
		len = IEEE802154_MTU;
	}

	at86rf230_rx(lp, buf + 2, len, buf[2 + len]);
}

static void
static void
at86rf230_rx_read_frame(struct at86rf230_local *lp)
at86rf230_rx_read_frame(struct at86rf230_local *lp)
{
{