diff options
author | Greg Kroah-Hartman <gregkh@suse.de> | 2008-05-21 15:52:33 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2008-07-22 00:54:44 -0400 |
commit | 05675de250c4efcefbd488e6a556639b63898af8 (patch) | |
tree | 13e87c0219764a0f125d296c3cadcbdc943074af /drivers/s390 | |
parent | e17da9c4c5fc19e16ddcb0ae020ee4425ca0c96f (diff) |
device create: s390: convert device_create to device_create_drvdata
device_create() is race-prone, so use the race-free
device_create_drvdata() instead as device_create() is going away.
Cc: Martin Schwidefsky <schwidefsky@de.ibm.com>
Cc: Heiko Carstens <heiko.carstens@de.ibm.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/s390')
-rw-r--r-- | drivers/s390/char/raw3270.c | 14 | ||||
-rw-r--r-- | drivers/s390/char/tape_class.c | 7 | ||||
-rw-r--r-- | drivers/s390/char/vmur.c | 5 |
3 files changed, 14 insertions, 12 deletions
diff --git a/drivers/s390/char/raw3270.c b/drivers/s390/char/raw3270.c index 81a96e019080..c3dee900a5c8 100644 --- a/drivers/s390/char/raw3270.c +++ b/drivers/s390/char/raw3270.c | |||
@@ -1168,17 +1168,19 @@ static int raw3270_create_attributes(struct raw3270 *rp) | |||
1168 | if (rc) | 1168 | if (rc) |
1169 | goto out; | 1169 | goto out; |
1170 | 1170 | ||
1171 | rp->clttydev = device_create(class3270, &rp->cdev->dev, | 1171 | rp->clttydev = device_create_drvdata(class3270, &rp->cdev->dev, |
1172 | MKDEV(IBM_TTY3270_MAJOR, rp->minor), | 1172 | MKDEV(IBM_TTY3270_MAJOR, rp->minor), |
1173 | "tty%s", rp->cdev->dev.bus_id); | 1173 | NULL, |
1174 | "tty%s", rp->cdev->dev.bus_id); | ||
1174 | if (IS_ERR(rp->clttydev)) { | 1175 | if (IS_ERR(rp->clttydev)) { |
1175 | rc = PTR_ERR(rp->clttydev); | 1176 | rc = PTR_ERR(rp->clttydev); |
1176 | goto out_ttydev; | 1177 | goto out_ttydev; |
1177 | } | 1178 | } |
1178 | 1179 | ||
1179 | rp->cltubdev = device_create(class3270, &rp->cdev->dev, | 1180 | rp->cltubdev = device_create_drvdata(class3270, &rp->cdev->dev, |
1180 | MKDEV(IBM_FS3270_MAJOR, rp->minor), | 1181 | MKDEV(IBM_FS3270_MAJOR, rp->minor), |
1181 | "tub%s", rp->cdev->dev.bus_id); | 1182 | NULL, |
1183 | "tub%s", rp->cdev->dev.bus_id); | ||
1182 | if (!IS_ERR(rp->cltubdev)) | 1184 | if (!IS_ERR(rp->cltubdev)) |
1183 | goto out; | 1185 | goto out; |
1184 | 1186 | ||
diff --git a/drivers/s390/char/tape_class.c b/drivers/s390/char/tape_class.c index 6dfdb7c17981..12c2a5aaf31b 100644 --- a/drivers/s390/char/tape_class.c +++ b/drivers/s390/char/tape_class.c | |||
@@ -69,10 +69,9 @@ struct tape_class_device *register_tape_dev( | |||
69 | if (rc) | 69 | if (rc) |
70 | goto fail_with_cdev; | 70 | goto fail_with_cdev; |
71 | 71 | ||
72 | tcd->class_device = device_create(tape_class, device, | 72 | tcd->class_device = device_create_drvdata(tape_class, device, |
73 | tcd->char_device->dev, | 73 | tcd->char_device->dev, |
74 | "%s", tcd->device_name | 74 | NULL, "%s", tcd->device_name); |
75 | ); | ||
76 | rc = IS_ERR(tcd->class_device) ? PTR_ERR(tcd->class_device) : 0; | 75 | rc = IS_ERR(tcd->class_device) ? PTR_ERR(tcd->class_device) : 0; |
77 | if (rc) | 76 | if (rc) |
78 | goto fail_with_cdev; | 77 | goto fail_with_cdev; |
diff --git a/drivers/s390/char/vmur.c b/drivers/s390/char/vmur.c index b0ac44b27127..c1f352b84868 100644 --- a/drivers/s390/char/vmur.c +++ b/drivers/s390/char/vmur.c | |||
@@ -896,8 +896,9 @@ static int ur_set_online(struct ccw_device *cdev) | |||
896 | goto fail_free_cdev; | 896 | goto fail_free_cdev; |
897 | } | 897 | } |
898 | 898 | ||
899 | urd->device = device_create(vmur_class, NULL, urd->char_device->dev, | 899 | urd->device = device_create_drvdata(vmur_class, NULL, |
900 | "%s", node_id); | 900 | urd->char_device->dev, NULL, |
901 | "%s", node_id); | ||
901 | if (IS_ERR(urd->device)) { | 902 | if (IS_ERR(urd->device)) { |
902 | rc = PTR_ERR(urd->device); | 903 | rc = PTR_ERR(urd->device); |
903 | TRACE("ur_set_online: device_create rc=%d\n", rc); | 904 | TRACE("ur_set_online: device_create rc=%d\n", rc); |