On Monday, December 01, 2014 07:44:10 PM Joe Perches wrote: > > In addition there's a rate limitted warning message. > > limited. Yes, thanks. > I think this _ratelimited bit should be a separate patch. > (and a suggestion / comment below too) Agree, sending v2 > #define wil_err_ratelimited(wil, fmt, ...) \ > do { \ > if (net_ratelimit()) \ > wil_err(wil, fmt, ##__VA_ARGS__); \ > } while (0) Oh no, this way arguments won't be evaluated for suppressed messages. The way it is done is less compact but do predictable job with arguments. Thanks, Vladimir -- 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