All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 00/15] rtc: use dev_get_platdata()
@ 2013-09-23  9:38 Jingoo Han
  2013-09-23  9:39 ` [PATCH 01/15] rtc: rtc-88pm80x: " Jingoo Han
                   ` (14 more replies)
  0 siblings, 15 replies; 17+ messages in thread
From: Jingoo Han @ 2013-09-23  9:38 UTC (permalink / raw)
  To: 'Andrew Morton'
  Cc: linux-kernel, 'Alessandro Zummo',
	rtc-linux, 'Jingoo Han'

Use the wrapper function for retrieving the platform data instead of
accessing dev->platform_data directly. This is a cosmetic change
to make the code simpler and enhance the readability.

---
 drivers/rtc/rtc-88pm80x.c         |    8 ++++----
 drivers/rtc/rtc-88pm860x.c        |    2 +-
 drivers/rtc/rtc-cmos.c            |    2 +-
 drivers/rtc/rtc-da9055.c          |    2 +-
 drivers/rtc/rtc-ds1305.c          |    2 +-
 drivers/rtc/rtc-ds1307.c          |    2 +-
 drivers/rtc/rtc-ds2404.c          |    2 +-
 drivers/rtc/rtc-ep93xx.c          |    6 +++---
 drivers/rtc/rtc-hid-sensor-time.c |    4 ++--
 drivers/rtc/rtc-m48t59.c          |   20 ++++++++++----------
 drivers/rtc/rtc-m48t86.c          |    8 ++++----
 drivers/rtc/rtc-pcf2123.c         |    2 +-
 drivers/rtc/rtc-rs5c348.c         |    4 ++--
 drivers/rtc/rtc-sh.c              |    5 +++--
 drivers/rtc/rtc-v3020.c           |    2 +-
 15 files changed, 36 insertions(+), 35 deletions(-)


^ permalink raw reply	[flat|nested] 17+ messages in thread

* [PATCH 01/15] rtc: rtc-88pm80x: use dev_get_platdata()
  2013-09-23  9:38 [PATCH 00/15] rtc: use dev_get_platdata() Jingoo Han
@ 2013-09-23  9:39 ` Jingoo Han
  2013-09-23  9:40 ` [PATCH 02/15] rtc: rtc-88pm860x: " Jingoo Han
                   ` (13 subsequent siblings)
  14 siblings, 0 replies; 17+ messages in thread
From: Jingoo Han @ 2013-09-23  9:39 UTC (permalink / raw)
  To: 'Andrew Morton'
  Cc: linux-kernel, 'Alessandro Zummo',
	rtc-linux, 'Jingoo Han'

Use the wrapper function for retrieving the platform data instead of
accessing dev->platform_data directly. This is a cosmetic change
to make the code simpler and enhance the readability.

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
---
 drivers/rtc/rtc-88pm80x.c |    8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/rtc/rtc-88pm80x.c b/drivers/rtc/rtc-88pm80x.c
index 354c937..0916089 100644
--- a/drivers/rtc/rtc-88pm80x.c
+++ b/drivers/rtc/rtc-88pm80x.c
@@ -251,14 +251,15 @@ static SIMPLE_DEV_PM_OPS(pm80x_rtc_pm_ops, pm80x_rtc_suspend, pm80x_rtc_resume);
 static int pm80x_rtc_probe(struct platform_device *pdev)
 {
 	struct pm80x_chip *chip = dev_get_drvdata(pdev->dev.parent);
-	struct pm80x_platform_data *pm80x_pdata;
+	struct pm80x_platform_data *pm80x_pdata =
+				dev_get_platdata(pdev->dev.parent);
 	struct pm80x_rtc_pdata *pdata = NULL;
 	struct pm80x_rtc_info *info;
 	struct rtc_time tm;
 	unsigned long ticks = 0;
 	int ret;
 
-	pdata = pdev->dev.platform_data;
+	pdata = dev_get_platdata(&pdev->dev);
 	if (pdata == NULL)
 		dev_warn(&pdev->dev, "No platform data!\n");
 
@@ -326,8 +327,7 @@ static int pm80x_rtc_probe(struct platform_device *pdev)
 	regmap_update_bits(info->map, PM800_RTC_CONTROL, PM800_RTC1_USE_XO,
 			   PM800_RTC1_USE_XO);
 
-	if (pdev->dev.parent->platform_data) {
-		pm80x_pdata = pdev->dev.parent->platform_data;
+	if (pm80x_pdata) {
 		pdata = pm80x_pdata->rtc;
 		if (pdata)
 			info->rtc_dev->dev.platform_data = &pdata->rtc_wakeup;
-- 
1.7.10.4



^ permalink raw reply related	[flat|nested] 17+ messages in thread

* [PATCH 02/15] rtc: rtc-88pm860x: use dev_get_platdata()
  2013-09-23  9:38 [PATCH 00/15] rtc: use dev_get_platdata() Jingoo Han
  2013-09-23  9:39 ` [PATCH 01/15] rtc: rtc-88pm80x: " Jingoo Han
@ 2013-09-23  9:40 ` Jingoo Han
  2013-10-10  3:41   ` Haojian Zhuang
  2013-09-23  9:41 ` [PATCH 03/15] rtc: rtc-cmos: " Jingoo Han
                   ` (12 subsequent siblings)
  14 siblings, 1 reply; 17+ messages in thread
From: Jingoo Han @ 2013-09-23  9:40 UTC (permalink / raw)
  To: 'Andrew Morton'
  Cc: linux-kernel, 'Alessandro Zummo',
	rtc-linux, 'Jingoo Han', 'Haojian Zhuang'

Use the wrapper function for retrieving the platform data instead of
accessing dev->platform_data directly. This is a cosmetic change
to make the code simpler and enhance the readability.

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
---
 drivers/rtc/rtc-88pm860x.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/rtc/rtc-88pm860x.c b/drivers/rtc/rtc-88pm860x.c
index 4e30c85..81650484 100644
--- a/drivers/rtc/rtc-88pm860x.c
+++ b/drivers/rtc/rtc-88pm860x.c
@@ -316,7 +316,7 @@ static int pm860x_rtc_probe(struct platform_device *pdev)
 	unsigned long ticks = 0;
 	int ret;
 
-	pdata = pdev->dev.platform_data;
+	pdata = dev_get_platdata(&pdev->dev);
 
 	info = devm_kzalloc(&pdev->dev, sizeof(struct pm860x_rtc_info),
 			    GFP_KERNEL);
-- 
1.7.10.4



^ permalink raw reply related	[flat|nested] 17+ messages in thread

* [PATCH 03/15] rtc: rtc-cmos: use dev_get_platdata()
  2013-09-23  9:38 [PATCH 00/15] rtc: use dev_get_platdata() Jingoo Han
  2013-09-23  9:39 ` [PATCH 01/15] rtc: rtc-88pm80x: " Jingoo Han
  2013-09-23  9:40 ` [PATCH 02/15] rtc: rtc-88pm860x: " Jingoo Han
@ 2013-09-23  9:41 ` Jingoo Han
  2013-09-23  9:43 ` [PATCH 04/15] rtc: rtc-da9055: " Jingoo Han
                   ` (11 subsequent siblings)
  14 siblings, 0 replies; 17+ messages in thread
From: Jingoo Han @ 2013-09-23  9:41 UTC (permalink / raw)
  To: 'Andrew Morton'
  Cc: linux-kernel, 'Alessandro Zummo',
	rtc-linux, 'Jingoo Han', 'Derek Basehore'

Use the wrapper function for retrieving the platform data instead of
accessing dev->platform_data directly. This is a cosmetic change
to make the code simpler and enhance the readability.

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
---
 drivers/rtc/rtc-cmos.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/rtc/rtc-cmos.c b/drivers/rtc/rtc-cmos.c
index 79680b1..f148762 100644
--- a/drivers/rtc/rtc-cmos.c
+++ b/drivers/rtc/rtc-cmos.c
@@ -595,7 +595,7 @@ static irqreturn_t cmos_interrupt(int irq, void *p)
 static int INITSECTION
 cmos_do_probe(struct device *dev, struct resource *ports, int rtc_irq)
 {
-	struct cmos_rtc_board_info	*info = dev->platform_data;
+	struct cmos_rtc_board_info	*info = dev_get_platdata(dev);
 	int				retval = 0;
 	unsigned char			rtc_control;
 	unsigned			address_space;
-- 
1.7.10.4



^ permalink raw reply related	[flat|nested] 17+ messages in thread

* [PATCH 04/15] rtc: rtc-da9055: use dev_get_platdata()
  2013-09-23  9:38 [PATCH 00/15] rtc: use dev_get_platdata() Jingoo Han
                   ` (2 preceding siblings ...)
  2013-09-23  9:41 ` [PATCH 03/15] rtc: rtc-cmos: " Jingoo Han
@ 2013-09-23  9:43 ` Jingoo Han
  2013-09-23  9:43 ` [PATCH 05/15] rtc: rtc-ds1305: " Jingoo Han
                   ` (10 subsequent siblings)
  14 siblings, 0 replies; 17+ messages in thread
From: Jingoo Han @ 2013-09-23  9:43 UTC (permalink / raw)
  To: 'Andrew Morton'
  Cc: linux-kernel, 'Alessandro Zummo',
	rtc-linux, 'Jingoo Han', 'David Dajun Chen'

Use the wrapper function for retrieving the platform data instead of
accessing dev->platform_data directly. This is a cosmetic change
to make the code simpler and enhance the readability.

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
---
 drivers/rtc/rtc-da9055.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/rtc/rtc-da9055.c b/drivers/rtc/rtc-da9055.c
index e00642b..48cb2ac3 100644
--- a/drivers/rtc/rtc-da9055.c
+++ b/drivers/rtc/rtc-da9055.c
@@ -278,7 +278,7 @@ static int da9055_rtc_probe(struct platform_device *pdev)
 		return -ENOMEM;
 
 	rtc->da9055 = dev_get_drvdata(pdev->dev.parent);
-	pdata = rtc->da9055->dev->platform_data;
+	pdata = dev_get_platdata(rtc->da9055->dev);
 	platform_set_drvdata(pdev, rtc);
 
 	ret = da9055_rtc_device_init(rtc->da9055, pdata);
-- 
1.7.10.4



^ permalink raw reply related	[flat|nested] 17+ messages in thread

* [PATCH 05/15] rtc: rtc-ds1305: use dev_get_platdata()
  2013-09-23  9:38 [PATCH 00/15] rtc: use dev_get_platdata() Jingoo Han
                   ` (3 preceding siblings ...)
  2013-09-23  9:43 ` [PATCH 04/15] rtc: rtc-da9055: " Jingoo Han
@ 2013-09-23  9:43 ` Jingoo Han
  2013-09-23  9:44 ` [PATCH 06/15] rtc: rtc-ds1307: " Jingoo Han
                   ` (9 subsequent siblings)
  14 siblings, 0 replies; 17+ messages in thread
From: Jingoo Han @ 2013-09-23  9:43 UTC (permalink / raw)
  To: 'Andrew Morton'
  Cc: linux-kernel, 'Alessandro Zummo',
	rtc-linux, 'Jingoo Han'

Use the wrapper function for retrieving the platform data instead of
accessing dev->platform_data directly. This is a cosmetic change
to make the code simpler and enhance the readability.

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
---
 drivers/rtc/rtc-ds1305.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/rtc/rtc-ds1305.c b/drivers/rtc/rtc-ds1305.c
index dd6170a..80f3237 100644
--- a/drivers/rtc/rtc-ds1305.c
+++ b/drivers/rtc/rtc-ds1305.c
@@ -606,7 +606,7 @@ static int ds1305_probe(struct spi_device *spi)
 	struct ds1305			*ds1305;
 	int				status;
 	u8				addr, value;
-	struct ds1305_platform_data	*pdata = spi->dev.platform_data;
+	struct ds1305_platform_data	*pdata = dev_get_platdata(&spi->dev);
 	bool				write_ctrl = false;
 
 	/* Sanity check board setup data.  This may be hooked up
-- 
1.7.10.4



^ permalink raw reply related	[flat|nested] 17+ messages in thread

* [PATCH 06/15] rtc: rtc-ds1307: use dev_get_platdata()
  2013-09-23  9:38 [PATCH 00/15] rtc: use dev_get_platdata() Jingoo Han
                   ` (4 preceding siblings ...)
  2013-09-23  9:43 ` [PATCH 05/15] rtc: rtc-ds1305: " Jingoo Han
@ 2013-09-23  9:44 ` Jingoo Han
  2013-09-23  9:45 ` [PATCH 07/15] rtc: rtc-ds2404: " Jingoo Han
                   ` (8 subsequent siblings)
  14 siblings, 0 replies; 17+ messages in thread
From: Jingoo Han @ 2013-09-23  9:44 UTC (permalink / raw)
  To: 'Andrew Morton'
  Cc: linux-kernel, 'Alessandro Zummo',
	rtc-linux, 'Jingoo Han'

Use the wrapper function for retrieving the platform data instead of
accessing dev->platform_data directly. This is a cosmetic change
to make the code simpler and enhance the readability.

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
---
 drivers/rtc/rtc-ds1307.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/rtc/rtc-ds1307.c b/drivers/rtc/rtc-ds1307.c
index e8ac439..bd94920 100644
--- a/drivers/rtc/rtc-ds1307.c
+++ b/drivers/rtc/rtc-ds1307.c
@@ -672,7 +672,7 @@ static int ds1307_probe(struct i2c_client *client,
 	struct i2c_adapter	*adapter = to_i2c_adapter(client->dev.parent);
 	int			want_irq = false;
 	unsigned char		*buf;
-	struct ds1307_platform_data *pdata = client->dev.platform_data;
+	struct ds1307_platform_data *pdata = dev_get_platdata(&client->dev);
 	static const int	bbsqi_bitpos[] = {
 		[ds_1337] = 0,
 		[ds_1339] = DS1339_BIT_BBSQI,
-- 
1.7.10.4



^ permalink raw reply related	[flat|nested] 17+ messages in thread

* [PATCH 07/15] rtc: rtc-ds2404: use dev_get_platdata()
  2013-09-23  9:38 [PATCH 00/15] rtc: use dev_get_platdata() Jingoo Han
                   ` (5 preceding siblings ...)
  2013-09-23  9:44 ` [PATCH 06/15] rtc: rtc-ds1307: " Jingoo Han
@ 2013-09-23  9:45 ` Jingoo Han
  2013-09-23  9:46 ` [PATCH 08/15] rtc: rtc-ep93xx: " Jingoo Han
                   ` (7 subsequent siblings)
  14 siblings, 0 replies; 17+ messages in thread
From: Jingoo Han @ 2013-09-23  9:45 UTC (permalink / raw)
  To: 'Andrew Morton'
  Cc: linux-kernel, 'Alessandro Zummo',
	rtc-linux, 'Jingoo Han'

Use the wrapper function for retrieving the platform data instead of
accessing dev->platform_data directly. This is a cosmetic change
to make the code simpler and enhance the readability.

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
---
 drivers/rtc/rtc-ds2404.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/rtc/rtc-ds2404.c b/drivers/rtc/rtc-ds2404.c
index 2ca5a23..fc209dc 100644
--- a/drivers/rtc/rtc-ds2404.c
+++ b/drivers/rtc/rtc-ds2404.c
@@ -224,7 +224,7 @@ static const struct rtc_class_ops ds2404_rtc_ops = {
 
 static int rtc_probe(struct platform_device *pdev)
 {
-	struct ds2404_platform_data *pdata = pdev->dev.platform_data;
+	struct ds2404_platform_data *pdata = dev_get_platdata(&pdev->dev);
 	struct ds2404 *chip;
 	int retval = -EBUSY;
 
-- 
1.7.10.4



^ permalink raw reply related	[flat|nested] 17+ messages in thread

* [PATCH 08/15] rtc: rtc-ep93xx: use dev_get_platdata()
  2013-09-23  9:38 [PATCH 00/15] rtc: use dev_get_platdata() Jingoo Han
                   ` (6 preceding siblings ...)
  2013-09-23  9:45 ` [PATCH 07/15] rtc: rtc-ds2404: " Jingoo Han
@ 2013-09-23  9:46 ` Jingoo Han
  2013-09-23  9:47 ` [PATCH 09/15] rtc: hid-sensor-time: " Jingoo Han
                   ` (6 subsequent siblings)
  14 siblings, 0 replies; 17+ messages in thread
From: Jingoo Han @ 2013-09-23  9:46 UTC (permalink / raw)
  To: 'Andrew Morton'
  Cc: linux-kernel, 'Alessandro Zummo',
	rtc-linux, 'Jingoo Han'

Use the wrapper function for retrieving the platform data instead of
accessing dev->platform_data directly. This is a cosmetic change
to make the code simpler and enhance the readability.

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
---
 drivers/rtc/rtc-ep93xx.c |    6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/rtc/rtc-ep93xx.c b/drivers/rtc/rtc-ep93xx.c
index 580e7b5..5e4f5dc 100644
--- a/drivers/rtc/rtc-ep93xx.c
+++ b/drivers/rtc/rtc-ep93xx.c
@@ -42,7 +42,7 @@ struct ep93xx_rtc {
 static int ep93xx_rtc_get_swcomp(struct device *dev, unsigned short *preload,
 				unsigned short *delete)
 {
-	struct ep93xx_rtc *ep93xx_rtc = dev->platform_data;
+	struct ep93xx_rtc *ep93xx_rtc = dev_get_platdata(dev);
 	unsigned long comp;
 
 	comp = __raw_readl(ep93xx_rtc->mmio_base + EP93XX_RTC_SWCOMP);
@@ -60,7 +60,7 @@ static int ep93xx_rtc_get_swcomp(struct device *dev, unsigned short *preload,
 
 static int ep93xx_rtc_read_time(struct device *dev, struct rtc_time *tm)
 {
-	struct ep93xx_rtc *ep93xx_rtc = dev->platform_data;
+	struct ep93xx_rtc *ep93xx_rtc = dev_get_platdata(dev);
 	unsigned long time;
 
 	 time = __raw_readl(ep93xx_rtc->mmio_base + EP93XX_RTC_DATA);
@@ -71,7 +71,7 @@ static int ep93xx_rtc_read_time(struct device *dev, struct rtc_time *tm)
 
 static int ep93xx_rtc_set_mmss(struct device *dev, unsigned long secs)
 {
-	struct ep93xx_rtc *ep93xx_rtc = dev->platform_data;
+	struct ep93xx_rtc *ep93xx_rtc = dev_get_platdata(dev);
 
 	__raw_writel(secs + 1, ep93xx_rtc->mmio_base + EP93XX_RTC_LOAD);
 	return 0;
-- 
1.7.10.4



^ permalink raw reply related	[flat|nested] 17+ messages in thread

* [PATCH 09/15] rtc: hid-sensor-time: use dev_get_platdata()
  2013-09-23  9:38 [PATCH 00/15] rtc: use dev_get_platdata() Jingoo Han
                   ` (7 preceding siblings ...)
  2013-09-23  9:46 ` [PATCH 08/15] rtc: rtc-ep93xx: " Jingoo Han
@ 2013-09-23  9:47 ` Jingoo Han
  2013-09-23  9:48 ` [PATCH 10/15] rtc: rtc-m48t59: " Jingoo Han
                   ` (5 subsequent siblings)
  14 siblings, 0 replies; 17+ messages in thread
From: Jingoo Han @ 2013-09-23  9:47 UTC (permalink / raw)
  To: 'Andrew Morton'
  Cc: linux-kernel, 'Alessandro Zummo',
	rtc-linux, 'Jingoo Han', 'Alexander Holler'

Use the wrapper function for retrieving the platform data instead of
accessing dev->platform_data directly. This is a cosmetic change
to make the code simpler and enhance the readability.

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
---
 drivers/rtc/rtc-hid-sensor-time.c |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/rtc/rtc-hid-sensor-time.c b/drivers/rtc/rtc-hid-sensor-time.c
index abbf5f3..22254f0 100644
--- a/drivers/rtc/rtc-hid-sensor-time.c
+++ b/drivers/rtc/rtc-hid-sensor-time.c
@@ -236,7 +236,7 @@ static const struct rtc_class_ops hid_time_rtc_ops = {
 static int hid_time_probe(struct platform_device *pdev)
 {
 	int ret = 0;
-	struct hid_sensor_hub_device *hsdev = pdev->dev.platform_data;
+	struct hid_sensor_hub_device *hsdev = dev_get_platdata(&pdev->dev);
 	struct hid_time_state *time_state = devm_kzalloc(&pdev->dev,
 		sizeof(struct hid_time_state), GFP_KERNEL);
 
@@ -298,7 +298,7 @@ static int hid_time_probe(struct platform_device *pdev)
 
 static int hid_time_remove(struct platform_device *pdev)
 {
-	struct hid_sensor_hub_device *hsdev = pdev->dev.platform_data;
+	struct hid_sensor_hub_device *hsdev = dev_get_platdata(&pdev->dev);
 
 	sensor_hub_remove_callback(hsdev, HID_USAGE_SENSOR_TIME);
 
-- 
1.7.10.4



^ permalink raw reply related	[flat|nested] 17+ messages in thread

* [PATCH 10/15] rtc: rtc-m48t59: use dev_get_platdata()
  2013-09-23  9:38 [PATCH 00/15] rtc: use dev_get_platdata() Jingoo Han
                   ` (8 preceding siblings ...)
  2013-09-23  9:47 ` [PATCH 09/15] rtc: hid-sensor-time: " Jingoo Han
@ 2013-09-23  9:48 ` Jingoo Han
  2013-09-23  9:48 ` [PATCH 11/15] rtc: rtc-m48t86: " Jingoo Han
                   ` (4 subsequent siblings)
  14 siblings, 0 replies; 17+ messages in thread
From: Jingoo Han @ 2013-09-23  9:48 UTC (permalink / raw)
  To: 'Andrew Morton'
  Cc: linux-kernel, 'Alessandro Zummo',
	rtc-linux, 'Jingoo Han'

Use the wrapper function for retrieving the platform data instead of
accessing dev->platform_data directly. This is a cosmetic change
to make the code simpler and enhance the readability.

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
---
 drivers/rtc/rtc-m48t59.c |   20 ++++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

diff --git a/drivers/rtc/rtc-m48t59.c b/drivers/rtc/rtc-m48t59.c
index fcb0329..11880c1 100644
--- a/drivers/rtc/rtc-m48t59.c
+++ b/drivers/rtc/rtc-m48t59.c
@@ -68,7 +68,7 @@ m48t59_mem_readb(struct device *dev, u32 ofs)
 static int m48t59_rtc_read_time(struct device *dev, struct rtc_time *tm)
 {
 	struct platform_device *pdev = to_platform_device(dev);
-	struct m48t59_plat_data *pdata = pdev->dev.platform_data;
+	struct m48t59_plat_data *pdata = dev_get_platdata(&pdev->dev);
 	struct m48t59_private *m48t59 = platform_get_drvdata(pdev);
 	unsigned long flags;
 	u8 val;
@@ -111,7 +111,7 @@ static int m48t59_rtc_read_time(struct device *dev, struct rtc_time *tm)
 static int m48t59_rtc_set_time(struct device *dev, struct rtc_time *tm)
 {
 	struct platform_device *pdev = to_platform_device(dev);
-	struct m48t59_plat_data *pdata = pdev->dev.platform_data;
+	struct m48t59_plat_data *pdata = dev_get_platdata(&pdev->dev);
 	struct m48t59_private *m48t59 = platform_get_drvdata(pdev);
 	unsigned long flags;
 	u8 val = 0;
@@ -158,7 +158,7 @@ static int m48t59_rtc_set_time(struct device *dev, struct rtc_time *tm)
 static int m48t59_rtc_readalarm(struct device *dev, struct rtc_wkalrm *alrm)
 {
 	struct platform_device *pdev = to_platform_device(dev);
-	struct m48t59_plat_data *pdata = pdev->dev.platform_data;
+	struct m48t59_plat_data *pdata = dev_get_platdata(&pdev->dev);
 	struct m48t59_private *m48t59 = platform_get_drvdata(pdev);
 	struct rtc_time *tm = &alrm->time;
 	unsigned long flags;
@@ -205,7 +205,7 @@ static int m48t59_rtc_readalarm(struct device *dev, struct rtc_wkalrm *alrm)
 static int m48t59_rtc_setalarm(struct device *dev, struct rtc_wkalrm *alrm)
 {
 	struct platform_device *pdev = to_platform_device(dev);
-	struct m48t59_plat_data *pdata = pdev->dev.platform_data;
+	struct m48t59_plat_data *pdata = dev_get_platdata(&pdev->dev);
 	struct m48t59_private *m48t59 = platform_get_drvdata(pdev);
 	struct rtc_time *tm = &alrm->time;
 	u8 mday, hour, min, sec;
@@ -266,7 +266,7 @@ static int m48t59_rtc_setalarm(struct device *dev, struct rtc_wkalrm *alrm)
 static int m48t59_rtc_alarm_irq_enable(struct device *dev, unsigned int enabled)
 {
 	struct platform_device *pdev = to_platform_device(dev);
-	struct m48t59_plat_data *pdata = pdev->dev.platform_data;
+	struct m48t59_plat_data *pdata = dev_get_platdata(&pdev->dev);
 	struct m48t59_private *m48t59 = platform_get_drvdata(pdev);
 	unsigned long flags;
 
@@ -283,7 +283,7 @@ static int m48t59_rtc_alarm_irq_enable(struct device *dev, unsigned int enabled)
 static int m48t59_rtc_proc(struct device *dev, struct seq_file *seq)
 {
 	struct platform_device *pdev = to_platform_device(dev);
-	struct m48t59_plat_data *pdata = pdev->dev.platform_data;
+	struct m48t59_plat_data *pdata = dev_get_platdata(&pdev->dev);
 	struct m48t59_private *m48t59 = platform_get_drvdata(pdev);
 	unsigned long flags;
 	u8 val;
@@ -304,7 +304,7 @@ static irqreturn_t m48t59_rtc_interrupt(int irq, void *dev_id)
 {
 	struct device *dev = (struct device *)dev_id;
 	struct platform_device *pdev = to_platform_device(dev);
-	struct m48t59_plat_data *pdata = pdev->dev.platform_data;
+	struct m48t59_plat_data *pdata = dev_get_platdata(&pdev->dev);
 	struct m48t59_private *m48t59 = platform_get_drvdata(pdev);
 	u8 event;
 
@@ -340,7 +340,7 @@ static ssize_t m48t59_nvram_read(struct file *filp, struct kobject *kobj,
 {
 	struct device *dev = container_of(kobj, struct device, kobj);
 	struct platform_device *pdev = to_platform_device(dev);
-	struct m48t59_plat_data *pdata = pdev->dev.platform_data;
+	struct m48t59_plat_data *pdata = dev_get_platdata(&pdev->dev);
 	struct m48t59_private *m48t59 = platform_get_drvdata(pdev);
 	ssize_t cnt = 0;
 	unsigned long flags;
@@ -360,7 +360,7 @@ static ssize_t m48t59_nvram_write(struct file *filp, struct kobject *kobj,
 {
 	struct device *dev = container_of(kobj, struct device, kobj);
 	struct platform_device *pdev = to_platform_device(dev);
-	struct m48t59_plat_data *pdata = pdev->dev.platform_data;
+	struct m48t59_plat_data *pdata = dev_get_platdata(&pdev->dev);
 	struct m48t59_private *m48t59 = platform_get_drvdata(pdev);
 	ssize_t cnt = 0;
 	unsigned long flags;
@@ -385,7 +385,7 @@ static struct bin_attribute m48t59_nvram_attr = {
 
 static int m48t59_rtc_probe(struct platform_device *pdev)
 {
-	struct m48t59_plat_data *pdata = pdev->dev.platform_data;
+	struct m48t59_plat_data *pdata = dev_get_platdata(&pdev->dev);
 	struct m48t59_private *m48t59 = NULL;
 	struct resource *res;
 	int ret = -ENOMEM;
-- 
1.7.10.4



^ permalink raw reply related	[flat|nested] 17+ messages in thread

* [PATCH 11/15] rtc: rtc-m48t86: use dev_get_platdata()
  2013-09-23  9:38 [PATCH 00/15] rtc: use dev_get_platdata() Jingoo Han
                   ` (9 preceding siblings ...)
  2013-09-23  9:48 ` [PATCH 10/15] rtc: rtc-m48t59: " Jingoo Han
@ 2013-09-23  9:48 ` Jingoo Han
  2013-09-23  9:49 ` [PATCH 12/15] rtc: rtc-pcf2123: " Jingoo Han
                   ` (3 subsequent siblings)
  14 siblings, 0 replies; 17+ messages in thread
From: Jingoo Han @ 2013-09-23  9:48 UTC (permalink / raw)
  To: 'Andrew Morton'
  Cc: linux-kernel, 'Alessandro Zummo',
	rtc-linux, 'Jingoo Han'

Use the wrapper function for retrieving the platform data instead of
accessing dev->platform_data directly. This is a cosmetic change
to make the code simpler and enhance the readability.

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
---
 drivers/rtc/rtc-m48t86.c |    8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/rtc/rtc-m48t86.c b/drivers/rtc/rtc-m48t86.c
index 2d30314..32f64c9 100644
--- a/drivers/rtc/rtc-m48t86.c
+++ b/drivers/rtc/rtc-m48t86.c
@@ -46,7 +46,7 @@ static int m48t86_rtc_read_time(struct device *dev, struct rtc_time *tm)
 {
 	unsigned char reg;
 	struct platform_device *pdev = to_platform_device(dev);
-	struct m48t86_ops *ops = pdev->dev.platform_data;
+	struct m48t86_ops *ops = dev_get_platdata(&pdev->dev);
 
 	reg = ops->readbyte(M48T86_REG_B);
 
@@ -84,7 +84,7 @@ static int m48t86_rtc_set_time(struct device *dev, struct rtc_time *tm)
 {
 	unsigned char reg;
 	struct platform_device *pdev = to_platform_device(dev);
-	struct m48t86_ops *ops = pdev->dev.platform_data;
+	struct m48t86_ops *ops = dev_get_platdata(&pdev->dev);
 
 	reg = ops->readbyte(M48T86_REG_B);
 
@@ -123,7 +123,7 @@ static int m48t86_rtc_proc(struct device *dev, struct seq_file *seq)
 {
 	unsigned char reg;
 	struct platform_device *pdev = to_platform_device(dev);
-	struct m48t86_ops *ops = pdev->dev.platform_data;
+	struct m48t86_ops *ops = dev_get_platdata(&pdev->dev);
 
 	reg = ops->readbyte(M48T86_REG_B);
 
@@ -147,7 +147,7 @@ static const struct rtc_class_ops m48t86_rtc_ops = {
 static int m48t86_rtc_probe(struct platform_device *dev)
 {
 	unsigned char reg;
-	struct m48t86_ops *ops = dev->dev.platform_data;
+	struct m48t86_ops *ops = dev_get_platdata(&dev->dev);
 	struct rtc_device *rtc;
 
 	rtc = devm_rtc_device_register(&dev->dev, "m48t86",
-- 
1.7.10.4



^ permalink raw reply related	[flat|nested] 17+ messages in thread

* [PATCH 12/15] rtc: rtc-pcf2123: use dev_get_platdata()
  2013-09-23  9:38 [PATCH 00/15] rtc: use dev_get_platdata() Jingoo Han
                   ` (10 preceding siblings ...)
  2013-09-23  9:48 ` [PATCH 11/15] rtc: rtc-m48t86: " Jingoo Han
@ 2013-09-23  9:49 ` Jingoo Han
  2013-09-23  9:50 ` [PATCH 13/15] rtc: rtc-rs5c348: " Jingoo Han
                   ` (2 subsequent siblings)
  14 siblings, 0 replies; 17+ messages in thread
From: Jingoo Han @ 2013-09-23  9:49 UTC (permalink / raw)
  To: 'Andrew Morton'
  Cc: linux-kernel, 'Alessandro Zummo',
	rtc-linux, 'Jingoo Han'

Use the wrapper function for retrieving the platform data instead of
accessing dev->platform_data directly. This is a cosmetic change
to make the code simpler and enhance the readability.

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
---
 drivers/rtc/rtc-pcf2123.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/rtc/rtc-pcf2123.c b/drivers/rtc/rtc-pcf2123.c
index 1725b50..d1953bb 100644
--- a/drivers/rtc/rtc-pcf2123.c
+++ b/drivers/rtc/rtc-pcf2123.c
@@ -327,7 +327,7 @@ kfree_exit:
 
 static int pcf2123_remove(struct spi_device *spi)
 {
-	struct pcf2123_plat_data *pdata = spi->dev.platform_data;
+	struct pcf2123_plat_data *pdata = dev_get_platdata(&spi->dev);
 	int i;
 
 	if (pdata) {
-- 
1.7.10.4



^ permalink raw reply related	[flat|nested] 17+ messages in thread

* [PATCH 13/15] rtc: rtc-rs5c348: use dev_get_platdata()
  2013-09-23  9:38 [PATCH 00/15] rtc: use dev_get_platdata() Jingoo Han
                   ` (11 preceding siblings ...)
  2013-09-23  9:49 ` [PATCH 12/15] rtc: rtc-pcf2123: " Jingoo Han
@ 2013-09-23  9:50 ` Jingoo Han
  2013-09-23  9:52 ` [PATCH 14/15] rtc: rtc-sh: " Jingoo Han
  2013-09-23  9:53 ` [PATCH 15/15] rtc: rtc-v3020: " Jingoo Han
  14 siblings, 0 replies; 17+ messages in thread
From: Jingoo Han @ 2013-09-23  9:50 UTC (permalink / raw)
  To: 'Andrew Morton'
  Cc: linux-kernel, 'Alessandro Zummo',
	rtc-linux, 'Jingoo Han', 'Atsushi Nemoto'

Use the wrapper function for retrieving the platform data instead of
accessing dev->platform_data directly. This is a cosmetic change
to make the code simpler and enhance the readability.

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
---
 drivers/rtc/rtc-rs5c348.c |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/rtc/rtc-rs5c348.c b/drivers/rtc/rtc-rs5c348.c
index f7a90a1..090a101 100644
--- a/drivers/rtc/rtc-rs5c348.c
+++ b/drivers/rtc/rtc-rs5c348.c
@@ -64,7 +64,7 @@ static int
 rs5c348_rtc_set_time(struct device *dev, struct rtc_time *tm)
 {
 	struct spi_device *spi = to_spi_device(dev);
-	struct rs5c348_plat_data *pdata = spi->dev.platform_data;
+	struct rs5c348_plat_data *pdata = dev_get_platdata(&spi->dev);
 	u8 txbuf[5+7], *txp;
 	int ret;
 
@@ -100,7 +100,7 @@ static int
 rs5c348_rtc_read_time(struct device *dev, struct rtc_time *tm)
 {
 	struct spi_device *spi = to_spi_device(dev);
-	struct rs5c348_plat_data *pdata = spi->dev.platform_data;
+	struct rs5c348_plat_data *pdata = dev_get_platdata(&spi->dev);
 	u8 txbuf[5], rxbuf[7];
 	int ret;
 
-- 
1.7.10.4



^ permalink raw reply related	[flat|nested] 17+ messages in thread

* [PATCH 14/15] rtc: rtc-sh: use dev_get_platdata()
  2013-09-23  9:38 [PATCH 00/15] rtc: use dev_get_platdata() Jingoo Han
                   ` (12 preceding siblings ...)
  2013-09-23  9:50 ` [PATCH 13/15] rtc: rtc-rs5c348: " Jingoo Han
@ 2013-09-23  9:52 ` Jingoo Han
  2013-09-23  9:53 ` [PATCH 15/15] rtc: rtc-v3020: " Jingoo Han
  14 siblings, 0 replies; 17+ messages in thread
From: Jingoo Han @ 2013-09-23  9:52 UTC (permalink / raw)
  To: 'Andrew Morton'
  Cc: linux-kernel, 'Alessandro Zummo',
	rtc-linux, 'Jingoo Han'

Use the wrapper function for retrieving the platform data instead of
accessing dev->platform_data directly. This is a cosmetic change
to make the code simpler and enhance the readability.

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
---
 drivers/rtc/rtc-sh.c |    5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/drivers/rtc/rtc-sh.c b/drivers/rtc/rtc-sh.c
index 6d87e26..d0d2b04 100644
--- a/drivers/rtc/rtc-sh.c
+++ b/drivers/rtc/rtc-sh.c
@@ -649,8 +649,9 @@ static int __init sh_rtc_probe(struct platform_device *pdev)
 	clk_enable(rtc->clk);
 
 	rtc->capabilities = RTC_DEF_CAPABILITIES;
-	if (pdev->dev.platform_data) {
-		struct sh_rtc_platform_info *pinfo = pdev->dev.platform_data;
+	if (dev_get_platdata(&pdev->dev)) {
+		struct sh_rtc_platform_info *pinfo =
+			dev_get_platdata(&pdev->dev);
 
 		/*
 		 * Some CPUs have special capabilities in addition to the
-- 
1.7.10.4



^ permalink raw reply related	[flat|nested] 17+ messages in thread

* [PATCH 15/15] rtc: rtc-v3020: use dev_get_platdata()
  2013-09-23  9:38 [PATCH 00/15] rtc: use dev_get_platdata() Jingoo Han
                   ` (13 preceding siblings ...)
  2013-09-23  9:52 ` [PATCH 14/15] rtc: rtc-sh: " Jingoo Han
@ 2013-09-23  9:53 ` Jingoo Han
  14 siblings, 0 replies; 17+ messages in thread
From: Jingoo Han @ 2013-09-23  9:53 UTC (permalink / raw)
  To: 'Andrew Morton'
  Cc: linux-kernel, 'Alessandro Zummo',
	rtc-linux, 'Jingoo Han', 'Raphael Assenat'

Use the wrapper function for retrieving the platform data instead of
accessing dev->platform_data directly. This is a cosmetic change
to make the code simpler and enhance the readability.

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
---
 drivers/rtc/rtc-v3020.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/rtc/rtc-v3020.c b/drivers/rtc/rtc-v3020.c
index d07d898..25222cd 100644
--- a/drivers/rtc/rtc-v3020.c
+++ b/drivers/rtc/rtc-v3020.c
@@ -303,7 +303,7 @@ static const struct rtc_class_ops v3020_rtc_ops = {
 
 static int rtc_probe(struct platform_device *pdev)
 {
-	struct v3020_platform_data *pdata = pdev->dev.platform_data;
+	struct v3020_platform_data *pdata = dev_get_platdata(&pdev->dev);
 	struct v3020 *chip;
 	int retval = -EBUSY;
 	int i;
-- 
1.7.10.4



^ permalink raw reply related	[flat|nested] 17+ messages in thread

* Re: [PATCH 02/15] rtc: rtc-88pm860x: use dev_get_platdata()
  2013-09-23  9:40 ` [PATCH 02/15] rtc: rtc-88pm860x: " Jingoo Han
@ 2013-10-10  3:41   ` Haojian Zhuang
  0 siblings, 0 replies; 17+ messages in thread
From: Haojian Zhuang @ 2013-10-10  3:41 UTC (permalink / raw)
  To: Jingoo Han; +Cc: Andrew Morton, linux-kernel, Alessandro Zummo, rtc-linux

On 23 September 2013 17:40, Jingoo Han <jg1.han@samsung.com> wrote:
> Use the wrapper function for retrieving the platform data instead of
> accessing dev->platform_data directly. This is a cosmetic change
> to make the code simpler and enhance the readability.
>
> Signed-off-by: Jingoo Han <jg1.han@samsung.com>
> ---
>  drivers/rtc/rtc-88pm860x.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/rtc/rtc-88pm860x.c b/drivers/rtc/rtc-88pm860x.c
> index 4e30c85..81650484 100644
> --- a/drivers/rtc/rtc-88pm860x.c
> +++ b/drivers/rtc/rtc-88pm860x.c
> @@ -316,7 +316,7 @@ static int pm860x_rtc_probe(struct platform_device *pdev)
>         unsigned long ticks = 0;
>         int ret;
>
> -       pdata = pdev->dev.platform_data;
> +       pdata = dev_get_platdata(&pdev->dev);
>
>         info = devm_kzalloc(&pdev->dev, sizeof(struct pm860x_rtc_info),
>                             GFP_KERNEL);
> --
> 1.7.10.4
>
>

Acked-by: Haojian Zhuang <haojian.zhuang@linaro.org>

^ permalink raw reply	[flat|nested] 17+ messages in thread

end of thread, other threads:[~2013-10-10  3:41 UTC | newest]

Thread overview: 17+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-09-23  9:38 [PATCH 00/15] rtc: use dev_get_platdata() Jingoo Han
2013-09-23  9:39 ` [PATCH 01/15] rtc: rtc-88pm80x: " Jingoo Han
2013-09-23  9:40 ` [PATCH 02/15] rtc: rtc-88pm860x: " Jingoo Han
2013-10-10  3:41   ` Haojian Zhuang
2013-09-23  9:41 ` [PATCH 03/15] rtc: rtc-cmos: " Jingoo Han
2013-09-23  9:43 ` [PATCH 04/15] rtc: rtc-da9055: " Jingoo Han
2013-09-23  9:43 ` [PATCH 05/15] rtc: rtc-ds1305: " Jingoo Han
2013-09-23  9:44 ` [PATCH 06/15] rtc: rtc-ds1307: " Jingoo Han
2013-09-23  9:45 ` [PATCH 07/15] rtc: rtc-ds2404: " Jingoo Han
2013-09-23  9:46 ` [PATCH 08/15] rtc: rtc-ep93xx: " Jingoo Han
2013-09-23  9:47 ` [PATCH 09/15] rtc: hid-sensor-time: " Jingoo Han
2013-09-23  9:48 ` [PATCH 10/15] rtc: rtc-m48t59: " Jingoo Han
2013-09-23  9:48 ` [PATCH 11/15] rtc: rtc-m48t86: " Jingoo Han
2013-09-23  9:49 ` [PATCH 12/15] rtc: rtc-pcf2123: " Jingoo Han
2013-09-23  9:50 ` [PATCH 13/15] rtc: rtc-rs5c348: " Jingoo Han
2013-09-23  9:52 ` [PATCH 14/15] rtc: rtc-sh: " Jingoo Han
2013-09-23  9:53 ` [PATCH 15/15] rtc: rtc-v3020: " Jingoo Han

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.