diff options
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/leds/leds-renesas-tpu.c | 23 |
1 files changed, 9 insertions, 14 deletions
diff --git a/drivers/leds/leds-renesas-tpu.c b/drivers/leds/leds-renesas-tpu.c index 32fe337d5c68..9ee12c28059a 100644 --- a/drivers/leds/leds-renesas-tpu.c +++ b/drivers/leds/leds-renesas-tpu.c | |||
@@ -243,31 +243,30 @@ static int __devinit r_tpu_probe(struct platform_device *pdev) | |||
243 | struct led_renesas_tpu_config *cfg = pdev->dev.platform_data; | 243 | struct led_renesas_tpu_config *cfg = pdev->dev.platform_data; |
244 | struct r_tpu_priv *p; | 244 | struct r_tpu_priv *p; |
245 | struct resource *res; | 245 | struct resource *res; |
246 | int ret = -ENXIO; | 246 | int ret; |
247 | 247 | ||
248 | if (!cfg) { | 248 | if (!cfg) { |
249 | dev_err(&pdev->dev, "missing platform data\n"); | 249 | dev_err(&pdev->dev, "missing platform data\n"); |
250 | goto err0; | 250 | goto err0; |
251 | } | 251 | } |
252 | 252 | ||
253 | p = kzalloc(sizeof(*p), GFP_KERNEL); | 253 | p = devm_kzalloc(&pdev->dev, sizeof(*p), GFP_KERNEL); |
254 | if (p == NULL) { | 254 | if (p == NULL) { |
255 | dev_err(&pdev->dev, "failed to allocate driver data\n"); | 255 | dev_err(&pdev->dev, "failed to allocate driver data\n"); |
256 | ret = -ENOMEM; | 256 | return -ENOMEM; |
257 | goto err0; | ||
258 | } | 257 | } |
259 | 258 | ||
260 | res = platform_get_resource(pdev, IORESOURCE_MEM, 0); | 259 | res = platform_get_resource(pdev, IORESOURCE_MEM, 0); |
261 | if (!res) { | 260 | if (!res) { |
262 | dev_err(&pdev->dev, "failed to get I/O memory\n"); | 261 | dev_err(&pdev->dev, "failed to get I/O memory\n"); |
263 | goto err1; | 262 | return -ENXIO; |
264 | } | 263 | } |
265 | 264 | ||
266 | /* map memory, let mapbase point to our channel */ | 265 | /* map memory, let mapbase point to our channel */ |
267 | p->mapbase = ioremap_nocache(res->start, resource_size(res)); | 266 | p->mapbase = ioremap_nocache(res->start, resource_size(res)); |
268 | if (p->mapbase == NULL) { | 267 | if (p->mapbase == NULL) { |
269 | dev_err(&pdev->dev, "failed to remap I/O memory\n"); | 268 | dev_err(&pdev->dev, "failed to remap I/O memory\n"); |
270 | goto err1; | 269 | return -ENXIO; |
271 | } | 270 | } |
272 | 271 | ||
273 | /* get hold of clock */ | 272 | /* get hold of clock */ |
@@ -275,7 +274,7 @@ static int __devinit r_tpu_probe(struct platform_device *pdev) | |||
275 | if (IS_ERR(p->clk)) { | 274 | if (IS_ERR(p->clk)) { |
276 | dev_err(&pdev->dev, "cannot get clock\n"); | 275 | dev_err(&pdev->dev, "cannot get clock\n"); |
277 | ret = PTR_ERR(p->clk); | 276 | ret = PTR_ERR(p->clk); |
278 | goto err2; | 277 | goto err0; |
279 | } | 278 | } |
280 | 279 | ||
281 | p->pdev = pdev; | 280 | p->pdev = pdev; |
@@ -294,7 +293,7 @@ static int __devinit r_tpu_probe(struct platform_device *pdev) | |||
294 | p->ldev.flags |= LED_CORE_SUSPENDRESUME; | 293 | p->ldev.flags |= LED_CORE_SUSPENDRESUME; |
295 | ret = led_classdev_register(&pdev->dev, &p->ldev); | 294 | ret = led_classdev_register(&pdev->dev, &p->ldev); |
296 | if (ret < 0) | 295 | if (ret < 0) |
297 | goto err3; | 296 | goto err1; |
298 | 297 | ||
299 | /* max_brightness may be updated by the LED core code */ | 298 | /* max_brightness may be updated by the LED core code */ |
300 | p->min_rate = p->ldev.max_brightness * p->refresh_rate; | 299 | p->min_rate = p->ldev.max_brightness * p->refresh_rate; |
@@ -302,14 +301,11 @@ static int __devinit r_tpu_probe(struct platform_device *pdev) | |||
302 | pm_runtime_enable(&pdev->dev); | 301 | pm_runtime_enable(&pdev->dev); |
303 | return 0; | 302 | return 0; |
304 | 303 | ||
305 | err3: | 304 | err1: |
306 | r_tpu_set_pin(p, R_TPU_PIN_UNUSED, LED_OFF); | 305 | r_tpu_set_pin(p, R_TPU_PIN_UNUSED, LED_OFF); |
307 | clk_put(p->clk); | 306 | clk_put(p->clk); |
308 | err2: | ||
309 | iounmap(p->mapbase); | ||
310 | err1: | ||
311 | kfree(p); | ||
312 | err0: | 307 | err0: |
308 | iounmap(p->mapbase); | ||
313 | return ret; | 309 | return ret; |
314 | } | 310 | } |
315 | 311 | ||
@@ -327,7 +323,6 @@ static int __devexit r_tpu_remove(struct platform_device *pdev) | |||
327 | clk_put(p->clk); | 323 | clk_put(p->clk); |
328 | 324 | ||
329 | iounmap(p->mapbase); | 325 | iounmap(p->mapbase); |
330 | kfree(p); | ||
331 | return 0; | 326 | return 0; |
332 | } | 327 | } |
333 | 328 | ||