summary refs log tree commit diff
path: root/drivers/ssb/main.c
diff options
context:
space:
mode:
authorYang Li <yang.lee@linux.alibaba.com>2021-04-29 18:47:10 +0800
committerKalle Valo <kvalo@codeaurora.org>2021-06-15 13:10:16 +0300
commit7557dfde1bd1251793fade20a52014f1105c1012 (patch)
tree905b41000e289cb0f7ee4a7edfb150bd6cd9110f /drivers/ssb/main.c
parentf30282129a4d47bf44805c712be1ac4f89eb8783 (diff)
downloadlinux-7557dfde1bd1251793fade20a52014f1105c1012.tar.gz
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 <abaci@linux.alibaba.com>
Signed-off-by: Yang Li <yang.lee@linux.alibaba.com>
Acked-by: Michael Büsch <m@bues.ch>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
Link: https://lore.kernel.org/r/1619693230-108804-1-git-send-email-yang.lee@linux.alibaba.com
Diffstat (limited to 'drivers/ssb/main.c')
-rw-r--r--drivers/ssb/main.c2
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/ssb/main.c b/drivers/ssb/main.c
index 0e180c36daa4..620cc9dccad7 100644
--- a/drivers/ssb/main.c
+++ b/drivers/ssb/main.c
@@ -1312,13 +1312,11 @@ static int __init ssb_modinit(void)
 	if (err) {
 		pr_err("Broadcom 43xx PCI-SSB-bridge initialization failed\n");
 		/* don't fail SSB init because of this */
-		err = 0;
 	}
 	err = ssb_host_pcmcia_init();
 	if (err) {
 		pr_err("PCMCIA host initialization failed\n");
 		/* don't fail SSB init because of this */
-		err = 0;
 	}
 	err = ssb_gige_init();
 	if (err) {