[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Thu, 3 Mar 2022 19:25:04 +0100
From: Miquel Raynal <miquel.raynal@...tlin.com>
To: Alexander Aring <alex.aring@...il.com>,
Stefan Schmidt <stefan@...enfreihafen.org>,
linux-wpan@...r.kernel.org
Cc: "David S. Miller" <davem@...emloft.net>,
Jakub Kicinski <kuba@...nel.org>, netdev@...r.kernel.org,
David Girault <david.girault@...vo.com>,
Romuald Despres <romuald.despres@...vo.com>,
Frederic Blain <frederic.blain@...vo.com>,
Nicolas Schodet <nico@...fr.eu.org>,
Thomas Petazzoni <thomas.petazzoni@...tlin.com>,
Miquel Raynal <miquel.raynal@...tlin.com>
Subject: [PATCH wpan-next v3 07/11] net: ieee802154: at86rf230: Provide meaningful error codes when possible
Either the spi operation failed, or the device encountered an error. In
both case, we know more or less what happened thanks to the spi call
return code or the content of the TRAC register otherwise. Use them in
order to propagate one step above the error.
Signed-off-by: Miquel Raynal <miquel.raynal@...tlin.com>
---
drivers/net/ieee802154/at86rf230.c | 25 +++++++++++++++++++++++--
1 file changed, 23 insertions(+), 2 deletions(-)
diff --git a/drivers/net/ieee802154/at86rf230.c b/drivers/net/ieee802154/at86rf230.c
index 12ee071057d2..5f19266b3045 100644
--- a/drivers/net/ieee802154/at86rf230.c
+++ b/drivers/net/ieee802154/at86rf230.c
@@ -370,7 +370,27 @@ static inline void
at86rf230_async_error(struct at86rf230_local *lp,
struct at86rf230_state_change *ctx, int rc)
{
- dev_err(&lp->spi->dev, "spi_async error %d\n", rc);
+ int reason;
+
+ switch (rc) {
+ case TRAC_CHANNEL_ACCESS_FAILURE:
+ reason = IEEE802154_CHANNEL_ACCESS_FAILURE;
+ break;
+ case TRAC_NO_ACK:
+ reason = IEEE802154_NO_ACK;
+ break;
+ case TRAC_INVALID:
+ reason = IEEE802154_SYSTEM_ERROR;
+ break;
+ default:
+ reason = rc;
+ }
+
+ if (reason < 0)
+ dev_err(&lp->spi->dev, "spi_async error %d\n", reason);
+ else
+ dev_err(&lp->spi->dev, "xceiver error %d\n", reason);
+
at86rf230_async_state_change(lp, ctx, STATE_FORCE_TRX_OFF,
at86rf230_async_error_recover);
@@ -693,6 +713,7 @@ at86rf230_tx_trac_check(void *context)
goto failure;
default:
WARN_ONCE(1, "received tx trac status %d\n", trac);
+ trac = TRAC_INVALID;
lp->trac.invalid++;
goto failure;
}
@@ -702,7 +723,7 @@ at86rf230_tx_trac_check(void *context)
return;
failure:
- at86rf230_async_error(lp, ctx, -EIO);
+ at86rf230_async_error(lp, ctx, trac);
}
static void
--
2.27.0
Powered by blists - more mailing lists