On 08/26/2012 08:14 AM, Robert P. J. Day wrote: > On Sun, 26 Aug 2012, Randy Dunlap wrote: > >> On 08/26/2012 04:47 AM, Robert P. J. Day wrote: >> >>> >>> Fix *obvious* errors in current kerneldoc in both hsi.h and hsi.c, >>> including that a comment in the middle of the structure seems to screw >>> up kerneldoc processing. I didn't know that. >> >> >> FYI, scripts/kernel-doc supports using >> /* private: */ >> and >> /* public: */ >> >> to mark fields of a struct that the owner/maintainer wishes >> to retain as private fields, undocumented, generally not to be >> used/touched by common drivers, just used by the core code. >> >> In this case it does appear that the /* private: */ line might >> as well be deleted since the fields are documented. > > but when i removed that line, some warnings disappeared. i'll take > another look at that. Yes, private fields are not expected to have kernel-doc entries, so that causes warnings. > rday > > p.s. i removed carlos chinea's email address since it bounced as > invalid. > -- ~Randy -- To unsubscribe from this list: send the line "unsubscribe linux-doc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
