Re: [PATCH 00/39] merge request for WiMAX kernel stack and i2400m driver v2

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Johannes Berg
Date: Thursday, December 4, 2008 - 2:00 am

On Tue, 2008-12-02 at 18:07 -0800, Inaky Perez-Gonzalez wrote:


But things like these are fairly easy to cover, just allow netlink
attributes to specify where to scan, and allow drivers to disregard
them, no harm caused. Maybe include a capability bit, like I'm adding to
nl80211 in my scanning patch (not included yet) that includes a
capability for how many SSIDs it can scan actively at once.


I have no idea about these things, obviously. But what's wrong with just
defining the scan operation with netlink attributes as you need them now
(say the scan returns NSPs) and then later when somebody needs to return
NAPs add a new attribute? Userspace will easily be able to figure out
which one it got by looking at which attributes are present.


Here's where I disagree, obviously, I think you should at least define a
subset of the imaginable interface, which is, in my opinion, _much_
better than defining no interface at all and hoping for the next guy to
figure it out, which is unlikely to happen when you haven't started with
something the next guy can understand.


Ok, I guess that makes sense then, I'm not aware of all the details.


Ok, thanks for the explanation.

johannes

--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH 00/39] merge request for WiMAX kernel stack and i24 ..., Inaky Perez-Gonzalez, (Wed Nov 26, 3:40 pm)
[PATCH 01/39] wimax: documentation for the stack, Inaky Perez-Gonzalez, (Wed Nov 26, 3:40 pm)
[PATCH 02/39] wimax: declarations for the in-kernel WiMAX API, Inaky Perez-Gonzalez, (Wed Nov 26, 3:40 pm)
[PATCH 03/39] wimax: constants and definitions to interact ..., Inaky Perez-Gonzalez, (Wed Nov 26, 3:40 pm)
[PATCH 04/39] wimax: internal API for the kernel space WiM ..., Inaky Perez-Gonzalez, (Wed Nov 26, 3:40 pm)
[PATCH 05/39] wimax: debug macros and debug settings for t ..., Inaky Perez-Gonzalez, (Wed Nov 26, 3:40 pm)
[PATCH 06/39] genetlink: export genl_unregister_mc_group(), Inaky Perez-Gonzalez, (Wed Nov 26, 3:40 pm)
[PATCH 07/39] wimax: generic WiMAX device management (regi ..., Inaky Perez-Gonzalez, (Wed Nov 26, 3:40 pm)
[PATCH 08/39] wimax: Mappping of generic netlink family ID ..., Inaky Perez-Gonzalez, (Wed Nov 26, 3:40 pm)
[PATCH 09/39] wimax: provides user space with information ..., Inaky Perez-Gonzalez, (Wed Nov 26, 3:40 pm)
[PATCH 10/39] wimax: Generic messaging interface between u ..., Inaky Perez-Gonzalez, (Wed Nov 26, 3:40 pm)
[PATCH 11/39] wimax: RF-kill framework integration, Inaky Perez-Gonzalez, (Wed Nov 26, 3:40 pm)
[PATCH 12/39] wimax: API call to reset a WiMAX device, Inaky Perez-Gonzalez, (Wed Nov 26, 3:40 pm)
[PATCH 13/39] wimax: Makefile, Kconfig and docbook linkage ..., Inaky Perez-Gonzalez, (Wed Nov 26, 3:40 pm)
[PATCH 14/39] i2400m: documentation and instructions for usage, Inaky Perez-Gonzalez, (Wed Nov 26, 3:40 pm)
[PATCH 15/39] i2400m: host-to-device protocol definitions, Inaky Perez-Gonzalez, (Wed Nov 26, 3:40 pm)
[PATCH 16/39] i2400m: core driver definitions and API, Inaky Perez-Gonzalez, (Wed Nov 26, 3:40 pm)
[PATCH 17/39] i2400m: Generic probe/disconnect, reset and ..., Inaky Perez-Gonzalez, (Wed Nov 26, 3:40 pm)
[PATCH 18/39] i2400m: linkage to the networking stack, Inaky Perez-Gonzalez, (Wed Nov 26, 3:40 pm)
[PATCH 19/39] i2400m: sysfs controls, Inaky Perez-Gonzalez, (Wed Nov 26, 3:40 pm)
[PATCH 20/39] i2400m: rfkill integration with the WiMAX stack, Inaky Perez-Gonzalez, (Wed Nov 26, 3:40 pm)
[PATCH 21/39] i2400m: firmware loading and bootrom initial ..., Inaky Perez-Gonzalez, (Wed Nov 26, 3:40 pm)
[PATCH 22/39] i2400m: handling of the data/control recepti ..., Inaky Perez-Gonzalez, (Wed Nov 26, 3:40 pm)
[PATCH 23/39] i2400m: handling of the data/control transmi ..., Inaky Perez-Gonzalez, (Wed Nov 26, 3:40 pm)
[PATCH 24/39] i2400m: various functions for device management, Inaky Perez-Gonzalez, (Wed Nov 26, 3:40 pm)
[PATCH 25/39] i2400m/USB: header for the USB bus driver, Inaky Perez-Gonzalez, (Wed Nov 26, 3:40 pm)
[PATCH 26/39] i2400m/USB: error density tracking, Inaky Perez-Gonzalez, (Wed Nov 26, 3:40 pm)
[PATCH 27/39] i2400m/USB: main probe/disconnect and backen ..., Inaky Perez-Gonzalez, (Wed Nov 26, 3:40 pm)
[PATCH 28/39] i2400m/USB: firmware upload backend, Inaky Perez-Gonzalez, (Wed Nov 26, 3:40 pm)
[PATCH 29/39] i2400m/USB: handling of notifications from t ..., Inaky Perez-Gonzalez, (Wed Nov 26, 3:40 pm)
[PATCH 30/39] i2400m/USB: read transactions from the USB d ..., Inaky Perez-Gonzalez, (Wed Nov 26, 3:40 pm)
[PATCH 31/39] i2400m/USB: write transactions to the USB device, Inaky Perez-Gonzalez, (Wed Nov 26, 3:40 pm)
[PATCH 32/39] i2400m/SDIO: header for the SDIO subdriver, Inaky Perez-Gonzalez, (Wed Nov 26, 3:40 pm)
[PATCH 33/39] i2400m/SDIO: main probe/disconnect and backe ..., Inaky Perez-Gonzalez, (Wed Nov 26, 3:40 pm)
[PATCH 34/39] i2400m/SDIO: firmware upload backend, Inaky Perez-Gonzalez, (Wed Nov 26, 3:40 pm)
[PATCH 35/39] i2400m/SDIO: read transactions from the SDIO ..., Inaky Perez-Gonzalez, (Wed Nov 26, 3:40 pm)
[PATCH 36/39] i2400m/SDIO: write transactions to the SDIO ..., Inaky Perez-Gonzalez, (Wed Nov 26, 3:40 pm)
[PATCH 37/39] i2400m: Makefile and Kconfig, Inaky Perez-Gonzalez, (Wed Nov 26, 3:40 pm)
[PATCH 38/39] wimax: export linux/wimax.h and linux/wimax/ ..., Inaky Perez-Gonzalez, (Wed Nov 26, 3:40 pm)
[PATCH 39/39] wimax/i2400m: add CREDITS and MAINTAINERS en ..., Inaky Perez-Gonzalez, (Wed Nov 26, 3:40 pm)
Re: [PATCH 19/39] i2400m: sysfs controls, Johannes Berg, (Thu Nov 27, 2:23 am)
Re: [PATCH 00/39] merge request for WiMAX kernel stack and ..., Inaky Perez-Gonzalez, (Thu Nov 27, 2:24 am)
Re: [PATCH 01/39] wimax: documentation for the stack, Johannes Berg, (Thu Nov 27, 2:29 am)
Re: [PATCH 11/39] wimax: RF-kill framework integration, Johannes Berg, (Thu Nov 27, 2:56 am)
Re: [PATCH 12/39] wimax: API call to reset a WiMAX device, Johannes Berg, (Thu Nov 27, 2:58 am)
Re: [PATCH 00/39] merge request for WiMAX kernel stack and ..., Arkadiusz Miskiewicz, (Thu Nov 27, 3:18 am)
Re: [PATCH 00/39] merge request for WiMAX kernel stack and ..., Inaky Perez-Gonzalez, (Thu Nov 27, 9:51 am)
Re: [PATCH 10/39] wimax: Generic messaging interface betwe ..., Inaky Perez-Gonzalez, (Tue Dec 2, 7:02 pm)
Re: [PATCH 11/39] wimax: RF-kill framework integration, Inaky Perez-Gonzalez, (Tue Dec 2, 7:03 pm)
Re: [PATCH 12/39] wimax: API call to reset a WiMAX device, Inaky Perez-Gonzalez, (Tue Dec 2, 7:05 pm)
Re: [PATCH 03/39] wimax: constants and definitions to inte ..., Inaky Perez-Gonzalez, (Tue Dec 2, 7:06 pm)
Re: [PATCH 09/39] wimax: provides user space with informat ..., Inaky Perez-Gonzalez, (Tue Dec 2, 7:06 pm)
Re: [PATCH 07/39] wimax: generic WiMAX device management ( ..., Inaky Perez-Gonzalez, (Tue Dec 2, 7:06 pm)
Re: [PATCH 08/39] wimax: Mappping of generic netlink famil ..., Inaky Perez-Gonzalez, (Tue Dec 2, 7:06 pm)
Re: [PATCH 14/39] i2400m: documentation and instructions f ..., Inaky Perez-Gonzalez, (Tue Dec 2, 7:06 pm)
Re: [PATCH 15/39] i2400m: host-to-device protocol definitions, Inaky Perez-Gonzalez, (Tue Dec 2, 7:06 pm)
Re: [PATCH 00/39] merge request for WiMAX kernel stack and ..., Inaky Perez-Gonzalez, (Tue Dec 2, 7:07 pm)
Re: [PATCH 02/39] wimax: declarations for the in-kernel Wi ..., Inaky Perez-Gonzalez, (Tue Dec 2, 7:07 pm)
Re: [PATCH 04/39] wimax: internal API for the kernel space ..., Inaky Perez-Gonzalez, (Tue Dec 2, 7:07 pm)
Re: [PATCH 05/39] wimax: debug macros and debug settings f ..., Inaky Perez-Gonzalez, (Tue Dec 2, 7:07 pm)
Re: [PATCH 01/39] wimax: documentation for the stack, Inaky Perez-Gonzalez, (Tue Dec 2, 7:07 pm)
Re: [PATCH 00/39] merge request for WiMAX kernel stack and ..., Inaky Perez-Gonzalez, (Tue Dec 2, 7:10 pm)
Re: [PATCH 00/39] merge request for WiMAX kernel stack and ..., Johannes Berg, (Thu Dec 4, 2:00 am)
Re: [PATCH 00/39] merge request for WiMAX kernel stack and ..., Inaky Perez-Gonzalez, (Thu Dec 4, 12:21 pm)
Re: [PATCH 04/39] wimax: internal API for the kernel space ..., Inaky Perez-Gonzalez, (Thu Dec 4, 12:22 pm)
Re: [PATCH 02/39] wimax: declarations for the in-kernel Wi ..., Inaky Perez-Gonzalez, (Thu Dec 4, 1:11 pm)