diff options
| author | Antonio Quartulli <a@unstable.cc> | 2020-12-10 09:53:21 +0100 |
|---|---|---|
| committer | Marc Kleine-Budde <mkl@pengutronix.de> | 2020-12-10 10:10:43 +0100 |
| commit | ecbaf5e13f45d8eebd3b772ba0032afcec1e519a (patch) | |
| tree | 6de7be0e3b6ad531b48ae7f53ce4ab758bc74ec6 | |
| parent | 921ca574cd382142add8b12d0a7117f495510de5 (diff) | |
| download | lwn-ecbaf5e13f45d8eebd3b772ba0032afcec1e519a.tar.gz lwn-ecbaf5e13f45d8eebd3b772ba0032afcec1e519a.zip | |
can: rx-offload: can_rx_offload_offload_one(): avoid double unlikely() notation when using IS_ERR()
The definition of IS_ERR() already applies the unlikely() notation when
checking the error status of the passed pointer. For this reason there is no
need to have the same notation outside of IS_ERR() itself.
Clean up code by removing redundant notation.
Signed-off-by: Antonio Quartulli <a@unstable.cc>
Link: https://lore.kernel.org/r/20201210085321.18693-1-a@unstable.cc
Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
| -rw-r--r-- | drivers/net/can/rx-offload.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/can/rx-offload.c b/drivers/net/can/rx-offload.c index 450c5cfcb3fc..3c1912c0430b 100644 --- a/drivers/net/can/rx-offload.c +++ b/drivers/net/can/rx-offload.c @@ -157,7 +157,7 @@ can_rx_offload_offload_one(struct can_rx_offload *offload, unsigned int n) /* There was a problem reading the mailbox, propagate * error value. */ - if (unlikely(IS_ERR(skb))) { + if (IS_ERR(skb)) { offload->dev->stats.rx_dropped++; offload->dev->stats.rx_fifo_errors++; |