leds: convert PCA9633 LED driver to devm_kzalloc()

Cc: Peter Meerwald <p.meerwald@bct-electronic.com> Signed-off-by: Bryan Wu <bryan.wu@canonical.com>

leds: convert PCA9633 LED driver to devm_kzalloc()
Cc: Peter Meerwald <p.meerwald@bct-electronic.com> Signed-off-by: Bryan Wu <bryan.wu@canonical.com>
Bryan Wu
1 parent ac63ff50ad
Showing 1 changed file with 1 additions and 5 deletions Side-by-side Diff
drivers/leds/leds-pca9633.c
@@ -108,7 +108,7 @@
 		}
 	}
  
-	pca9633 = kcalloc(4, sizeof(*pca9633), GFP_KERNEL);
+	pca9633 = devm_kzalloc(&client->dev, 4 * sizeof(*pca9633), GFP_KERNEL);
 	if (!pca9633)
 		return -ENOMEM;
  
@@ -156,8 +156,6 @@
 		cancel_work_sync(&pca9633[i].work);
 	}
  
-	kfree(pca9633);
-
 	return err;
 }
  
@@ -170,8 +168,6 @@
 		led_classdev_unregister(&pca9633[i].led_cdev);
 		cancel_work_sync(&pca9633[i].work);
 	}
-
-	kfree(pca9633);
  
 	return 0;
 }
...	...	@@ -108,7 +108,7 @@
108	108	}
109	109	}
110	110
111		- pca9633 = kcalloc(4, sizeof(*pca9633), GFP_KERNEL);
	111	+ pca9633 = devm_kzalloc(&client->dev, 4 * sizeof(*pca9633), GFP_KERNEL);
112	112	if (!pca9633)
113	113	return -ENOMEM;
114	114
...	...	@@ -156,8 +156,6 @@
156	156	cancel_work_sync(&pca9633[i].work);
157	157	}
158	158
159		- kfree(pca9633);
160		-
161	159	return err;
162	160	}
163	161
...	...	@@ -170,8 +168,6 @@
170	168	led_classdev_unregister(&pca9633[i].led_cdev);
171	169	cancel_work_sync(&pca9633[i].work);
172	170	}
173		-
174		- kfree(pca9633);
175	171
176	172	return 0;
177	173	}