Re: [PATCH 00/14] RFC: Driver for Wireless RNDIS USB devices.

2008-01-25 Thread David Brownell
On Friday 25 January 2008, Jussi Kivilinna wrote:
 On Thu, 2008-01-24 at 17:19 -0800, David Brownell wrote:
   13. [rndis_host] blacklist known wireless RNDIS devices
  
  That will be a headache over time though ... can't you just
  let the probe succeed enough to recogize it's wireless (using
  the media flag) and then bail, so the next driver can try?
 
 Sure, that works too (but causes a little bit more message flood).

Excess messages can be dealt with, or at worst, ignored.

Thanks.

--
To unsubscribe from this list: send the line unsubscribe netdev in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH 00/14] RFC: Driver for Wireless RNDIS USB devices.

2008-01-25 Thread Jussi Kivilinna
On Thu, 2008-01-24 at 17:19 -0800, David Brownell wrote:
  13. [rndis_host] blacklist known wireless RNDIS devices
 
 That will be a headache over time though ... can't you just
 let the probe succeed enough to recogize it's wireless (using
 the media flag) and then bail, so the next driver can try?

Sure, that works too (but causes a little bit more message flood).


--
To unsubscribe from this list: send the line unsubscribe netdev in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH 00/14] RFC: Driver for Wireless RNDIS USB devices.

2008-01-24 Thread David Brownell
On Saturday 19 January 2008, Jussi Kivilinna wrote:
 Hello,
 
 This is second try on wireless RNDIS patchset started by Bjorge Dijkstra. 
 Since
 Bjorge has disappeared, I claim maintainership of rndis_wext and this patchset
 until he returns.
 
 This patchset adds support for various 802.11 USB devices based on Broadcom
 4320 chip. Chip uses RNDIS to communicate with the host, so module depend
 heavily on rndis_host/cdc_ether/usbnet and needs some changes on these 
 modules in order to work.
 
 Patches 1-6 are from first patchset:
  1. Fix sparse warning: returning void valued expression
  2. [cdc_ether] Hardwire CDC descriptors when missing
  3. [rndis_host] Use 1KB buffer in rndis_unbind
  4. [rndis_host] Halt device if rndis_bind fails
  5. [rndis_host] Fix rndis packet filter flags
  6. [usbnet] Use wlan device name for RNDIS wireless devices
 
 Of these 1, 3 and 4 are not required for this version of rndis_wext to work.
 
 Actual wireless part is changed from extension on rndis_host to separate 
 driver. Different devices are detected by device specific USB vendor/product 
 IDs as the way done with Windows drivers instead of detecting RNDIS media 
 type 
 like in first patchset.
 
 New patches 7-14:
  7. [rndis_host] Split up rndis_host.c
  8. [rndis_host] export functions
  9. [usbnet] add driver_priv pointer to 'struct usbnet'

So far as I'm concerned patches 1-9 can go in any time.

The other patches I won't ack yet; see below.


 10. [rndis_host] Add rndis_early_init function pointer to 'struct rndis_data'.
 11. [rndis_host] Add rndis_link_change function pointer to 'struct 
 rndis_data'.

Those aren't added to struct rndis_data ... they're added to
the struct at the core of the usbnet framework.  So they should
not be RNDIS-specific ... even though the only current user will
be the RNDIS host code.  Rename those methods and I'll be happy.


 12. Move usbnet.h and rndis_host.h to include/linux/usb

No problem with that, except that fixing #10 and #11 will
break them.


 13. [rndis_host] blacklist known wireless RNDIS devices

That will be a headache over time though ... can't you just
let the probe succeed enough to recogize it's wireless (using
the media flag) and then bail, so the next driver can try?


 14. Add new driver 'rndis_wext' for wireless RNDIS devices.

The real goods!  :)

 
 Patches should be applied in order, series apply cleanly to 2.6.24-rc8.
 
  - Jussi Kivilinna
 


--
To unsubscribe from this list: send the line unsubscribe netdev in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH 00/14] RFC: Driver for Wireless RNDIS USB devices.

2008-01-19 Thread Jussi Kivilinna
Hello,

This is second try on wireless RNDIS patchset started by Bjorge Dijkstra. Since
Bjorge has disappeared, I claim maintainership of rndis_wext and this patchset
until he returns.

This patchset adds support for various 802.11 USB devices based on Broadcom
4320 chip. Chip uses RNDIS to communicate with the host, so module depend
heavily on rndis_host/cdc_ether/usbnet and needs some changes on these 
modules in order to work.

Patches 1-6 are from first patchset:
 1. Fix sparse warning: returning void valued expression
 2. [cdc_ether] Hardwire CDC descriptors when missing
 3. [rndis_host] Use 1KB buffer in rndis_unbind
 4. [rndis_host] Halt device if rndis_bind fails
 5. [rndis_host] Fix rndis packet filter flags
 6. [usbnet] Use wlan device name for RNDIS wireless devices

Of these 1, 3 and 4 are not required for this version of rndis_wext to work.

Actual wireless part is changed from extension on rndis_host to separate 
driver. Different devices are detected by device specific USB vendor/product 
IDs as the way done with Windows drivers instead of detecting RNDIS media type 
like in first patchset.

New patches 7-14:
 7. [rndis_host] Split up rndis_host.c
 8. [rndis_host] export functions
 9. [usbnet] add driver_priv pointer to 'struct usbnet'
10. [rndis_host] Add rndis_early_init function pointer to 'struct rndis_data'.
11. [rndis_host] Add rndis_link_change function pointer to 'struct rndis_data'.
12. Move usbnet.h and rndis_host.h to include/linux/usb
13. [rndis_host] blacklist known wireless RNDIS devices
14. Add new driver 'rndis_wext' for wireless RNDIS devices.

Patches should be applied in order, series apply cleanly to 2.6.24-rc8.

 - Jussi Kivilinna

--
To unsubscribe from this list: send the line unsubscribe netdev in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html