Johannes Berg wrote: > > A bunch > > of parameters are already set with a single function, only config > > and config_interface need to be replaced by parameter-specific > > functions. Reset should be removed. > > Not sure what reset is for. Now, we are at least two. > > add_interface() and > > remove_interface() are comparable to my enable_mac() and > > disable_mac() as you discovered. > > Aha, but they're not fully compatible. First of all, you were talking > about multiple MACs which isn't something we're going to support > (multiple MAC addresses yes). I would state we should the support the functionality that the device supports. Everything else is "form follows function". > Secondly, a WDS and AP interface will be > running with the same MAC address but have quite different > configuration. Thirdly, always doing lookups by MAC address (even if it > were possible which it is not due to WDS vs AP) is rather inefficient. The MAC address lookup issue could be handled. Assuming that the device would be able to create beacons on a specific MAC address, would it need to know about WDS or AP mode? > > Tell the driver on the real interface, which packets it should > > report to the mac80211 stack. > > That's what I proposed. > > > Don't bother the driver with virtual > > interfaces. The stack should handle it completely transparently. > > What I'm trying to explain to you is that basically you cannot do that. > AP vs WDS interfaces, lookups by MAC address, beaconing, ... Beaconing needs to be supported by the device. AP and WDS are working on the same MAC address, but why should the people know about it. It doesn't look into the packets transmitted. > > Don't call the driver before open() or after stop() is called. > > Do we do that? In any case, it's rather orthogonal to the issue at hand. Sure. Some drivers support the configuration of the channel before the device is up. There has been the request for the zd1211rw driver to shadow the configuration in the device data structure. But I believe that is something the stack should handle. Ciao, Uli -- Uli Kunitz - To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
