net: neterion: remove redundant assignment to variable tmp64

Colin Ian King · kuba-moo · commit 465aa30420bc · 2020-04-12T11:44:02.000-07:00
The variable tmp64 is being initialized with a value that is never read
and it is being updated later with a new value.  The initialization is
redundant and can be removed.

Addresses-Coverity: ("Unused value")
Signed-off-by: Colin Ian King &lt;colin.king@canonical.com&gt;
Signed-off-by: Jakub Kicinski &lt;kuba@kernel.org&gt;
diff --git a/drivers/net/ethernet/neterion/s2io.c b/drivers/net/ethernet/neterion/s2io.c
@@ -5155,7 +5155,7 @@ static int do_s2io_delete_unicast_mc(struct s2io_nic *sp, u64 addr)
 /* read mac entries from CAM */
 static u64 do_s2io_read_unicast_mc(struct s2io_nic *sp, int offset)
 {
-	u64 tmp64 = 0xffffffffffff0000ULL, val64;
+	u64 tmp64, val64;
 	struct XENA_dev_config __iomem *bar0 = sp->bar0;
 
 	/* read mac addr */

Original file line number	Diff line number	Diff line change
`@@ -5155,7 +5155,7 @@ static int do_s2io_delete_unicast_mc(struct s2io_nic *sp, u64 addr)`
`5155`	`5155`	`/* read mac entries from CAM */`
`5156`	`5156`	`static u64 do_s2io_read_unicast_mc(struct s2io_nic *sp, int offset)`
`5157`	`5157`	`{`
`5158`		`- u64 tmp64 = 0xffffffffffff0000ULL, val64;`
	`5158`	`+ u64 tmp64, val64;`
`5159`	`5159`	`struct XENA_dev_config __iomem *bar0 = sp->bar0;`
`5160`	`5160`
`5161`	`5161`	`/* read mac addr */`