]> err.no Git - linux-2.6/commit
usb: u132-hcd driver style clean up
authorDaniel Walker <dwalker@mvista.com>
Sun, 23 Mar 2008 07:00:01 +0000 (00:00 -0700)
committerGreg Kroah-Hartman <gregkh@suse.de>
Fri, 25 Apr 2008 04:16:44 +0000 (21:16 -0700)
commitb40f8d3980d3eef26b3bd77da5314728a5b30aea
tree6b44b58d2b0c916701799044cf9c2b5ea8712a00
parent619a6f1d1423d08e74ed2b8a2113f12ef18e4373
usb: u132-hcd driver style clean up

I was converting a semaphore in this file to a mutex when I noticed that
this file has some fairly rampant style problems. Practically every line
has spaces instead of tabs .. Once I cleared that up, checkpatch.pl showed
a number of other problem.. I think this file might be a good one to review
for new style checks that could be added..

Below are the only two remaining which I didn't remove.

#5083: FILE: drivers/usb/host/u132-hcd.c:2907:
+               error:

WARNING: labels should not be indented
#5087: FILE: drivers/usb/host/u132-hcd.c:2911:
+               stall:

These labels are actually inside a switch statement, and they are right
under "default:". "default:" appears to be exempt and these other label
should be too, or default shouldn't be exempt.

I also deleted a few lines due to single statements inside { } ,

if (is_error()) {
return;
}

becomes,

if (is_error())
return;

with one line deleted.

Signed-off-by: Daniel Walker <dwalker@mvista.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/usb/host/u132-hcd.c