Re: [PATCH 2/6] USB: OHCI: make ohci-omap a separate driver

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

 



On Wed, 24 Jul 2013, Manjunath Goudar wrote:

> > I suspect there's a mistake here, and the omap_ohci_clock_power() call
> > perhaps should be moved after the "err3:" label.  But that mistake (if
> > it is a mistake) was present in the original code, and this patch
> > shouldn't change it.
> >
> 
>  Before omap_ohci_clock_power(0) was calling dependence on host_enabled
>  value was set or not. We rid the of host_enabled.

I know.  I was the person who told you to do that.

> Now my approach is as below, to avoid the omap_ohci_clock_power(0) issue:
> 
>     retval = usb_add_hcd(hcd, irq, 0);
>  +       if (retval == 0)
>     +            return retval;
> 
> /* I thought  here clock disable is required before any of
> errX statement execute
>     please comment on this*/
>    + omap_ohci_clock_power(0);
> err3:
>        iounmap(hcd->regs);
> 
> What about your suggestion?

I don't know.  You should ask the person who wrote this driver 
originally.

Alan Stern

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