[PATCH net-next] mlxsw: spectrum_flower: use PTR_ERR_OR_ZERO()

Subsystems: mellanox ethernet switch drivers, networking drivers, the rest

STALE2920d

3 messages, 3 authors, 2018-08-09 · open the first message on its own page

[PATCH net-next] mlxsw: spectrum_flower: use PTR_ERR_OR_ZERO()

From: YueHaibing <hidden>
Date: 2018-08-08 12:29:26

Fix ptr_ret.cocci warnings:
 drivers/net/ethernet/mellanox/mlxsw/spectrum_flower.c:543:1-3: WARNING: PTR_ERR_OR_ZERO can be used

Signed-off-by: YueHaibing <redacted>
---
 drivers/net/ethernet/mellanox/mlxsw/spectrum_flower.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/net/ethernet/mellanox/mlxsw/spectrum_flower.c b/drivers/net/ethernet/mellanox/mlxsw/spectrum_flower.c
index 8213cb7..ddcd508 100644
--- a/drivers/net/ethernet/mellanox/mlxsw/spectrum_flower.c
+++ b/drivers/net/ethernet/mellanox/mlxsw/spectrum_flower.c
@@ -540,10 +540,9 @@ int mlxsw_sp_flower_tmplt_create(struct mlxsw_sp *mlxsw_sp,
 					   f->common.chain_index,
 					   MLXSW_SP_ACL_PROFILE_FLOWER,
 					   &rulei.values.elusage);
-	if (IS_ERR(ruleset))
-		return PTR_ERR(ruleset);
+
 	/* keep the reference to the ruleset */
-	return 0;
+	return PTR_ERR_OR_ZERO(ruleset);
 }
 
 void mlxsw_sp_flower_tmplt_destroy(struct mlxsw_sp *mlxsw_sp,
-- 
2.7.0

Re: [PATCH net-next] mlxsw: spectrum_flower: use PTR_ERR_OR_ZERO()

From: Ido Schimmel <hidden>
Date: 2018-08-08 15:08:54

On Wed, Aug 08, 2018 at 08:29:08PM +0800, YueHaibing wrote:
Fix ptr_ret.cocci warnings:
 drivers/net/ethernet/mellanox/mlxsw/spectrum_flower.c:543:1-3: WARNING: PTR_ERR_OR_ZERO can be used

Signed-off-by: YueHaibing <redacted>
Reviewed-by: Ido Schimmel <redacted>

Thanks!

Re: [PATCH net-next] mlxsw: spectrum_flower: use PTR_ERR_OR_ZERO()

From: David Miller <davem@davemloft.net>
Date: 2018-08-09 21:14:25

From: YueHaibing <redacted>
Date: Wed, 8 Aug 2018 20:29:08 +0800
Fix ptr_ret.cocci warnings:
 drivers/net/ethernet/mellanox/mlxsw/spectrum_flower.c:543:1-3: WARNING: PTR_ERR_OR_ZERO can be used

Signed-off-by: YueHaibing <redacted>
Applied.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help