On Thu, 2016-08-11 at 11:38 -0500, Denis Kenzior wrote: > Hi Johannes, > > On 08/11/2016 07:47 AM, Johannes Berg wrote: > > > > On Wed, 2016-08-03 at 17:02 -0500, Denis Kenzior wrote: > > > > > > > > > +static int nl80211_dump_interface_parse(struct sk_buff *skb, > > > + struct netlink_callback *cb, > > > + int *filter_wiphy) > > > > Wrong indentation :) > > Sorry :) > > Speaking of indentation, can you point me to a doc of the rules I > should follow? You've seen Documentation/CodingStyle? > I can confirm that I sanity checked this patch. Both ATTR_WIPHY, > ATTR_WDEV and wildcard dumps seemed to produce expected results. I think it probably works due to the other conditions? > I noticed you applied this patch. Is there a particular scenario > where it goes wrong or did you convince yourself it is correct? > No, that was a mistake :( I've removed it now. johannes -- 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
