> -----Original Message----- > From: Bowers, AndrewX > Sent: Friday, May 17, 2019 3:34 PM > To: kernel-janitors@xxxxxxxxxxxxxxx; intel-wired-lan@xxxxxxxxxxxxxxxx > Subject: RE: [Intel-wired-lan] [PATCH] i40e/i40evf: cleanup > i40e_update_nvm_checksum() > > > -----Original Message----- > > From: Intel-wired-lan [mailto:intel-wired-lan-bounces@xxxxxxxxxx] On > > Behalf Of Dan Carpenter > > Sent: Thursday, May 9, 2019 3:11 AM > > To: Kirsher, Jeffrey T <jeffrey.t.kirsher@xxxxxxxxx> > > Cc: kernel-janitors@xxxxxxxxxxxxxxx; intel-wired-lan@xxxxxxxxxxxxxxxx > > Subject: [Intel-wired-lan] [PATCH] i40e/i40evf: cleanup > > i40e_update_nvm_checksum() > > > > Smatch complains that we use 'checksum' when it's uninitialized. > > > > drivers/net/ethernet/intel/i40e/i40e_nvm.c:581 > > i40e_update_nvm_checksum() > > error: uninitialized symbol 'checksum'. > > > > This is true, but it harmless because we don't use it again. > > > > Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> > > --- > > drivers/net/ethernet/intel/i40e/i40e_nvm.c | 9 ++++----- > > 1 file changed, 4 insertions(+), 5 deletions(-) > > kernel-janitors@xxxxxxxxxxxxxxx; intel-wired-lan@xxxxxxxxxxxxxxxx Tested-by: Andrew Bowers <andrewx.bowers@xxxxxxxxx> Helps to paste in my "Tested by" and not the addresses I'm sending it to...