e1000e: do not ignore variables which get set a value

Browse source

Static analysis with cppcheck has shown a few instances of a variable which
is assigned a value that is never used.  A number of these are the return
status of various driver function calls which should be passed back to the
caller of the current function.

Signed-off-by: Bruce Allan <bruce.w.allan@intel.com>
Tested-by: Aaron Brown <aaron.f.brown@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>

...

This commit is contained in:

Bruce Allan 2013-01-05 05:08:31 +00:00 • committed by Jeff Kirsher

parent 1f96012d34

commit 7dbbe5d5a5

4 changed files with 14 additions and 6 deletions

Show all changes Ignore whitespace when comparing lines Ignore changes in amount of whitespace Ignore changes in whitespace at EOL

Download patch file Download diff file Expand all files Collapse all files

5

drivers/net/ethernet/intel/e1000e/80003es2lan.c

View file

8

drivers/net/ethernet/intel/e1000e/82571.c

View file

5

drivers/net/ethernet/intel/e1000e/ich8lan.c

View file

2

drivers/net/ethernet/intel/e1000e/phy.c

View file