]> git.ipfire.org Git - thirdparty/linux.git/commitdiff
char: ipmi: remove redundant variable 'type' and check
authorColin Ian King <colin.i.king@gmail.com>
Tue, 8 Jul 2025 15:18:05 +0000 (16:18 +0100)
committerCorey Minyard <corey@minyard.net>
Tue, 8 Jul 2025 17:15:44 +0000 (12:15 -0500)
The variable 'type' is assigned the value SI_INVALID which is zero
and later checks of 'type' is non-zero (which is always false). The
variable is not referenced anywhere else, so it is redundant and
so is the check, so remove these.

Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
Message-ID: <20250708151805.1893858-1-colin.i.king@gmail.com>
Signed-off-by: Corey Minyard <corey@minyard.net>
drivers/char/ipmi/ipmi_si_intf.c

index 7fe891783a378c053c19fa1c36e5fcb1f47177a0..064cc463313d70ca477c725bf35ce71c2ce5e8b9 100644 (file)
@@ -2107,7 +2107,6 @@ static bool __init ipmi_smi_info_same(struct smi_info *e1, struct smi_info *e2)
 static int __init init_ipmi_si(void)
 {
        struct smi_info *e, *e2;
-       enum ipmi_addr_src type = SI_INVALID;
 
        if (initialized)
                return 0;
@@ -2189,9 +2188,6 @@ static int __init init_ipmi_si(void)
        initialized = true;
        mutex_unlock(&smi_infos_lock);
 
-       if (type)
-               return 0;
-
        mutex_lock(&smi_infos_lock);
        if (unload_when_empty && list_empty(&smi_infos)) {
                mutex_unlock(&smi_infos_lock);