BTW you right, got nervous me too.. configure_filter is called too many times.. mostly in managed mode! is that correct behavior? If its correct, the function needs a little optimization, in my opinion..I mean, I noticed that, after the first few calls, later the value of priv->rx_conf becomes redundant, IN value of priv->rx_conf is the same as OUT.. and rtl818x_iowrite32_async unleashes a hell of stack at every call.. :P Is it worth putting one more condition such as, only if priv->rx_conf IN!=OUT then call rtl818x_iowrite32_async ? Thanks :) On Wed, Dec 4, 2013, at 12:49 PM, Larry Finger wrote: > Until I fixed your patch, I could not be assured that it would work. I > get > nervous when that small a patch generates that many warhings. -- http://www.fastmail.fm - Does exactly what it says on the tin -- 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
