Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Subject: Re: [PATCH] net/mlx4_core : remove unneeded semicolon
To:     trix@redhat.com, davem@davemloft.net, kuba@kernel.org
Cc:     netdev@vger.kernel.org, linux-rdma@vger.kernel.org,
        linux-kernel@vger.kernel.org
References: <20201101140528.2279424-1-trix@redhat.com>
From:   Tariq Toukan <ttoukan.linux@gmail.com>
Message-ID: <78cd4150-0040-44a7-81cf-02c17d61f463@gmail.com>
Date:   Mon, 2 Nov 2020 11:31:57 +0200
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:68.0) Gecko/20100101
 Thunderbird/68.12.1
MIME-Version: 1.0
In-Reply-To: <20201101140528.2279424-1-trix@redhat.com>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Precedence: bulk


On 11/1/2020 4:05 PM, trix@redhat.com wrote:
> From: Tom Rix <trix@redhat.com>
> 
> A semicolon is not needed after a switch statement.
> 
> Signed-off-by: Tom Rix <trix@redhat.com>
> ---
>   drivers/net/ethernet/mellanox/mlx4/resource_tracker.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/net/ethernet/mellanox/mlx4/resource_tracker.c b/drivers/net/ethernet/mellanox/mlx4/resource_tracker.c
> index 1187ef1375e2..394f43add85c 100644
> --- a/drivers/net/ethernet/mellanox/mlx4/resource_tracker.c
> +++ b/drivers/net/ethernet/mellanox/mlx4/resource_tracker.c
> @@ -300,7 +300,7 @@ static const char *resource_str(enum mlx4_resource rt)
>   	case RES_FS_RULE: return "RES_FS_RULE";
>   	case RES_XRCD: return "RES_XRCD";
>   	default: return "Unknown resource type !!!";
> -	};
> +	}
>   }
>   
>   static void rem_slave_vlans(struct mlx4_dev *dev, int slave);
> 

Reviewed-by: Tariq Toukan <tariqt@nvidia.com>

Thanks,
Tariq