Daniel
On 4/3/20 10:08 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 | 6 +-----
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/drivers/net/can/m_can/tcan4x5x.c b/drivers/net/can/m_can/tcan4x5x.c
index eacd428e07e9..07f1d4a627fa 100644
--- a/drivers/net/can/m_can/tcan4x5x.c
+++ b/drivers/net/can/m_can/tcan4x5x.c
@@ -328,12 +328,8 @@ static int tcan4x5x_clear_interrupts(struct m_can_classdev *cdev)
if (ret)
return ret;
- ret = tcan4x5x_write_tcan_reg(cdev, TCAN4X5X_ERROR_STATUS,
+ return tcan4x5x_write_tcan_reg(cdev, TCAN4X5X_ERROR_STATUS,
TCAN4X5X_CLEAR_ALL_INT);
- if (ret)
- return ret;
-
- return ret;
}
static int tcan4x5x_init(struct m_can_classdev *cdev)
Acked-by: Dan Murphy <dmurphy@xxxxxx>
