Re: [PATCH] ACPI: remove unused ACPI_PROCFS Kconfig param

[Lan Tianyu]

2013/10/27 Michael Opdenacker <michael.opdenacker@xxxxxxxxxxxxxxxxxx>:
> This patch removes the ACPI_PROCFS parameter
> (support for legacy /proc/acpi), which is defined
> but no longer used anywhere in the makefiles and source code.
>

I think this should be not removed now. Since there are still interfaces
under /proc/acpi (E.G, /proc/acpi/wakeup and /proc/acpi/lid/*).

The origin commit which introduces the kconfig is to make some
interfaces that have backup interfaces in the sysfs configurable before
removing them actually.

So we still need this kconfig when try to remove remaining /proc/acpi
interfaces.

commit b981c591891dc8885de36498d38fa8d8a5481069
Author: Zhang Rui <rui.zhang@xxxxxxxxx>
Date:   Mon Jan 29 11:02:30 2007 +0800

    ACPI: add a Kconfig option for ACPI procfs interface

    Add a kconfig option CONFIG_ACPI_PROCFS to make procfs interface
    a configurable attribute of ACPI.

    No procfs interface is actually deprecated,
    and no sysfs interface is added in this patch.

    CONGI_ACPI_PROCFS is used to mark procfs interface as deprecated
    once the same function is duplicated in sysfs.


> Signed-off-by: Michael Opdenacker <michael.opdenacker@xxxxxxxxxxxxxxxxxx>
> ---
>  drivers/acpi/Kconfig | 13 -------------
>  1 file changed, 13 deletions(-)
>
> diff --git a/drivers/acpi/Kconfig b/drivers/acpi/Kconfig
> index 6efe2ac..201cfb7 100644
> --- a/drivers/acpi/Kconfig
> +++ b/drivers/acpi/Kconfig
> @@ -43,19 +43,6 @@ config ACPI_SLEEP
>         depends on SUSPEND || HIBERNATION
>         default y
>
> -config ACPI_PROCFS
> -       bool "Deprecated /proc/acpi files"
> -       depends on PROC_FS
> -       help
> -         For backwards compatibility, this option allows
> -         deprecated /proc/acpi/ files to exist, even when
> -         they have been replaced by functions in /sys.
> -
> -         This option has no effect on /proc/acpi/ files
> -         and functions which do not yet exist in /sys.
> -
> -         Say N to delete /proc/acpi/ files that have moved to /sys/
> -
>  config ACPI_PROCFS_POWER
>         bool "Deprecated power /proc/acpi directories"
>         depends on PROC_FS
> --
> 1.8.1.2
>
> --
> 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



-- 
Best regards
Tianyu Lan
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/