[hermon] Fix inconsistent information in HERMON_SET_PORT_GENERAL_PARAM

pptx and pfctx should not be set together, nor should pprx and pfcrx.

Signed-off-by: Michael Brown <mcb30@ipxe.org>

diff --git a/src/drivers/infiniband/hermon.c b/src/drivers/infiniband/hermon.c
index 91c7455097fd634f0a0757679e010823b8974985..65b148cc277d00ff547777deb642ebe2d80d0ed2 100644 (file)
--- a/src/drivers/infiniband/hermon.c
+++ b/src/drivers/infiniband/hermon.c
@@ -2369,12 +2369,10 @@ static int hermon_eth_open ( struct net_device *netdev ) {
                     v_pptx, 1 );
        MLX_FILL_1 ( &set_port.general, 1,
                     mtu, ( ETH_FRAME_LEN + 40 /* Used by card */ ) );
-       MLX_FILL_2 ( &set_port.general, 2,
-                    pfctx, ( 1 << FCOE_VLAN_PRIORITY ),
-                    pptx, 1 );
-       MLX_FILL_2 ( &set_port.general, 3,
-                    pfcrx, ( 1 << FCOE_VLAN_PRIORITY ),
-                    pprx, 1 );
+       MLX_FILL_1 ( &set_port.general, 2,
+                    pfctx, ( 1 << FCOE_VLAN_PRIORITY ) );
+       MLX_FILL_1 ( &set_port.general, 3,
+                    pfcrx, ( 1 << FCOE_VLAN_PRIORITY ) );
        if ( ( rc = hermon_cmd_set_port ( hermon, 1,
                                          ( HERMON_SET_PORT_GENERAL_PARAM |
                                            ibdev->port ),