[Intel-wired-lan] [PATCH] i40e/i40evf: cleanup i40e_update_nvm_checksum()

Bowers, AndrewX andrewx.bowers at intel.com
Wed May 22 22:14:26 UTC 2019


> -----Original Message-----
> From: Bowers, AndrewX
> Sent: Friday, May 17, 2019 3:34 PM
> To: kernel-janitors at vger.kernel.org; intel-wired-lan at lists.osuosl.org
> Subject: RE: [Intel-wired-lan] [PATCH] i40e/i40evf: cleanup
> i40e_update_nvm_checksum()
> 
> > -----Original Message-----
> > From: Intel-wired-lan [mailto:intel-wired-lan-bounces at osuosl.org] On
> > Behalf Of Dan Carpenter
> > Sent: Thursday, May 9, 2019 3:11 AM
> > To: Kirsher, Jeffrey T <jeffrey.t.kirsher at intel.com>
> > Cc: kernel-janitors at vger.kernel.org; intel-wired-lan at lists.osuosl.org
> > 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 at oracle.com>
> > ---
> >  drivers/net/ethernet/intel/i40e/i40e_nvm.c | 9 ++++-----
> >  1 file changed, 4 insertions(+), 5 deletions(-)
> 
> kernel-janitors at vger.kernel.org; intel-wired-lan at lists.osuosl.org

Tested-by: Andrew Bowers <andrewx.bowers at intel.com>

Helps to paste in my "Tested by" and not the addresses I'm sending it to... 



More information about the Intel-wired-lan mailing list