be2net: clarify promiscuous cmd with a comment
authorSathya Perla <sathyap@serverengines.com>
Sun, 11 Apr 2010 22:35:27 +0000 (22:35 +0000)
committerDavid S. Miller <davem@davemloft.net>
Tue, 13 Apr 2010 10:32:43 +0000 (03:32 -0700)
The promiscous cmd config code gives an impression that
setting a port to promisc mode will unset the other port.
This is not the case and is clarified with a comment.

Signed-off-by: Sathya Perla <sathyap@serverengines.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/benet/be_cmds.c

index da8793026bb1b09512250a55469c677e33e03c3d..e79bf8b9af3bc5e0a3034d9b3c9ab7d6f9dfef39 100644 (file)
@@ -1113,6 +1113,10 @@ int be_cmd_promiscuous_config(struct be_adapter *adapter, u8 port_num, bool en)
        be_cmd_hdr_prepare(&req->hdr, CMD_SUBSYSTEM_ETH,
                OPCODE_ETH_PROMISCUOUS, sizeof(*req));
 
+       /* In FW versions X.102.149/X.101.487 and later,
+        * the port setting associated only with the
+        * issuing pci function will take effect
+        */
        if (port_num)
                req->port1_promiscuous = en;
        else