Commit 7f13bbf7 authored by Sachin Kamat's avatar Sachin Kamat Committed by Bryan Wu

leds: Use devm_kzalloc in leds-max8997.c file

devm_kzalloc() makes code cleanup simpler.
Signed-off-by: default avatarSachin Kamat <sachin.kamat@linaro.org>
Signed-off-by: default avatarBryan Wu <bryan.wu@canonical.com>
parent d4158965
...@@ -276,11 +276,9 @@ static int __devinit max8997_led_probe(struct platform_device *pdev) ...@@ -276,11 +276,9 @@ static int __devinit max8997_led_probe(struct platform_device *pdev)
return -ENODEV; return -ENODEV;
} }
led = kzalloc(sizeof(*led), GFP_KERNEL); led = devm_kzalloc(&pdev->dev, sizeof(*led), GFP_KERNEL);
if (led == NULL) { if (led == NULL)
ret = -ENOMEM; return -ENOMEM;
goto err_mem;
}
led->id = pdev->id; led->id = pdev->id;
snprintf(name, sizeof(name), "max8997-led%d", pdev->id); snprintf(name, sizeof(name), "max8997-led%d", pdev->id);
...@@ -315,23 +313,17 @@ static int __devinit max8997_led_probe(struct platform_device *pdev) ...@@ -315,23 +313,17 @@ static int __devinit max8997_led_probe(struct platform_device *pdev)
ret = led_classdev_register(&pdev->dev, &led->cdev); ret = led_classdev_register(&pdev->dev, &led->cdev);
if (ret < 0) if (ret < 0)
goto err_led; return ret;
ret = device_create_file(led->cdev.dev, &dev_attr_mode); ret = device_create_file(led->cdev.dev, &dev_attr_mode);
if (ret != 0) { if (ret != 0) {
dev_err(&pdev->dev, dev_err(&pdev->dev,
"failed to create file: %d\n", ret); "failed to create file: %d\n", ret);
goto err_file; led_classdev_unregister(&led->cdev);
return ret;
} }
return 0; return 0;
err_file:
led_classdev_unregister(&led->cdev);
err_led:
kfree(led);
err_mem:
return ret;
} }
static int __devexit max8997_led_remove(struct platform_device *pdev) static int __devexit max8997_led_remove(struct platform_device *pdev)
...@@ -340,7 +332,6 @@ static int __devexit max8997_led_remove(struct platform_device *pdev) ...@@ -340,7 +332,6 @@ static int __devexit max8997_led_remove(struct platform_device *pdev)
device_remove_file(led->cdev.dev, &dev_attr_mode); device_remove_file(led->cdev.dev, &dev_attr_mode);
led_classdev_unregister(&led->cdev); led_classdev_unregister(&led->cdev);
kfree(led);
return 0; return 0;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment