Skip to content

Commit

Permalink
staging: usbip: userspace: rename usbip device and interface
Browse files Browse the repository at this point in the history
Add prefix of usbip_ to internal usb device and interface to avoid
confusion with the kernel types. This also identifies the types as
being part of the usbip library.

Signed-off-by: matt mooney <mfm@muteddisk.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
  • Loading branch information
matt mooney authored and Greg Kroah-Hartman committed Jun 7, 2011
1 parent 74ce259 commit 35dd0c2
Show file tree
Hide file tree
Showing 10 changed files with 44 additions and 41 deletions.
5 changes: 3 additions & 2 deletions drivers/staging/usbip/userspace/libsrc/stub_driver.c
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ static struct sysfs_driver *open_sysfs_stub_driver(void)
#define SYSFS_OPEN_RETRIES 100

/* only the first interface value is true! */
static int32_t read_attr_usbip_status(struct usb_device *udev)
static int32_t read_attr_usbip_status(struct usbip_usb_device *udev)
{
char attrpath[SYSFS_PATH_MAX];
struct sysfs_attribute *attr;
Expand Down Expand Up @@ -145,7 +145,8 @@ static struct usbip_exported_device *usbip_exported_device_new(char *sdevpath)
goto err;

/* reallocate buffer to include usb interface data */
size_t size = sizeof(*edev) + edev->udev.bNumInterfaces * sizeof(struct usb_interface);
size_t size = sizeof(*edev) + edev->udev.bNumInterfaces *
sizeof(struct usbip_usb_interface);
edev = (struct usbip_exported_device *) realloc(edev, size);
if (!edev) {
err("alloc device");
Expand Down
4 changes: 2 additions & 2 deletions drivers/staging/usbip/userspace/libsrc/stub_driver.h
Original file line number Diff line number Diff line change
Expand Up @@ -19,8 +19,8 @@ struct usbip_exported_device {
struct sysfs_device *sudev;

int32_t status;
struct usb_device udev;
struct usb_interface uinf[];
struct usbip_usb_device udev;
struct usbip_usb_interface uinf[];
};


Expand Down
9 changes: 5 additions & 4 deletions drivers/staging/usbip/userspace/libsrc/usbip_common.c
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,7 @@ const char *usbip_speed_string(int num)
#define DBG_UINF_INTEGER(name)\
dbg("%-20s = %x", to_string(name), (int) uinf->name)

void dump_usb_interface(struct usb_interface *uinf)
void dump_usb_interface(struct usbip_usb_interface *uinf)
{
char buff[100];
usbip_names_get_class(buff, sizeof(buff),
Expand All @@ -74,7 +74,7 @@ void dump_usb_interface(struct usb_interface *uinf)
dbg("%-20s = %s", "Interface(C/SC/P)", buff);
}

void dump_usb_device(struct usb_device *udev)
void dump_usb_device(struct usbip_usb_device *udev)
{
char buff[100];

Expand Down Expand Up @@ -181,7 +181,7 @@ int read_attr_speed(struct sysfs_device *dev)
do { (object)->name = (type) read_attr_value(dev, to_string(name), format); } while (0)


int read_usb_device(struct sysfs_device *sdev, struct usb_device *udev)
int read_usb_device(struct sysfs_device *sdev, struct usbip_usb_device *udev)
{
uint32_t busnum, devnum;

Expand Down Expand Up @@ -209,7 +209,8 @@ int read_usb_device(struct sysfs_device *sdev, struct usb_device *udev)
return 0;
}

int read_usb_interface(struct usb_device *udev, int i, struct usb_interface *uinf)
int read_usb_interface(struct usbip_usb_device *udev, int i,
struct usbip_usb_interface *uinf)
{
char busid[SYSFS_BUS_ID_SIZE];
struct sysfs_device *sif;
Expand Down
13 changes: 7 additions & 6 deletions drivers/staging/usbip/userspace/libsrc/usbip_common.h
Original file line number Diff line number Diff line change
Expand Up @@ -104,7 +104,7 @@ extern int usbip_use_debug ;
#define BUG() do { err("sorry, it's a bug"); abort(); } while (0)


struct usb_interface {
struct usbip_usb_interface {
uint8_t bInterfaceClass;
uint8_t bInterfaceSubClass;
uint8_t bInterfaceProtocol;
Expand All @@ -113,7 +113,7 @@ struct usb_interface {



struct usb_device {
struct usbip_usb_device {
char path[SYSFS_PATH_MAX];
char busid[SYSFS_BUS_ID_SIZE];

Expand All @@ -135,11 +135,12 @@ struct usb_device {

#define to_string(s) #s

void dump_usb_interface(struct usb_interface *);
void dump_usb_device(struct usb_device *);
int read_usb_device(struct sysfs_device *sdev, struct usb_device *udev);
void dump_usb_interface(struct usbip_usb_interface *);
void dump_usb_device(struct usbip_usb_device *);
int read_usb_device(struct sysfs_device *sdev, struct usbip_usb_device *udev);
int read_attr_value(struct sysfs_device *dev, const char *name, const char *format);
int read_usb_interface(struct usb_device *udev, int i, struct usb_interface *uinf);
int read_usb_interface(struct usbip_usb_device *udev, int i,
struct usbip_usb_interface *uinf);

const char *usbip_speed_string(int num);
const char *usbip_status_string(int32_t status);
Expand Down
2 changes: 1 addition & 1 deletion drivers/staging/usbip/userspace/libsrc/vhci_driver.h
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ struct usbip_imported_device {

/* usbip_class_device list */
struct dlist *cdev_list;
struct usb_device udev;
struct usbip_usb_device udev;
};

struct usbip_vhci_driver {
Expand Down
2 changes: 1 addition & 1 deletion drivers/staging/usbip/userspace/src/usbip_attach.c
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,7 @@ static int record_connection(char *host, char *port, char *busid, int rhport)
return 0;
}

static int import_device(int sockfd, struct usb_device *udev)
static int import_device(int sockfd, struct usbip_usb_device *udev)
{
int rc;
int port;
Expand Down
8 changes: 4 additions & 4 deletions drivers/staging/usbip/userspace/src/usbip_list.c
Original file line number Diff line number Diff line change
Expand Up @@ -77,13 +77,13 @@ static int query_exported_devices(int sockfd)
for (unsigned int i=0; i < rep.ndev; i++) {
char product_name[100];
char class_name[100];
struct usb_device udev;
struct usbip_usb_device udev;

memset(&udev, 0, sizeof(udev));

ret = usbip_recv(sockfd, (void *) &udev, sizeof(udev));
if (ret < 0) {
err("recv usb_device[%d]", i);
err("recv usbip_usb_device[%d]", i);
return -1;
}
pack_usb_device(0, &udev);
Expand All @@ -99,11 +99,11 @@ static int query_exported_devices(int sockfd)
printf("%8s: %s\n", " ", class_name);

for (int j=0; j < udev.bNumInterfaces; j++) {
struct usb_interface uinf;
struct usbip_usb_interface uinf;

ret = usbip_recv(sockfd, (void *) &uinf, sizeof(uinf));
if (ret < 0) {
err("recv usb_interface[%d]", j);
err("recv usbip_usb_interface[%d]", j);
return -1;
}

Expand Down
16 changes: 8 additions & 8 deletions drivers/staging/usbip/userspace/src/usbip_network.c
Original file line number Diff line number Diff line change
Expand Up @@ -4,10 +4,10 @@
*/

#include <sys/socket.h>
#include <arpa/inet.h>

#include <string.h>

#include <arpa/inet.h>
#include <netdb.h>
#include <netinet/tcp.h>
#include <unistd.h>
Expand Down Expand Up @@ -39,7 +39,7 @@ void pack_uint16_t(int pack, uint16_t *num)
*num = i;
}

void pack_usb_device(int pack, struct usb_device *udev)
void pack_usb_device(int pack, struct usbip_usb_device *udev)
{
pack_uint32_t(pack, &udev->busnum);
pack_uint32_t(pack, &udev->devnum);
Expand All @@ -51,7 +51,7 @@ void pack_usb_device(int pack, struct usb_device *udev)
}

void pack_usb_interface(int pack __attribute__((unused)),
struct usb_interface *udev __attribute__((unused)))
struct usbip_usb_interface *udev __attribute__((unused)))
{
/* uint8_t members need nothing */
}
Expand Down Expand Up @@ -102,15 +102,15 @@ int usbip_send_op_common(int sockfd, uint32_t code, uint32_t status)

memset(&op_common, 0, sizeof(op_common));

op_common.version = USBIP_VERSION;
op_common.code = code;
op_common.status = status;
op_common.version = USBIP_VERSION;
op_common.code = code;
op_common.status = status;

PACK_OP_COMMON(1, &op_common);

ret = usbip_send(sockfd, (void *) &op_common, sizeof(op_common));
if (ret < 0) {
err("send op_common");
err("usbip_send has failed");
return -1;
}

Expand All @@ -126,7 +126,7 @@ int usbip_recv_op_common(int sockfd, uint16_t *code)

ret = usbip_recv(sockfd, (void *) &op_common, sizeof(op_common));
if (ret < 0) {
err("recv op_common, %d", ret);
err("usbip_recv has failed ret=%d", ret);
goto err;
}

Expand Down
20 changes: 10 additions & 10 deletions drivers/staging/usbip/userspace/src/usbip_network.h
Original file line number Diff line number Diff line change
Expand Up @@ -56,8 +56,8 @@ struct op_devinfo_request {
} __attribute__((packed));

struct op_devinfo_reply {
struct usb_device udev;
struct usb_interface uinf[];
struct usbip_usb_device udev;
struct usbip_usb_interface uinf[];
} __attribute__((packed));

/* ---------------------------------------------------------------------- */
Expand All @@ -71,8 +71,8 @@ struct op_import_request {
} __attribute__((packed));

struct op_import_reply {
struct usb_device udev;
// struct usb_interface uinf[];
struct usbip_usb_device udev;
// struct usbip_usb_interface uinf[];
} __attribute__((packed));

#define PACK_OP_IMPORT_REQUEST(pack, request) do {\
Expand All @@ -89,7 +89,7 @@ struct op_import_reply {
#define OP_REP_EXPORT (OP_REPLY | OP_EXPORT)

struct op_export_request {
struct usb_device udev;
struct usbip_usb_device udev;
} __attribute__((packed));

struct op_export_reply {
Expand All @@ -111,7 +111,7 @@ struct op_export_reply {
#define OP_REP_UNEXPORT (OP_REPLY | OP_UNEXPORT)

struct op_unexport_request {
struct usb_device udev;
struct usbip_usb_device udev;
} __attribute__((packed));

struct op_unexport_reply {
Expand Down Expand Up @@ -156,8 +156,8 @@ struct op_devlist_reply {
} __attribute__((packed));

struct op_devlist_reply_extra {
struct usb_device udev;
struct usb_interface uinf[];
struct usbip_usb_device udev;
struct usbip_usb_interface uinf[];
} __attribute__((packed));

#define PACK_OP_DEVLIST_REQUEST(pack, request) do {\
Expand All @@ -169,8 +169,8 @@ struct op_devlist_reply_extra {

void pack_uint32_t(int pack, uint32_t *num);
void pack_uint16_t(int pack, uint16_t *num);
void pack_usb_device(int pack, struct usb_device *udev);
void pack_usb_interface(int pack, struct usb_interface *uinf);
void pack_usb_device(int pack, struct usbip_usb_device *udev);
void pack_usb_interface(int pack, struct usbip_usb_interface *uinf);

ssize_t usbip_recv(int sockfd, void *buff, size_t bufflen);
ssize_t usbip_send(int sockfd, void *buff, size_t bufflen);
Expand Down
6 changes: 3 additions & 3 deletions drivers/staging/usbip/userspace/src/usbipd.c
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,7 @@ static int send_reply_devlist(int sockfd)
}

dlist_for_each_data(stub_driver->edev_list, edev, struct usbip_exported_device) {
struct usb_device pdu_udev;
struct usbip_usb_device pdu_udev;

dump_usb_device(&edev->udev);
memcpy(&pdu_udev, &edev->udev, sizeof(pdu_udev));
Expand All @@ -77,7 +77,7 @@ static int send_reply_devlist(int sockfd)
}

for (int i=0; i < edev->udev.bNumInterfaces; i++) {
struct usb_interface pdu_uinf;
struct usbip_usb_interface pdu_uinf;

dump_usb_interface(&edev->uinf[i]);
memcpy(&pdu_uinf, &edev->uinf[i], sizeof(pdu_uinf));
Expand Down Expand Up @@ -167,7 +167,7 @@ static int recv_request_import(int sockfd)
}

if (!error) {
struct usb_device pdu_udev;
struct usbip_usb_device pdu_udev;

memcpy(&pdu_udev, &edev->udev, sizeof(pdu_udev));
pack_usb_device(1, &pdu_udev);
Expand Down

0 comments on commit 35dd0c2

Please sign in to comment.