Re: [PATCH v2 5/5] ACPICA: Remove deprecated _LINUX definitions for ACPICA.

From: Rafael J. Wysocki
Date: Sun Apr 27 2014 - 17:22:27 EST


On Wednesday, April 23, 2014 02:54:22 PM Lv Zheng wrote:
> There are _LINUX defined so that when Linux kernel is compiled using broken
> compilers that having not __linux__ defined can still include
> <acpi/platform/aclinux.h> from <acpi/platform/acenv.h>.
>
> This behavior is deprecated as all drivers/acpi/acpica files are compiled
> without including <linux/acpi.h>, thus without _LINUX defined. As there is
> no issues encountered when we compile ACPICA code without _LINUX defined,
> it is OK to remove _LINUX from <linux/acpi.h> now.
>
> Signed-off-by: Lv Zheng <lv.zheng@xxxxxxxxx>
> ---
> include/linux/acpi.h | 4 ----
> 1 file changed, 4 deletions(-)
>
> diff --git a/include/linux/acpi.h b/include/linux/acpi.h
> index 7a8f2cd..9c559f7 100644
> --- a/include/linux/acpi.h
> +++ b/include/linux/acpi.h
> @@ -31,10 +31,6 @@
>
> #ifdef CONFIG_ACPI
>
> -#ifndef _LINUX
> -#define _LINUX
> -#endif
> -
> #include <linux/list.h>
> #include <linux/mod_devicetable.h>

What about <acpi/platform/acenv.h>? Should it still check if _LINUX is defined
after this change?


--
I speak only for myself.
Rafael J. Wysocki, Intel Open Source Technology Center.
--
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/