> From: Brown, Aaron F > Sent: Thursday, November 9, 2017 7:16 PM > To: 'Colin King' <colin.king@xxxxxxxxxxxxx>; Kirsher, Jeffrey T > <jeffrey.t.kirsher@xxxxxxxxx>; intel-wired-lan@xxxxxxxxxxxxxxxx; > netdev@xxxxxxxxxxxxxxx > Cc: kernel-janitors@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx > Subject: RE: [Intel-wired-lan] [PATCH] i40e: remove redundant initialization > of read_size > > > From: Intel-wired-lan [mailto:intel-wired-lan-bounces@xxxxxxxxxx] On > Behalf > > Of Colin King > > Sent: Sunday, November 5, 2017 5:04 AM > > To: Kirsher, Jeffrey T <jeffrey.t.kirsher@xxxxxxxxx>; intel-wired- > > lan@xxxxxxxxxxxxxxxx; netdev@xxxxxxxxxxxxxxx > > Cc: kernel-janitors@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx > > Subject: [Intel-wired-lan] [PATCH] i40e: remove redundant initialization of > > read_size > > > > From: Colin Ian King <colin.king@xxxxxxxxxxxxx> > > > > 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.king@xxxxxxxxxxxxx> > > s/intialization/initialization/g Other than that typo, looks fine: Tested-by: Aaron Brown <aaron.f.brown@xxxxxxxxx> -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
