Skip to content

Commit

Permalink
Staging: vt665x: cleanup USB definitions
Browse files Browse the repository at this point in the history 
Remove gratuitous macros re-defining usb functions.
One result is to make kcompat.h now identical in both driver dirs

Signed-off-by: Jim Lieb <lieb@canonical.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
  • Loading branch information
Jim Lieb authored and Greg Kroah-Hartman committed Sep 15, 2009
1 parent f9de7e7 commit dad72fe
Show file tree
Hide file tree
Showing 3 changed files with 20 additions and 25 deletions.
5 changes: 0 additions & 5 deletions drivers/staging/vt6656/kcompat.h
View file
Original file line number Diff line number Diff line change
Expand Up @@ -35,10 +35,5 @@
#define netdev_priv(dev) (dev->priv)
#endif

#define vntwusb_submit_urb(val) usb_submit_urb(val, GFP_ATOMIC)
#define vntwusb_alloc_urb(val) usb_alloc_urb(val, GFP_ATOMIC)

#define vntwusb_unlink_urb(val) usb_kill_urb(val)

#endif

26 changes: 13 additions & 13 deletions drivers/staging/vt6656/main_usb.c
View file
Original file line number Diff line number Diff line change
Expand Up @@ -859,7 +859,7 @@ static VOID device_free_tx_bufs(PSDevice pDevice) {
pTxContext = pDevice->apTD[ii];
//de-allocate URBs
if (pTxContext->pUrb) {
vntwusb_unlink_urb(pTxContext->pUrb);
usb_kill_urb(pTxContext->pUrb);
usb_free_urb(pTxContext->pUrb);
}
if (pTxContext)
Expand All @@ -878,7 +878,7 @@ static VOID device_free_rx_bufs(PSDevice pDevice) {
pRCB = pDevice->apRCB[ii];
//de-allocate URBs
if (pRCB->pUrb) {
vntwusb_unlink_urb(pRCB->pUrb);
usb_kill_urb(pRCB->pUrb);
usb_free_urb(pRCB->pUrb);
}
//de-allocate skb
Expand Down Expand Up @@ -926,7 +926,7 @@ static BOOL device_alloc_bufs(PSDevice pDevice) {
pDevice->apTD[ii] = pTxContext;
pTxContext->pDevice = (PVOID) pDevice;
//allocate URBs
pTxContext->pUrb = vntwusb_alloc_urb(0);;
pTxContext->pUrb = usb_alloc_urb(0, GFP_ATOMIC);
if (pTxContext->pUrb == NULL) {
DBG_PRT(MSG_LEVEL_ERR,KERN_ERR "alloc tx urb failed\n");
goto free_tx;
Expand Down Expand Up @@ -955,7 +955,7 @@ static BOOL device_alloc_bufs(PSDevice pDevice) {
pDevice->apRCB[ii] = pRCB;
pRCB->pDevice = (PVOID) pDevice;
//allocate URBs
pRCB->pUrb = vntwusb_alloc_urb(0);
pRCB->pUrb = usb_alloc_urb(0, GFP_ATOMIC);

if (pRCB->pUrb == NULL) {
DBG_PRT(MSG_LEVEL_ERR,KERN_ERR" Failed to alloc rx urb\n");
Expand All @@ -974,25 +974,25 @@ static BOOL device_alloc_bufs(PSDevice pDevice) {
}


pDevice->pControlURB = vntwusb_alloc_urb(0);
pDevice->pControlURB = usb_alloc_urb(0, GFP_ATOMIC);
if (pDevice->pControlURB == NULL) {
DBG_PRT(MSG_LEVEL_ERR,KERN_ERR"Failed to alloc control urb\n");
goto free_rx_tx;
}

pDevice->pInterruptURB = vntwusb_alloc_urb(0);
pDevice->pInterruptURB = usb_alloc_urb(0, GFP_ATOMIC);
if (pDevice->pInterruptURB == NULL) {
DBG_PRT(MSG_LEVEL_ERR,KERN_ERR"Failed to alloc int urb\n");
vntwusb_unlink_urb(pDevice->pControlURB);
usb_kill_urb(pDevice->pControlURB);
usb_free_urb(pDevice->pControlURB);
goto free_rx_tx;
}

pDevice->intBuf.pDataBuf = kmalloc(MAX_INTERRUPT_SIZE, GFP_KERNEL);
if (pDevice->intBuf.pDataBuf == NULL) {
DBG_PRT(MSG_LEVEL_ERR,KERN_ERR"Failed to alloc int buf\n");
vntwusb_unlink_urb(pDevice->pControlURB);
vntwusb_unlink_urb(pDevice->pInterruptURB);
usb_kill_urb(pDevice->pControlURB);
usb_kill_urb(pDevice->pInterruptURB);
usb_free_urb(pDevice->pControlURB);
usb_free_urb(pDevice->pInterruptURB);
goto free_rx_tx;
Expand Down Expand Up @@ -1193,8 +1193,8 @@ static int device_open(struct net_device *dev) {
device_free_rx_bufs(pDevice);
device_free_tx_bufs(pDevice);
device_free_int_bufs(pDevice);
vntwusb_unlink_urb(pDevice->pControlURB);
vntwusb_unlink_urb(pDevice->pInterruptURB);
usb_kill_urb(pDevice->pControlURB);
usb_kill_urb(pDevice->pInterruptURB);
usb_free_urb(pDevice->pControlURB);
usb_free_urb(pDevice->pInterruptURB);

Expand Down Expand Up @@ -1280,8 +1280,8 @@ device_release_WPADEV(pDevice);
device_free_int_bufs(pDevice);
device_free_frag_bufs(pDevice);

vntwusb_unlink_urb(pDevice->pControlURB);
vntwusb_unlink_urb(pDevice->pInterruptURB);
usb_kill_urb(pDevice->pControlURB);
usb_kill_urb(pDevice->pInterruptURB);
usb_free_urb(pDevice->pControlURB);
usb_free_urb(pDevice->pInterruptURB);

Expand Down
14 changes: 7 additions & 7 deletions drivers/staging/vt6656/usbpipe.c
View file
Original file line number Diff line number Diff line change
Expand Up @@ -193,7 +193,7 @@ PIPEnsControlOut(
usb_sndctrlpipe(pDevice->usb , 0), (char *) &pDevice->sUsbCtlRequest,
pbyBuffer, wLength, s_nsControlInUsbIoCompleteWrite, pDevice);

if ((ntStatus = vntwusb_submit_urb(pDevice->pControlURB) != 0)) {
if ((ntStatus = usb_submit_urb(pDevice->pControlURB, GFP_ATOMIC)) != 0) {
DBG_PRT(MSG_LEVEL_DEBUG, KERN_INFO"control send request submission failed: %d\n", ntStatus);
return STATUS_FAILURE;
}
Expand Down Expand Up @@ -251,7 +251,7 @@ PIPEnsControlIn(
usb_rcvctrlpipe(pDevice->usb , 0), (char *) &pDevice->sUsbCtlRequest,
pbyBuffer, wLength, s_nsControlInUsbIoCompleteRead, pDevice);

if ((ntStatus = vntwusb_submit_urb(pDevice->pControlURB) != 0)) {
if ((ntStatus = usb_submit_urb(pDevice->pControlURB, GFP_ATOMIC)) != 0) {
DBG_PRT(MSG_LEVEL_DEBUG, KERN_INFO"control request submission failed: %d\n", ntStatus);
}else {
MP_SET_FLAG(pDevice, fMP_CONTROL_READS);
Expand Down Expand Up @@ -414,7 +414,7 @@ usb_fill_bulk_urb(pDevice->pInterruptURB,
#endif
#endif

if ((ntStatus = vntwusb_submit_urb(pDevice->pInterruptURB)) != 0) {
if ((ntStatus = usb_submit_urb(pDevice->pInterruptURB, GFP_ATOMIC)) != 0) {
DBG_PRT(MSG_LEVEL_DEBUG, KERN_INFO"Submit int URB failed %d\n", ntStatus);
}

Expand Down Expand Up @@ -494,7 +494,7 @@ s_nsInterruptUsbIoCompleteRead(

if (pDevice->fKillEventPollingThread != TRUE) {
#if 0 //reserve int URB submit
if ((ntStatus = vntwusb_submit_urb(urb)) != 0) {
if ((ntStatus = usb_submit_urb(urb, GFP_ATOMIC)) != 0) {
DBG_PRT(MSG_LEVEL_DEBUG, KERN_INFO"Re-Submit int URB failed %d\n", ntStatus);
}
#else //replace int URB submit by bulk transfer
Expand All @@ -507,7 +507,7 @@ s_nsInterruptUsbIoCompleteRead(
s_nsInterruptUsbIoCompleteRead,
pDevice);

if ((ntStatus = vntwusb_submit_urb(pDevice->pInterruptURB)) != 0) {
if ((ntStatus = usb_submit_urb(pDevice->pInterruptURB, GFP_ATOMIC)) != 0) {
DBG_PRT(MSG_LEVEL_DEBUG, KERN_INFO"Submit int URB failed %d\n", ntStatus);
}

Expand Down Expand Up @@ -572,7 +572,7 @@ PIPEnsBulkInUsbRead(
s_nsBulkInUsbIoCompleteRead,
pRCB);

if((ntStatus = vntwusb_submit_urb(pUrb)!=0)){
if((ntStatus = usb_submit_urb(pUrb, GFP_ATOMIC)) != 0){
DBG_PRT(MSG_LEVEL_DEBUG, KERN_INFO"Submit Rx URB failed %d\n", ntStatus);
return STATUS_FAILURE ;
}
Expand Down Expand Up @@ -718,7 +718,7 @@ PIPEnsSendBulkOut(
s_nsBulkOutIoCompleteWrite,
pContext);

if((status = vntwusb_submit_urb(pUrb))!=0)
if((status = usb_submit_urb(pUrb, GFP_ATOMIC))!=0)
{
DBG_PRT(MSG_LEVEL_DEBUG, KERN_INFO"Submit Tx URB failed %d\n", status);
return STATUS_FAILURE;
Expand Down

0 comments on commit dad72fe

Please sign in to comment.