On Fri, Nov 16, 2012 at 03:22:34PM +0200, Alexander Shishkin wrote: > Michael Grzeschik <m.grzeschik@xxxxxxxxxxxxxx> writes: > > > This adds mx53 as the next user of the usbmisc driver and makes it > > possible to disable the overcurrent-detection of the internal phy. > > > > Signed-off-by: Michael Grzeschik <m.grzeschik@xxxxxxxxxxxxxx> > > Signed-off-by: Marc Kleine-Budde <mkl@xxxxxxxxxxxxxx> > > --- > > drivers/usb/chipidea/usbmisc_imx.c | 28 ++++++++++++++++++++++++++++ > > 1 file changed, 28 insertions(+) > > > > diff --git a/drivers/usb/chipidea/usbmisc_imx.c b/drivers/usb/chipidea/usbmisc_imx.c > > index 913e414..9718500 100644 > > --- a/drivers/usb/chipidea/usbmisc_imx.c > > +++ b/drivers/usb/chipidea/usbmisc_imx.c > > @@ -54,6 +54,29 @@ static struct usbmisc_usb_device *get_usbdev(struct device *dev) > > return &usbmisc->usbdev[i]; > > } > > > > +static int usbmisc_imx53_init(struct device *dev) > > +{ > > + struct usbmisc_usb_device *usbdev; > > + void __iomem *reg; > > + unsigned long flags; > > + u32 val; > > + > > + usbdev = get_usbdev(dev); > > + if (IS_ERR(usbdev)) > > + return PTR_ERR(usbdev); > > + > > + reg = usbmisc->base + usbdev->index * 8; > > + > > + if (usbdev->disable_oc) { > > + spin_lock_irqsave(&usbmisc->lock, flags); > > + val = readl(reg); > > + writel(val | (1 << 5), reg); > > We know now that (1 << 5) probably means "disable overcurrent > detection", but is there a definition for this register's bits > somewhere? Will add defines for the magic numbers used. Thanks, Michael -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | -- 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