* ipw2100 wireless driver
@ 2004-07-14 11:41 Pavel Machek
2004-07-14 11:48 ` Jeff Chua
` (2 more replies)
0 siblings, 3 replies; 31+ messages in thread
From: Pavel Machek @ 2004-07-14 11:41 UTC (permalink / raw)
To: netdev, kernel list
Hi!
What is the status of ipw2100? Is there chance that it would be pushed
into mainline?
I have few problems with that:
* it will not compile with gcc-2.95. Attached patch fixes one problem
but more remain.
--- ipw2100-ofic/ieee80211.h 2004-07-09 06:32:17.000000000 +0200
+++ ipw2100-0.49/ieee80211.h 2004-07-14 13:18:50.000000000 +0200
@@ -440,7 +440,7 @@
u16 reserved;
u16 frag_size;
u16 payload_size;
- struct sk_buff *fragments[];
+ struct sk_buff *fragments[1];
};
extern struct ieee80211_txb *ieee80211_skb_to_txb(struct ieee80211_device *ieee,
* it requires CONFIG_CRYPTO, but fails to force it in Kconfig.
Pavel
--
People were complaining that M$ turns users into beta-testers...
...jr ghea gurz vagb qrirybcref, naq gurl frrz gb yvxr vg gung jnl!
^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: ipw2100 wireless driver
2004-07-14 11:41 ipw2100 wireless driver Pavel Machek
@ 2004-07-14 11:48 ` Jeff Chua
2004-07-14 11:55 ` Pavel Machek
2004-07-14 12:30 ` Tomas Szepe
2004-07-14 12:27 ` Pavel Machek
2004-07-14 13:15 ` Vojtech Pavlik
2 siblings, 2 replies; 31+ messages in thread
From: Jeff Chua @ 2004-07-14 11:48 UTC (permalink / raw)
To: Pavel Machek; +Cc: netdev, kernel list
On Wed, 14 Jul 2004, Pavel Machek wrote:
> Hi!
>
> What is the status of ipw2100? Is there chance that it would be pushed
> into mainline?
>
> I have few problems with that:
>
> * it will not compile with gcc-2.95. Attached patch fixes one problem
> but more remain.
I've given up hope on that. Don't think it'll ever compile on 2.95. I'm
using ndiswrapper and it works nicely.
Jeff.
^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: ipw2100 wireless driver
2004-07-14 11:48 ` Jeff Chua
@ 2004-07-14 11:55 ` Pavel Machek
2004-08-09 20:15 ` Tomas Szepe
2004-07-14 12:30 ` Tomas Szepe
1 sibling, 1 reply; 31+ messages in thread
From: Pavel Machek @ 2004-07-14 11:55 UTC (permalink / raw)
To: Jeff Chua; +Cc: netdev, kernel list
Hi!
> >What is the status of ipw2100? Is there chance that it would be pushed
> >into mainline?
> >
> >I have few problems with that:
> >
> >* it will not compile with gcc-2.95. Attached patch fixes one problem
> >but more remain.
>
> I've given up hope on that. Don't think it'll ever compile on 2.95. I'm
> using ndiswrapper and it works nicely.
No, I think that can be fixed... I'll rather fix ipw2100 than use
ndiswrapper.
Pavel
--
People were complaining that M$ turns users into beta-testers...
...jr ghea gurz vagb qrirybcref, naq gurl frrz gb yvxr vg gung jnl!
^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: ipw2100 wireless driver
2004-07-14 11:41 ipw2100 wireless driver Pavel Machek
2004-07-14 11:48 ` Jeff Chua
@ 2004-07-14 12:27 ` Pavel Machek
2004-07-14 13:15 ` Vojtech Pavlik
2 siblings, 0 replies; 31+ messages in thread
From: Pavel Machek @ 2004-07-14 12:27 UTC (permalink / raw)
To: netdev, kernel list
Hi!
> What is the status of ipw2100? Is there chance that it would be pushed
> into mainline?
Hmm, it has very "nice" interface in proc, where it tells you (in
english *sentences*) if radio kill switch is enabled or not. Ouch.
iwconfig eth1 mode ad-hoc
followed by
iwconfig eth1
makes it crash in ipw2100_wx_get_power called from
wireless_process_ioctl.
Pavel
--
People were complaining that M$ turns users into beta-testers...
...jr ghea gurz vagb qrirybcref, naq gurl frrz gb yvxr vg gung jnl!
^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: ipw2100 wireless driver
2004-07-14 11:48 ` Jeff Chua
2004-07-14 11:55 ` Pavel Machek
@ 2004-07-14 12:30 ` Tomas Szepe
1 sibling, 0 replies; 31+ messages in thread
From: Tomas Szepe @ 2004-07-14 12:30 UTC (permalink / raw)
To: Jeff Chua; +Cc: Pavel Machek, netdev, kernel list
On Jul-14 2004, Wed, 19:48 +0800
Jeff Chua <jeffchua@silk.corp.fedex.com> wrote:
> >I have few problems with that:
> >
> >* it will not compile with gcc-2.95. Attached patch fixes one problem
> >but more remain.
>
> I've given up hope on that. Don't think it'll ever compile on 2.95. I'm
> using ndiswrapper and it works nicely.
I'd rather fiddle around with gcc versions than risk stack overflows
from running Windows drivers in Linux kernel space.
--
Tomas Szepe <szepe@pinerecords.com>
^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: ipw2100 wireless driver
2004-07-14 11:41 ipw2100 wireless driver Pavel Machek
2004-07-14 11:48 ` Jeff Chua
2004-07-14 12:27 ` Pavel Machek
@ 2004-07-14 13:15 ` Vojtech Pavlik
2004-07-14 13:42 ` Pavel Machek
2 siblings, 1 reply; 31+ messages in thread
From: Vojtech Pavlik @ 2004-07-14 13:15 UTC (permalink / raw)
To: Pavel Machek; +Cc: netdev, kernel list
On Wed, Jul 14, 2004 at 01:41:35PM +0200, Pavel Machek wrote:
> Hi!
>
> What is the status of ipw2100? Is there chance that it would be pushed
> into mainline?
>
> I have few problems with that:
>
> * it will not compile with gcc-2.95. Attached patch fixes one problem
> but more remain.
Wouldn't "struct sk_buff **fragments" be a more correct fix?
> --- ipw2100-ofic/ieee80211.h 2004-07-09 06:32:17.000000000 +0200
> +++ ipw2100-0.49/ieee80211.h 2004-07-14 13:18:50.000000000 +0200
> @@ -440,7 +440,7 @@
> u16 reserved;
> u16 frag_size;
> u16 payload_size;
> - struct sk_buff *fragments[];
> + struct sk_buff *fragments[1];
> };
>
> extern struct ieee80211_txb *ieee80211_skb_to_txb(struct ieee80211_device *ieee,
>
> * it requires CONFIG_CRYPTO, but fails to force it in Kconfig.
>
> Pavel
> --
> People were complaining that M$ turns users into beta-testers...
> ...jr ghea gurz vagb qrirybcref, naq gurl frrz gb yvxr vg gung jnl!
> -
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
>
--
Vojtech Pavlik
SuSE Labs, SuSE CR
^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: ipw2100 wireless driver
2004-07-14 13:15 ` Vojtech Pavlik
@ 2004-07-14 13:42 ` Pavel Machek
0 siblings, 0 replies; 31+ messages in thread
From: Pavel Machek @ 2004-07-14 13:42 UTC (permalink / raw)
To: Vojtech Pavlik; +Cc: netdev, kernel list
Hi!
> > What is the status of ipw2100? Is there chance that it would be pushed
> > into mainline?
> >
> > I have few problems with that:
> >
> > * it will not compile with gcc-2.95. Attached patch fixes one problem
> > but more remain.
>
> Wouldn't "struct sk_buff **fragments" be a more correct fix?
Yep, that would certainly be better. I'll wait if I get some reply
from ipw2100 people. If so I'll update the patch.
Pavel
--
People were complaining that M$ turns users into beta-testers...
...jr ghea gurz vagb qrirybcref, naq gurl frrz gb yvxr vg gung jnl!
^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: ipw2100 wireless driver
2004-07-14 11:55 ` Pavel Machek
@ 2004-08-09 20:15 ` Tomas Szepe
2004-08-10 5:02 ` Jeff Chua
0 siblings, 1 reply; 31+ messages in thread
From: Tomas Szepe @ 2004-08-09 20:15 UTC (permalink / raw)
To: Pavel Machek; +Cc: Jeff Chua, netdev, kernel list
On Jul-14 2004, Wed, 13:55 +0200
Pavel Machek <pavel@suse.cz> wrote:
> > >What is the status of ipw2100? Is there chance that it would be pushed
> > >into mainline?
> > >
> > >I have few problems with that:
> > >
> > >* it will not compile with gcc-2.95. Attached patch fixes one problem
> > >but more remain.
> >
> > I've given up hope on that. Don't think it'll ever compile on 2.95. I'm
> > using ndiswrapper and it works nicely.
>
> No, I think that can be fixed... I'll rather fix ipw2100 than use
> ndiswrapper.
ipw2100 0.51 from ipw2100.sf.net builds using gcc-2.95.3 "out of the box."
Also make sure to use the attached patch for 2.6.8pre.
--
Tomas Szepe <szepe@pinerecords.com>
diff -urN linux-2.6.7/drivers/net/wireless/ipw2100/ipw2100_fw.c linux-2.6.7.x/drivers/net/wireless/ipw2100/ipw2100_fw.c
--- linux-2.6.7/drivers/net/wireless/ipw2100/ipw2100_fw.c 2004-08-09 21:37:11.000000000 +0200
+++ linux-2.6.7.x/drivers/net/wireless/ipw2100/ipw2100_fw.c 2004-08-09 21:36:15.000000000 +0200
@@ -200,7 +200,7 @@
goto fail;
}
- if (read(fd, c->buf, c->len) != c->len) {
+ if (sys_read(fd, c->buf, c->len) != c->len) {
printk(KERN_INFO "Failed to read chunk firmware "
"chunk %d.\n", i);
goto fail;
@@ -231,17 +231,17 @@
INIT_LIST_HEAD(&fw->fw.chunk_list);
INIT_LIST_HEAD(&fw->uc.chunk_list);
- fd = open(fn, 0, 0);
+ fd = sys_open(fn, 0, 0);
if (fd == -1) {
printk(KERN_INFO "Unable to load '%s'.\n", fn);
return 1;
}
- l = lseek(fd, 0L, 2);
- lseek(fd, 0L, 0);
+ l = sys_lseek(fd, 0L, 2);
+ sys_lseek(fd, 0L, 0);
IPW2100_DEBUG_FW("Loading %ld bytes for firmware '%s'\n", l, fn);
- if (read(fd, (char *)&h, sizeof(h)) != sizeof(h)) {
+ if (sys_read(fd, (char *)&h, sizeof(h)) != sizeof(h)) {
printk(KERN_INFO "Failed to read '%s'.\n", fn);
goto fail;
}
@@ -262,12 +262,12 @@
if (ipw2100_fw_load(fd, &fw->uc, h.uc_size))
goto fail;
- close(fd);
+ sys_close(fd);
return 0;
fail:
ipw2100_fw_free(fw);
- close(fd);
+ sys_close(fd);
return 1;
}
^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: ipw2100 wireless driver
2004-08-09 20:15 ` Tomas Szepe
@ 2004-08-10 5:02 ` Jeff Chua
2004-08-10 6:55 ` Christoph Hellwig
0 siblings, 1 reply; 31+ messages in thread
From: Jeff Chua @ 2004-08-10 5:02 UTC (permalink / raw)
To: Tomas Szepe; +Cc: Pavel Machek, Jeff Chua, netdev, kernel list
On Mon, 9 Aug 2004, Tomas Szepe wrote:
> ipw2100 0.51 from ipw2100.sf.net builds using gcc-2.95.3 "out of the box."
Well, this is really good news!
I just downloaded 0.51 compiled with gcc-2.95.3 and got it working on my
IBM X31 with WEP. Even better, 0.51 doesn't need hostap-driver.
Looks good.
Thanks,
Jeff
^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: ipw2100 wireless driver
2004-08-10 5:02 ` Jeff Chua
@ 2004-08-10 6:55 ` Christoph Hellwig
2004-08-10 10:16 ` Pavel Machek
0 siblings, 1 reply; 31+ messages in thread
From: Christoph Hellwig @ 2004-08-10 6:55 UTC (permalink / raw)
To: Jeff Chua; +Cc: Tomas Szepe, Pavel Machek, netdev, kernel list
On Tue, Aug 10, 2004 at 01:02:07PM +0800, Jeff Chua wrote:
>
> On Mon, 9 Aug 2004, Tomas Szepe wrote:
>
> > ipw2100 0.51 from ipw2100.sf.net builds using gcc-2.95.3 "out of the box."
>
> Well, this is really good news!
>
> I just downloaded 0.51 compiled with gcc-2.95.3 and got it working on my
> IBM X31 with WEP. Even better, 0.51 doesn't need hostap-driver.
Btw, any vounteer for merging the hostap-based generic ieee80211_* files
from the ipw2100 driver with the hostap driver in the wireless-2.6 tree?
^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: ipw2100 wireless driver
2004-08-10 6:55 ` Christoph Hellwig
@ 2004-08-10 10:16 ` Pavel Machek
2004-08-10 10:34 ` Christoph Hellwig
2004-08-11 10:16 ` James Ketrenos
0 siblings, 2 replies; 31+ messages in thread
From: Pavel Machek @ 2004-08-10 10:16 UTC (permalink / raw)
To: Christoph Hellwig, Jeff Chua, Tomas Szepe, netdev, kernel list
Hi!
> > > ipw2100 0.51 from ipw2100.sf.net builds using gcc-2.95.3 "out of the box."
> >
> > Well, this is really good news!
> >
> > I just downloaded 0.51 compiled with gcc-2.95.3 and got it working on my
> > IBM X31 with WEP. Even better, 0.51 doesn't need hostap-driver.
>
> Btw, any vounteer for merging the hostap-based generic ieee80211_* files
> from the ipw2100 driver with the hostap driver in the wireless-2.6 tree?
I know very little about wireless-2.6 tree (where to get it without
bitkeeper?), but...
task is to take ipw2100 driver, drop ieee80211_* files from it, and
make it work with ieee80211* files from wireless-2.6?
Pavel
--
Horseback riding is like software...
...vgf orggre jura vgf serr.
^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: ipw2100 wireless driver
2004-08-10 10:16 ` Pavel Machek
@ 2004-08-10 10:34 ` Christoph Hellwig
2004-08-11 12:17 ` Pavel Machek
2004-08-11 10:16 ` James Ketrenos
1 sibling, 1 reply; 31+ messages in thread
From: Christoph Hellwig @ 2004-08-10 10:34 UTC (permalink / raw)
To: Pavel Machek; +Cc: Jeff Chua, Tomas Szepe, netdev, kernel list
On Tue, Aug 10, 2004 at 12:16:40PM +0200, Pavel Machek wrote:
> I know very little about wireless-2.6 tree (where to get it without
> bitkeeper?), but...
http://gkernel.bkbits.net:8080/wireless-2.6 it the bkweb interface, that's
the only thing I've looked at myself so far.
> task is to take ipw2100 driver, drop ieee80211_* files from it, and
> make it work with ieee80211* files from wireless-2.6?
there's no ieee80211_* files in the wireless-2.6 tree, the code is part
of hostap_*
and btw, I think hostap_* is the wrong name for the prism-specific files,
too. I'd rather call those prism2_* or something.
^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: ipw2100 wireless driver
2004-08-10 10:16 ` Pavel Machek
2004-08-10 10:34 ` Christoph Hellwig
@ 2004-08-11 10:16 ` James Ketrenos
2004-08-11 10:44 ` Christoph Hellwig
2004-08-11 17:51 ` Jeff Garzik
1 sibling, 2 replies; 31+ messages in thread
From: James Ketrenos @ 2004-08-11 10:16 UTC (permalink / raw)
To: Pavel Machek
Cc: Christoph Hellwig, Jeff Chua, Tomas Szepe, netdev, kernel list
Pavel Machek wrote:
> Hi!
>
>>>>ipw2100 0.51 from ipw2100.sf.net builds using gcc-2.95.3 "out of the box."
>>>
>>>Well, this is really good news!
>>>
>>>I just downloaded 0.51 compiled with gcc-2.95.3 and got it working on my
>>>IBM X31 with WEP. Even better, 0.51 doesn't need hostap-driver.
>>
>>Btw, any vounteer for merging the hostap-based generic ieee80211_* files
>>from the ipw2100 driver with the hostap driver in the wireless-2.6 tree?
>
> I know very little about wireless-2.6 tree (where to get it without
> bitkeeper?), but...
>
> task is to take ipw2100 driver, drop ieee80211_* files from it, and
> make it work with ieee80211* files from wireless-2.6?
The ieee80211_* files in the ipw2100 were partially based on the hostap source,
with any HW specific code pulled out. It handles Tx and Rx right now for BSS,
IBSS, and MONITOR mode, supporting wep, etc. It doesn't have all the hooks
needed for the host based AP mode that Host AP currently has, nor does it
support wpa yet.
We're also using the stack for the ipw2200 project (which we hope to get another
snapshot release out soon) We've been talking about pulling ieee80211 into its
own project as others have expressed an interest in using it in their drivers as
well.
The general goal for the ieee80211 stack has been to be able to take an skb from
the xmit handler, fragment and encrypt that skb, and provide that list of those
802.11 fragments to the driver for transmission. On the Rx side, the ieee80211
stack expects a standard 802.11 data frame which it then performs any decrypt /
defrag and then passes up to the kernel.
Additional frame code is slowly being added to ieee80211 to support handling of
probe response / beacons to manage scan results, etc. and provide default WE
handlers for exposing that information.
ieee80211_wx provides some generic implementation for the WE handlers that every
driver has to implement the same way.
We're currently working to clean up ipw2100 and ieee80211 code for submission to
netdev for discussion and hopefully inclusion in the future. The ieee80211 code
is still being heavily developed, but its usable. If anyone wants to help out,
or if folks feel its ready as-is to get pulled into wireless-2.6, let me know.
Thanks,
James
^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: ipw2100 wireless driver
2004-08-11 10:16 ` James Ketrenos
@ 2004-08-11 10:44 ` Christoph Hellwig
2004-08-11 10:53 ` Wichert Akkerman
2004-08-11 16:21 ` James Ketrenos
2004-08-11 17:51 ` Jeff Garzik
1 sibling, 2 replies; 31+ messages in thread
From: Christoph Hellwig @ 2004-08-11 10:44 UTC (permalink / raw)
To: James Ketrenos
Cc: Pavel Machek, Christoph Hellwig, Jeff Chua, Tomas Szepe, netdev,
kernel list
On Wed, Aug 11, 2004 at 05:16:35AM -0500, James Ketrenos wrote:
> We're currently working to clean up ipw2100 and ieee80211 code for submission to
> netdev for discussion and hopefully inclusion in the future. The ieee80211 code
> is still being heavily developed, but its usable. If anyone wants to help out,
> or if folks feel its ready as-is to get pulled into wireless-2.6, let me know.
Maybe we should switch to your ieee802.11 for a generic wireless stack then
instead of the original hostap code. At least it seems more actively
maintained right now and supports two drivers already.
Btw, I've looked at the ipw2100 and have to concerns regarding the firmware,
a) yo'ure not using the proper firmware loader but some horrible
handcrafted code using sys_open/sys_read & co that's not namespace
safe at all
b) the firmware has an extremly complicated and hard to comply with license,
I'm not sure we want a driver that can't work without a so strangely
licensed blob in the kernel. Can you talk to intel lawyers and put it on
simple redristribution and binary modification for allowed for all purposes
license please?
> Thanks,
> James
>
>
>
---end quoted text---
^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: ipw2100 wireless driver
2004-08-11 10:44 ` Christoph Hellwig
@ 2004-08-11 10:53 ` Wichert Akkerman
2004-08-11 10:59 ` Christoph Hellwig
2004-08-11 16:21 ` James Ketrenos
1 sibling, 1 reply; 31+ messages in thread
From: Wichert Akkerman @ 2004-08-11 10:53 UTC (permalink / raw)
To: Christoph Hellwig, James Ketrenos, Pavel Machek, Jeff Chua,
Tomas Szepe, netdev, kernel list
Previously Christoph Hellwig wrote:
> a) yo'ure not using the proper firmware loader but some horrible
> handcrafted code using sys_open/sys_read & co that's not namespace
> safe at all
It can use standard hotplug firmware load as well.
Wichert.
--
Wichert Akkerman <wichert@wiggy.net> It is simple to make things.
http://www.wiggy.net/ It is hard to make things simple.
^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: ipw2100 wireless driver
2004-08-11 10:53 ` Wichert Akkerman
@ 2004-08-11 10:59 ` Christoph Hellwig
0 siblings, 0 replies; 31+ messages in thread
From: Christoph Hellwig @ 2004-08-11 10:59 UTC (permalink / raw)
To: James Ketrenos, Pavel Machek, Jeff Chua, Tomas Szepe, netdev,
kernel list
On Wed, Aug 11, 2004 at 12:53:38PM +0200, Wichert Akkerman wrote:
> Previously Christoph Hellwig wrote:
> > a) yo'ure not using the proper firmware loader but some horrible
> > handcrafted code using sys_open/sys_read & co that's not namespace
> > safe at all
>
> It can use standard hotplug firmware load as well.
Okay, I'll take that back.
^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: ipw2100 wireless driver
2004-08-10 10:34 ` Christoph Hellwig
@ 2004-08-11 12:17 ` Pavel Machek
2004-08-11 17:54 ` Jeff Garzik
0 siblings, 1 reply; 31+ messages in thread
From: Pavel Machek @ 2004-08-11 12:17 UTC (permalink / raw)
To: Christoph Hellwig, Jeff Chua, Tomas Szepe, netdev, kernel list
Hi!
> > I know very little about wireless-2.6 tree (where to get it without
> > bitkeeper?), but...
>
> http://gkernel.bkbits.net:8080/wireless-2.6 it the bkweb interface, that's
> the only thing I've looked at myself so far.
Hmm, strange, one merge in last 8 weeks. That's not too active
project. Are you sure this is the right tree to work against?
Are there plain diffs somewhere?
Pavel
--
People were complaining that M$ turns users into beta-testers...
...jr ghea gurz vagb qrirybcref, naq gurl frrz gb yvxr vg gung jnl!
^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: ipw2100 wireless driver
2004-08-11 10:44 ` Christoph Hellwig
2004-08-11 10:53 ` Wichert Akkerman
@ 2004-08-11 16:21 ` James Ketrenos
2004-08-11 16:30 ` Stephen Hemminger
2004-08-12 3:47 ` Luis R. Rodriguez
1 sibling, 2 replies; 31+ messages in thread
From: James Ketrenos @ 2004-08-11 16:21 UTC (permalink / raw)
To: Christoph Hellwig
Cc: Pavel Machek, Jeff Chua, Tomas Szepe, netdev, kernel list
Christoph Hellwig wrote:
> On Wed, Aug 11, 2004 at 05:16:35AM -0500, James Ketrenos wrote:
>
>>We're currently working to clean up ipw2100 and ieee80211 code for submission to
>>netdev for discussion and hopefully inclusion in the future. The ieee80211 code
>>is still being heavily developed, but its usable. If anyone wants to help out,
>>or if folks feel its ready as-is to get pulled into wireless-2.6, let me know.
>
> Maybe we should switch to your ieee802.11 for a generic wireless stack then
> instead of the original hostap code. At least it seems more actively
> maintained right now and supports two drivers already.
This would be ideal for those working on the projects using that stack. If
others agree I'll put together patches that introduce the ieee80211* module into
wireless-2.6 once I get the next driver snapshots out for ipw2100 and ipw2200.
> Btw, I've looked at the ipw2100 and have to concerns regarding the firmware,
>
> a) yo'ure not using the proper firmware loader but some horrible
> handcrafted code using sys_open/sys_read & co that's not namespace
> safe at all
The driver supports (and defaults to) using firmware_class for loading the
firmware. The driver also supports a legacy loading approach for folks that
have problems with using hotplug to load the firmware (which represents a fair
number of users).
> b) the firmware has an extremly complicated and hard to comply with license,
> I'm not sure we want a driver that can't work without a so strangely
> licensed blob in the kernel. Can you talk to intel lawyers and put it on
> simple redristribution and binary modification for allowed for all purposes
> license please?
The firmware license supports redistribution, and complying with the license
shouldn't be too hard (I agree it may not be worded the most clearly, but few
legal documents are). If you have issues or questions about specific terms
please email me offlist and I can try and address them.
Just to re-answer some others may be wondering regarding the firmware:
1) the firmware does not use nor is it dependent (at all) on the kernel. no
part of the firmware executes on the host CPU.
2) the firmware is loaded from disk vs. having to have non-volatile storage on
the NIC and requiring a firmware flashing utility, etc.
3) the firmware, as per its license, can be redistributed by OSDs, ISVs, etc.
Thanks,
James
^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: ipw2100 wireless driver
2004-08-11 16:21 ` James Ketrenos
@ 2004-08-11 16:30 ` Stephen Hemminger
2004-08-11 16:33 ` Tomas Szepe
2004-08-12 3:47 ` Luis R. Rodriguez
1 sibling, 1 reply; 31+ messages in thread
From: Stephen Hemminger @ 2004-08-11 16:30 UTC (permalink / raw)
To: James Ketrenos
Cc: Christoph Hellwig, Pavel Machek, Jeff Chua, Tomas Szepe, netdev,
kernel list
> The driver supports (and defaults to) using firmware_class for loading the
> firmware. The driver also supports a legacy loading approach for folks that
> have problems with using hotplug to load the firmware (which represents a fair
> number of users).
>
When and if you submit it into mainline, please remove the legacy loading
approach. Let's get to the cause of the problem and fix it, not bandaid
around it.
^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: ipw2100 wireless driver
2004-08-11 16:30 ` Stephen Hemminger
@ 2004-08-11 16:33 ` Tomas Szepe
2004-08-11 16:51 ` Christoph Hellwig
2004-08-11 23:55 ` Jeff Chua
0 siblings, 2 replies; 31+ messages in thread
From: Tomas Szepe @ 2004-08-11 16:33 UTC (permalink / raw)
To: Stephen Hemminger
Cc: James Ketrenos, Christoph Hellwig, Pavel Machek, Jeff Chua,
netdev, kernel list
On Aug-11 2004, Wed, 09:30 -0700
Stephen Hemminger <shemminger@osdl.org> wrote:
> > The driver supports (and defaults to) using firmware_class for loading the
> > firmware. The driver also supports a legacy loading approach for folks that
> > have problems with using hotplug to load the firmware (which represents a fair
> > number of users).
> When and if you submit it into mainline, please remove the legacy loading
> approach. Let's get to the cause of the problem and fix it, not bandaid
> around it.
Or better yet, let's not!
There are many people who don't want to mess around with hotplug just
to get a single driver to load.
--
Tomas Szepe <szepe@pinerecords.com>
^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: ipw2100 wireless driver
2004-08-11 16:33 ` Tomas Szepe
@ 2004-08-11 16:51 ` Christoph Hellwig
2004-08-11 17:02 ` Tomas Szepe
2004-08-11 23:55 ` Jeff Chua
1 sibling, 1 reply; 31+ messages in thread
From: Christoph Hellwig @ 2004-08-11 16:51 UTC (permalink / raw)
To: Tomas Szepe
Cc: Stephen Hemminger, James Ketrenos, Christoph Hellwig,
Pavel Machek, Jeff Chua, netdev, kernel list
On Wed, Aug 11, 2004 at 06:33:33PM +0200, Tomas Szepe wrote:
> There are many people who don't want to mess around with hotplug just
> to get a single driver to load.
Then use a distribution that gets it right for you. Having gazillions
of diffferent firmware loaders just because people are too lazy to set
up the canonical one isn't where we want to go.
^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: ipw2100 wireless driver
2004-08-11 16:51 ` Christoph Hellwig
@ 2004-08-11 17:02 ` Tomas Szepe
2004-08-11 17:11 ` Christoph Hellwig
2004-08-11 18:55 ` John Stoffel
0 siblings, 2 replies; 31+ messages in thread
From: Tomas Szepe @ 2004-08-11 17:02 UTC (permalink / raw)
To: Christoph Hellwig, Stephen Hemminger, James Ketrenos,
Pavel Machek, Jeff Chua, netdev, kernel list
On Aug-11 2004, Wed, 17:51 +0100
Christoph Hellwig <hch@infradead.org> wrote:
> On Wed, Aug 11, 2004 at 06:33:33PM +0200, Tomas Szepe wrote:
> > There are many people who don't want to mess around with hotplug just
> > to get a single driver to load.
>
> Then use a distribution that gets it right for you. Having gazillions
> of diffferent firmware loaders just because people are too lazy to set
> up the canonical one isn't where we want to go.
Agreed. But the point is, in the actual case of ipw2100, will the removal
of 40 or so lines of code justify killing the functionality for those (lots)
that use it? I don't think so. A nice /* duplicate this in another driver
and die */ comment in the right place will do the job just fine IMHO.
--
Tomas Szepe <szepe@pinerecords.com>
^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: ipw2100 wireless driver
2004-08-11 17:02 ` Tomas Szepe
@ 2004-08-11 17:11 ` Christoph Hellwig
2004-08-11 17:22 ` Tomas Szepe
2004-08-11 18:55 ` John Stoffel
1 sibling, 1 reply; 31+ messages in thread
From: Christoph Hellwig @ 2004-08-11 17:11 UTC (permalink / raw)
To: Tomas Szepe
Cc: Christoph Hellwig, Stephen Hemminger, James Ketrenos,
Pavel Machek, Jeff Chua, netdev, kernel list
On Wed, Aug 11, 2004 at 07:02:08PM +0200, Tomas Szepe wrote:
> Agreed. But the point is, in the actual case of ipw2100, will the removal
> of 40 or so lines of code justify killing the functionality for those (lots)
> that use it?
Yes.
^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: ipw2100 wireless driver
2004-08-11 17:11 ` Christoph Hellwig
@ 2004-08-11 17:22 ` Tomas Szepe
0 siblings, 0 replies; 31+ messages in thread
From: Tomas Szepe @ 2004-08-11 17:22 UTC (permalink / raw)
To: Christoph Hellwig, Stephen Hemminger, James Ketrenos,
Pavel Machek, Jeff Chua, netdev, kernel list
On Aug-11 2004, Wed, 18:11 +0100
Christoph Hellwig <hch@infradead.org> wrote:
> On Wed, Aug 11, 2004 at 07:02:08PM +0200, Tomas Szepe wrote:
> > Agreed. But the point is, in the actual case of ipw2100, will the removal
> > of 40 or so lines of code justify killing the functionality for those (lots)
> > that use it?
>
> Yes.
Well, "I'm not young enough to know everything." Too bad.
And btw, mails to hch@infradead.org bounce.
--
Tomas Szepe <szepe@pinerecords.com>
^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: ipw2100 wireless driver
2004-08-11 10:16 ` James Ketrenos
2004-08-11 10:44 ` Christoph Hellwig
@ 2004-08-11 17:51 ` Jeff Garzik
1 sibling, 0 replies; 31+ messages in thread
From: Jeff Garzik @ 2004-08-11 17:51 UTC (permalink / raw)
To: James Ketrenos
Cc: Pavel Machek, Christoph Hellwig, Jeff Chua, Tomas Szepe, netdev,
kernel list
James Ketrenos wrote:
> We're currently working to clean up ipw2100 and ieee80211 code for
> submission to netdev for discussion and hopefully inclusion in the
> future. The ieee80211 code is still being heavily developed, but its
> usable. If anyone wants to help out, or if folks feel its ready as-is
> to get pulled into wireless-2.6, let me know.
Well, wireless-2.6 is a development tree, so I would rather things land
there sooner rather than later.
Everybody needs to be developing "on the same page".
Jeff
^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: ipw2100 wireless driver
2004-08-11 12:17 ` Pavel Machek
@ 2004-08-11 17:54 ` Jeff Garzik
2004-08-11 20:27 ` Pavel Machek
0 siblings, 1 reply; 31+ messages in thread
From: Jeff Garzik @ 2004-08-11 17:54 UTC (permalink / raw)
To: Pavel Machek
Cc: Christoph Hellwig, Jeff Chua, Tomas Szepe, netdev, kernel list,
Jouni Malinen
Pavel Machek wrote:
> Hmm, strange, one merge in last 8 weeks. That's not too active
> project. Are you sure this is the right tree to work against?
It's the right tree, but wireless development hasn't coalesced yet on
that tree it appears. It sounds like the Intel folks are interested in
working on that tree, and others have mentioned they are interested as well.
Waiting for people step up, and put their code where their mouth is...
Jeff
^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: ipw2100 wireless driver
2004-08-11 17:02 ` Tomas Szepe
2004-08-11 17:11 ` Christoph Hellwig
@ 2004-08-11 18:55 ` John Stoffel
1 sibling, 0 replies; 31+ messages in thread
From: John Stoffel @ 2004-08-11 18:55 UTC (permalink / raw)
To: Tomas Szepe
Cc: Christoph Hellwig, Stephen Hemminger, James Ketrenos,
Pavel Machek, Jeff Chua, netdev, kernel list
Tomas> On Aug-11 2004, Wed, 17:51 +0100
Tomas> Christoph Hellwig <hch@infradead.org> wrote:
>> On Wed, Aug 11, 2004 at 06:33:33PM +0200, Tomas Szepe wrote:
>> > There are many people who don't want to mess around with hotplug just
>> > to get a single driver to load.
>>
>> Then use a distribution that gets it right for you. Having gazillions
>> of diffferent firmware loaders just because people are too lazy to set
>> up the canonical one isn't where we want to go.
Tomas> Agreed. But the point is, in the actual case of ipw2100, will
Tomas> the removal of 40 or so lines of code justify killing the
Tomas> functionality for those (lots) that use it? I don't think so.
You can't have your cake and eat it too Tomas! You agree that having
multiple firmware loaders in the kernel is bad, yet you still want to
have your own special one in there? What's so difficult about setting
up hotplug for this purpose anyway?
John
^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: ipw2100 wireless driver
2004-08-11 17:54 ` Jeff Garzik
@ 2004-08-11 20:27 ` Pavel Machek
2004-08-11 22:01 ` Jeff Garzik
0 siblings, 1 reply; 31+ messages in thread
From: Pavel Machek @ 2004-08-11 20:27 UTC (permalink / raw)
To: Jeff Garzik
Cc: Pavel Machek, Christoph Hellwig, Jeff Chua, Tomas Szepe, netdev,
kernel list, Jouni Malinen
Hi!
> >Hmm, strange, one merge in last 8 weeks. That's not too active
> >project. Are you sure this is the right tree to work against?
>
>
> It's the right tree, but wireless development hasn't coalesced yet on
> that tree it appears. It sounds like the Intel folks are interested
> in working on that tree, and others have mentioned they are
> interested as well.
Is there way to access that tree without bk? Daily patches maybe?
--
64 bytes from 195.113.31.123: icmp_seq=28 ttl=51 time=448769.1 ms
^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: ipw2100 wireless driver
2004-08-11 20:27 ` Pavel Machek
@ 2004-08-11 22:01 ` Jeff Garzik
0 siblings, 0 replies; 31+ messages in thread
From: Jeff Garzik @ 2004-08-11 22:01 UTC (permalink / raw)
To: Pavel Machek
Cc: Pavel Machek, Christoph Hellwig, Jeff Chua, Tomas Szepe, netdev,
kernel list, Jouni Malinen
Pavel Machek wrote:
> Hi!
>
>
>>>Hmm, strange, one merge in last 8 weeks. That's not too active
>>>project. Are you sure this is the right tree to work against?
>>
>>
>>It's the right tree, but wireless development hasn't coalesced yet on
>>that tree it appears. It sounds like the Intel folks are interested
>>in working on that tree, and others have mentioned they are
>>interested as well.
>
>
> Is there way to access that tree without bk? Daily patches maybe?
Just posted to netdev. No regular snapshots, though.
Jeff
^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: ipw2100 wireless driver
2004-08-11 16:33 ` Tomas Szepe
2004-08-11 16:51 ` Christoph Hellwig
@ 2004-08-11 23:55 ` Jeff Chua
1 sibling, 0 replies; 31+ messages in thread
From: Jeff Chua @ 2004-08-11 23:55 UTC (permalink / raw)
To: Tomas Szepe
Cc: Stephen Hemminger, James Ketrenos, Christoph Hellwig,
Pavel Machek, Jeff Chua, netdev, kernel list
On Wed, 11 Aug 2004, Tomas Szepe wrote:
> Or better yet, let's not!
>
> There are many people who don't want to mess around with hotplug just
> to get a single driver to load.
Agreed. Please leave the legacy-loader in there. I don't need hotplug
scripts just to load a single firmware on my X31.
Thanks,
Jeff
^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: ipw2100 wireless driver
2004-08-11 16:21 ` James Ketrenos
2004-08-11 16:30 ` Stephen Hemminger
@ 2004-08-12 3:47 ` Luis R. Rodriguez
1 sibling, 0 replies; 31+ messages in thread
From: Luis R. Rodriguez @ 2004-08-12 3:47 UTC (permalink / raw)
To: James Ketrenos
Cc: Christoph Hellwig, Pavel Machek, Jeff Chua, Tomas Szepe, netdev,
kernel list
On Wed, Aug 11, 2004 at 11:21:34AM -0500, James Ketrenos wrote:
> Just to re-answer some others may be wondering regarding the firmware:
<-- 1 and 2 and -->
> 3) the firmware, as per its license, can be redistributed by OSDs, ISVs,
> etc.
Would it be possible to get a better licensing for the firmware, ie one that
would allow free redistribution, and be transferable?
Luis
--
GnuPG Key fingerprint = 113F B290 C6D2 0251 4D84 A34A 6ADD 4937 E20A 525E
^ permalink raw reply [flat|nested] 31+ messages in thread
end of thread, other threads:[~2004-08-12 3:47 UTC | newest]
Thread overview: 31+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2004-07-14 11:41 ipw2100 wireless driver Pavel Machek
2004-07-14 11:48 ` Jeff Chua
2004-07-14 11:55 ` Pavel Machek
2004-08-09 20:15 ` Tomas Szepe
2004-08-10 5:02 ` Jeff Chua
2004-08-10 6:55 ` Christoph Hellwig
2004-08-10 10:16 ` Pavel Machek
2004-08-10 10:34 ` Christoph Hellwig
2004-08-11 12:17 ` Pavel Machek
2004-08-11 17:54 ` Jeff Garzik
2004-08-11 20:27 ` Pavel Machek
2004-08-11 22:01 ` Jeff Garzik
2004-08-11 10:16 ` James Ketrenos
2004-08-11 10:44 ` Christoph Hellwig
2004-08-11 10:53 ` Wichert Akkerman
2004-08-11 10:59 ` Christoph Hellwig
2004-08-11 16:21 ` James Ketrenos
2004-08-11 16:30 ` Stephen Hemminger
2004-08-11 16:33 ` Tomas Szepe
2004-08-11 16:51 ` Christoph Hellwig
2004-08-11 17:02 ` Tomas Szepe
2004-08-11 17:11 ` Christoph Hellwig
2004-08-11 17:22 ` Tomas Szepe
2004-08-11 18:55 ` John Stoffel
2004-08-11 23:55 ` Jeff Chua
2004-08-12 3:47 ` Luis R. Rodriguez
2004-08-11 17:51 ` Jeff Garzik
2004-07-14 12:30 ` Tomas Szepe
2004-07-14 12:27 ` Pavel Machek
2004-07-14 13:15 ` Vojtech Pavlik
2004-07-14 13:42 ` Pavel Machek
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).