i40e: Don't check operational or sync bit for App TLV
authorNeerav Parikh <neerav.parikh@intel.com>
Thu, 26 Feb 2015 16:12:00 +0000 (16:12 +0000)
committerJeff Kirsher <jeffrey.t.kirsher@intel.com>
Thu, 5 Mar 2015 09:53:22 +0000 (01:53 -0800)
In CEE mode the firmware does not set the operational status bit of
the application TLV status as returned from the "Get CEE DCBX Oper Cfg"
AQ command. This occurs whenever a DCBX configuration is changed.

This is a workaround to remove the check for the operational and sync bits
of the application TLV status till a firmware fix is provided.

Change-ID: I1a31ff2fcadcb06feb5b55776a33593afc6ea176
Signed-off-by: Neerav Parikh <neerav.parikh@intel.com>
Acked-by: Shannon Nelson <shannon.nelson@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
drivers/net/ethernet/intel/i40e/i40e_dcb.c

index 3ce43588592d99c7c5449b6cd0fb84a31177d84f..6e146675676097562985ea3cc7924562499938a3 100644 (file)
@@ -459,7 +459,7 @@ static void i40e_cee_to_dcb_v1_config(
        sync = (status & I40E_TLV_STATUS_SYNC) ? 1 : 0;
        oper = (status & I40E_TLV_STATUS_OPER) ? 1 : 0;
        /* Add APPs if Error is False and Oper/Sync is True */
-       if (!err && sync && oper) {
+       if (!err) {
                /* CEE operating configuration supports FCoE/iSCSI/FIP only */
                dcbcfg->numapps = I40E_CEE_OPER_MAX_APPS;