Re: [PATCH v2 05/11] usb: musb: move musb_start to musb_virthub.c

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



>>>>> "Daniel" == Daniel Mack <zonque@xxxxxxxxx> writes:

 Daniel> This function has its only user in musb_virthub.c, so let's move it
 Daniel> there and make it static.

 Daniel> Signed-off-by: Daniel Mack <zonque@xxxxxxxxx>

Acked-by: Peter Korsgaard <jacmet@xxxxxxxxxx>

 Daniel> ---
 Daniel>  drivers/usb/musb/musb_core.c    | 45 -----------------------------------------
 Daniel>  drivers/usb/musb/musb_core.h    |  1 -
 Daniel>  drivers/usb/musb/musb_virthub.c | 44 ++++++++++++++++++++++++++++++++++++++++
 Daniel>  3 files changed, 44 insertions(+), 46 deletions(-)

 Daniel> diff --git a/drivers/usb/musb/musb_core.c b/drivers/usb/musb/musb_core.c
 Daniel> index 13341c6..adf069d 100644
 Daniel> --- a/drivers/usb/musb/musb_core.c
 Daniel> +++ b/drivers/usb/musb/musb_core.c
 Daniel> @@ -905,51 +905,6 @@ b_host:
 
 Daniel>  /*-------------------------------------------------------------------------*/
 
 Daniel> -/*
 Daniel> -* Program the HDRC to start (enable interrupts, dma, etc.).
 Daniel> -*/
 Daniel> -void musb_start(struct musb *musb)
 Daniel> -{
 Daniel> -	void __iomem	*regs = musb->mregs;
 Daniel> -	u8		devctl = musb_readb(regs, MUSB_DEVCTL);
 Daniel> -
 Daniel> -	dev_dbg(musb->controller, "<== devctl %02x\n", devctl);
 Daniel> -
 Daniel> -	/*  Set INT enable registers, enable interrupts */
 Daniel> -	musb->intrtxe = musb->epmask;
 Daniel> -	musb_writew(regs, MUSB_INTRTXE, musb->intrtxe);
 Daniel> -	musb->intrrxe = musb->epmask & 0xfffe;
 Daniel> -	musb_writew(regs, MUSB_INTRRXE, musb->intrrxe);
 Daniel> -	musb_writeb(regs, MUSB_INTRUSBE, 0xf7);
 Daniel> -
 Daniel> -	musb_writeb(regs, MUSB_TESTMODE, 0);
 Daniel> -
 Daniel> -	/* put into basic highspeed mode and start session */
 Daniel> -	musb_writeb(regs, MUSB_POWER, MUSB_POWER_ISOUPDATE
 Daniel> -						| MUSB_POWER_HSENAB
 Daniel> -						/* ENSUSPEND wedges tusb */
 Daniel> -						/* | MUSB_POWER_ENSUSPEND */
 Daniel> -						);
 Daniel> -
 Daniel> -	musb->is_active = 0;
 Daniel> -	devctl = musb_readb(regs, MUSB_DEVCTL);
 Daniel> -	devctl &= ~MUSB_DEVCTL_SESSION;
 Daniel> -
 Daniel> -	/* session started after:
 Daniel> -	 * (a) ID-grounded irq, host mode;
 Daniel> -	 * (b) vbus present/connect IRQ, peripheral mode;
 Daniel> -	 * (c) peripheral initiates, using SRP
 Daniel> -	 */
 Daniel> -	if ((devctl & MUSB_DEVCTL_VBUS) == MUSB_DEVCTL_VBUS)
 Daniel> -		musb->is_active = 1;
 Daniel> -	else
 Daniel> -		devctl |= MUSB_DEVCTL_SESSION;
 Daniel> -
 Daniel> -	musb_platform_enable(musb);
 Daniel> -	musb_writeb(regs, MUSB_DEVCTL, devctl);
 Daniel> -}
 Daniel> -
 Daniel> -
 Daniel>  static void musb_generic_disable(struct musb *musb)
 Daniel>  {
 Daniel>  	void __iomem	*mbase = musb->mregs;
 Daniel> diff --git a/drivers/usb/musb/musb_core.h b/drivers/usb/musb/musb_core.h
 Daniel> index 04d8974..87da041 100644
 Daniel> --- a/drivers/usb/musb/musb_core.h
 Daniel> +++ b/drivers/usb/musb/musb_core.h
 Daniel> @@ -503,7 +503,6 @@ static inline void musb_configure_ep0(struct musb *musb)
 
 Daniel>  extern const char musb_driver_name[];
 
 Daniel> -extern void musb_start(struct musb *musb);
 Daniel>  extern void musb_stop(struct musb *musb);
 
 Daniel>  extern void musb_write_fifo(struct musb_hw_ep *ep, u16 len, const u8 *src);
 Daniel> diff --git a/drivers/usb/musb/musb_virthub.c b/drivers/usb/musb/musb_virthub.c
 Daniel> index ef7d110..1e17c20 100644
 Daniel> --- a/drivers/usb/musb/musb_virthub.c
 Daniel> +++ b/drivers/usb/musb/musb_virthub.c
 Daniel> @@ -44,6 +44,50 @@
 
 Daniel>  #include "musb_core.h"
 
 Daniel> +/*
 Daniel> +* Program the HDRC to start (enable interrupts, dma, etc.).
 Daniel> +*/
 Daniel> +static void musb_start(struct musb *musb)
 Daniel> +{
 Daniel> +	void __iomem	*regs = musb->mregs;
 Daniel> +	u8		devctl = musb_readb(regs, MUSB_DEVCTL);
 Daniel> +
 Daniel> +	dev_dbg(musb->controller, "<== devctl %02x\n", devctl);
 Daniel> +
 Daniel> +	/*  Set INT enable registers, enable interrupts */
 Daniel> +	musb->intrtxe = musb->epmask;
 Daniel> +	musb_writew(regs, MUSB_INTRTXE, musb->intrtxe);
 Daniel> +	musb->intrrxe = musb->epmask & 0xfffe;
 Daniel> +	musb_writew(regs, MUSB_INTRRXE, musb->intrrxe);
 Daniel> +	musb_writeb(regs, MUSB_INTRUSBE, 0xf7);
 Daniel> +
 Daniel> +	musb_writeb(regs, MUSB_TESTMODE, 0);
 Daniel> +
 Daniel> +	/* put into basic highspeed mode and start session */
 Daniel> +	musb_writeb(regs, MUSB_POWER, MUSB_POWER_ISOUPDATE
 Daniel> +						| MUSB_POWER_HSENAB
 Daniel> +						/* ENSUSPEND wedges tusb */
 Daniel> +						/* | MUSB_POWER_ENSUSPEND */
 Daniel> +						);
 Daniel> +
 Daniel> +	musb->is_active = 0;
 Daniel> +	devctl = musb_readb(regs, MUSB_DEVCTL);
 Daniel> +	devctl &= ~MUSB_DEVCTL_SESSION;
 Daniel> +
 Daniel> +	/* session started after:
 Daniel> +	 * (a) ID-grounded irq, host mode;
 Daniel> +	 * (b) vbus present/connect IRQ, peripheral mode;
 Daniel> +	 * (c) peripheral initiates, using SRP
 Daniel> +	 */
 Daniel> +	if ((devctl & MUSB_DEVCTL_VBUS) == MUSB_DEVCTL_VBUS) {
 Daniel> +		musb->is_active = 1;
 Daniel> +	} else {
 Daniel> +		devctl |= MUSB_DEVCTL_SESSION;
 Daniel> +	}
 Daniel> +
 Daniel> +	musb_platform_enable(musb);
 Daniel> +	musb_writeb(regs, MUSB_DEVCTL, devctl);
 Daniel> +}
 
 Daniel>  static void musb_port_suspend(struct musb *musb, bool do_suspend)
 Daniel>  {
 Daniel> -- 
 Daniel> 1.8.1.4



-- 
Bye, Peter Korsgaard
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux