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
