diff options
Diffstat (limited to 'drivers/rtc/rtc-bfin.c')
| -rw-r--r-- | drivers/rtc/rtc-bfin.c | 34 |
1 files changed, 11 insertions, 23 deletions
diff --git a/drivers/rtc/rtc-bfin.c b/drivers/rtc/rtc-bfin.c index 4ec614b0954d..ad44ec5dc29a 100644 --- a/drivers/rtc/rtc-bfin.c +++ b/drivers/rtc/rtc-bfin.c | |||
| @@ -352,14 +352,14 @@ static int bfin_rtc_probe(struct platform_device *pdev) | |||
| 352 | dev_dbg_stamp(dev); | 352 | dev_dbg_stamp(dev); |
| 353 | 353 | ||
| 354 | /* Allocate memory for our RTC struct */ | 354 | /* Allocate memory for our RTC struct */ |
| 355 | rtc = kzalloc(sizeof(*rtc), GFP_KERNEL); | 355 | rtc = devm_kzalloc(dev, sizeof(*rtc), GFP_KERNEL); |
| 356 | if (unlikely(!rtc)) | 356 | if (unlikely(!rtc)) |
| 357 | return -ENOMEM; | 357 | return -ENOMEM; |
| 358 | platform_set_drvdata(pdev, rtc); | 358 | platform_set_drvdata(pdev, rtc); |
| 359 | device_init_wakeup(dev, 1); | 359 | device_init_wakeup(dev, 1); |
| 360 | 360 | ||
| 361 | /* Register our RTC with the RTC framework */ | 361 | /* Register our RTC with the RTC framework */ |
| 362 | rtc->rtc_dev = rtc_device_register(pdev->name, dev, &bfin_rtc_ops, | 362 | rtc->rtc_dev = devm_rtc_device_register(dev, pdev->name, &bfin_rtc_ops, |
| 363 | THIS_MODULE); | 363 | THIS_MODULE); |
| 364 | if (unlikely(IS_ERR(rtc->rtc_dev))) { | 364 | if (unlikely(IS_ERR(rtc->rtc_dev))) { |
| 365 | ret = PTR_ERR(rtc->rtc_dev); | 365 | ret = PTR_ERR(rtc->rtc_dev); |
| @@ -367,9 +367,10 @@ static int bfin_rtc_probe(struct platform_device *pdev) | |||
| 367 | } | 367 | } |
| 368 | 368 | ||
| 369 | /* Grab the IRQ and init the hardware */ | 369 | /* Grab the IRQ and init the hardware */ |
| 370 | ret = request_irq(IRQ_RTC, bfin_rtc_interrupt, 0, pdev->name, dev); | 370 | ret = devm_request_irq(dev, IRQ_RTC, bfin_rtc_interrupt, 0, |
| 371 | pdev->name, dev); | ||
| 371 | if (unlikely(ret)) | 372 | if (unlikely(ret)) |
| 372 | goto err_reg; | 373 | goto err; |
| 373 | /* sometimes the bootloader touched things, but the write complete was not | 374 | /* sometimes the bootloader touched things, but the write complete was not |
| 374 | * enabled, so let's just do a quick timeout here since the IRQ will not fire ... | 375 | * enabled, so let's just do a quick timeout here since the IRQ will not fire ... |
| 375 | */ | 376 | */ |
| @@ -381,32 +382,23 @@ static int bfin_rtc_probe(struct platform_device *pdev) | |||
| 381 | 382 | ||
| 382 | return 0; | 383 | return 0; |
| 383 | 384 | ||
| 384 | err_reg: | ||
| 385 | rtc_device_unregister(rtc->rtc_dev); | ||
| 386 | err: | 385 | err: |
| 387 | kfree(rtc); | ||
| 388 | return ret; | 386 | return ret; |
| 389 | } | 387 | } |
| 390 | 388 | ||
| 391 | static int bfin_rtc_remove(struct platform_device *pdev) | 389 | static int bfin_rtc_remove(struct platform_device *pdev) |
| 392 | { | 390 | { |
| 393 | struct bfin_rtc *rtc = platform_get_drvdata(pdev); | ||
| 394 | struct device *dev = &pdev->dev; | 391 | struct device *dev = &pdev->dev; |
| 395 | 392 | ||
| 396 | bfin_rtc_reset(dev, 0); | 393 | bfin_rtc_reset(dev, 0); |
| 397 | free_irq(IRQ_RTC, dev); | ||
| 398 | rtc_device_unregister(rtc->rtc_dev); | ||
| 399 | platform_set_drvdata(pdev, NULL); | 394 | platform_set_drvdata(pdev, NULL); |
| 400 | kfree(rtc); | ||
| 401 | 395 | ||
| 402 | return 0; | 396 | return 0; |
| 403 | } | 397 | } |
| 404 | 398 | ||
| 405 | #ifdef CONFIG_PM | 399 | #ifdef CONFIG_PM_SLEEP |
| 406 | static int bfin_rtc_suspend(struct platform_device *pdev, pm_message_t state) | 400 | static int bfin_rtc_suspend(struct device *dev) |
| 407 | { | 401 | { |
| 408 | struct device *dev = &pdev->dev; | ||
| 409 | |||
| 410 | dev_dbg_stamp(dev); | 402 | dev_dbg_stamp(dev); |
| 411 | 403 | ||
| 412 | if (device_may_wakeup(dev)) { | 404 | if (device_may_wakeup(dev)) { |
| @@ -418,10 +410,8 @@ static int bfin_rtc_suspend(struct platform_device *pdev, pm_message_t state) | |||
| 418 | return 0; | 410 | return 0; |
| 419 | } | 411 | } |
| 420 | 412 | ||
| 421 | static int bfin_rtc_resume(struct platform_device *pdev) | 413 | static int bfin_rtc_resume(struct device *dev) |
| 422 | { | 414 | { |
| 423 | struct device *dev = &pdev->dev; | ||
| 424 | |||
| 425 | dev_dbg_stamp(dev); | 415 | dev_dbg_stamp(dev); |
| 426 | 416 | ||
| 427 | if (device_may_wakeup(dev)) | 417 | if (device_may_wakeup(dev)) |
| @@ -440,20 +430,18 @@ static int bfin_rtc_resume(struct platform_device *pdev) | |||
| 440 | 430 | ||
| 441 | return 0; | 431 | return 0; |
| 442 | } | 432 | } |
| 443 | #else | ||
| 444 | # define bfin_rtc_suspend NULL | ||
| 445 | # define bfin_rtc_resume NULL | ||
| 446 | #endif | 433 | #endif |
| 447 | 434 | ||
| 435 | static SIMPLE_DEV_PM_OPS(bfin_rtc_pm_ops, bfin_rtc_suspend, bfin_rtc_resume); | ||
| 436 | |||
| 448 | static struct platform_driver bfin_rtc_driver = { | 437 | static struct platform_driver bfin_rtc_driver = { |
| 449 | .driver = { | 438 | .driver = { |
| 450 | .name = "rtc-bfin", | 439 | .name = "rtc-bfin", |
| 451 | .owner = THIS_MODULE, | 440 | .owner = THIS_MODULE, |
| 441 | .pm = &bfin_rtc_pm_ops, | ||
| 452 | }, | 442 | }, |
| 453 | .probe = bfin_rtc_probe, | 443 | .probe = bfin_rtc_probe, |
| 454 | .remove = bfin_rtc_remove, | 444 | .remove = bfin_rtc_remove, |
| 455 | .suspend = bfin_rtc_suspend, | ||
| 456 | .resume = bfin_rtc_resume, | ||
| 457 | }; | 445 | }; |
| 458 | 446 | ||
| 459 | module_platform_driver(bfin_rtc_driver); | 447 | module_platform_driver(bfin_rtc_driver); |
