mirror of
https://github.com/torvalds/linux.git
synced 2024-12-30 14:52:05 +00:00
drivers/usb: Remove unnecessary return's from void functions
Greg prefers this to go through the trivial tree. http://lkml.org/lkml/2010/6/24/1 There are about 2500 void functions in drivers/usb Only a few used return; at end of function. Standardize them a bit. Moved a statement down a line in drivers/usb/host/u132-hcd.c Signed-off-by: Joe Perches <joe@perches.com> Signed-off-by: Jiri Kosina <jkosina@suse.cz>
This commit is contained in:
parent
429d646d9a
commit
7f26b3a753
@ -1266,7 +1266,6 @@ write_in:
|
||||
csr |= AT91_UDP_TXPKTRDY;
|
||||
__raw_writel(csr, creg);
|
||||
udc->req_pending = 0;
|
||||
return;
|
||||
}
|
||||
|
||||
static void handle_ep0(struct at91_udc *udc)
|
||||
|
@ -317,8 +317,6 @@ static void f_audio_playback_work(struct work_struct *data)
|
||||
|
||||
u_audio_playback(&audio->card, play_buf->buf, play_buf->actual);
|
||||
f_audio_buffer_free(play_buf);
|
||||
|
||||
return;
|
||||
}
|
||||
|
||||
static int f_audio_out_ep_complete(struct usb_ep *ep, struct usb_request *req)
|
||||
|
@ -318,8 +318,6 @@ static void hidg_set_report_complete(struct usb_ep *ep, struct usb_request *req)
|
||||
spin_unlock(&hidg->spinlock);
|
||||
|
||||
wake_up(&hidg->read_queue);
|
||||
|
||||
return;
|
||||
}
|
||||
|
||||
static int hidg_setup(struct usb_function *f,
|
||||
@ -413,8 +411,6 @@ static void hidg_disable(struct usb_function *f)
|
||||
|
||||
usb_ep_disable(hidg->in_ep);
|
||||
hidg->in_ep->driver_data = NULL;
|
||||
|
||||
return;
|
||||
}
|
||||
|
||||
static int hidg_set_alt(struct usb_function *f, unsigned intf, unsigned alt)
|
||||
|
@ -287,8 +287,6 @@ static void dr_controller_run(struct fsl_udc *udc)
|
||||
temp = fsl_readl(&dr_regs->usbcmd);
|
||||
temp |= USB_CMD_RUN_STOP;
|
||||
fsl_writel(temp, &dr_regs->usbcmd);
|
||||
|
||||
return;
|
||||
}
|
||||
|
||||
static void dr_controller_stop(struct fsl_udc *udc)
|
||||
@ -308,8 +306,6 @@ static void dr_controller_stop(struct fsl_udc *udc)
|
||||
tmp = fsl_readl(&dr_regs->usbcmd);
|
||||
tmp &= ~USB_CMD_RUN_STOP;
|
||||
fsl_writel(tmp, &dr_regs->usbcmd);
|
||||
|
||||
return;
|
||||
}
|
||||
|
||||
static void dr_ep_setup(unsigned char ep_num, unsigned char dir,
|
||||
@ -416,8 +412,6 @@ static void struct_ep_qh_setup(struct fsl_udc *udc, unsigned char ep_num,
|
||||
p_QH->max_pkt_length = cpu_to_le32(tmp);
|
||||
p_QH->next_dtd_ptr = 1;
|
||||
p_QH->size_ioc_int_sts = 0;
|
||||
|
||||
return;
|
||||
}
|
||||
|
||||
/* Setup qh structure and ep register for ep0. */
|
||||
|
@ -284,7 +284,6 @@ static void ep0_reset(struct langwell_udc *dev)
|
||||
}
|
||||
|
||||
VDBG(dev, "<--- %s()\n", __func__);
|
||||
return;
|
||||
}
|
||||
|
||||
|
||||
@ -1486,7 +1485,6 @@ static void langwell_udc_start(struct langwell_udc *dev)
|
||||
writel(usbcmd, &dev->op_regs->usbcmd);
|
||||
|
||||
DBG(dev, "<--- %s()\n", __func__);
|
||||
return;
|
||||
}
|
||||
|
||||
|
||||
@ -1509,7 +1507,6 @@ static void langwell_udc_stop(struct langwell_udc *dev)
|
||||
writel(usbcmd, &dev->op_regs->usbcmd);
|
||||
|
||||
DBG(dev, "<--- %s()\n", __func__);
|
||||
return;
|
||||
}
|
||||
|
||||
|
||||
|
@ -1394,8 +1394,6 @@ static void pxa_ep_fifo_flush(struct usb_ep *_ep)
|
||||
}
|
||||
|
||||
spin_unlock_irqrestore(&ep->lock, flags);
|
||||
|
||||
return;
|
||||
}
|
||||
|
||||
/**
|
||||
|
@ -916,8 +916,6 @@ void rndis_deregister (int configNr)
|
||||
|
||||
if (configNr >= RNDIS_MAX_CONFIGS) return;
|
||||
rndis_per_dev_params [configNr].used = 0;
|
||||
|
||||
return;
|
||||
}
|
||||
|
||||
int rndis_set_param_dev(u8 configNr, struct net_device *dev, u16 *cdc_filter)
|
||||
|
@ -1041,7 +1041,6 @@ nogood:
|
||||
ep->hcpriv = NULL;
|
||||
done:
|
||||
spin_unlock_irqrestore (&ehci->lock, flags);
|
||||
return;
|
||||
}
|
||||
|
||||
static void
|
||||
|
@ -370,7 +370,6 @@ sanitize:
|
||||
}
|
||||
ep->hcpriv = NULL;
|
||||
spin_unlock_irqrestore (&ohci->lock, flags);
|
||||
return;
|
||||
}
|
||||
|
||||
static int ohci_get_frame (struct usb_hcd *hcd)
|
||||
|
@ -544,8 +544,6 @@ static void oxu_buf_free(struct oxu_hcd *oxu, struct ehci_qtd *qtd)
|
||||
qtd->buffer = NULL;
|
||||
|
||||
spin_unlock(&oxu->mem_lock);
|
||||
|
||||
return;
|
||||
}
|
||||
|
||||
static inline void ehci_qtd_init(struct ehci_qtd *qtd, dma_addr_t dma)
|
||||
@ -571,8 +569,6 @@ static inline void oxu_qtd_free(struct oxu_hcd *oxu, struct ehci_qtd *qtd)
|
||||
oxu->qtd_used[index] = 0;
|
||||
|
||||
spin_unlock(&oxu->mem_lock);
|
||||
|
||||
return;
|
||||
}
|
||||
|
||||
static struct ehci_qtd *ehci_qtd_alloc(struct oxu_hcd *oxu)
|
||||
@ -615,8 +611,6 @@ static void oxu_qh_free(struct oxu_hcd *oxu, struct ehci_qh *qh)
|
||||
oxu->qh_used[index] = 0;
|
||||
|
||||
spin_unlock(&oxu->mem_lock);
|
||||
|
||||
return;
|
||||
}
|
||||
|
||||
static void qh_destroy(struct kref *kref)
|
||||
@ -693,8 +687,6 @@ static void oxu_murb_free(struct oxu_hcd *oxu, struct oxu_murb *murb)
|
||||
oxu->murb_used[index] = 0;
|
||||
|
||||
spin_unlock(&oxu->mem_lock);
|
||||
|
||||
return;
|
||||
}
|
||||
|
||||
static struct oxu_murb *oxu_murb_alloc(struct oxu_hcd *oxu)
|
||||
@ -3073,7 +3065,6 @@ nogood:
|
||||
ep->hcpriv = NULL;
|
||||
done:
|
||||
spin_unlock_irqrestore(&oxu->lock, flags);
|
||||
return;
|
||||
}
|
||||
|
||||
static int oxu_get_frame(struct usb_hcd *hcd)
|
||||
|
@ -338,8 +338,6 @@ static void __devinit quirk_usb_disable_ehci(struct pci_dev *pdev)
|
||||
writel(0x3f, op_reg_base + EHCI_USBSTS);
|
||||
|
||||
iounmap(base);
|
||||
|
||||
return;
|
||||
}
|
||||
|
||||
/*
|
||||
|
@ -316,7 +316,6 @@ static void u132_ring_requeue_work(struct u132 *u132, struct u132_ring *ring,
|
||||
} else if (queue_delayed_work(workqueue, &ring->scheduler, 0))
|
||||
return;
|
||||
kref_put(&u132->kref, u132_hcd_delete);
|
||||
return;
|
||||
}
|
||||
|
||||
static void u132_ring_queue_work(struct u132 *u132, struct u132_ring *ring,
|
||||
@ -324,7 +323,6 @@ static void u132_ring_queue_work(struct u132 *u132, struct u132_ring *ring,
|
||||
{
|
||||
kref_get(&u132->kref);
|
||||
u132_ring_requeue_work(u132, ring, delta);
|
||||
return;
|
||||
}
|
||||
|
||||
static void u132_ring_cancel_work(struct u132 *u132, struct u132_ring *ring)
|
||||
@ -543,7 +541,6 @@ static void u132_hcd_giveback_urb(struct u132 *u132, struct u132_endp *endp,
|
||||
mutex_unlock(&u132->scheduler_lock);
|
||||
u132_endp_put_kref(u132, endp);
|
||||
usb_hcd_giveback_urb(hcd, urb, status);
|
||||
return;
|
||||
}
|
||||
|
||||
static void u132_hcd_forget_urb(struct u132 *u132, struct u132_endp *endp,
|
||||
@ -574,8 +571,8 @@ static void u132_hcd_abandon_urb(struct u132 *u132, struct u132_endp *endp,
|
||||
endp->active = 0;
|
||||
spin_unlock_irqrestore(&endp->queue_lock.slock, irqs);
|
||||
kfree(urbq);
|
||||
} usb_hcd_giveback_urb(hcd, urb, status);
|
||||
return;
|
||||
}
|
||||
usb_hcd_giveback_urb(hcd, urb, status);
|
||||
}
|
||||
|
||||
static inline int edset_input(struct u132 *u132, struct u132_ring *ring,
|
||||
@ -3085,7 +3082,6 @@ static void u132_initialise(struct u132 *u132, struct platform_device *pdev)
|
||||
u132->endp[endps] = NULL;
|
||||
|
||||
mutex_unlock(&u132->sw_lock);
|
||||
return;
|
||||
}
|
||||
|
||||
static int __devinit u132_probe(struct platform_device *pdev)
|
||||
|
@ -398,7 +398,6 @@ void mts_int_submit_urb (struct urb* transfer,
|
||||
context->srb->result = DID_ERROR << 16;
|
||||
mts_transfer_cleanup(transfer);
|
||||
}
|
||||
return;
|
||||
}
|
||||
|
||||
|
||||
@ -409,7 +408,6 @@ static void mts_transfer_cleanup( struct urb *transfer )
|
||||
|
||||
if ( likely(context->final_callback != NULL) )
|
||||
context->final_callback(context->srb);
|
||||
|
||||
}
|
||||
|
||||
static void mts_transfer_done( struct urb *transfer )
|
||||
@ -420,8 +418,6 @@ static void mts_transfer_done( struct urb *transfer )
|
||||
context->srb->result |= (unsigned)(*context->scsi_status)<<1;
|
||||
|
||||
mts_transfer_cleanup(transfer);
|
||||
|
||||
return;
|
||||
}
|
||||
|
||||
|
||||
@ -452,8 +448,6 @@ static void mts_data_done( struct urb* transfer )
|
||||
}
|
||||
|
||||
mts_get_status(transfer);
|
||||
|
||||
return;
|
||||
}
|
||||
|
||||
|
||||
@ -496,8 +490,6 @@ static void mts_command_done( struct urb *transfer )
|
||||
mts_get_status(transfer);
|
||||
}
|
||||
}
|
||||
|
||||
return;
|
||||
}
|
||||
|
||||
static void mts_do_sg (struct urb* transfer)
|
||||
@ -522,7 +514,6 @@ static void mts_do_sg (struct urb* transfer)
|
||||
sg[context->fragment].length,
|
||||
context->fragment + 1 == scsi_sg_count(context->srb) ?
|
||||
mts_data_done : mts_do_sg);
|
||||
return;
|
||||
}
|
||||
|
||||
static const u8 mts_read_image_sig[] = { 0x28, 00, 00, 00 };
|
||||
|
@ -456,7 +456,6 @@ static void ftdi_elan_cancel_targets(struct usb_ftdi *ftdi)
|
||||
static void ftdi_elan_kick_command_queue(struct usb_ftdi *ftdi)
|
||||
{
|
||||
ftdi_command_queue_work(ftdi, 0);
|
||||
return;
|
||||
}
|
||||
|
||||
static void ftdi_elan_command_work(struct work_struct *work)
|
||||
@ -483,7 +482,6 @@ static void ftdi_elan_command_work(struct work_struct *work)
|
||||
static void ftdi_elan_kick_respond_queue(struct usb_ftdi *ftdi)
|
||||
{
|
||||
ftdi_respond_queue_work(ftdi, 0);
|
||||
return;
|
||||
}
|
||||
|
||||
static void ftdi_elan_respond_work(struct work_struct *work)
|
||||
|
@ -90,7 +90,6 @@ static void mon_bus_submit(struct mon_bus *mbus, struct urb *urb)
|
||||
r->rnf_submit(r->r_data, urb);
|
||||
}
|
||||
spin_unlock_irqrestore(&mbus->lock, flags);
|
||||
return;
|
||||
}
|
||||
|
||||
static void mon_submit(struct usb_bus *ubus, struct urb *urb)
|
||||
@ -117,7 +116,6 @@ static void mon_bus_submit_error(struct mon_bus *mbus, struct urb *urb, int erro
|
||||
r->rnf_error(r->r_data, urb, error);
|
||||
}
|
||||
spin_unlock_irqrestore(&mbus->lock, flags);
|
||||
return;
|
||||
}
|
||||
|
||||
static void mon_submit_error(struct usb_bus *ubus, struct urb *urb, int error)
|
||||
|
@ -829,7 +829,6 @@ void musb_g_rx(struct musb *musb, u8 epnum)
|
||||
DBG(3, "packet waiting for %s%s request\n",
|
||||
musb_ep->desc ? "" : "inactive ",
|
||||
musb_ep->end_point.name);
|
||||
return;
|
||||
}
|
||||
|
||||
/* ------------------------------------------------------------ */
|
||||
|
@ -1320,8 +1320,6 @@ continue_read:
|
||||
cypress_set_dead(port);
|
||||
}
|
||||
}
|
||||
|
||||
return;
|
||||
} /* cypress_read_int_callback */
|
||||
|
||||
|
||||
|
@ -2065,7 +2065,6 @@ static void ftdi_set_termios(struct tty_struct *tty,
|
||||
/* lower DTR/RTS */
|
||||
clear_mctrl(port, TIOCM_DTR | TIOCM_RTS);
|
||||
}
|
||||
return;
|
||||
}
|
||||
|
||||
static int ftdi_tiocmget(struct tty_struct *tty, struct file *file)
|
||||
|
@ -1264,7 +1264,6 @@ static void garmin_read_bulk_callback(struct urb *urb)
|
||||
garmin_data_p->flags &= ~FLAGS_BULK_IN_ACTIVE;
|
||||
spin_unlock_irqrestore(&garmin_data_p->lock, flags);
|
||||
}
|
||||
return;
|
||||
}
|
||||
|
||||
|
||||
|
@ -1465,8 +1465,6 @@ static void edge_throttle(struct tty_struct *tty)
|
||||
if (status != 0)
|
||||
return;
|
||||
}
|
||||
|
||||
return;
|
||||
}
|
||||
|
||||
|
||||
@ -1770,8 +1768,6 @@ static void edge_break(struct tty_struct *tty, int break_state)
|
||||
dbg("%s - error sending break set/clear command.",
|
||||
__func__);
|
||||
}
|
||||
|
||||
return;
|
||||
}
|
||||
|
||||
|
||||
@ -2042,7 +2038,6 @@ static void process_rcvd_status(struct edgeport_serial *edge_serial,
|
||||
dbg("%s - Unrecognized IOSP status code %u", __func__, code);
|
||||
break;
|
||||
}
|
||||
return;
|
||||
}
|
||||
|
||||
|
||||
@ -2095,8 +2090,6 @@ static void handle_new_msr(struct edgeport_port *edge_port, __u8 newMsr)
|
||||
|
||||
/* Save the new modem status */
|
||||
edge_port->shadowMSR = newMsr & 0xf0;
|
||||
|
||||
return;
|
||||
}
|
||||
|
||||
|
||||
@ -2143,8 +2136,6 @@ static void handle_new_lsr(struct edgeport_port *edge_port, __u8 lsrData,
|
||||
icount->parity++;
|
||||
if (newLsr & LSR_FRM_ERR)
|
||||
icount->frame++;
|
||||
|
||||
return;
|
||||
}
|
||||
|
||||
|
||||
@ -2720,7 +2711,6 @@ static void change_port_settings(struct tty_struct *tty,
|
||||
baud = tty_termios_baud_rate(old_termios);
|
||||
tty_encode_baud_rate(tty, baud, baud);
|
||||
}
|
||||
return;
|
||||
}
|
||||
|
||||
|
||||
@ -2922,7 +2912,6 @@ static void load_application_firmware(struct edgeport_serial *edge_serial)
|
||||
0x40, 0x4000, 0x0001, NULL, 0, 3000);
|
||||
|
||||
release_firmware(fw);
|
||||
return;
|
||||
}
|
||||
|
||||
|
||||
|
@ -1571,8 +1571,6 @@ static void handle_new_msr(struct edgeport_port *edge_port, __u8 msr)
|
||||
}
|
||||
}
|
||||
tty_kref_put(tty);
|
||||
|
||||
return;
|
||||
}
|
||||
|
||||
static void handle_new_lsr(struct edgeport_port *edge_port, int lsr_data,
|
||||
@ -2424,7 +2422,6 @@ static void change_port_settings(struct tty_struct *tty,
|
||||
dbg("%s - error %d when trying to write config to device",
|
||||
__func__, status);
|
||||
kfree(config);
|
||||
return;
|
||||
}
|
||||
|
||||
static void edge_set_termios(struct tty_struct *tty,
|
||||
@ -2445,7 +2442,6 @@ static void edge_set_termios(struct tty_struct *tty,
|
||||
return;
|
||||
/* change the port settings to the new ones specified */
|
||||
change_port_settings(tty, edge_port, old_termios);
|
||||
return;
|
||||
}
|
||||
|
||||
static int edge_tiocmset(struct tty_struct *tty, struct file *file,
|
||||
|
@ -801,7 +801,6 @@ static void read_rxcmd_callback(struct urb *urb)
|
||||
iuu_uart_read_callback, port);
|
||||
result = usb_submit_urb(port->read_urb, GFP_ATOMIC);
|
||||
dbg("%s - submit result = %d", __func__, result);
|
||||
return;
|
||||
}
|
||||
|
||||
static int iuu_uart_on(struct usb_serial_port *port)
|
||||
|
@ -292,7 +292,6 @@ static void keyspan_pda_rx_unthrottle(struct tty_struct *tty)
|
||||
port->interrupt_in_urb->dev = port->serial->dev;
|
||||
if (usb_submit_urb(port->interrupt_in_urb, GFP_KERNEL))
|
||||
dbg(" usb_submit_urb(read urb) failed");
|
||||
return;
|
||||
}
|
||||
|
||||
|
||||
|
@ -827,7 +827,6 @@ exit:
|
||||
dev_err(&urb->dev->dev,
|
||||
"%s - Error %d submitting control urb\n",
|
||||
__func__, result);
|
||||
return;
|
||||
}
|
||||
|
||||
/*
|
||||
@ -907,7 +906,6 @@ exit:
|
||||
dev_err(&urb->dev->dev,
|
||||
"%s - Error %d submitting control urb\n",
|
||||
__func__, result);
|
||||
return;
|
||||
}
|
||||
|
||||
/*
|
||||
@ -1227,8 +1225,6 @@ static void mos7720_break(struct tty_struct *tty, int break_state)
|
||||
mos7720_port->shadowLCR = data;
|
||||
write_mos_reg(serial, port->number - port->serial->minor,
|
||||
LCR, mos7720_port->shadowLCR);
|
||||
|
||||
return;
|
||||
}
|
||||
|
||||
/*
|
||||
@ -1746,7 +1742,6 @@ static void change_port_settings(struct tty_struct *tty,
|
||||
dbg("usb_submit_urb(read bulk) failed, status = %d",
|
||||
status);
|
||||
}
|
||||
return;
|
||||
}
|
||||
|
||||
/*
|
||||
@ -1803,7 +1798,6 @@ static void mos7720_set_termios(struct tty_struct *tty,
|
||||
dbg("usb_submit_urb(read bulk) failed, status = %d",
|
||||
status);
|
||||
}
|
||||
return;
|
||||
}
|
||||
|
||||
/*
|
||||
|
@ -1355,8 +1355,6 @@ static void mos7840_break(struct tty_struct *tty, int break_state)
|
||||
mos7840_port->shadowLCR);
|
||||
mos7840_set_uart_reg(port, LINE_CONTROL_REGISTER,
|
||||
mos7840_port->shadowLCR);
|
||||
|
||||
return;
|
||||
}
|
||||
|
||||
/*****************************************************************************
|
||||
@ -1587,8 +1585,6 @@ static void mos7840_throttle(struct tty_struct *tty)
|
||||
if (status < 0)
|
||||
return;
|
||||
}
|
||||
|
||||
return;
|
||||
}
|
||||
|
||||
/*****************************************************************************
|
||||
@ -2063,8 +2059,6 @@ static void mos7840_change_port_settings(struct tty_struct *tty,
|
||||
mos7840_port->delta_msr_cond = 1;
|
||||
dbg("mos7840_change_port_settings mos7840_port->shadowLCR is End %x",
|
||||
mos7840_port->shadowLCR);
|
||||
|
||||
return;
|
||||
}
|
||||
|
||||
/*****************************************************************************
|
||||
@ -2133,7 +2127,6 @@ static void mos7840_set_termios(struct tty_struct *tty,
|
||||
mos7840_port->read_urb_busy = false;
|
||||
}
|
||||
}
|
||||
return;
|
||||
}
|
||||
|
||||
/*****************************************************************************
|
||||
|
@ -246,8 +246,6 @@ static void omninet_read_bulk_callback(struct urb *urb)
|
||||
dev_err(&port->dev,
|
||||
"%s - failed resubmitting read urb, error %d\n",
|
||||
__func__, result);
|
||||
|
||||
return;
|
||||
}
|
||||
|
||||
static int omninet_write(struct tty_struct *tty, struct usb_serial_port *port,
|
||||
|
@ -620,8 +620,6 @@ static void sierra_indat_callback(struct urb *urb)
|
||||
dev_err(&port->dev, "resubmit read urb failed."
|
||||
"(%d)\n", err);
|
||||
}
|
||||
|
||||
return;
|
||||
}
|
||||
|
||||
static void sierra_instat_callback(struct urb *urb)
|
||||
|
@ -441,7 +441,6 @@ static void spcp8x5_set_termios(struct tty_struct *tty,
|
||||
spcp8x5_set_workMode(serial->dev, 0x000a,
|
||||
SET_WORKING_MODE_U2C, priv->type);
|
||||
}
|
||||
return;
|
||||
}
|
||||
|
||||
/* open the serial port. do some usb system call. set termios and get the line
|
||||
|
@ -234,7 +234,6 @@ static void usb_wwan_indat_callback(struct urb *urb)
|
||||
}
|
||||
|
||||
}
|
||||
return;
|
||||
}
|
||||
|
||||
static void usb_wwan_outdat_callback(struct urb *urb)
|
||||
|
@ -655,8 +655,6 @@ static void whiteheat_release(struct usb_serial *serial)
|
||||
}
|
||||
kfree(info);
|
||||
}
|
||||
|
||||
return;
|
||||
}
|
||||
|
||||
static int whiteheat_open(struct tty_struct *tty, struct usb_serial_port *port)
|
||||
@ -955,8 +953,6 @@ static void whiteheat_throttle(struct tty_struct *tty)
|
||||
spin_lock_irq(&info->lock);
|
||||
info->flags |= THROTTLED;
|
||||
spin_unlock_irq(&info->lock);
|
||||
|
||||
return;
|
||||
}
|
||||
|
||||
|
||||
@ -975,8 +971,6 @@ static void whiteheat_unthrottle(struct tty_struct *tty)
|
||||
|
||||
if (actually_throttled)
|
||||
rx_data_softint(&info->rx_work);
|
||||
|
||||
return;
|
||||
}
|
||||
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user