diff options
author | Greg Kroah-Hartman <gregkh@suse.de> | 2009-05-04 15:40:54 -0400 |
---|---|---|
committer | Takashi Iwai <tiwai@suse.de> | 2009-05-12 03:20:54 -0400 |
commit | ae31c1fbdbb18d917b0a1139497c2dbd35886989 (patch) | |
tree | cdc2e881e4195858a72a88627b28461900477757 /sound/aoa | |
parent | 091bf7624d1c90cec9e578a18529f615213ff847 (diff) |
sound: remove driver_data direct access of struct device
In the near future, the driver core is going to not allow direct access
to the driver_data pointer in struct device. Instead, the functions
dev_get_drvdata() and dev_set_drvdata() should be used. These functions
have been around since the beginning, so are backwards compatible with
all older kernel versions.
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Acked-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Diffstat (limited to 'sound/aoa')
-rw-r--r-- | sound/aoa/fabrics/layout.c | 8 | ||||
-rw-r--r-- | sound/aoa/soundbus/i2sbus/core.c | 8 |
2 files changed, 8 insertions, 8 deletions
diff --git a/sound/aoa/fabrics/layout.c b/sound/aoa/fabrics/layout.c index fbf5c933baa4..586965f9605f 100644 --- a/sound/aoa/fabrics/layout.c +++ b/sound/aoa/fabrics/layout.c | |||
@@ -1037,7 +1037,7 @@ static int aoa_fabric_layout_probe(struct soundbus_dev *sdev) | |||
1037 | } | 1037 | } |
1038 | ldev->selfptr_headphone.ptr = ldev; | 1038 | ldev->selfptr_headphone.ptr = ldev; |
1039 | ldev->selfptr_lineout.ptr = ldev; | 1039 | ldev->selfptr_lineout.ptr = ldev; |
1040 | sdev->ofdev.dev.driver_data = ldev; | 1040 | dev_set_drvdata(&sdev->ofdev.dev, ldev); |
1041 | list_add(&ldev->list, &layouts_list); | 1041 | list_add(&ldev->list, &layouts_list); |
1042 | layouts_list_items++; | 1042 | layouts_list_items++; |
1043 | 1043 | ||
@@ -1081,7 +1081,7 @@ static int aoa_fabric_layout_probe(struct soundbus_dev *sdev) | |||
1081 | 1081 | ||
1082 | static int aoa_fabric_layout_remove(struct soundbus_dev *sdev) | 1082 | static int aoa_fabric_layout_remove(struct soundbus_dev *sdev) |
1083 | { | 1083 | { |
1084 | struct layout_dev *ldev = sdev->ofdev.dev.driver_data; | 1084 | struct layout_dev *ldev = dev_get_drvdata(&sdev->ofdev.dev); |
1085 | int i; | 1085 | int i; |
1086 | 1086 | ||
1087 | for (i=0; i<MAX_CODECS_PER_BUS; i++) { | 1087 | for (i=0; i<MAX_CODECS_PER_BUS; i++) { |
@@ -1114,7 +1114,7 @@ static int aoa_fabric_layout_remove(struct soundbus_dev *sdev) | |||
1114 | #ifdef CONFIG_PM | 1114 | #ifdef CONFIG_PM |
1115 | static int aoa_fabric_layout_suspend(struct soundbus_dev *sdev, pm_message_t state) | 1115 | static int aoa_fabric_layout_suspend(struct soundbus_dev *sdev, pm_message_t state) |
1116 | { | 1116 | { |
1117 | struct layout_dev *ldev = sdev->ofdev.dev.driver_data; | 1117 | struct layout_dev *ldev = dev_get_drvdata(&sdev->ofdev.dev); |
1118 | 1118 | ||
1119 | if (ldev->gpio.methods && ldev->gpio.methods->all_amps_off) | 1119 | if (ldev->gpio.methods && ldev->gpio.methods->all_amps_off) |
1120 | ldev->gpio.methods->all_amps_off(&ldev->gpio); | 1120 | ldev->gpio.methods->all_amps_off(&ldev->gpio); |
@@ -1124,7 +1124,7 @@ static int aoa_fabric_layout_suspend(struct soundbus_dev *sdev, pm_message_t sta | |||
1124 | 1124 | ||
1125 | static int aoa_fabric_layout_resume(struct soundbus_dev *sdev) | 1125 | static int aoa_fabric_layout_resume(struct soundbus_dev *sdev) |
1126 | { | 1126 | { |
1127 | struct layout_dev *ldev = sdev->ofdev.dev.driver_data; | 1127 | struct layout_dev *ldev = dev_get_drvdata(&sdev->ofdev.dev); |
1128 | 1128 | ||
1129 | if (ldev->gpio.methods && ldev->gpio.methods->all_amps_off) | 1129 | if (ldev->gpio.methods && ldev->gpio.methods->all_amps_off) |
1130 | ldev->gpio.methods->all_amps_restore(&ldev->gpio); | 1130 | ldev->gpio.methods->all_amps_restore(&ldev->gpio); |
diff --git a/sound/aoa/soundbus/i2sbus/core.c b/sound/aoa/soundbus/i2sbus/core.c index 418c84c99d69..4e3b819d4993 100644 --- a/sound/aoa/soundbus/i2sbus/core.c +++ b/sound/aoa/soundbus/i2sbus/core.c | |||
@@ -358,14 +358,14 @@ static int i2sbus_probe(struct macio_dev* dev, const struct of_device_id *match) | |||
358 | return -ENODEV; | 358 | return -ENODEV; |
359 | } | 359 | } |
360 | 360 | ||
361 | dev->ofdev.dev.driver_data = control; | 361 | dev_set_drvdata(&dev->ofdev.dev, control); |
362 | 362 | ||
363 | return 0; | 363 | return 0; |
364 | } | 364 | } |
365 | 365 | ||
366 | static int i2sbus_remove(struct macio_dev* dev) | 366 | static int i2sbus_remove(struct macio_dev* dev) |
367 | { | 367 | { |
368 | struct i2sbus_control *control = dev->ofdev.dev.driver_data; | 368 | struct i2sbus_control *control = dev_get_drvdata(&dev->ofdev.dev); |
369 | struct i2sbus_dev *i2sdev, *tmp; | 369 | struct i2sbus_dev *i2sdev, *tmp; |
370 | 370 | ||
371 | list_for_each_entry_safe(i2sdev, tmp, &control->list, item) | 371 | list_for_each_entry_safe(i2sdev, tmp, &control->list, item) |
@@ -377,7 +377,7 @@ static int i2sbus_remove(struct macio_dev* dev) | |||
377 | #ifdef CONFIG_PM | 377 | #ifdef CONFIG_PM |
378 | static int i2sbus_suspend(struct macio_dev* dev, pm_message_t state) | 378 | static int i2sbus_suspend(struct macio_dev* dev, pm_message_t state) |
379 | { | 379 | { |
380 | struct i2sbus_control *control = dev->ofdev.dev.driver_data; | 380 | struct i2sbus_control *control = dev_get_drvdata(&dev->ofdev.dev); |
381 | struct codec_info_item *cii; | 381 | struct codec_info_item *cii; |
382 | struct i2sbus_dev* i2sdev; | 382 | struct i2sbus_dev* i2sdev; |
383 | int err, ret = 0; | 383 | int err, ret = 0; |
@@ -407,7 +407,7 @@ static int i2sbus_suspend(struct macio_dev* dev, pm_message_t state) | |||
407 | 407 | ||
408 | static int i2sbus_resume(struct macio_dev* dev) | 408 | static int i2sbus_resume(struct macio_dev* dev) |
409 | { | 409 | { |
410 | struct i2sbus_control *control = dev->ofdev.dev.driver_data; | 410 | struct i2sbus_control *control = dev_get_drvdata(&dev->ofdev.dev); |
411 | struct codec_info_item *cii; | 411 | struct codec_info_item *cii; |
412 | struct i2sbus_dev* i2sdev; | 412 | struct i2sbus_dev* i2sdev; |
413 | int err, ret = 0; | 413 | int err, ret = 0; |