On Mon, 21 Mar 2011 21:38:12 +0100 Johannes Berg wrote: > On Mon, 2011-03-21 at 13:28 -0700, Randy Dunlap wrote: > > > > +/** > > > + * DOC: Core Utility Functions > > > This is not kernel-doc notation. Change /** to /*. > > Actually, it is, if you prefix with DOC: you can embed that into the > generated DocBook with > !P<file> <heading name> > and include the text. I did that in b112e0f7 :-) Yes, I'm familiar with that notation. bah. You omitted a few lines of comments: > +/** This is not kernel-doc notation. Change /** to /*. > + * Manufacturer Ids > + */ > > > +/** > > > > ditto > > > > > + * Component Classes > > You're right for most of the other ones of course. --- ~Randy *** Remember to use Documentation/SubmitChecklist when testing your code *** -- 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