From: Kumar Gala <galak@kernel.crashing.org>
To: David Brownell <david-b@pacbell.net>
Cc: linux-usb-devel@lists.sourceforge.net, Greg KH <gregkh@suse.de>,
LKML mailing list <linux-kernel@vger.kernel.org>
Subject: Re: [linux-usb-devel] compile error when building multiple EHCI host controllers as modules
Date: Mon, 3 Apr 2006 23:23:29 -0500 [thread overview]
Message-ID: <074E7D37-773D-4203-BB09-20040C5D5D5B@kernel.crashing.org> (raw)
In-Reply-To: <200604031948.38101.david-b@pacbell.net>
On Apr 3, 2006, at 9:48 PM, David Brownell wrote:
> On Friday 24 March 2006 12:32 pm, Kumar Gala wrote:
>>> The issue I have this is that it makes two (or more) things that
>>> were
>>> independent now dependent. What about just moving the module_init/
>>> exit() functions into files that are built separately. For the
>>> ehci-
>>> fsl case it was trivial, need to look at ehci-pci case.
>>
>> Ok, my idea required exporting things I didn't really want to
>> export, so
>> what about something like this or where you thinking of some more
>> sophisticated?
>>
>> If this is good, I'll do the same for ohci.
>
> How about this one instead? It requires fewer per-SOC hacks in
> generic
> code when adding a new SOC. And it also removes a platform device
> naming
> goof for your mpc83xx support ... that's a case where you should
> just let
> the platform device IDs distinguish things.
Let me test this patch out. I'm ok with the changes for handling
both PCI and platform driver. However, I need to take a look at the
renaming of the fsl driver. The "dr" device supports device and OTG
modes. I'm concerned about how we distinguish that in the future.
(also, we need to fixup arch/powerpc/sysdev/fsl_soc.c)
- k
next prev parent reply other threads:[~2006-04-04 4:23 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-03-23 22:26 Kumar Gala
2006-03-24 6:33 ` [linux-usb-devel] " David Brownell
2006-03-24 16:59 ` Kumar Gala
2006-03-24 20:32 ` Kumar Gala
2006-03-28 16:18 ` Kumar Gala
2006-03-28 17:51 ` Matthieu CASTET
2006-04-04 2:48 ` [linux-usb-devel] " David Brownell
2006-04-04 4:23 ` Kumar Gala [this message]
2006-04-05 17:32 ` Kumar Gala
2006-04-10 19:11 ` David Brownell
2006-04-10 19:16 ` Kumar Gala
2006-04-11 15:07 ` [PATCH] usb: allow multiple types of EHCI controllers to be built " Kumar Gala
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=074E7D37-773D-4203-BB09-20040C5D5D5B@kernel.crashing.org \
--to=galak@kernel.crashing.org \
--cc=david-b@pacbell.net \
--cc=gregkh@suse.de \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-usb-devel@lists.sourceforge.net \
/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
all inboxes | Powered by JetHome®