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

[Dan Murphy <dmurphy@xxxxxx>]

Sean

On 12/11/19 8:16 AM, Sean Nyekjaer wrote:
> Signed-off-by: Sean Nyekjaer <sean@xxxxxxxxxx>
> ---
>   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 8ed2813f227e..0fd83045f371 100644
> --- a/drivers/net/can/m_can/tcan4x5x.c
> +++ b/drivers/net/can/m_can/tcan4x5x.c
> @@ -327,12 +327,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>