>>>>> "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