On Wed, Feb 08, 2006 at 08:58:03AM -0800, Greg KH wrote:
> > +void pm_set_ac_status(int (*ac_status_function)(void))
>
> No extra line in there please.
That matches the style used in the rest of the file.
> And perhaps a bit more description of what this is used for?
Ok.
> > +{
> > + 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...
> > + get_ac_status = ac_status_function;
> > + up(&pm_sem);
> > +}
> > +
> > +EXPORT_SYMBOL(pm_set_ac_status);
>
> No extra line between the function and the EXPORT_SYMBOL please.
Sure.
> Also, how about EXPORT_SYMBOL_GPL()?
If that's considered reasonable, sure.
> And, who will be using this interface, and what for?
The backlight interface only supports exporting and setting the current
brightness. For various bits of hardware, the AC and DC brightnesses are
stored separately. Drivers would need to know which brightness value to
export to userspace. I have an HP backlight driver here which would
benefit from this, and I'm looking at the same issue for a Panasonic
one.
Here's an updated version, anyway.
Signed-Off-By: Matthew Garrett <[email protected]>
diff --git a/include/linux/pm.h b/include/linux/pm.h
index 5be87ba..59ece0f 100644
--- a/include/linux/pm.h
+++ b/include/linux/pm.h
@@ -129,6 +129,8 @@ struct pm_ops {
extern void pm_set_ops(struct pm_ops *);
extern struct pm_ops *pm_ops;
+extern void pm_set_ac_status(int (*ac_status_function)(void));
+extern int pm_get_ac_status(void);
extern int pm_suspend(suspend_state_t state);
diff --git a/kernel/power/main.c b/kernel/power/main.c
index 46110d5..929a3c0 100644
--- a/kernel/power/main.c
+++ b/kernel/power/main.c
@@ -15,7 +15,7 @@
#include <linux/errno.h>
#include <linux/init.h>
#include <linux/pm.h>
-
+#include <linux/module.h>
#include "power.h"
@@ -25,6 +25,7 @@
DECLARE_MUTEX(pm_sem);
struct pm_ops *pm_ops = NULL;
+int (*get_ac_status)(void);
suspend_disk_method_t pm_disk_mode = PM_DISK_SHUTDOWN;
/**
@@ -39,6 +40,36 @@ void pm_set_ops(struct pm_ops * ops)
up(&pm_sem);
}
+/**
+ * pm_set_ac_status - Set the ac status callback. Returns true if the
+ * system is on AC and has a registered callback.
+ * @ops: Pointer to function
+ */
+
+void pm_set_ac_status(int (*ac_status_function)(void))
+{
+ down(&pm_sem);
+ get_ac_status = ac_status_function;
+ up(&pm_sem);
+}
+EXPORT_SYMBOL_GPL(pm_set_ac_status);
+
+/**
+ * pm_get_ac_status - return true if the system is on AC
+ */
+
+int pm_get_ac_status(void)
+{
+ int status=0;
+
+ down (&pm_sem);
+ if (get_ac_status)
+ status = get_ac_status();
+ up (&pm_sem);
+
+ return status;
+}
+EXPORT_SYMBOL_GPL(pm_get_ac_status);
/**
* suspend_prepare - Do prep work before entering low-power state.
--
Matthew Garrett | [email protected]
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
