2020-11-01 14:08:39

by Tom Rix

Subject: [PATCH] net/mlx4_core : remove unneeded semicolon

From: Tom Rix <[email protected]>

A semicolon is not needed after a switch statement.

Signed-off-by: Tom Rix <[email protected]>
---
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);
--
2.18.1

2020-11-02 09:33:45

by Tariq Toukan

Subject: Re: [PATCH] net/mlx4_core : remove unneeded semicolon

On 11/1/2020 4:05 PM, [email protected] wrote:
> From: Tom Rix <[email protected]>
>
> A semicolon is not needed after a switch statement.
>
> Signed-off-by: Tom Rix <[email protected]>
> ---
> 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 <[email protected]>

Thanks,
Tariq

2020-11-03 01:55:39

by Jakub Kicinski

Subject: Re: [PATCH] net/mlx4_core : remove unneeded semicolon

On Mon, 2 Nov 2020 11:31:57 +0200 Tariq Toukan wrote:
> On 11/1/2020 4:05 PM, [email protected] wrote:
> > From: Tom Rix <[email protected]>
> >
> > A semicolon is not needed after a switch statement.
> >
> > Signed-off-by: Tom Rix <[email protected]>
>
> Reviewed-by: Tariq Toukan <[email protected]>

Applied.