From: Marek Vasut <marek.vasut@gmail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 1/1] USB: EHCI: Initialize multiple USB controllers at once
Date: Mon, 30 Jul 2012 14:02:13 +0200 [thread overview]
Message-ID: <201207301402.13546.marek.vasut@gmail.com> (raw)
In-Reply-To: <4B9C9637D5087840A465BDCB251780E9E2D6EDA40C@HKMAIL02.nvidia.com>
Dear Jim Lin,
> >-----Original Message-----
> >From: Marek Vasut [mailto:marek.vasut at gmail.com]
> >Sent: Friday, July 27, 2012 5:26 PM
> >To: u-boot at lists.denx.de
> >Cc: Jim Lin; Wolfgang Denk; Tom Warren
> >Subject: Re: [U-Boot] [PATCH 1/1] USB: EHCI: Initialize multiple USB
> >controllers at once
> >
> >> Wolfgang,
> >> Is there any chance to get this feature in after Stephen explained to
> >> you?
> >
> >I'll poke into it. Can we not get rid of the added ifdef, eg. by setting
> >up the controller number to 1 for those that don't define
> >CONFIG_USB_MULTI now (everyone)
>
> I assume you want me to add these in next patch.
> 1 . Use CONFIG_USB_MULTI, instead of CONFIG_USB_INIT_MULTI
> (current patch),
> 2. If CONFIG_USB_MULTI is not defined, then define
> CONFIG_USB_MAX_CONTROLLER_COUNT as 1.
Yes, is that a problem? Aka. have this always enabled and don't treat not having
multiple controllers as a special case.
> > and adding some wrapping goo?
>
> Don't understand what you meant.
> Could you provide an example?
Just don't treat having 1 controller as a special case, that's all I meant.
> Thanks.
>
> nvpublic
Best regards,
Marek Vasut
next prev parent reply other threads:[~2012-07-30 12:02 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-06-26 10:15 [U-Boot] [PATCH 1/1] USB: EHCI: Initialize multiple USB controllers at once Jim Lin
2012-06-26 21:29 ` Wolfgang Denk
[not found] ` <4B9C9637D5087840A465BDCB251780E9E2D6EDA3EC@HKMAIL02.nvidia.com>
[not found] ` <4B9C9637D5087840A465BDCB251780E9E2D6EDA3F0@HKMAIL02.nvidia.com>
[not found] ` <4B9C9637D5087840A465BDCB251780E9E2D6EDA3F1@HKMAIL02.nvidia.com>
[not found] ` <4B9C9637D5087840A465BDCB251780E9E2D6EDA3F2@HKMAIL02.nvidia.com>
[not found] ` <4B9C9637D5087840A465BDCB251780E9E2D6EDA3F5@HKMAIL02.nvidia.com>
2012-06-27 6:55 ` Wolfgang Denk
2012-06-27 16:57 ` Stephen Warren
2012-07-02 23:29 ` Jim Lin
2012-07-27 2:47 ` Jim Lin
2012-07-27 9:25 ` Marek Vasut
2012-07-30 7:20 ` Jim Lin
2012-07-30 12:02 ` Marek Vasut [this message]
2012-07-31 4:07 ` Jim Lin
2012-07-31 5:31 ` Marek Vasut
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=201207301402.13546.marek.vasut@gmail.com \
--to=marek.vasut@gmail.com \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
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