ata: use platform_{get,set}_drvdata()

Use the wrapper functions for getting and setting the driver data using
platform_device instead of using dev_{get,set}_drvdata() with &pdev->dev,
so we can directly pass a struct platform_device.

Also, unnecessary dev_set_drvdata() is removed, because the driver core
clears the driver data to NULL after device_release or on probe failure.

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
Signed-off-by: Tejun Heo <tj@kernel.org>

1 files changed, 4 insertions, 6 deletions

diff --git a/drivers/ata/pata_bf54x.c b/drivers/ata/pata_bf54x.c
index 8d43510c6bec..ba0d8a29dc23 100644
--- a/drivers/ata/pata_bf54x.c
+++ b/drivers/ata/pata_bf54x.c
@@ -1596,7 +1596,7 @@ static int bfin_atapi_probe(struct platform_device *pdev)
                 return -ENODEV;
         }
 
-        dev_set_drvdata(&pdev->dev, host);
+        platform_set_drvdata(pdev, host);
 
         return 0;
 }
@@ -1610,11 +1610,9 @@ static int bfin_atapi_probe(struct platform_device *pdev)
  */
 static int bfin_atapi_remove(struct platform_device *pdev)
 {
-        struct device *dev = &pdev->dev;
-        struct ata_host *host = dev_get_drvdata(dev);
+        struct ata_host *host = platform_get_drvdata(pdev);
 
         ata_host_detach(host);
-        dev_set_drvdata(&pdev->dev, NULL);
 
         peripheral_free_list(atapi_io_port);
 
@@ -1624,7 +1622,7 @@ static int bfin_atapi_remove(struct platform_device *pdev)
 #ifdef CONFIG_PM
 static int bfin_atapi_suspend(struct platform_device *pdev, pm_message_t state)
 {
-        struct ata_host *host = dev_get_drvdata(&pdev->dev);
+        struct ata_host *host = platform_get_drvdata(pdev);
         if (host)
                 return ata_host_suspend(host, state);
         else
@@ -1633,7 +1631,7 @@ static int bfin_atapi_suspend(struct platform_device *pdev, pm_message_t state)
 
 static int bfin_atapi_resume(struct platform_device *pdev)
 {
-        struct ata_host *host = dev_get_drvdata(&pdev->dev);
+        struct ata_host *host = platform_get_drvdata(pdev);
         int ret;
 
         if (host) {