Skip to contentSkip to conversation

drivers/hwmon - switch to using

PATCH In review Dmitry Torokhov 3 participants 4 messages May 8, 2007 – May 10, 2007

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 drivers/hwmon - switch to using

In preparation for struct class_device -> struct device input core conversion, switch to using input_dev->dev.parent when specifying device position in sysfs tree.

---

 drivers/hwmon/ams/ams-input.c |    2 +-
 drivers/hwmon/hdaps.c         |    2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

Index: work/drivers/hwmon/ams/ams-input.c =================================--- work.orig/drivers/hwmon/ams/ams-input.c +++ work/drivers/hwmon/ams/ams-input.c
@@ -87,7 +87,7 @@ static void ams_input_enable(void)
ams_info.idev->id.vendor = 0;
ams_info.idev->open = ams_input_open; ams_info.idev->close = ams_input_close;
- ams_info.idev->cdev.dev = &ams_info.of_dev->dev;
+ ams_info.idev->dev.parent = &ams_info.of_dev->dev;

input_set_abs_params(ams_info.idev, ABS_X, -50, 50, 3, 0); input_set_abs_params(ams_info.idev, ABS_Y, -50, 50, 3, 0);
Index: work/drivers/hwmon/hdaps.c =================================--- work.orig/drivers/hwmon/hdaps.c +++ work/drivers/hwmon/hdaps.c
@@ -572,7 +572,7 @@ static int __init hdaps_init(void)

/* initialize the input class */
hdaps_idev->name = "hdaps";
- hdaps_idev->cdev.dev = &pdev->dev;
+ hdaps_idev->dev.parent = &pdev->dev;
hdaps_idev->evbit[0] = BIT(EV_ABS); input_set_abs_params(hdaps_idev, ABS_X,
-256, 256, HDAPS_INPUT_FUZZ, HDAPS_INPUT_FLAT);

_______________________________________________
lm-sensors mailing list
[email protected]
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors

#1 lore raw

PATCH 1/1 drivers/hwmon - switch to using

In preparation for struct class_device -> struct device input core conversion, switch to using input_dev->dev.parent when specifying device position in sysfs tree.

---

[Added applesmc to the mix]

 drivers/hwmon/ams/ams-input.c |    2 +-
 drivers/hwmon/applesmc.c      |    2 +-
 drivers/hwmon/hdaps.c         |    2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

Index: work/drivers/hwmon/ams/ams-input.c =================================--- work.orig/drivers/hwmon/ams/ams-input.c +++ work/drivers/hwmon/ams/ams-input.c
@@ -87,7 +87,7 @@ static void ams_input_enable(void)
ams_info.idev->id.vendor = 0;
ams_info.idev->open = ams_input_open; ams_info.idev->close = ams_input_close;
- ams_info.idev->cdev.dev = &ams_info.of_dev->dev;
+ ams_info.idev->dev.parent = &ams_info.of_dev->dev;

input_set_abs_params(ams_info.idev, ABS_X, -50, 50, 3, 0); input_set_abs_params(ams_info.idev, ABS_Y, -50, 50, 3, 0);
Index: work/drivers/hwmon/hdaps.c =================================--- work.orig/drivers/hwmon/hdaps.c +++ work/drivers/hwmon/hdaps.c
@@ -574,7 +574,7 @@ static int __init hdaps_init(void)

/* initialize the input class */
hdaps_idev->name = "hdaps";
- hdaps_idev->cdev.dev = &pdev->dev;
+ hdaps_idev->dev.parent = &pdev->dev;
hdaps_idev->evbit[0] = BIT(EV_ABS); input_set_abs_params(hdaps_idev, ABS_X,
-256, 256, HDAPS_INPUT_FUZZ, HDAPS_INPUT_FLAT);
Index: work/drivers/hwmon/applesmc.c =================================--- work.orig/drivers/hwmon/applesmc.c +++ work/drivers/hwmon/applesmc.c
@@ -1100,7 +1100,7 @@ static int applesmc_create_accelerometer
/* initialize the input class */
applesmc_idev->name = "applesmc"; applesmc_idev->id.bustype = BUS_HOST;
- applesmc_idev->cdev.dev = &pdev->dev;
+ applesmc_idev->dev.parent = &pdev->dev;
applesmc_idev->evbit[0] = BIT(EV_ABS); applesmc_idev->open = applesmc_idev_open; applesmc_idev->close = applesmc_idev_close;

_______________________________________________
lm-sensors mailing list
[email protected]
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors

Acked-by on v1 1/1
#2 lore raw
In reply to Dmitry Torokhov

Hi Dmitry,

Quoted 5 lines · Dmitry Torokhov

In preparation for struct class_device -> struct device input core conversion, switch to using input_dev->dev.parent when specifying device position in sysfs tree.

Quoted 37 lines

---

[Added applesmc to the mix]

 drivers/hwmon/ams/ams-input.c |    2 +-
 drivers/hwmon/applesmc.c      |    2 +-
 drivers/hwmon/hdaps.c         |    2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

Index: work/drivers/hwmon/ams/ams-input.c =================================> --- work.orig/drivers/hwmon/ams/ams-input.c +++ work/drivers/hwmon/ams/ams-input.c

(quoted diff) +1 -1
@@ -87,7 +87,7 @@ static void ams_input_enable(void)
 	ams_info.idev->id.vendor = 0;
 	ams_info.idev->open = ams_input_open;
 	ams_info.idev->close = ams_input_close;
-	ams_info.idev->cdev.dev = &ams_info.of_dev->dev;
+	ams_info.idev->dev.parent = &ams_info.of_dev->dev;
 
 	input_set_abs_params(ams_info.idev, ABS_X, -50, 50, 3, 0);
 	input_set_abs_params(ams_info.idev, ABS_Y, -50, 50, 3, 0);

Index: work/drivers/hwmon/hdaps.c =================================> --- work.orig/drivers/hwmon/hdaps.c +++ work/drivers/hwmon/hdaps.c

(quoted diff) +1 -1
@@ -574,7 +574,7 @@ static int __init hdaps_init(void)
 
 	/* initialize the input class */
 	hdaps_idev->name = "hdaps";
-	hdaps_idev->cdev.dev = &pdev->dev;
+	hdaps_idev->dev.parent = &pdev->dev;
 	hdaps_idev->evbit[0] = BIT(EV_ABS);
 	input_set_abs_params(hdaps_idev, ABS_X,
 			-256, 256, HDAPS_INPUT_FUZZ, HDAPS_INPUT_FLAT);

Index: work/drivers/hwmon/applesmc.c =================================> --- work.orig/drivers/hwmon/applesmc.c +++ work/drivers/hwmon/applesmc.c

Review static int applesmc_create_accelerometer · line 1106

Quoted 8 lines
(quoted diff) +1 -1
@@ -1100,7 +1100,7 @@ static int applesmc_create_accelerometer
 	/* initialize the input class */
 	applesmc_idev->name = "applesmc";
 	applesmc_idev->id.bustype = BUS_HOST;
-	applesmc_idev->cdev.dev = &pdev->dev;
+	applesmc_idev->dev.parent = &pdev->dev;
 	applesmc_idev->evbit[0] = BIT(EV_ABS);
 	applesmc_idev->open = applesmc_idev_open;
 	applesmc_idev->close = applesmc_idev_close;

Fine with me, however don't you think this patch would be better carried through the input tree? If it goes through the hwmon tree, it won't go upstream before 2.6.23-rc1, and you might find it difficult to synchronize the other related input patches.

Thanks,

Signature · 6 lines
Jean Delvare

_______________________________________________
lm-sensors mailing list
[email protected]
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors
#3 lore raw
In reply to Dmitry Torokhov
Quoted 10 lines · On Thu, 10 May 2007 08:13:10 +0200 Jean Delvare <[email protected]> wrote:

Hi Dmitry,

Quoted 5 lines · Dmitry Torokhov

In preparation for struct class_device -> struct device input core conversion, switch to using input_dev->dev.parent when specifying device position in sysfs tree.

Thanks.

Quoted 5 lines

...
Fine with me, however don't you think this patch would be better carried through the input tree? If it goes through the hwmon tree, it won't go upstream before 2.6.23-rc1, and you might find it difficult to synchronize the other related input patches.

I'll merge it in the next batch, tomorrow I guess.

_______________________________________________
lm-sensors mailing list
[email protected]
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors