igc: Remove duplicate code in MAC filtering logic
authorAndre Guedes <andre.guedes@intel.com>
Wed, 18 Mar 2020 23:00:51 +0000 (16:00 -0700)
committerJeff Kirsher <jeffrey.t.kirsher@intel.com>
Sun, 19 Apr 2020 18:39:35 +0000 (11:39 -0700)
commit0c2e060859aa1cb8b13376554d802a251f750fa9
tree6d757d68152a5230333abe6efca6552166c2ab09
parent632fbd5eb5b0e01f03f1acb90a2b9ac1352b5dc7
igc: Remove duplicate code in MAC filtering logic

This patch does a code refactoring in the MAC address filtering logic to
get rid of some duplicate code.

IGC driver has two functions to add MAC address filters that are pretty
much the same: igc_add_mac_filter() and igc_add_mac_filter_flags(). The
only difference is that the latter allows the callee to specify the
'flags' parameter while the former has it hard coded as zero. The same
rationale applies to filter deletion counterparts.

So this patch refactors igc_add_mac_filter() and igc_del_mac_filter() so
they handle the 'flags' parameters, removes the _flags() functions, and
fixes callees accordingly.

Signed-off-by: Andre Guedes <andre.guedes@intel.com>
Tested-by: Aaron Brown <aaron.f.brown@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
drivers/net/ethernet/intel/igc/igc_main.c