On Wednesday 27 October 2004 02:54, Ben Dooks wrote:
> On Tue, Oct 19, 2004 at 03:33:52PM -0700, David Brownell wrote:
> > On Tuesday 19 October 2004 2:14 pm, Ben Dooks wrote:
> > > > If you're up for it, I'd rather see a generic hook for
> > > > turning port N power on/off ... just hang a method in
> > > > the "struct ohci_hcd", and when it's there ohci-hub.c
> > > > would call it.  It's not just S3C2410 boards needing
> > > > custom power switching (and overcurrent) handlers!
> > > 
> > > ...
> 
> I was giving this some more thought, and would it be better to
> do an new file with an helper functions to allow an ohci
> implementation to override hc_driver.hub_status_data and
> hc_driver.hub_control?

I'm not sure I follow. Why a new file?  If that's how you
want to plug things in, it's easy enough ... ohci-s3c2410.c
(or whatever!) provides the hc_driver structure, so it can
just fill in versions of those methods that switch on the
request, delegating all but a few to the generic code.


> I'd be happy with this way, as it would be less intrusive
> into the usb hub code, and (iicc) should be as good as
> changing the hub code itself.

It was also the way you were already doing things, ISTR!
No problem.

- Dave


-------------------------------------------------------
This SF.Net email is sponsored by:
Sybase ASE Linux Express Edition - download now for FREE
LinuxWorld Reader's Choice Award Winner for best database on Linux.
http://ads.osdn.com/?ad_id=5588&alloc_id=12065&op=click
_______________________________________________
[EMAIL PROTECTED]
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel

Reply via email to