netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* 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).