> From: Brown, Aaron F > Sent: Thursday, November 9, 2017 7:16 PM > To: 'Colin King' <colin.k...@canonical.com>; Kirsher, Jeffrey T > <jeffrey.t.kirs...@intel.com>; intel-wired-...@lists.osuosl.org; > netdev@vger.kernel.org > Cc: kernel-janit...@vger.kernel.org; linux-ker...@vger.kernel.org > Subject: RE: [Intel-wired-lan] [PATCH] i40e: remove redundant initialization > of read_size > > > From: Intel-wired-lan [mailto:intel-wired-lan-boun...@osuosl.org] On > Behalf > > Of Colin King > > Sent: Sunday, November 5, 2017 5:04 AM > > To: Kirsher, Jeffrey T <jeffrey.t.kirs...@intel.com>; intel-wired- > > l...@lists.osuosl.org; netdev@vger.kernel.org > > Cc: kernel-janit...@vger.kernel.org; linux-ker...@vger.kernel.org > > Subject: [Intel-wired-lan] [PATCH] i40e: remove redundant initialization of > > read_size > > > > From: Colin Ian King <colin.k...@canonical.com> > > > > Variable read_size is initialized and this value is never read, it is > > instead set inside the do-loop, hence the intialization is redundant > > and can be removed. Cleans up clang warning: > > > > drivers/net/ethernet/intel/i40e/i40e_nvm.c:390:6: warning: Value stored > > to 'read_size' during its initialization is never read > > > > Signed-off-by: Colin Ian King <colin.k...@canonical.com> > > s/intialization/initialization/g
Other than that typo, looks fine: Tested-by: Aaron Brown <aaron.f.br...@intel.com>
