From: Pavel Roskin <proski@gnu.org>
To: David Gibson <hermes@gibson.dropbear.id.au>
Cc: Faidon Liambotis <paravoid@debian.org>, linux-wireless@vger.kernel.org
Subject: Re: [PATCH 2.6.23 3/3] [wireless] orinoco: create a Kconfig option for Prism2
Date: Mon, 23 Jul 2007 02:10:40 -0400 [thread overview]
Message-ID: <1185171040.2108.46.camel@dv> (raw)
In-Reply-To: <20070723014210.GB3272@localhost.localdomain>
On Mon, 2007-07-23 at 11:42 +1000, David Gibson wrote:
> > Create an option to disable support for Prism devices in orinoco driver which
> > disabled the IDs for the PCMCIA module and creates a Kconfig dependency for
> > the Prism PCI modules.
>
> I very much dislike this being a compile time only CONFIG option. A
> module parameter which enables or disables the device IDs would be
> much nicer, if possible. It can default to off (orinoco will not
> claim Prism cards).
I'm afraid it's not possible. There is only one device map in the
driver, and it's read by depmod to generate module map tables.
If the run-time device table is different from the compile-time table,
this would be really confusing. Either the module would be loaded to
support the device but would refuse to do so, or the module would need
to be loaded manually for some devices, even if no other module is
available to support the device.
Besides, orinoco_pci would become useless by default.
Please see my message. For TMD bridges, we don't know the firmware
flavor before we know the firmware version. For PLX bridges, we can
read the CIS first and match the PCMCIA table, but it would need to be
done manually, without the PCMCIA code.
I don't see any clear solution without some kind of the device handover
from Orinoco to hostap or vice versa.
I would probably start with cutting Agere and Symbol devices from
hostap, which is a much more clear-cut case than cutting working Prism
support from Orinoco.
--
Regards,
Pavel Roskin
next prev parent reply other threads:[~2007-07-23 6:10 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-07-22 13:17 [PATCH 2.6.23 3/3] [wireless] orinoco: create a Kconfig option for Prism2 Faidon Liambotis
2007-07-22 17:14 ` Pavel Roskin
2007-07-23 22:07 ` Faidon Liambotis
2007-07-25 5:30 ` Pavel Roskin
2007-07-25 6:19 ` Faidon Liambotis
2007-07-25 16:32 ` Pavel Roskin
2007-08-01 3:33 ` Jouni Malinen
2007-08-01 5:55 ` Pavel Roskin
2007-07-23 1:42 ` David Gibson
2007-07-23 6:10 ` Pavel Roskin [this message]
2007-07-23 6:20 ` David Gibson
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=1185171040.2108.46.camel@dv \
--to=proski@gnu.org \
--cc=hermes@gibson.dropbear.id.au \
--cc=linux-wireless@vger.kernel.org \
--cc=paravoid@debian.org \
/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;
as well as URLs for NNTP newsgroup(s).