On 04/19/2016 10:57 PM, Jeff Kirsher wrote: > On Tue, 2016-04-19 at 14:34 +0200, Denys Vlasenko wrote: >> "incvalue" variable holds a result of "er32(TIMINCA) & >> E1000_TIMINCA_INCVALUE_MASK" >> and used in "do_div(temp, incvalue)" as a divisor. >> >> Thus, "u64 incvalue" declaration is probably a mistake. >> Even though it seems to be a harmless one, let's fix it. >> >> Signed-off-by: Denys Vlasenko <dvlas...@redhat.com> >> CC: Jeff Kirsher <jeffrey.t.kirs...@intel.com> >> CC: Jesse Brandeburg <jesse.brandeb...@intel.com> >> CC: Shannon Nelson <shannon.nel...@intel.com> >> CC: Carolyn Wyborny <carolyn.wybo...@intel.com> >> CC: Don Skidmore <donald.c.skidm...@intel.com> >> CC: Bruce Allan <bruce.w.al...@intel.com> >> CC: John Ronciak <john.ronc...@intel.com> >> CC: Mitch Williams <mitch.a.willi...@intel.com> >> CC: David S. Miller <da...@davemloft.net> >> CC: LKML <linux-ker...@vger.kernel.org> >> CC: netdev@vger.kernel.org >> --- >> drivers/net/ethernet/intel/e1000e/netdev.c | 3 ++- >> 1 file changed, 2 insertions(+), 1 deletion(-) > > First of all, trimmed down the recipient list since almost all of the > reviewers you added have nothing to do with e1000e.
I took the list here, MAINTAINERS: INTEL ETHERNET DRIVERS M: Jeff Kirsher <jeffrey.t.kirs...@intel.com> R: Jesse Brandeburg <jesse.brandeb...@intel.com> R: Shannon Nelson <shannon.nel...@intel.com> R: Carolyn Wyborny <carolyn.wybo...@intel.com> R: Don Skidmore <donald.c.skidm...@intel.com> R: Bruce Allan <bruce.w.al...@intel.com> R: John Ronciak <john.ronc...@intel.com> R: Mitch Williams <mitch.a.willi...@intel.com> No more specific information who's e1000e reviewer. Sorry.
