Skip to content

Commit 7557dfd

Browse files
Yang LiKalle Valo
authored andcommitted
ssb: Remove redundant assignment to err
Variable 'err' is set to zero but this value is never read as it is overwritten with a new value later on, hence it is a redundant assignment and can be removed. Clean up the following clang-analyzer warning: drivers/ssb/main.c:1306:3: warning: Value stored to 'err' is never read [clang-analyzer-deadcode.DeadStores] drivers/ssb/main.c:1312:3: warning: Value stored to 'err' is never read [clang-analyzer-deadcode.DeadStores] Reported-by: Abaci Robot <[email protected]> Signed-off-by: Yang Li <[email protected]> Acked-by: Michael Büsch <[email protected]> Signed-off-by: Kalle Valo <[email protected]> Link: https://lore.kernel.org/r/[email protected]
1 parent f302821 commit 7557dfd

File tree

1 file changed

+0
-2
lines changed

1 file changed

+0
-2
lines changed

drivers/ssb/main.c

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1312,13 +1312,11 @@ static int __init ssb_modinit(void)
13121312
if (err) {
13131313
pr_err("Broadcom 43xx PCI-SSB-bridge initialization failed\n");
13141314
/* don't fail SSB init because of this */
1315-
err = 0;
13161315
}
13171316
err = ssb_host_pcmcia_init();
13181317
if (err) {
13191318
pr_err("PCMCIA host initialization failed\n");
13201319
/* don't fail SSB init because of this */
1321-
err = 0;
13221320
}
13231321
err = ssb_gige_init();
13241322
if (err) {

0 commit comments

Comments
 (0)