linux.git/drivers/staging/usbip/userspace, branch v3.10

staging: usbip: userspace: show product name in `list -l' command

2013-04-05T21:22:52+00:00

The `usbip list -l' command shows your local usb-devices.
Example:
 $ usbip list -l
 $ Local USB devices
 $ =================
 $ - busid 1-1 (13fe:1d00)
 $         1-1:1.0 -> usb-storage
 $
 $ - busid 1-2 (0409:55aa)
 $         1-2:1.0 -> hub

However this list command doesn't show which device is connected
to this busid. Therefore you have to use another tool e.g. lsusb
to determine that.

This patches adds the possibility to see which device that is.
Example:
 $ usbip list -l
 $ Local USB devices
 $ =================
 $ - busid 1-1 (13fe:1d00)
 $   Kingston Technology Company Inc. : DataTraveler 2.0 1GB/4GB Flash Drive / Patriot Xporter 4GB Flash
 $         1-1:1.0 -> usb-storage
 $
 $ - busid 1-2 (0409:55aa)
 $   NEC Corp. : Hub (0409:55aa)
 $         1-2:1.0 -> hub

If parsable is specified the info will be not printed.

Signed-off-by: Kurt Kanzenbach 
Signed-off-by: Stefan Reif 
Signed-off-by: Greg Kroah-Hartman

staging: usbip: userspace: unified command line arguments

2013-04-05T21:22:52+00:00

The command `usbip attach' uses --host for specifing
the remote host, while `usbip list' uses --remote.
This is confusing and this patch adapts this.
In Addition changed the manpage and README accordingly.

Before:
 $ usbip attach --host  -b 
 $ usbip list --remote 

Now:
 $ usbip attach --remote  -b 
 $ usbip list --remote 

Signed-off-by: Kurt Kanzenbach 
Signed-off-by: Stefan Reif 
Signed-off-by: Greg Kroah-Hartman

staging: usbip: userspace: libsrc: cleanup parsing

2013-04-05T21:22:51+00:00

Since the names.c/names.h are taken from another project, some
functions which names.c provides aren't used by usbipd.
This patch fixes:
 - removed useless comments
 - unified debug/error messages by using the macros
   provided by usbip_common.h
 - removed unnused code

The code cleanup includes:
 - remove unused data structures
 - remove code to create them
 - remove code to access them

The file names.c is used to parse the `usb.ids' file. The parser
stores a lot of information about usb devices that is never used.

The `usb.ids' file has several sections. Some variables (like
`lasthut') store the ID of the current section, and those variables
are used to decide which section is currently being parsed (i.e. in
which data structure the current line will be  stored).

We removed the code to read those IDs because they are never used
anyway. We replaced them by the pseudo-ID `1' (instead of reading the
ID from the file) to indicate that the parser is in a section that
can be ignored. If the parser is in such a section, the current line
(which contains sub-items for this section) is discarded.

Signed-off-by: Kurt Kanzenbach 
Signed-off-by: Stefan Reif 
Signed-off-by: Greg Kroah-Hartman

staging: usbip: userspace: removed unnecessary code

2013-04-05T21:22:51+00:00

Since no usbip_name function is used in usbipd, it's not
necessary to parse "usb.ids" file at startup.

Signed-off-by: Kurt Kanzenbach 
Signed-off-by: Stefan Reif 
Signed-off-by: Greg Kroah-Hartman

staging: usbip: userspace: avoid memory leaks

2013-04-05T21:22:51+00:00

Call freeaddrinfo when connect/listen fails.
Call usbip_host_driver_close on error.

Signed-off-by: Stefan Reif 
Signed-off-by: Greg Kroah-Hartman

staging: usbip: userspace: libsrc: replace numbers by ascii

2013-04-05T21:22:49+00:00

replace numbers in code by ascii text constants as suggested
by Dan Carpenter:
http://driverdev.linuxdriverproject.org/pipermail/devel/2013-February/035907.html

Signed-off-by: Stefan Reif 
Signed-off-by: Greg Kroah-Hartman

staging: usbip: userspace: fix whitespace errors

2013-03-11T16:45:29+00:00

This patch fixes the following checkpatch errors:
-ERROR: space required after that ','
-ERROR: spaces required around that '='
-ERROR: space prohibited before that close parenthesis
-WARNING: please, no space before tabs

Signed-off-by: Stefan Reif 
Signed-off-by: Kurt Kanzenbach 
Signed-off-by: Greg Kroah-Hartman

staging: usbip: userspace: libsrc: added missing space

2013-03-11T16:45:29+00:00

This patch fixes the following checkpatch warning:
-WARNING: missing space after enum definition

Signed-off-by: Kurt Kanzenbach 
Signed-off-by: Greg Kroah-Hartman

staging: usbip: userspace: libsrc: removed assignments in if conditions

2013-03-11T16:45:28+00:00

This patch fixes the following checkpatch error:
-ERROR: do not use assignment in if condition

Signed-off-by: Kurt Kanzenbach 
Signed-off-by: Greg Kroah-Hartman

staging: usbip: userspace: libsrc: replaced lines over 80 characters

2013-03-11T16:45:28+00:00

This patch fixes some of the following checkpatch warnings:
-WARNING: line over 80 characters

We did not split format strings for readability.

Signed-off-by: Kurt Kanzenbach 
Signed-off-by: Greg Kroah-Hartman