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:

quoted

quoted

+{
+	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