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:43 -0400 |
commit | f71674a09f374bb0b5371b3e7b943029867477c5 (patch) | |
tree | 876b0daddad7b033cc3f91afe87c93b3180c80f1 | |
parent | 6ecaaf94b831827405613ee9110d8fb8eff92482 (diff) |
device create: ieee1394: 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: Ben Collins <ben.collins@ubuntu.com>
Acked-by: Stefan Richter <stefanr@s5r6.in-berlin.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-rw-r--r-- | drivers/ieee1394/dv1394.c | 7 | ||||
-rw-r--r-- | drivers/ieee1394/raw1394.c | 4 | ||||
-rw-r--r-- | drivers/ieee1394/video1394.c | 6 |
3 files changed, 9 insertions, 8 deletions
diff --git a/drivers/ieee1394/dv1394.c b/drivers/ieee1394/dv1394.c index 9d19aec5820a..b6eb2cf25914 100644 --- a/drivers/ieee1394/dv1394.c +++ b/drivers/ieee1394/dv1394.c | |||
@@ -2296,9 +2296,10 @@ static void dv1394_add_host(struct hpsb_host *host) | |||
2296 | 2296 | ||
2297 | ohci = (struct ti_ohci *)host->hostdata; | 2297 | ohci = (struct ti_ohci *)host->hostdata; |
2298 | 2298 | ||
2299 | device_create(hpsb_protocol_class, NULL, MKDEV( | 2299 | device_create_drvdata(hpsb_protocol_class, NULL, |
2300 | IEEE1394_MAJOR, IEEE1394_MINOR_BLOCK_DV1394 * 16 + (id<<2)), | 2300 | MKDEV(IEEE1394_MAJOR, |
2301 | "dv1394-%d", id); | 2301 | IEEE1394_MINOR_BLOCK_DV1394 * 16 + (id<<2)), NULL, |
2302 | "dv1394-%d", id); | ||
2302 | 2303 | ||
2303 | dv1394_init(ohci, DV1394_NTSC, MODE_RECEIVE); | 2304 | dv1394_init(ohci, DV1394_NTSC, MODE_RECEIVE); |
2304 | dv1394_init(ohci, DV1394_NTSC, MODE_TRANSMIT); | 2305 | dv1394_init(ohci, DV1394_NTSC, MODE_TRANSMIT); |
diff --git a/drivers/ieee1394/raw1394.c b/drivers/ieee1394/raw1394.c index 96f2847b0405..6fa9e4a21840 100644 --- a/drivers/ieee1394/raw1394.c +++ b/drivers/ieee1394/raw1394.c | |||
@@ -3010,10 +3010,10 @@ static int __init init_raw1394(void) | |||
3010 | hpsb_register_highlevel(&raw1394_highlevel); | 3010 | hpsb_register_highlevel(&raw1394_highlevel); |
3011 | 3011 | ||
3012 | if (IS_ERR | 3012 | if (IS_ERR |
3013 | (device_create( | 3013 | (device_create_drvdata( |
3014 | hpsb_protocol_class, NULL, | 3014 | hpsb_protocol_class, NULL, |
3015 | MKDEV(IEEE1394_MAJOR, IEEE1394_MINOR_BLOCK_RAW1394 * 16), | 3015 | MKDEV(IEEE1394_MAJOR, IEEE1394_MINOR_BLOCK_RAW1394 * 16), |
3016 | RAW1394_DEVICE_NAME))) { | 3016 | NULL, RAW1394_DEVICE_NAME))) { |
3017 | ret = -EFAULT; | 3017 | ret = -EFAULT; |
3018 | goto out_unreg; | 3018 | goto out_unreg; |
3019 | } | 3019 | } |
diff --git a/drivers/ieee1394/video1394.c b/drivers/ieee1394/video1394.c index 069b9f6bf16d..25db6e67fa4e 100644 --- a/drivers/ieee1394/video1394.c +++ b/drivers/ieee1394/video1394.c | |||
@@ -1341,9 +1341,9 @@ static void video1394_add_host (struct hpsb_host *host) | |||
1341 | hpsb_set_hostinfo_key(&video1394_highlevel, host, ohci->host->id); | 1341 | hpsb_set_hostinfo_key(&video1394_highlevel, host, ohci->host->id); |
1342 | 1342 | ||
1343 | minor = IEEE1394_MINOR_BLOCK_VIDEO1394 * 16 + ohci->host->id; | 1343 | minor = IEEE1394_MINOR_BLOCK_VIDEO1394 * 16 + ohci->host->id; |
1344 | device_create(hpsb_protocol_class, NULL, | 1344 | device_create_drvdata(hpsb_protocol_class, NULL, |
1345 | MKDEV(IEEE1394_MAJOR, minor), | 1345 | MKDEV(IEEE1394_MAJOR, minor), NULL, |
1346 | "%s-%d", VIDEO1394_DRIVER_NAME, ohci->host->id); | 1346 | "%s-%d", VIDEO1394_DRIVER_NAME, ohci->host->id); |
1347 | } | 1347 | } |
1348 | 1348 | ||
1349 | 1349 | ||