2017-11-10 03:40:15

by Brown, Aaron F

[permalink] [raw]
Subject: RE: [Intel-wired-lan] [PATCH] i40e: remove redundant initialization of read_size

> From: Brown, Aaron F
> Sent: Thursday, November 9, 2017 7:16 PM
> To: 'Colin King' <[email protected]>; Kirsher, Jeffrey T
> <[email protected]>; [email protected];
> [email protected]
> Cc: [email protected]; [email protected]
> Subject: RE: [Intel-wired-lan] [PATCH] i40e: remove redundant initialization
> of read_size
>
> > From: Intel-wired-lan [mailto:[email protected]] On
> Behalf
> > Of Colin King
> > Sent: Sunday, November 5, 2017 5:04 AM
> > To: Kirsher, Jeffrey T <[email protected]>; intel-wired-
> > [email protected]; [email protected]
> > Cc: [email protected]; [email protected]
> > Subject: [Intel-wired-lan] [PATCH] i40e: remove redundant initialization of
> > read_size
> >
> > From: Colin Ian King <[email protected]>
> >
> > 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 <[email protected]>
>
> s/intialization/initialization/g

Other than that typo, looks fine:
Tested-by: Aaron Brown <[email protected]>

From 1583647357956083563@xxx Fri Nov 10 03:16:51 +0000 2017
X-GM-THRID: 1583233154789670684
X-Gmail-Labels: Inbox,Category Forums,HistoricalUnread