public inbox for linuxppc-dev@ozlabs.org 
 help / color / mirror / Atom feed
From: Peter Korsgaard <jacmet@sunsite•dk>
To: David Brownell <david-b@pacbell•net>
Cc: linuxppc-dev@ozlabs•org, linux-usb@vger•kernel.org
Subject: Re: [patch v4 0/4] Cypress c67x00 (EZ-Host/EZ-OTG) support
Date: Wed, 23 Jan 2008 22:20:32 +0100	[thread overview]
Message-ID: <873asofdcv.fsf@macbook.be.48ers.dk> (raw)
In-Reply-To: <200801230953.05921.david-b@pacbell.net> (David Brownell's message of "Wed\, 23 Jan 2008 09\:53\:05 -0800")

>>>>> "David" =3D=3D David Brownell <david-b@pacbell•net> writes:

 David> On Wednesday 23 January 2008, Grant Likely wrote:
 >> The question is about the device structure which used to be provided
 >> by the platform device instances and now there just uses the c67x00's
 >> device struct. =C2=A0I was under the impression that each USB HCD needs=
 to
 >> have it's own struct device. =C2=A0I take it that's not true?

 David> Each root hub necessarily is a unique device, representing a set
 David> of downstream links.  Unless Peter didn't test something relevant,
 David> it would seem we have observational proof that two root hubs can
 David> share the same device node for an upstream link.

I haven't actually tested a setup with HCDs on both SIEs as I don't
have a board with host connectors on both, but I've done the change
based on your feedback:

http://article.gmane.org/gmane.linux.usb.devel/53496

 David> I can't think of a reason to demand multiple upstream links, though
 David> sharing them between root hubs like that isn't a common structure.=
=20=20

 David> - Dave


--=20
Bye, Peter Korsgaard

  reply	other threads:[~2008-01-23 21:20 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-01-21 10:34 [patch v4 0/4] Cypress c67x00 (EZ-Host/EZ-OTG) support Peter Korsgaard
2008-01-21 10:34 ` [patch v4 1/4] USB: add Cypress c67x00 low level interface code Peter Korsgaard
2008-01-21 10:34 ` [patch v4 2/4] USB: add Cypress c67x00 OTG controller core driver Peter Korsgaard
2008-01-21 10:34 ` [patch v4 3/4] USB: add Cypress c67x00 OTG controller HCD driver Peter Korsgaard
2008-01-21 10:34 ` [patch v4 4/4] USB: add Cypress c67x00 OTG controller gadget driver Peter Korsgaard
2008-01-21 17:11 ` [patch v4 0/4] Cypress c67x00 (EZ-Host/EZ-OTG) support Grant Likely
2008-01-21 20:16   ` Peter Korsgaard
2008-01-21 20:51     ` David Brownell
2008-01-21 20:53       ` Grant Likely
2008-01-21 20:01 ` David Brownell
2008-01-21 20:14   ` Grant Likely
2008-01-21 21:13     ` Peter Korsgaard
2008-01-21 21:16     ` Peter Korsgaard
2008-01-23 17:12       ` Grant Likely
2008-01-23 17:53         ` David Brownell
2008-01-23 21:20           ` Peter Korsgaard [this message]
2008-01-28 20:40           ` Grant Likely
2008-01-28 21:01             ` Peter Korsgaard
2008-01-23 21:18         ` Peter Korsgaard
2008-01-21 21:08   ` Peter Korsgaard

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=873asofdcv.fsf@macbook.be.48ers.dk \
    --to=jacmet@sunsite$(echo .)dk \
    --cc=david-b@pacbell$(echo .)net \
    --cc=linux-usb@vger$(echo .)kernel.org \
    --cc=linuxppc-dev@ozlabs$(echo .)org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox