Re: [PATCH] can: tcan4x5x: remove redundant return statement

[Dan Murphy <dmurphy@xxxxxx>]

Sean

On 12/11/19 8:03 AM, Sean Nyekjaer wrote:
> Signed-off-by: Sean Nyekjaer <sean@xxxxxxxxxx>
> ---
>   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 8ed2813f227e..9a33fcaaf067 100644
> --- a/drivers/net/can/m_can/tcan4x5x.c
> +++ b/drivers/net/can/m_can/tcan4x5x.c
> @@ -329,8 +329,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;
>   
>   	return ret;
>   }
How about return tcan4x5x_write_tcan_reg(...)?