diff options
author | Pete Zaitcev <zaitcev@redhat.com> | 2007-03-08 20:02:26 -0800 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2007-04-27 13:28:34 -0700 |
commit | 64e35d92367d8cd376946eb924838de1dd3287c7 (patch) | |
tree | 9100e11f70dcbca6696ec29e1c9da91f9331f793 /drivers/usb/storage | |
parent | 643616e6780b26dd8c9cea0b9344bb5d7aeae29d (diff) | |
download | op-kernel-dev-64e35d92367d8cd376946eb924838de1dd3287c7.zip op-kernel-dev-64e35d92367d8cd376946eb924838de1dd3287c7.tar.gz |
libusual: change block scope variable to function scope
Someone changed the code to kthread and used his style instead of mine.
The problem with the block variables is that they provoke shadowing,
which is actually exactly what has happened in my other tree which
has the class patch.
Signed-off-by: Pete Zaitcev <zaitcev@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/usb/storage')
-rw-r--r-- | drivers/usb/storage/libusual.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/usb/storage/libusual.c b/drivers/usb/storage/libusual.c index 599ad10..06d1107 100644 --- a/drivers/usb/storage/libusual.c +++ b/drivers/usb/storage/libusual.c @@ -117,6 +117,7 @@ EXPORT_SYMBOL_GPL(usb_usual_check_type); static int usu_probe(struct usb_interface *intf, const struct usb_device_id *id) { + int rc; unsigned long type; struct task_struct* task; unsigned long flags; @@ -135,7 +136,7 @@ static int usu_probe(struct usb_interface *intf, task = kthread_run(usu_probe_thread, (void*)type, "libusual_%d", type); if (IS_ERR(task)) { - int rc = PTR_ERR(task); + rc = PTR_ERR(task); printk(KERN_WARNING "libusual: " "Unable to start the thread for %s: %d\n", bias_names[type], rc); |