lkml.org 
[lkml]   [2014]   [Jul]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v2 1/3] usb: host: st-hcd: Add USB HCD support for STi SoCs
Hi Arnd,


<snip>
> >
> > I'm of course happy to change it if required. I see looking through that a lot
> > of other platforms do it the way you describe with a
> >
> > ehci-<platname>.c and ohci-<platname>.c
>
> Right. We are trying to gradually move some of them over to use the
> generic *hci-platform.c drivers though.

Right, ok I understand.

> > > Depending on what kind of special requirements the ST version has,
> > > this can be done either by using the generic ohci/ehci bindings
> > > with extensions where necessary, or by creating a new binding and
> > > new driver files that use 'ohci_init_driver'/'ehci_init_driver'
> > > to inherit from the common code.
> > >
> > > The first of the two approaches is preferred.
> >
> > We don't have anything particularly special, just a couple of reset lines,
> > clock, phy, etc.
>
> Ok, good. Please see Documentation/devicetree/bindings/usb/usb-?hci.txt
> then. You might actually be able to just use the existing drivers
> without new code by just adding the proper DT nodes that follow these
> bindings.

The only issues I can see with the generic versions are: -
1) We also have a powerdown line in addition to the reset line both of which are
exposed via reset controller API, so I would need to add that into ehci-platform
and ohci-platform.

2) We have a magic poke in st_ehci_configure_bus of the current driver to configure
the AHB to ST bus protocol convertor IP. I'm not quite sure where I could hook that
in (sorry... slightly pulling back on my "nothing special comment" a bit ;-).

3) We also set the rate of the ohci clock, which the generic driver doesn't do.

regards,

Peter.


\
 
 \ /
  Last update: 2014-07-24 17:01    [W:0.058 / U:6.376 seconds]
©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site