diff options
| author | Alexander Aring <alex.aring@gmail.com> | 2014-04-24 19:09:17 +0200 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2014-04-26 12:20:34 -0400 |
| commit | 1cc9fc53b3e6f323c4194157fcc606e30f2a0b6e (patch) | |
| tree | 3ef70706a80833a8a1761898d3560bc70b8a4e43 /drivers/net/ieee802154/at86rf230.c | |
| parent | 57e48ffce90aa6105f2fe032336b6d7e86bec5af (diff) | |
| download | lwn-1cc9fc53b3e6f323c4194157fcc606e30f2a0b6e.tar.gz lwn-1cc9fc53b3e6f323c4194157fcc606e30f2a0b6e.zip | |
at86rf230: remove unnecessary state read
This patch removes a unnecessary state read. The status value is never
evaluate after reading the state.
Also rename the status variable to dvdd, because this variable will be
used later.
Signed-off-by: Alexander Aring <alex.aring@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ieee802154/at86rf230.c')
| -rw-r--r-- | drivers/net/ieee802154/at86rf230.c | 10 |
1 files changed, 3 insertions, 7 deletions
diff --git a/drivers/net/ieee802154/at86rf230.c b/drivers/net/ieee802154/at86rf230.c index 67c01bf1421e..1331152561f9 100644 --- a/drivers/net/ieee802154/at86rf230.c +++ b/drivers/net/ieee802154/at86rf230.c @@ -969,13 +969,9 @@ static int at86rf230_irq_polarity(struct at86rf230_local *lp, int pol) static int at86rf230_hw_init(struct at86rf230_local *lp) { int rc, irq_pol, irq_type; - u8 status; + u8 dvdd; u8 csma_seed[2]; - rc = at86rf230_read_subreg(lp, SR_TRX_STATUS, &status); - if (rc) - return rc; - rc = at86rf230_write_subreg(lp, SR_TRX_CMD, STATE_FORCE_TRX_OFF); if (rc) return rc; @@ -1015,10 +1011,10 @@ static int at86rf230_hw_init(struct at86rf230_local *lp) /* Wait the next SLEEP cycle */ msleep(100); - rc = at86rf230_read_subreg(lp, SR_DVDD_OK, &status); + rc = at86rf230_read_subreg(lp, SR_DVDD_OK, &dvdd); if (rc) return rc; - if (!status) { + if (!dvdd) { dev_err(&lp->spi->dev, "DVDD error\n"); return -EINVAL; } |