Messages in this thread | | | Date | Mon, 13 Aug 2012 19:00:27 -0700 | From | Greg KH <> | Subject | Re: [PATCH 4/5] Redocument some functions in android/logger.c |
| |
On Wed, Aug 01, 2012 at 02:54:19PM +1000, Cruz Julian Bishop wrote: > I will document the rest later if they remain unchanged > Normally, I would do them all at once, but I don't have the chance to do them all at the moment > > Signed-off-by: Cruz Julian Bishop <cruzjbishop@gmail.com> > --- > drivers/staging/android/logger.c | 90 +++++++++++++++++++++++++------------- > 1 file changed, 60 insertions(+), 30 deletions(-) > > diff --git a/drivers/staging/android/logger.c b/drivers/staging/android/logger.c > index 1d5ed47..226d8b5 100644 > --- a/drivers/staging/android/logger.c > +++ b/drivers/staging/android/logger.c > @@ -78,15 +78,20 @@ struct logger_reader { > size_t r_off; > }; > > -/* logger_offset - returns index 'n' into the log via (optimized) modulus */ > +/** > + * logger_offset() - returns index 'n' into the log via (optimized) modulus > + * @log: The log being referenced > + * @n: The index number being referenced > + */ > static size_t logger_offset(struct logger_log *log, size_t n)
There is no need to document static functions in this style, unless you really feel it is needed.
For simple things like this, it isn't needed at all, so I'm not going to apply this patch, sorry.
greg k-h
| |