Messages in this thread | | | Date | Wed, 8 Feb 2006 09:16:49 -0800 | From | Greg KH <> | Subject | Re: [linux-pm] Re: [PATCH, RFC] [1/3] Generic in-kernel AC status |
| |
On Wed, Feb 08, 2006 at 05:08:58PM +0000, Matthew Garrett wrote: > On Wed, Feb 08, 2006 at 08:58:03AM -0800, Greg KH wrote: > > > +{ > > > + down(&pm_sem); > > > > Shouldn't this be a mutex? > > It is, isn't it? The name's somewhat misleading, but I was just using > what already existed...
Ah, ok, nevermind, I thought this was new.
> +/** > + * pm_set_ac_status - Set the ac status callback. Returns true if the > + * system is on AC and has a registered callback.
kerneldoc will not work with this. It needs to be a one line, short, description. Put the full description below the function paramaters, it can be as many lines as you want there.
thanks,
greg k-h - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |