Hi, "Brown, Aaron F" <aaron.f.br...@intel.com> writes:
>> --- a/drivers/net/ethernet/intel/igb/igb_main.c >> +++ b/drivers/net/ethernet/intel/igb/igb_main.c >> @@ -8747,12 +8747,19 @@ static void igb_rar_set_index(struct igb_adapter >> *adapter, u32 index) >> if (is_valid_ether_addr(addr)) >> rar_high |= E1000_RAH_AV; >> >> - if (hw->mac.type == e1000_82575) >> + switch (hw->mac.type) { >> + case e1000_82575: >> + case e1000_i210: >> + case e1000_i211: >> + rar_high |= E1000_RAH_QSEL_ENABLE; >> rar_high |= E1000_RAH_POOL_1 * >> - adapter->mac_table[index].queue; >> - else >> + adapter->mac_table[index].queue; >> + break; >> + default: >> rar_high |= E1000_RAH_POOL_1 << >> - adapter->mac_table[index].queue; >> + adapter->mac_table[index].queue; > > Small nit. Shouldn't this line be indented more to be a few spaces > past the "|=" operator as above? I don't know why my editor seemed to disagree, I will fix. Thank you, -- Vinicius