[dpdk-dev,5/5] net/mlx5: remove old MLNX_OFED 3.3 verification

Message ID f0533cf454c4496fbc7e575807c3513d1d0b6000.1501588970.git.nelio.laranjeiro@6wind.com (mailing list archive)
State Superseded, archived
Delegated to: Ferruh Yigit
Headers

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation success Compilation OK

Commit Message

Nélio Laranjeiro Aug. 1, 2017, 12:09 p.m. UTC
  This version of MLNX_OFED is no more supported.

Signed-off-by: Nelio Laranjeiro <nelio.laranjeiro@6wind.com>
Acked-by: Shahaf Shuler <shahafs@mellanox.com>
---
 drivers/net/mlx5/mlx5.h | 5 -----
 1 file changed, 5 deletions(-)
  

Comments

Ferruh Yigit Aug. 17, 2017, 2:38 p.m. UTC | #1
On 8/1/2017 1:09 PM, Nelio Laranjeiro wrote:
> This version of MLNX_OFED is no more supported.

Does it make sense to document minimum supported MLNX_OFED version?

> 
> Signed-off-by: Nelio Laranjeiro <nelio.laranjeiro@6wind.com>
> Acked-by: Shahaf Shuler <shahafs@mellanox.com>

<...>
  
Nélio Laranjeiro Aug. 22, 2017, 8:25 a.m. UTC | #2
On Thu, Aug 17, 2017 at 03:38:05PM +0100, Ferruh Yigit wrote:
> On 8/1/2017 1:09 PM, Nelio Laranjeiro wrote:
> > This version of MLNX_OFED is no more supported.
> 
> Does it make sense to document minimum supported MLNX_OFED version?
> 
> > 
> > Signed-off-by: Nelio Laranjeiro <nelio.laranjeiro@6wind.com>
> > Acked-by: Shahaf Shuler <shahafs@mellanox.com>

Hi Ferruh,

This is already documented in the NIC guides, see section "20.5.
Prerequisites" [1].

Thanks,

[1] http://dpdk.org/doc/guides/nics/mlx5.html
  

Patch

diff --git a/drivers/net/mlx5/mlx5.h b/drivers/net/mlx5/mlx5.h
index 2392be5..e89aba8 100644
--- a/drivers/net/mlx5/mlx5.h
+++ b/drivers/net/mlx5/mlx5.h
@@ -63,11 +63,6 @@ 
 #include "mlx5_autoconf.h"
 #include "mlx5_defs.h"
 
-#if !defined(HAVE_VERBS_IBV_EXP_CQ_COMPRESSED_CQE) || \
-	!defined(HAVE_VERBS_MLX5_ETH_VLAN_INLINE_HEADER_SIZE)
-#error Mellanox OFED >= 3.3 is required, please refer to the documentation.
-#endif
-
 enum {
 	PCI_VENDOR_ID_MELLANOX = 0x15b3,
 };