| Message ID | 20181021200039.1933-2-wsa+renesas@sang-engineering.com |
|---|---|
| State | Accepted, archived |
| Delegated to: | David Miller |
| Headers | show |
| Series | ptp: simplify getting .driver_data | expand |
On Sun, Oct 21, 2018 at 10:00:39PM +0200, Wolfram Sang wrote: > We should get 'driver_data' from 'struct device' directly. Going via > platform_device is an unneeded step back and forth. Acked-by: Richard Cochran <richardcochran@gmail.com>
From: Wolfram Sang <wsa+renesas@sang-engineering.com> Date: Sun, 21 Oct 2018 22:00:39 +0200 > We should get 'driver_data' from 'struct device' directly. Going via > platform_device is an unneeded step back and forth. > > Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com> Applied.
diff --git a/drivers/ptp/ptp_dte.c b/drivers/ptp/ptp_dte.c index a7dc43368df4..5b6393e3ea27 100644 --- a/drivers/ptp/ptp_dte.c +++ b/drivers/ptp/ptp_dte.c @@ -288,8 +288,7 @@ static int ptp_dte_remove(struct platform_device *pdev) #ifdef CONFIG_PM_SLEEP static int ptp_dte_suspend(struct device *dev) { - struct platform_device *pdev = to_platform_device(dev); - struct ptp_dte *ptp_dte = platform_get_drvdata(pdev); + struct ptp_dte *ptp_dte = dev_get_drvdata(dev); u8 i; for (i = 0; i < DTE_NUM_REGS_TO_RESTORE; i++) { @@ -305,8 +304,7 @@ static int ptp_dte_suspend(struct device *dev) static int ptp_dte_resume(struct device *dev) { - struct platform_device *pdev = to_platform_device(dev); - struct ptp_dte *ptp_dte = platform_get_drvdata(pdev); + struct ptp_dte *ptp_dte = dev_get_drvdata(dev); u8 i; for (i = 0; i < DTE_NUM_REGS_TO_RESTORE; i++) {
We should get 'driver_data' from 'struct device' directly. Going via platform_device is an unneeded step back and forth. Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com> --- Build tested only. buildbot is happy. drivers/ptp/ptp_dte.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-)