Re: [IWN] Review

2013-10-23 Thread Adrian Chadd
Hi,

Ok. I've reviewed this stuff in more depth.

Split-6 is still way too big to commit as one commit.

I was also hoping that we could get the updated hardware support into this
without necessarily adding the PAN support. But, you're the one driving
this, so it's up to you. :)

* There's DPRINTF() and the debug flags that you've moved into if_iwnreg.h,
which is the wrong place for it!
* .. same as iwn_intr_str()
* You still have style(9) issues:
  + the if constructs should be if (), not if();
  + you need spaces between things - eg, if (a == b) rather than if(a==b)
or if (a==b);
  + the } else { should be on one line, not separate on multiple lines.

*  What's the story behind sc-ctx? When is it being set/changed?
* And then there's also ivp-ctx; that's the current VAP context, right? If
that's the case ,why are we bothering checking unit number? Why don't we
consistently check the vap context?

So, let's break up split_6 into this:

* create if_iwn_debug.h; put the debug macros, enum and such into that;
* submit a patch _just_ for the debug work, that's easy to do.

* Then, fix up the style(9) issues.
* Then, help me figure out what the story is with sc-ctx so I understand
what's going on there. I'd like to try and remove that if possible.

Thanks,



-adrian



On 28 September 2013 10:23, Cedric GROSS c...@cgross.info wrote:

 Hello,



 I'm get some free time. So I setted up my github for split work.



 So on https://github.com/KreizIT/FreeBSD-IWN/



 You will find 2 new branch : split_6 and split_7.



 Split_6 is iwn -HEAD with split 6 applied. Patch for that is also available
 in this branch.

 Split_7 is iwn -HEAD with split 6 and new split 7 applied.



 Split_7 start parameters task. So no massive change except that it's
 manage NIC without BT.



 Regards

 Cedric

 ___
 freebsd-wireless@freebsd.org mailing list
 http://lists.freebsd.org/mailman/listinfo/freebsd-wireless
 To unsubscribe, send any mail to freebsd-wireless-unsubscr...@freebsd.org
 

___
freebsd-wireless@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-wireless
To unsubscribe, send any mail to freebsd-wireless-unsubscr...@freebsd.org


Re: [IWN] Review

2013-10-23 Thread Adrian Chadd
.. and I've just done the if_iwn_debug.h stuff, so you should update your
patchset to include that.

Thanks,



-adrian



On 23 October 2013 17:43, Adrian Chadd adr...@freebsd.org wrote:

 Hi,

 Ok. I've reviewed this stuff in more depth.

 Split-6 is still way too big to commit as one commit.

 I was also hoping that we could get the updated hardware support into this
 without necessarily adding the PAN support. But, you're the one driving
 this, so it's up to you. :)

 * There's DPRINTF() and the debug flags that you've moved into
 if_iwnreg.h, which is the wrong place for it!
 * .. same as iwn_intr_str()
 * You still have style(9) issues:
   + the if constructs should be if (), not if();
   + you need spaces between things - eg, if (a == b) rather than if(a==b)
 or if (a==b);
   + the } else { should be on one line, not separate on multiple lines.

 *  What's the story behind sc-ctx? When is it being set/changed?
 * And then there's also ivp-ctx; that's the current VAP context, right?
 If that's the case ,why are we bothering checking unit number? Why don't we
 consistently check the vap context?

 So, let's break up split_6 into this:

 * create if_iwn_debug.h; put the debug macros, enum and such into that;
 * submit a patch _just_ for the debug work, that's easy to do.

 * Then, fix up the style(9) issues.
 * Then, help me figure out what the story is with sc-ctx so I understand
 what's going on there. I'd like to try and remove that if possible.

 Thanks,



 -adrian



 On 28 September 2013 10:23, Cedric GROSS c...@cgross.info wrote:

 Hello,



 I'm get some free time. So I setted up my github for split work.



 So on https://github.com/KreizIT/FreeBSD-IWN/



 You will find 2 new branch : split_6 and split_7.



 Split_6 is iwn -HEAD with split 6 applied. Patch for that is also
 available
 in this branch.

 Split_7 is iwn -HEAD with split 6 and new split 7 applied.



 Split_7 start parameters task. So no massive change except that it's
 manage NIC without BT.



 Regards

 Cedric

 ___
 freebsd-wireless@freebsd.org mailing list
 http://lists.freebsd.org/mailman/listinfo/freebsd-wireless
 To unsubscribe, send any mail to 
 freebsd-wireless-unsubscr...@freebsd.org



___
freebsd-wireless@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-wireless
To unsubscribe, send any mail to freebsd-wireless-unsubscr...@freebsd.org


Re: [IWN] Review

2013-10-15 Thread Adrian Chadd
Ok, so what, I should test out split_8, get that into the tree, then merge
in more stuff from you?


-adrian



On 14 October 2013 08:17, Cedric GROSS c...@cgross.info wrote:

  De : Sean Bruno [mailto:sean_br...@yahoo.com]
  À : Cedric GROSS
  Cc : 'Adrian Chadd'; freebsd-wireless@freebsd.org
  Objet : RE: [IWN] Review
 
  On Mon, 2013-09-30 at 16:47 +0200, Cedric GROSS wrote:
   Hello,
   I forgot your lastest commit but it?s done now.
  
   So split_6 and split_7 are sync with most recent ?HEAD.
   Cedric
 
  Cedric:
 Hi Sean,

 
  Which branch should we be testing?
 

 As Adrian wish to smoothly include changes into FreeBSD repo, split_6 an
 split_7 doesn't change big things regarding latest NIC.

 Instead Split_8, is minimal version for managing 2200 and so on NIC series.
 Could be a good idea to test this version with à lot of NIC.

  sean

 Thanks
 Cedric


___
freebsd-wireless@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-wireless
To unsubscribe, send any mail to freebsd-wireless-unsubscr...@freebsd.org


RE: [IWN] Review

2013-09-30 Thread Cedric GROSS
Hello,

 

I forgot your lastest commit but it’s done now.

 

So split_6 and split_7 are sync with most recent –HEAD.

 

Cedric

 

De : adrian.ch...@gmail.com [mailto:adrian.ch...@gmail.com] De la part de
Adrian Chadd
Envoyé : dimanche 29 septembre 2013 10:36
À : Cedric GROSS
Cc : freebsd-wireless@freebsd.org
Objet : Re: [IWN] Review

 

.. is this diff (split_6) against the most recent -HEAD? There looks like
there's some extra stuff in there that I committed recently (ie, the rate
control change and deref/free)

 

I'll go check it out in more depth soon.


Thanks,

 

 

-a

 

On 29 September 2013 01:34, Adrian Chadd adr...@freebsd.org wrote:

Cool, I'll review it now!

 

 

-a

 

On 28 September 2013 10:23, Cedric GROSS c...@cgross.info wrote:

Hello,



I'm get some free time. So I setted up my github for split work.



So on https://github.com/KreizIT/FreeBSD-IWN/



You will find 2 new branch : split_6 and split_7.



Split_6 is iwn -HEAD with split 6 applied. Patch for that is also available
in this branch.

Split_7 is iwn -HEAD with split 6 and new split 7 applied.



Split_7 start parameters task. So no massive change except that it's
manage NIC without BT.



Regards

Cedric

___
freebsd-wireless@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-wireless
To unsubscribe, send any mail to freebsd-wireless-unsubscr...@freebsd.org

 

 

___
freebsd-wireless@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-wireless
To unsubscribe, send any mail to freebsd-wireless-unsubscr...@freebsd.org


RE: [iwn]Review Split 6

2013-08-13 Thread Cedric GROSS
 
  I guess '_u1' here is 'PAN VAP'. We'll eventually rename this stuff
  and maybe shift it into a separate source file so all the PAN stuff
  clearly lives elsewhere.
 
 Ok I'll do that.
 

Here is new version of split 6 with new files and new function renamed.

Cedric


pan2.patch
Description: Binary data
___
freebsd-wireless@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-wireless
To unsubscribe, send any mail to freebsd-wireless-unsubscr...@freebsd.org

RE: [iwn]Review Split 6

2013-08-12 Thread Cedric GROSS

 De : adrian.ch...@gmail.com [mailto:adrian.ch...@gmail.com] De la part
 de Adrian Chadd
 Envoyé : dimanche 11 août 2013 19:09
 À : Cedric GROSS
 Cc : freebsd-wireless@freebsd.org
 Objet : Re: [iwn]Review Split 6
 
 On 11 August 2013 10:06, Cedric GROSS c...@cgross.info wrote:
 
  U1 stand for unit 1.
  As you see, all function with this prefix will be call when unit==1
  within iwn_vap_create.
 
  As previously, I haven't investigate a lot this side. I plan to do so
  when standard run will be ok.
 
 Right. But what's going on, per se? Is it two separate NICs that show
 up as separate PCI devices? Or is this a hack so the firmware can
 present two virtual NICs via the firmware API?
 
Not a separate PCI device but yes ability to create 2 wlan NIC :

iwn0: flags=8843UP,BROADCAST,RUNNING,SIMPLEX,MULTICAST metric 0 mtu 2290
ether 60:36:dd:93:34:56
nd6 options=29PERFORMNUD,IFDISABLED,AUTO_LINKLOCAL
media: IEEE 802.11 Wireless Ethernet autoselect mode 11ng
status: associated
wlan0: flags=8943UP,BROADCAST,RUNNING,PROMISC,SIMPLEX,MULTICAST metric 0
mtu 1500
ether 60:36:dd:93:34:56
inet 192.168.1.20 netmask 0xff00 broadcast 192.168.1.255
nd6 options=29PERFORMNUD,IFDISABLED,AUTO_LINKLOCAL
media: IEEE 802.11 Wireless Ethernet MCS mode 11ng
status: associated
ssid WLenine channel 5 (2432 MHz 11g ht/20) bssid 00:24:d4:97:80:20
country US authmode WPA2/802.11i privacy ON deftxkey UNDEF
AES-CCM 2:128-bit txpower 15 bmiss 10 scanvalid 450 bgscan
bgscanintvl 300 bgscanidle 250 roam:rssi 7 roam:rate 64 protmode CTS
ampdulimit 64k -amsdutx amsdurx shortgi wme roaming MANUAL bintval
96
wlan1: flags=8802BROADCAST,SIMPLEX,MULTICAST metric 0 mtu 1500
ether 60:36:dd:93:34:57
nd6 options=29PERFORMNUD,IFDISABLED,AUTO_LINKLOCAL
media: IEEE 802.11 Wireless Ethernet autoselect (autoselect)
status: no carrier
ssid  channel 5 (2432 MHz 11g ht/20)
country US authmode OPEN privacy OFF txpower 15 bmiss 10 scanvalid
60
bgscan bgscanintvl 300 bgscanidle 250 roam:rssi 7 roam:rate 64
protmode CTS ampdulimit 8k -amsdutx amsdurx shortgi wme bintval 0

___
freebsd-wireless@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-wireless
To unsubscribe, send any mail to freebsd-wireless-unsubscr...@freebsd.org


Re: [iwn]Review Split 6

2013-08-12 Thread Adrian Chadd
On 12 August 2013 10:41, Cedric GROSS c...@cgross.info wrote:

 But again, as current version is not stable in normal operation, I haven't
 test it further.

 What could be interest of this ?
 Wifi relay, bridge or whatever usage of 2 wireless but only with 1 card.

It's likely for STA on one VAP and P2P-GO on the other VAP.



-adrian
___
freebsd-wireless@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-wireless
To unsubscribe, send any mail to freebsd-wireless-unsubscr...@freebsd.org


Re: [iwn]Review Split 6

2013-08-12 Thread Adrian Chadd
Ok, so this PAN support is .. pretty terrifying. We'll have to tidy it
up once the rest of this work is debugged.

I guess '_u1' here is 'PAN VAP'. We'll eventually rename this stuff
and maybe shift it into a separate source file so all the PAN stuff
clearly lives elsewhere.

I wonder which NICs support PAN - ie, how I would even begin going
about testing it out. Unfortunately our net80211 stack doesn't yet
have P2P support - it would be nice if someone wrote up the basic
infrastructure for that.

Anyway. I'll test this out on the iwn(4) NICs I now have and I'll get
back to you with any regressions that my light testing finds. As
always, thanks!



-adrian
___
freebsd-wireless@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-wireless
To unsubscribe, send any mail to freebsd-wireless-unsubscr...@freebsd.org


RE: [iwn]Review Split 6

2013-08-12 Thread Cedric GROSS

 -Message d'origine-
 De : adrian.ch...@gmail.com [mailto:adrian.ch...@gmail.com] De la part
 de Adrian Chadd
 Envoyé : lundi 12 août 2013 20:25
 À : Cedric GROSS
 Cc : freebsd-wireless@freebsd.org
 Objet : Re: [iwn]Review Split 6
 
 Ok, so this PAN support is .. pretty terrifying. We'll have to tidy it
 up once the rest of this work is debugged.
 
 I guess '_u1' here is 'PAN VAP'. We'll eventually rename this stuff and
 maybe shift it into a separate source file so all the PAN stuff clearly
 lives elsewhere.

Ok I'll do that. 

 
 I wonder which NICs support PAN - ie, how I would even begin going
 about testing it out. Unfortunately our net80211 stack doesn't yet have
 P2P support - it would be nice if someone wrote up the basic
 infrastructure for that.

PAN support is found within firmware: IWN_FW_TLV_PAN

 
 Anyway. I'll test this out on the iwn(4) NICs I now have and I'll get
 back to you with any regressions that my light testing finds. As
 always, thanks!
 

Should be none I guess ;)

 
 
 -adrian

___
freebsd-wireless@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-wireless
To unsubscribe, send any mail to freebsd-wireless-unsubscr...@freebsd.org


Re: [iwn]Review Split 6

2013-08-11 Thread Adrian Chadd
Hi!

What's 'u1' as a prefix supposed to represent?



-adrian

On 11 August 2013 06:09, Cedric GROSS c...@cgross.info wrote:
 Hello,



 Here is patch split 6.



 It's follow split 5 as it implement PAN context.

 This patch is purely based on work done by Sean Bruno.



 Cedric


 ___
 freebsd-wireless@freebsd.org mailing list
 http://lists.freebsd.org/mailman/listinfo/freebsd-wireless
 To unsubscribe, send any mail to freebsd-wireless-unsubscr...@freebsd.org
___
freebsd-wireless@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-wireless
To unsubscribe, send any mail to freebsd-wireless-unsubscr...@freebsd.org


RE: [iwn]Review Split 6

2013-08-11 Thread Cedric GROSS


 -Message d'origine-
 De : adrian.ch...@gmail.com [mailto:adrian.ch...@gmail.com] De la part
 de Adrian Chadd
 Envoyé : dimanche 11 août 2013 18:34
 À : Cedric GROSS
 Cc : freebsd-wireless@freebsd.org
 Objet : Re: [iwn]Review Split 6
 
 Hi!
 
 What's 'u1' as a prefix supposed to represent?
 

U1 stand for unit 1. 
As you see, all function with this prefix will be call when unit==1 within
iwn_vap_create.

As previously, I haven't investigate a lot this side. I plan to do so when
standard run will be ok.

 
 
 -adrian
 
  It's follow split 5 as it implement PAN context.
 
  This patch is purely based on work done by Sean Bruno.
 
  Cedric

___
freebsd-wireless@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-wireless
To unsubscribe, send any mail to freebsd-wireless-unsubscr...@freebsd.org


Re: [iwn]Review Split 6

2013-08-11 Thread Adrian Chadd
On 11 August 2013 10:06, Cedric GROSS c...@cgross.info wrote:

 U1 stand for unit 1.
 As you see, all function with this prefix will be call when unit==1 within
 iwn_vap_create.

 As previously, I haven't investigate a lot this side. I plan to do so when
 standard run will be ok.

Right. But what's going on, per se? Is it two separate NICs that show
up as separate PCI devices? Or is this a hack so the firmware can
present two virtual NICs via the firmware API?



-adrian
___
freebsd-wireless@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-wireless
To unsubscribe, send any mail to freebsd-wireless-unsubscr...@freebsd.org


Re: [iwn] Review split 5

2013-08-09 Thread Adrian Chadd
On 8 August 2013 03:43, Cedric GROSS c...@cgross.info wrote:

 So please go and fix that!

 Done.

Cool. I'll test this out on the 4965 and 5100 today/tomorrow and get
back to you.



-adrian
___
freebsd-wireless@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-wireless
To unsubscribe, send any mail to freebsd-wireless-unsubscr...@freebsd.org


Re: [iwn] Review split 5

2013-08-05 Thread Adrian Chadd
.. so what's this PAN context stuff?



-adrian

On 5 August 2013 10:23, Cedric GROSS c...@cgross.info wrote:
 Hello,



 Here is another split.



 This one update if_iwnvar.h with new structure for later but also modify
 iwn_softc for context selection.

 Context will be used later when PAN context will be set in next diff.



 uCode TLV detection has been updated also.



 Cedric


 ___
 freebsd-wireless@freebsd.org mailing list
 http://lists.freebsd.org/mailman/listinfo/freebsd-wireless
 To unsubscribe, send any mail to freebsd-wireless-unsubscr...@freebsd.org
___
freebsd-wireless@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-wireless
To unsubscribe, send any mail to freebsd-wireless-unsubscr...@freebsd.org


[iwn] Review split 4

2013-08-02 Thread Cedric GROSS
Hello,

 

After split 3 applied and committed , you will find split 4.

This one modify if_iwnreg.h

 

Add nearly everything we will need for later use.

 

Cedric



define.patch
Description: Binary data
___
freebsd-wireless@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-wireless
To unsubscribe, send any mail to freebsd-wireless-unsubscr...@freebsd.org

Re: [iwn] Review split 4

2013-08-02 Thread Adrian Chadd
Committed!

Thanks!



-adrian


On 2 August 2013 04:12, Cedric GROSS c...@cgross.info wrote:
 Hello,



 After split 3 applied and committed , you will find split 4.

 This one modify if_iwnreg.h



 Add nearly everything we will need for later use.



 Cedric


 ___
 freebsd-wireless@freebsd.org mailing list
 http://lists.freebsd.org/mailman/listinfo/freebsd-wireless
 To unsubscribe, send any mail to freebsd-wireless-unsubscr...@freebsd.org
___
freebsd-wireless@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-wireless
To unsubscribe, send any mail to freebsd-wireless-unsubscr...@freebsd.org


[iwn] Review split 3

2013-07-29 Thread Cedric GROSS
Hello,

 

As discuss we Adrian, previous big patch is too heavy for direct application
in repo.

So I resume splitting modification. 

 

Here is split 3. 

This one create if_iwn_devif.h for storing device ID and subdevice ID for
later use.

If_iwn.c and If_iwnreg.h modified as needed.

 

Cedric



deviceid.patch
Description: Binary data
___
freebsd-wireless@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-wireless
To unsubscribe, send any mail to freebsd-wireless-unsubscr...@freebsd.org

Re: [iwn] Review split 3

2013-07-29 Thread Adrian Chadd
Hi!

Cool! ok, the style niggles.

* Your #define IWN_blah needs a TAB between #define and the IWN_blah
value. Right now you have spaces.
* $FreeBSD$ should be after the copyright header, not before
* Your new file is missing the standard single-include stuff.. ie

#ifndef __IF_IWN_DEVID_H__
#define __IF_IWN_DEVID_H__

(header contents go here..)

#endif /* __IF_IWN_DEVID_H__ */



-adrian

On 29 July 2013 08:49, Cedric GROSS c...@cgross.info wrote:
 Hello,



 As discuss we Adrian, previous big patch is too heavy for direct application
 in repo.

 So I resume splitting modification.



 Here is split 3.

 This one create if_iwn_devif.h for storing device ID and subdevice ID for
 later use.

 If_iwn.c and If_iwnreg.h modified as needed.



 Cedric


 ___
 freebsd-wireless@freebsd.org mailing list
 http://lists.freebsd.org/mailman/listinfo/freebsd-wireless
 To unsubscribe, send any mail to freebsd-wireless-unsubscr...@freebsd.org
___
freebsd-wireless@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-wireless
To unsubscribe, send any mail to freebsd-wireless-unsubscr...@freebsd.org


Re: [iwn] Review split 3

2013-07-29 Thread Warren Block

On Mon, 29 Jul 2013, Adrian Chadd wrote:


Hi!

Cool! ok, the style niggles.

* Your #define IWN_blah needs a TAB between #define and the IWN_blah
value. Right now you have spaces.


There is whitespace at the end of some of the comment lines also.

textproc/igor will find problems like that:

  igor -R filename | less -RS
___
freebsd-wireless@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-wireless
To unsubscribe, send any mail to freebsd-wireless-unsubscr...@freebsd.org


RE: [iwn] Review split 3

2013-07-29 Thread Cedric GROSS


 De : Warren Block [mailto:wbl...@wonkity.com]
 Envoyé : lundi 29 juillet 2013 20:11
 À : Adrian Chadd
 Cc : Cedric GROSS; freebsd-wireless@freebsd.org
 
 On Mon, 29 Jul 2013, Adrian Chadd wrote:
 
  Hi!
 
  Cool! ok, the style niggles.
 
  * Your #define IWN_blah needs a TAB between #define and the IWN_blah
  value. Right now you have spaces.

Ok, I tough you need spaces between #define and values...

standard single-include stuff : Done
$FreeBSD$ : Moved

Each file is different. Really hard to find something common. Man 9 style is
quite useful but enough precise for tab and space usage I think.

 
 There is whitespace at the end of some of the comment lines also.
 
 textproc/igor will find problems like that:
 
igor -R filename | less -RS

Thanks for that Warren !


deviceid.patch
Description: Binary data
___
freebsd-wireless@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-wireless
To unsubscribe, send any mail to freebsd-wireless-unsubscr...@freebsd.org

IWN Review split 1

2013-07-23 Thread Cedric GROSS
Hello there,

 

As discuss this morning on Efnet, here is the first small patch for IWN.

This one add an hint value for early debugging IWN.

 

Early debug feature will start printing information from attach function and
further.

After attachment, debug verbosity is controlled by sysctl value.

 

Cedric



hint_debug.patch
Description: Binary data
___
freebsd-wireless@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-wireless
To unsubscribe, send any mail to freebsd-wireless-unsubscr...@freebsd.org

RE: IWN Review

2013-07-21 Thread Cedric GROSS
 De : adrian.ch...@gmail.com [mailto:adrian.ch...@gmail.com] De la part
 de Adrian Chadd
 Envoyé : dimanche 21 juillet 2013 06:06
 À : Cedric GROSS
 Cc : freebsd-wireless@freebsd.org; freebsd-driv...@freebsd.org
 
 Hi!
 
 so, my feedback!
 
 * Why'd you disable including opt_wlan.h ? It's required
Just because I worked with a 9.1-release and I don't reach to compile iwn
with this include. Forget to enable it again.

 * What merge issues did you have? I'd like to get this tree updated to
 the latest -HEAD, including whatever my recent changes for rate
 selection, so we can get this stuff tidied up for -HEAD
Ok I will try.

 * Why was ieee80211_resume_all() commented out? Did you do this?
This things was not in original IWL so I commented it to be sure that my
version of IWN work as iwl do. Plan to uncomment when merging to HEAD.

 
 There's some stuff we can merge in right now into -HEAD:
 
 * the debug improvements
 * the changes from hard-coded checks for features based on the chipset,
 to a bitmap of features that get set during probe/attach
 
 So, let's try to get your stuff updated to the latest -HEAD, then get
 it tested out. I'll then start merging this stuff into -HEAD as we
 figure out what all these pieces do.
 
 Thanks for all this hard work!
 
 
 
 -adrian
 
 
 On 20 July 2013 07:18, Cedric GROSS c...@cgross.info wrote:
  Hello dear all,
 
 
 
  I'm please to announce that I completed my work on IWN.
 
 
 
  My version available here https://github.com/KreizIT/FreeBSD-IWN
  include improvement issued by seanbruno
  (https://github.com/seanbruno/freebsd-iwl)
  but also mine to support centrino N 2230 NIC (it's my NIC)
 
 
 
  I set up a new parameters system which more sensitive. I port back
 all
  NIC parameters to this new system. I hope that I haven't do some
 mistake.
 
  So I'll be really please if Intel's NIC owner could check my work and
  let me know.
 
 
 
  By the way, if commiters would include my work, please do.
 
 
 
  Cheers
 
  Cedric
 
 
 
  ___
  freebsd-wireless@freebsd.org mailing list
  http://lists.freebsd.org/mailman/listinfo/freebsd-wireless
  To unsubscribe, send any mail to freebsd-wireless-
 unsubscr...@freebsd.org

___
freebsd-wireless@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-wireless
To unsubscribe, send any mail to freebsd-wireless-unsubscr...@freebsd.org


IWN Review

2013-07-20 Thread Cedric GROSS
Hello dear all,

 

I'm please to announce that I completed my work on IWN.

 

My version available here https://github.com/KreizIT/FreeBSD-IWN include
improvement issued by seanbruno (https://github.com/seanbruno/freebsd-iwl)
but also mine to support centrino N 2230 NIC (it's my NIC)

 

I set up a new parameters system which more sensitive. I port back all NIC
parameters to this new system. I hope that I haven't do some mistake.

So I'll be really please if Intel's NIC owner could check my work and let me
know.

 

By the way, if commiters would include my work, please do.

 

Cheers

Cedric

 

___
freebsd-wireless@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-wireless
To unsubscribe, send any mail to freebsd-wireless-unsubscr...@freebsd.org


RE: IWN Review

2013-07-20 Thread Cedric GROSS
 De : Andrey Fesenko [mailto:f0and...@gmail.com]
 Envoyé : samedi 20 juillet 2013 16:29
 À : Cedric GROSS
 Cc : freebsd-wireless@freebsd.org; freebsd-driv...@freebsd.org
 
 On Sat, Jul 20, 2013 at 6:18 PM, Cedric GROSS c...@cgross.info wrote:
  Hello dear all,
 
 
 
  I'm please to announce that I completed my work on IWN.
 
 
 
  My version available here https://github.com/KreizIT/FreeBSD-IWN
  include improvement issued by seanbruno
  (https://github.com/seanbruno/freebsd-iwl)
  but also mine to support centrino N 2230 NIC (it's my NIC)
 
 
 
  I set up a new parameters system which more sensitive. I port back
 all
  NIC parameters to this new system. I hope that I haven't do some
 mistake.
 
  So I'll be really please if Intel's NIC owner could check my work and
  let me know.
 
 
 
  By the way, if commiters would include my work, please do.
 
 
 
  Cheers
 
  Cedric
 
 
 
 
 Great news, whether the driver supports n-mode?

What do you mean by 'n-mode' ? 802.11n mode ? 
If so, answer is yes :

#dmesg |grep iwn0
iwn0: Intel Centrino Wireless-N 2230 mem 0xf7c0-0xf7c01fff irq 18 at 
device 0.0 on pci4
#ifconfig wlan0
wlan0: flags=8843UP,BROADCAST,RUNNING,SIMPLEX,MULTICAST metric 0 mtu 1500
nd6 options=29PERFORMNUD,IFDISABLED,AUTO_LINKLOCAL
media: IEEE 802.11 Wireless Ethernet DS/1Mbps mode 11ng
status: associated
ssid FreeWifi channel 5 (2432 MHz 11g ht/20) 
Cedric

___
freebsd-wireless@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-wireless
To unsubscribe, send any mail to freebsd-wireless-unsubscr...@freebsd.org

Re: IWN Review

2013-07-20 Thread Andrey Fesenko
On Sat, Jul 20, 2013 at 7:30 PM, Cedric GROSS c...@cgross.info wrote:
 De : Andrey Fesenko [mailto:f0and...@gmail.com]
 Envoyé : samedi 20 juillet 2013 16:29
 À : Cedric GROSS
 Cc : freebsd-wireless@freebsd.org; freebsd-driv...@freebsd.org

 On Sat, Jul 20, 2013 at 6:18 PM, Cedric GROSS c...@cgross.info wrote:
  Hello dear all,
 
 
 
  I'm please to announce that I completed my work on IWN.
 
 
 
  My version available here https://github.com/KreizIT/FreeBSD-IWN
  include improvement issued by seanbruno
  (https://github.com/seanbruno/freebsd-iwl)
  but also mine to support centrino N 2230 NIC (it's my NIC)
 
 
 
  I set up a new parameters system which more sensitive. I port back
 all
  NIC parameters to this new system. I hope that I haven't do some
 mistake.
 
  So I'll be really please if Intel's NIC owner could check my work and
  let me know.
 
 
 
  By the way, if commiters would include my work, please do.
 
 
 
  Cheers
 
  Cedric
 
 
 

 Great news, whether the driver supports n-mode?

 What do you mean by 'n-mode' ? 802.11n mode ?
 If so, answer is yes :

 #dmesg |grep iwn0
 iwn0: Intel Centrino Wireless-N 2230 mem 0xf7c0-0xf7c01fff irq 18 at 
 device 0.0 on pci4
 #ifconfig wlan0
 wlan0: flags=8843UP,BROADCAST,RUNNING,SIMPLEX,MULTICAST metric 0 mtu 1500
 nd6 options=29PERFORMNUD,IFDISABLED,AUTO_LINKLOCAL
 media: IEEE 802.11 Wireless Ethernet DS/1Mbps mode 11ng
 status: associated
 ssid FreeWifi channel 5 (2432 MHz 11g ht/20)
 Cedric


Nice, ht/40 support to?
Do you plan to support other cards Intel 6235 and supporting more than
one range (5GHz)?

I think whether to change AR9285 base card or not :)
___
freebsd-wireless@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-wireless
To unsubscribe, send any mail to freebsd-wireless-unsubscr...@freebsd.org

Re: IWN Review

2013-07-20 Thread Lev Serebryakov
Hello, Andrey.
You wrote 20 июля 2013 г., 19:42:41:

AF I think whether to change AR9285 base card or not :)
  Atheros is better. And, yes, Intel doesn't support host AP mode.

-- 
// Black Lion AKA Lev Serebryakov l...@freebsd.org

___
freebsd-wireless@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-wireless
To unsubscribe, send any mail to freebsd-wireless-unsubscr...@freebsd.org

RE: IWN Review

2013-07-20 Thread Cedric GROSS


 De : Andrey Fesenko [mailto:f0and...@gmail.com]
[...]
 
  Great news, whether the driver supports n-mode?
 
  What do you mean by 'n-mode' ? 802.11n mode ?
  If so, answer is yes :
 
  #dmesg |grep iwn0
  iwn0: Intel Centrino Wireless-N 2230 mem 0xf7c0-0xf7c01fff irq
  18 at device 0.0 on pci4 #ifconfig wlan0
  wlan0: flags=8843UP,BROADCAST,RUNNING,SIMPLEX,MULTICAST metric 0
 mtu 1500
  nd6 options=29PERFORMNUD,IFDISABLED,AUTO_LINKLOCAL
  media: IEEE 802.11 Wireless Ethernet DS/1Mbps mode 11ng
  status: associated
  ssid FreeWifi channel 5 (2432 MHz 11g ht/20) Cedric
 
 
 Nice, ht/40 support to?

I don't know. I'm too noob in iwn to say that. I let someone else answer.
 Do you plan to support other cards Intel 6235 and supporting more than
 one range (5GHz)?
6235 is already supported by my version. I just wait for a friend to test it 
but it should be ok.

For other range, as for ht/40, I'm not enough expert for doing that, sorry.

 
 I think whether to change AR9285 base card or not :)
--
Cedric

___
freebsd-wireless@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-wireless
To unsubscribe, send any mail to freebsd-wireless-unsubscr...@freebsd.org


Re: IWN Review

2013-07-20 Thread Adrian Chadd
Hi!

This is great news!

Are you able to merge this into the iwn driver? Is it feasible? I'd
rather not have two drivers in the tree for the same family of NICs.

Thanks!



-adrian

On 20 July 2013 08:54, Cedric GROSS c...@cgross.info wrote:


 De : Andrey Fesenko [mailto:f0and...@gmail.com]
 [...]
 
  Great news, whether the driver supports n-mode?
 
  What do you mean by 'n-mode' ? 802.11n mode ?
  If so, answer is yes :
 
  #dmesg |grep iwn0
  iwn0: Intel Centrino Wireless-N 2230 mem 0xf7c0-0xf7c01fff irq
  18 at device 0.0 on pci4 #ifconfig wlan0
  wlan0: flags=8843UP,BROADCAST,RUNNING,SIMPLEX,MULTICAST metric 0
 mtu 1500
  nd6 options=29PERFORMNUD,IFDISABLED,AUTO_LINKLOCAL
  media: IEEE 802.11 Wireless Ethernet DS/1Mbps mode 11ng
  status: associated
  ssid FreeWifi channel 5 (2432 MHz 11g ht/20) Cedric
 

 Nice, ht/40 support to?

 I don't know. I'm too noob in iwn to say that. I let someone else answer.
 Do you plan to support other cards Intel 6235 and supporting more than
 one range (5GHz)?
 6235 is already supported by my version. I just wait for a friend to test it 
 but it should be ok.

 For other range, as for ht/40, I'm not enough expert for doing that, sorry.


 I think whether to change AR9285 base card or not :)
 --
 Cedric

 ___
 freebsd-wireless@freebsd.org mailing list
 http://lists.freebsd.org/mailman/listinfo/freebsd-wireless
 To unsubscribe, send any mail to freebsd-wireless-unsubscr...@freebsd.org
___
freebsd-wireless@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-wireless
To unsubscribe, send any mail to freebsd-wireless-unsubscr...@freebsd.org