lkml.org 
[lkml]   [2005]   [Dec]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
SubjectRe: remove incorrect dependancy on CONFIG_APM
On Thu, Dec 22, 2005 at 09:22:27PM -0500, Dave Jones wrote:

> > CC arch/i386/kernel/apm.o
> > arch/i386/kernel/apm.c: In function 'apm_init':
> > arch/i386/kernel/apm.c:2304: error: 'pm_active' undeclared (first use in this function)
> > arch/i386/kernel/apm.c:2304: error: (Each undeclared identifier is reported only once
> > arch/i386/kernel/apm.c:2304: error: for each function it appears in.)
> > arch/i386/kernel/apm.c: In function 'apm_exit':
> > arch/i386/kernel/apm.c:2410: error: 'pm_active' undeclared (first use in this function)
> > make[1]: *** [arch/i386/kernel/apm.o] Error 1
> >
> > <-- snip -->
> >
> > If PM_LEGACY causes user confusion for APM users, commit
> > bca73e4bf8563d83f7856164caa44d5f42e44cca should be reverted.
>
> Yeah, I realised that earlier too, my change was untested.
>
> Hrmph. For now I've enabled PM_LEGACY, but silently taking options
> away like this is what surprises users.

Living dangerously with another not-compile-tested patch.
(It's building right now, but I'm about to go eat, and
with my goldfish like attention span, I'll forget about it later)

The ifdef's are a bit ugly, but given it's a legacy interface,
maybe pm_active & co will all go away completely one day.

Signed-off-by: Dave Jones <davej@redhat.com>

--- linux-2.6.14/arch/i386/Kconfig~ 2005-12-22 22:06:10.000000000 -0500
+++ linux-2.6.14/arch/i386/Kconfig 2005-12-22 22:06:16.000000000 -0500
@@ -710,7 +710,7 @@ depends on PM && !X86_VISWS

config APM
tristate "APM (Advanced Power Management) BIOS support"
- depends on PM && PM_LEGACY
+ depends on PM
---help---
APM is a BIOS specification for saving power using several different
techniques. This is mostly useful for battery powered laptops with
--- linux-2.6.14/arch/i386/kernel/apm.c~ 2005-12-22 21:53:43.000000000 -0500
+++ linux-2.6.14/arch/i386/kernel/apm.c 2005-12-22 21:54:02.000000000 -0500
@@ -2301,7 +2301,9 @@ static int __init apm_init(void)
apm_info.disabled = 1;
return -ENODEV;
}
+#ifdef CONFIG_PM_LEGACY
pm_active = 1;
+#endif

/*
* Set up a segment that references the real mode segment 0x40
@@ -2407,7 +2409,9 @@ static void __exit apm_exit(void)
exit_kapmd = 1;
while (kapmd_running)
schedule();
+#ifdef CONFIG_PM_LEGACY
pm_active = 0;
+#endif
}

module_init(apm_init);
-
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/
\
 
 \ /
  Last update: 2005-12-23 04:18    [W:0.038 / U:0.112 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site