[RFC] CONFIG_PM_LEGACY = compiler warnings for deprecation ... solution?

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



w/ CONFIG_PM_LEGACY .config option set, you get following batch of
compile errors because it marks the functions as 'deprecated'. w/o 
it set, it just uses stubs. compiler warnings for refernece (from 
2.6.23-1, though appears to have been around since at least 2.6.11 
from another mail list post).

However, since these functions are required if you have
CONFIG_PM_LEGACY option marked...

I propose that within the include/linux/pm_legacy.h file, functions
in the CONFIG_PM_LEGACY aren't marked 'deprecated' and functions
outside the #if are marked deprecated.  Then the handful of calls
(12 or so) that actually call the pm_legacy apis can get wrapped in
the CONFIG_PM_LEGACY flag.

This is basically already being done in one case since as of
2.6.20.1 the pm_register function is only used in
drivers/serial/68328serial.c.

The pm_send_all is not wrapped around all uses yet, but that is
pretty easy to patch.  Then the compile will actually complain
should someone try to call it and not wrap w/ the CONFIG_PM_LEGACY
ifdef.

Happy to hear thoughts or other solutions.

------------------------------
Regards,
Scott Thompson / postfail@xxxxxxxxxxxx
------------------------------

--
Free information on vast array of home improvement projects.  Click here.
http://tagline.hushmail.com/fc/Ioyw6h4eNIBTzslyd6sFcem0DGn1xrajHGmTczUmHzuMeqA2wEMUHS/

-
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux IBM ACPI]     [Linux Power Management]     [Linux Kernel]     [Linux Laptop]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux