Hey Luca, > > Unfortunately I didn't have much time to look into your AP patches yet. > We need to review this very carefully. It touches so many parts of the > code that I'm a bit worried that this would break normal STA > functionality. Most of the patches add AP related functionality without touching the flow in STA mode. In fact the only patch that touches STA is "[PATCH 06/18] wl1271: AP mode - init sequence". The init sequence has been modified to unify the STA and AP common parts and separate the different parts into different functions. For STA mode this means a slight change in the order of initialization. Other changes are basically: 1. STA function renames 2. removing STA functionality when operating in AP mode by protecting code segments with "if (!is_ap)" 3. adding AP mode only functionality. This includes code segments with "if (is_ap)" in common STA/AP code. > > How well have you tested these patches in STA mode? Do you think there > would be any possibility of making this Kconfigurable so that there > would be no (or very little) binary difference in the code if AP is not > enabled? We have performed extensive stress tests on the init sequence part (constant on/off for several days). Basic throughput and stress tests were performed for STA mode operation (as well as AP mode operation). A third kind of test was a stress test of the switch between STA and AP modes. We feel making the AP mode Kconfigurable will make the code uglier and hurt maintainability. We were hoping we could avoid it by structuring the code as it is. Regards, Arik -- 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
