diff options
| author | Yang Li <yang.lee@linux.alibaba.com> | 2021-04-29 13:47:10 +0300 |
|---|---|---|
| committer | Kalle Valo <kvalo@codeaurora.org> | 2021-06-15 13:10:16 +0300 |
| commit | 7557dfde1bd1251793fade20a52014f1105c1012 (patch) | |
| tree | 905b41000e289cb0f7ee4a7edfb150bd6cd9110f | |
| parent | f30282129a4d47bf44805c712be1ac4f89eb8783 (diff) | |
| download | linux-7557dfde1bd1251793fade20a52014f1105c1012.tar.xz | |
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
| -rw-r--r-- | drivers/ssb/main.c | 2 |
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) { |