Hi 2011/11/24 Johannes Berg <johannes@xxxxxxxxxxxxxxxx>: > On Thu, 2011-11-24 at 13:33 -0500, Nikolay Martynov wrote: > >> > Looks OK to me. Did you run it through sparse too? :-) >> > >> Please forgive my lack of experience, but could you please point me >> to some docs where I can read about sparse and how to use it? I'd >> really appreciate this. > > Sure, FWIW, I'm just asking because I received lots of patches recently > that later got sparse warnings. I guess you normally run something like > > make M=net/mac80211 > > To run sparse, you add C=1 (just run sparse if cc runs) or C=2 (run > sparse on all files). Obviously you need to install sparse first, your > distribution probably has packages. > I wasn't able to find any sparse warnings introduced by my changes, everything looks fine. -- Truthfully yours, Martynov Nikolay. Email: mar.kolya@xxxxxxxxx -- 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