rfkill: fix error return code in rfkill_gpio_probe()
authorWei Yongjun <yongjun_wei@trendmicro.com.cn>
Thu, 18 Apr 2013 02:31:16 +0000 (10:31 +0800)
committerJohannes Berg <johannes.berg@intel.com>
Mon, 22 Apr 2013 13:42:18 +0000 (15:42 +0200)
Fix to return a negative error code from the error handling
case instead of 0, as returned elsewhere in this function.

Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn>
[fix some indentation on the way]
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
net/rfkill/rfkill-gpio.c

index 78fc093..fb076cd 100644 (file)
@@ -131,6 +131,7 @@ static int rfkill_gpio_probe(struct platform_device *pdev)
                rfkill->pwr_clk = clk_get(&pdev->dev, pdata->power_clk_name);
                if (IS_ERR(rfkill->pwr_clk)) {
                        pr_warn("%s: can't find pwr_clk.\n", __func__);
+                       ret = PTR_ERR(rfkill->pwr_clk);
                        goto fail_shutdown_name;
                }
        }
@@ -152,9 +153,11 @@ static int rfkill_gpio_probe(struct platform_device *pdev)
        }
 
        rfkill->rfkill_dev = rfkill_alloc(pdata->name, &pdev->dev, pdata->type,
-                               &rfkill_gpio_ops, rfkill);
-       if (!rfkill->rfkill_dev)
+                                         &rfkill_gpio_ops, rfkill);
+       if (!rfkill->rfkill_dev) {
+               ret = -ENOMEM;
                goto fail_shutdown;
+       }
 
        ret = rfkill_register(rfkill->rfkill_dev);
        if (ret < 0)