Re: [PATCH] usb: ohci: Allow ohci on omap5 also

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

 



On Fri, 9 Sep 2016, Alan Stern wrote:

> On Fri, 9 Sep 2016, Tony Lindgren wrote:
> 
> > * Alan Stern <stern@xxxxxxxxxxxxxxxxxxx> [160909 13:41]:
> > > On Fri, 9 Sep 2016, Tony Lindgren wrote:
> > > 
> > > > * Alan Stern <stern@xxxxxxxxxxxxxxxxxxx> [160909 12:47]:
> > > > > You know, as far as I can tell the only thing that is OMAP-specific
> > > > > about the ohci-omap3 driver is the initialization of OHCI_CTRL_RWC.  
> > > > > This could be added to the platform data or the DT bindings, after
> > > > > which this driver wouldn't be needed at all -- the ohci_platform driver
> > > > > would work.  Does this seem reasonable?
> > > > 
> > > > Sure makes sense to me. Do we have some standard binding for
> > > > OCHI_CTRL_RWC?
> > > 
> > > No, one would have to be created.
> > 
> > OK I can do a patch for that, what should be binding be?
> > 
> > ohci-ctrl-rwc-something?
> 
> OHCI_CTRL_RWC is just a single-bit flag; the RWC part stands for
> "Remote Wakeup Control".

Sorry, careless mistake on my part.  It actually stands for "Remote 
Wakeup Connected"; meaning that the remote-wakeup output signal from 
the controller is connected to something really can wake up the system.
This is exactly the sort of hardware implementation detail that the 
firmware is supposed to know.

Alan Stern

>  The flag indicates whether the controller
> hardware is able to wake up the system, and it is supposed to be set
> initially by the firmware during boot-up.  Under DT, the driver will
> have to set it manually.
> 
> So you could call it ohci-ctrl-rwc-on, or ohci-ctrl-rwc-enable, or 
> anything similar.  Use your own judgment.
> 
> 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