ipmi_si: Remove bogus err_free label
authorAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Fri, 2 Apr 2021 17:43:26 +0000 (20:43 +0300)
committerCorey Minyard <cminyard@mvista.com>
Fri, 2 Apr 2021 17:53:19 +0000 (12:53 -0500)
There is no more 'free' in the error path, so drop the label and
return errors inline.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Message-Id: <20210402174334.13466-3-andriy.shevchenko@linux.intel.com>
Signed-off-by: Corey Minyard <cminyard@mvista.com>
drivers/char/ipmi/ipmi_si_platform.c

index 8f2fefc..e5cd500 100644 (file)
@@ -316,7 +316,6 @@ static int acpi_ipmi_probe(struct platform_device *pdev)
        acpi_status status;
        unsigned long long tmp;
        struct resource *res;
-       int rv = -EINVAL;
 
        if (!si_tryacpi)
                return -ENODEV;
@@ -336,7 +335,7 @@ static int acpi_ipmi_probe(struct platform_device *pdev)
        if (ACPI_FAILURE(status)) {
                dev_err(&pdev->dev,
                        "Could not find ACPI IPMI interface type\n");
-               goto err_free;
+               return -EINVAL;
        }
 
        switch (tmp) {
@@ -350,21 +349,18 @@ static int acpi_ipmi_probe(struct platform_device *pdev)
                io.si_type = SI_BT;
                break;
        case 4: /* SSIF, just ignore */
-               rv = -ENODEV;
-               goto err_free;
+               return -ENODEV;
        default:
                dev_info(&pdev->dev, "unknown IPMI type %lld\n", tmp);
-               goto err_free;
+               return -EINVAL;
        }
 
        io.regsize = DEFAULT_REGSIZE;
        io.regshift = 0;
 
        res = ipmi_get_info_from_resources(pdev, &io);
-       if (!res) {
-               rv = -EINVAL;
-               goto err_free;
-       }
+       if (!res)
+               return -EINVAL;
 
        /* If _GPE exists, use it; otherwise use standard interrupts */
        status = acpi_evaluate_integer(handle, "_GPE", NULL, &tmp);
@@ -390,9 +386,6 @@ static int acpi_ipmi_probe(struct platform_device *pdev)
        request_module("acpi_ipmi");
 
        return ipmi_si_add_smi(&io);
-
-err_free:
-       return rv;
 }
 
 static const struct acpi_device_id acpi_ipmi_match[] = {