1 files changed, 41 insertions, 65 deletions

diff --git a/drivers/net/ethernet/intel/igb/e1000_82575.c b/drivers/net/ethernet/intel/igb/e1000_82575.c
index 9057d10a698a..8d79face0717 100644
--- a/drivers/net/ethernet/intel/igb/e1000_82575.c
+++ b/drivers/net/ethernet/intel/igb/e1000_82575.c
@@ -238,6 +238,7 @@ static s32 igb_init_nvm_params_82575(struct e1000_hw *hw)
 
         size = (u16)((eecd & E1000_EECD_SIZE_EX_MASK) >>
                      E1000_EECD_SIZE_EX_SHIFT);
+
         /* Added to a constant, "size" becomes the left-shift value
          * for setting word_size.
          */
@@ -250,86 +251,52 @@ static s32 igb_init_nvm_params_82575(struct e1000_hw *hw)
                 size = 15;
 
         nvm->word_size = 1 << size;
-        if (hw->mac.type < e1000_i210) {
-                nvm->opcode_bits = 8;
-                nvm->delay_usec = 1;
-
-                switch (nvm->override) {
-                case e1000_nvm_override_spi_large:
-                        nvm->page_size = 32;
-                        nvm->address_bits = 16;
-                        break;
-                case e1000_nvm_override_spi_small:
-                        nvm->page_size = 8;
-                        nvm->address_bits = 8;
-                        break;
-                default:
-                        nvm->page_size = eecd & E1000_EECD_ADDR_BITS ? 32 : 8;
-                        nvm->address_bits = eecd & E1000_EECD_ADDR_BITS ?
-                                            16 : 8;
-                        break;
-                }
-                if (nvm->word_size == (1 << 15))
-                        nvm->page_size = 128;
+        nvm->opcode_bits = 8;
+        nvm->delay_usec = 1;
 
-                nvm->type = e1000_nvm_eeprom_spi;
-        } else {
-                nvm->type = e1000_nvm_flash_hw;
+        switch (nvm->override) {
+        case e1000_nvm_override_spi_large:
+                nvm->page_size = 32;
+                nvm->address_bits = 16;
+                break;
+        case e1000_nvm_override_spi_small:
+                nvm->page_size = 8;
+                nvm->address_bits = 8;
+                break;
+        default:
+                nvm->page_size = eecd & E1000_EECD_ADDR_BITS ? 32 : 8;
+                nvm->address_bits = eecd & E1000_EECD_ADDR_BITS ?
+                                    16 : 8;
+                break;
         }
+        if (nvm->word_size == (1 << 15))
+                nvm->page_size = 128;
+
+        nvm->type = e1000_nvm_eeprom_spi;
 
         /* NVM Function Pointers */
+        nvm->ops.acquire = igb_acquire_nvm_82575;
+        nvm->ops.release = igb_release_nvm_82575;
+        nvm->ops.write = igb_write_nvm_spi;
+        nvm->ops.validate = igb_validate_nvm_checksum;
+        nvm->ops.update = igb_update_nvm_checksum;
+        if (nvm->word_size < (1 << 15))
+                nvm->ops.read = igb_read_nvm_eerd;
+        else
+                nvm->ops.read = igb_read_nvm_spi;
+
+        /* override generic family function pointers for specific descendants */
         switch (hw->mac.type) {
         case e1000_82580:
                 nvm->ops.validate = igb_validate_nvm_checksum_82580;
                 nvm->ops.update = igb_update_nvm_checksum_82580;
-                nvm->ops.acquire = igb_acquire_nvm_82575;
-                nvm->ops.release = igb_release_nvm_82575;
-                if (nvm->word_size < (1 << 15))
-                        nvm->ops.read = igb_read_nvm_eerd;
-                else
-                        nvm->ops.read = igb_read_nvm_spi;
-                nvm->ops.write = igb_write_nvm_spi;
                 break;
         case e1000_i354:
         case e1000_i350:
                 nvm->ops.validate = igb_validate_nvm_checksum_i350;
                 nvm->ops.update = igb_update_nvm_checksum_i350;
-                nvm->ops.acquire = igb_acquire_nvm_82575;
-                nvm->ops.release = igb_release_nvm_82575;
-                if (nvm->word_size < (1 << 15))
-                        nvm->ops.read = igb_read_nvm_eerd;
-                else
-                        nvm->ops.read = igb_read_nvm_spi;
-                nvm->ops.write = igb_write_nvm_spi;
-                break;
-        case e1000_i210:
-                nvm->ops.validate = igb_validate_nvm_checksum_i210;
-                nvm->ops.update   = igb_update_nvm_checksum_i210;
-                nvm->ops.acquire = igb_acquire_nvm_i210;
-                nvm->ops.release = igb_release_nvm_i210;
-                nvm->ops.read    = igb_read_nvm_srrd_i210;
-                nvm->ops.write   = igb_write_nvm_srwr_i210;
-                nvm->ops.valid_led_default = igb_valid_led_default_i210;
-                break;
-        case e1000_i211:
-                nvm->ops.acquire  = igb_acquire_nvm_i210;
-                nvm->ops.release  = igb_release_nvm_i210;
-                nvm->ops.read     = igb_read_nvm_i211;
-                nvm->ops.valid_led_default = igb_valid_led_default_i210;
-                nvm->ops.validate = NULL;
-                nvm->ops.update   = NULL;
-                nvm->ops.write    = NULL;
                 break;
         default:
-                nvm->ops.validate = igb_validate_nvm_checksum;
-                nvm->ops.update = igb_update_nvm_checksum;
-                nvm->ops.acquire = igb_acquire_nvm_82575;
-                nvm->ops.release = igb_release_nvm_82575;
-                if (nvm->word_size < (1 << 15))
-                        nvm->ops.read = igb_read_nvm_eerd;
-                else
-                        nvm->ops.read = igb_read_nvm_spi;
-                nvm->ops.write = igb_write_nvm_spi;
                 break;
         }
 
@@ -601,6 +568,15 @@ static s32 igb_get_invariants_82575(struct e1000_hw *hw)
 
         /* NVM initialization */
         ret_val = igb_init_nvm_params_82575(hw);
+        switch (hw->mac.type) {
+        case e1000_i210:
+        case e1000_i211:
+                ret_val = igb_init_nvm_params_i210(hw);
+                break;
+        default:
+                break;
+        }
+
         if (ret_val)
                 goto out;