Daniel
On 4/3/20 9:46 AM, Daniels Umanovskis wrote:
The redundant statement seems to be a copy-paste error from other places
in the file where the same pattern is used
Signed-off-by: Daniels Umanovskis <daniels@xxxxxxxxxxxxx>
---
drivers/net/can/m_can/tcan4x5x.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/net/can/m_can/tcan4x5x.c b/drivers/net/can/m_can/tcan4x5x.c
index eacd428e07e9..1065b0518c70 100644
--- a/drivers/net/can/m_can/tcan4x5x.c
+++ b/drivers/net/can/m_can/tcan4x5x.c
@@ -330,8 +330,6 @@ static int tcan4x5x_clear_interrupts(struct m_can_classdev *cdev)
ret = tcan4x5x_write_tcan_reg(cdev, TCAN4X5X_ERROR_STATUS,
TCAN4X5X_CLEAR_ALL_INT);
- if (ret)
- return ret;
How about just returning the value of tcan4x5x_write.. and remove the
return ret; below
Dan
return ret;
}
