From mboxrd@z Thu Jan 1 00:00:00 1970 Content-Type: multipart/mixed; boundary="===============3030431144339696949==" MIME-Version: 1.0 From: Al Stone Subject: Re: [Devel] [PATCH 7/7] ACPI: use Linux as ACPI_OS_NAME for _OS on ARM64 Date: Fri, 23 Jan 2015 16:34:54 +0000 Message-ID: <54C2782A.8090200@redhat.com> In-Reply-To: 3375613.hf7nCUoE0P@vostro.rjw.lan List-ID: To: devel@acpica.org --===============3030431144339696949== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable On 01/23/2015 08:55 AM, Rafael J. Wysocki wrote: > On Thursday, January 22, 2015 05:44:44 PM al.stone(a)linaro.org wrote: >> From: Hanjun Guo >> >> ACPI_OS_NAME is globally defined as "Microsoft Windows NT" for now. >> That doesn't make much sense in the ARM context, so set it to "Linux" >> when CONFIG_ARM64. >> >> If it is necessary to change the return value from \_OS_ (that is, return >> some value other than the default in ACPI_OS_NAME), use the kernel param= eter >> "acpi_os_name=3D". >> >> Signed-off-by: Hanjun Guo >> --- >> arch/arm64/Kconfig | 1 + >> drivers/acpi/Kconfig | 8 ++++++++ >> include/acpi/acconfig.h | 2 ++ >> include/acpi/platform/aclinux.h | 4 ++++ >> 4 files changed, 15 insertions(+) >> >> diff --git a/arch/arm64/Kconfig b/arch/arm64/Kconfig >> index 915aa16..20677d1 100644 >> --- a/arch/arm64/Kconfig >> +++ b/arch/arm64/Kconfig >> @@ -1,5 +1,6 @@ > = > If you added > = > config ACPI_OS_NAME > string > default "Linux" > depends on ACPI > = > to this file, then you don't need to add the new select below and -> > = >> config ARM64 >> def_bool y >> + select ACPI_OS_NAME_LINUX if ACPI >> select ACPI_REDUCED_HARDWARE_ONLY if ACPI >> select ARCH_BINFMT_ELF_RANDOMIZE_PIE >> select ARCH_HAS_ATOMIC64_DEC_IF_POSITIVE >> diff --git a/drivers/acpi/Kconfig b/drivers/acpi/Kconfig >> index 3e3bd35..024c5f0 100644 >> --- a/drivers/acpi/Kconfig >> +++ b/drivers/acpi/Kconfig >> @@ -369,6 +369,14 @@ config ACPI_REDUCED_HARDWARE_ONLY >> = >> If you are unsure what to do, do not enable this option. >> = >> +config ACPI_OS_NAME_LINUX >> + bool "Using Linux for _OS method" if EXPERT >> + def_bool n >> + When used, this option makes sure the _OS method returns >> + the value "Linux". >> + >> + If you are unsure what to do, do not enable this option. >> + > = > -> you don't need this at all ->> > = >> source "drivers/acpi/apei/Kconfig" >> = >> config ACPI_EXTLOG >> diff --git a/include/acpi/acconfig.h b/include/acpi/acconfig.h >> index 5a0a3e5..1980bf4 100644 >> --- a/include/acpi/acconfig.h >> +++ b/include/acpi/acconfig.h >> @@ -69,7 +69,9 @@ >> * code that will not execute the _OSI method unless _OS matches the st= ring >> * below. Therefore, change this string at your own risk. >> */ >> +#ifndef ACPI_OS_NAME >> #define ACPI_OS_NAME "Microsoft Windows NT" >> +#endif >> = >> /* Maximum objects in the various object caches */ >> = >> diff --git a/include/acpi/platform/aclinux.h b/include/acpi/platform/acl= inux.h >> index 1ba7c19..a8a7ee3 100644 >> --- a/include/acpi/platform/aclinux.h >> +++ b/include/acpi/platform/aclinux.h >> @@ -69,6 +69,10 @@ >> #define ACPI_REDUCED_HARDWARE 1 >> #endif >> = >> +#ifdef CONFIG_ACPI_OS_NAME_LINUX >> +#define ACPI_OS_NAME "Linux" >> +#endif > = > ->> and here you can do > = > +#ifdef CONFIG_ACPI_OS_NAME > +#define ACPI_OS_NAME CONFIG_ACPI_OS_NAME > +#endif > = > unless I'm missing something. > = >> + >> #include >> #include >> #include >> > = D'oh. Yup. I'll correct it. -- = ciao, al ----------------------------------- Al Stone Software Engineer Red Hat, Inc. ahs3(a)redhat.com ----------------------------------- --===============3030431144339696949==-- From mboxrd@z Thu Jan 1 00:00:00 1970 From: Al Stone Subject: Re: [PATCH 7/7] ACPI: use Linux as ACPI_OS_NAME for _OS on ARM64 Date: Fri, 23 Jan 2015 09:34:50 -0700 Message-ID: <54C2782A.8090200@redhat.com> References: <1421973884-13029-1-git-send-email-al.stone@linaro.org> <1421973884-13029-8-git-send-email-al.stone@linaro.org> <3375613.hf7nCUoE0P@vostro.rjw.lan> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Return-path: Received: from mx1.redhat.com ([209.132.183.28]:36030 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755889AbbAWQfe (ORCPT ); Fri, 23 Jan 2015 11:35:34 -0500 In-Reply-To: <3375613.hf7nCUoE0P@vostro.rjw.lan> Sender: linux-acpi-owner@vger.kernel.org List-Id: linux-acpi@vger.kernel.org To: "Rafael J. Wysocki" , al.stone@linaro.org Cc: tony.luck@intel.com, fenghua.yu@intel.com, catalin.marinas@arm.com, will.deacon@arm.com, tglx@linutronix.de, mingo@redhat.com, hpa@zytor.com, lenb@kernel.org, robert.moore@intel.com, linux-ia64@vger.kernel.org, linux-kernel@vger.kernel.org, linux-acpi@vger.kernel.org, linux-arm-kernel@lists.infradead.org, devel@acpica.org, linaro-acpi@lists.linaro.org, linaro-kernel@lists.linaro.org, patches@linaro.org On 01/23/2015 08:55 AM, Rafael J. Wysocki wrote: > On Thursday, January 22, 2015 05:44:44 PM al.stone@linaro.org wrote: >> From: Hanjun Guo >> >> ACPI_OS_NAME is globally defined as "Microsoft Windows NT" for now. >> That doesn't make much sense in the ARM context, so set it to "Linux" >> when CONFIG_ARM64. >> >> If it is necessary to change the return value from \_OS_ (that is, return >> some value other than the default in ACPI_OS_NAME), use the kernel parameter >> "acpi_os_name=". >> >> Signed-off-by: Hanjun Guo >> --- >> arch/arm64/Kconfig | 1 + >> drivers/acpi/Kconfig | 8 ++++++++ >> include/acpi/acconfig.h | 2 ++ >> include/acpi/platform/aclinux.h | 4 ++++ >> 4 files changed, 15 insertions(+) >> >> diff --git a/arch/arm64/Kconfig b/arch/arm64/Kconfig >> index 915aa16..20677d1 100644 >> --- a/arch/arm64/Kconfig >> +++ b/arch/arm64/Kconfig >> @@ -1,5 +1,6 @@ > > If you added > > config ACPI_OS_NAME > string > default "Linux" > depends on ACPI > > to this file, then you don't need to add the new select below and -> > >> config ARM64 >> def_bool y >> + select ACPI_OS_NAME_LINUX if ACPI >> select ACPI_REDUCED_HARDWARE_ONLY if ACPI >> select ARCH_BINFMT_ELF_RANDOMIZE_PIE >> select ARCH_HAS_ATOMIC64_DEC_IF_POSITIVE >> diff --git a/drivers/acpi/Kconfig b/drivers/acpi/Kconfig >> index 3e3bd35..024c5f0 100644 >> --- a/drivers/acpi/Kconfig >> +++ b/drivers/acpi/Kconfig >> @@ -369,6 +369,14 @@ config ACPI_REDUCED_HARDWARE_ONLY >> >> If you are unsure what to do, do not enable this option. >> >> +config ACPI_OS_NAME_LINUX >> + bool "Using Linux for _OS method" if EXPERT >> + def_bool n >> + When used, this option makes sure the _OS method returns >> + the value "Linux". >> + >> + If you are unsure what to do, do not enable this option. >> + > > -> you don't need this at all ->> > >> source "drivers/acpi/apei/Kconfig" >> >> config ACPI_EXTLOG >> diff --git a/include/acpi/acconfig.h b/include/acpi/acconfig.h >> index 5a0a3e5..1980bf4 100644 >> --- a/include/acpi/acconfig.h >> +++ b/include/acpi/acconfig.h >> @@ -69,7 +69,9 @@ >> * code that will not execute the _OSI method unless _OS matches the string >> * below. Therefore, change this string at your own risk. >> */ >> +#ifndef ACPI_OS_NAME >> #define ACPI_OS_NAME "Microsoft Windows NT" >> +#endif >> >> /* Maximum objects in the various object caches */ >> >> diff --git a/include/acpi/platform/aclinux.h b/include/acpi/platform/aclinux.h >> index 1ba7c19..a8a7ee3 100644 >> --- a/include/acpi/platform/aclinux.h >> +++ b/include/acpi/platform/aclinux.h >> @@ -69,6 +69,10 @@ >> #define ACPI_REDUCED_HARDWARE 1 >> #endif >> >> +#ifdef CONFIG_ACPI_OS_NAME_LINUX >> +#define ACPI_OS_NAME "Linux" >> +#endif > > ->> and here you can do > > +#ifdef CONFIG_ACPI_OS_NAME > +#define ACPI_OS_NAME CONFIG_ACPI_OS_NAME > +#endif > > unless I'm missing something. > >> + >> #include >> #include >> #include >> > D'oh. Yup. I'll correct it. -- ciao, al ----------------------------------- Al Stone Software Engineer Red Hat, Inc. ahs3@redhat.com ----------------------------------- From mboxrd@z Thu Jan 1 00:00:00 1970 From: Al Stone Date: Fri, 23 Jan 2015 16:34:50 +0000 Subject: Re: [PATCH 7/7] ACPI: use Linux as ACPI_OS_NAME for _OS on ARM64 Message-Id: <54C2782A.8090200@redhat.com> List-Id: References: <1421973884-13029-1-git-send-email-al.stone@linaro.org> <1421973884-13029-8-git-send-email-al.stone@linaro.org> <3375613.hf7nCUoE0P@vostro.rjw.lan> In-Reply-To: <3375613.hf7nCUoE0P@vostro.rjw.lan> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: "Rafael J. Wysocki" , al.stone@linaro.org Cc: tony.luck@intel.com, fenghua.yu@intel.com, catalin.marinas@arm.com, will.deacon@arm.com, tglx@linutronix.de, mingo@redhat.com, hpa@zytor.com, lenb@kernel.org, robert.moore@intel.com, linux-ia64@vger.kernel.org, linux-kernel@vger.kernel.org, linux-acpi@vger.kernel.org, linux-arm-kernel@lists.infradead.org, devel@acpica.org, linaro-acpi@lists.linaro.org, linaro-kernel@lists.linaro.org, patches@linaro.org On 01/23/2015 08:55 AM, Rafael J. Wysocki wrote: > On Thursday, January 22, 2015 05:44:44 PM al.stone@linaro.org wrote: >> From: Hanjun Guo >> >> ACPI_OS_NAME is globally defined as "Microsoft Windows NT" for now. >> That doesn't make much sense in the ARM context, so set it to "Linux" >> when CONFIG_ARM64. >> >> If it is necessary to change the return value from \_OS_ (that is, return >> some value other than the default in ACPI_OS_NAME), use the kernel parameter >> "acpi_os_name=". >> >> Signed-off-by: Hanjun Guo >> --- >> arch/arm64/Kconfig | 1 + >> drivers/acpi/Kconfig | 8 ++++++++ >> include/acpi/acconfig.h | 2 ++ >> include/acpi/platform/aclinux.h | 4 ++++ >> 4 files changed, 15 insertions(+) >> >> diff --git a/arch/arm64/Kconfig b/arch/arm64/Kconfig >> index 915aa16..20677d1 100644 >> --- a/arch/arm64/Kconfig >> +++ b/arch/arm64/Kconfig >> @@ -1,5 +1,6 @@ > > If you added > > config ACPI_OS_NAME > string > default "Linux" > depends on ACPI > > to this file, then you don't need to add the new select below and -> > >> config ARM64 >> def_bool y >> + select ACPI_OS_NAME_LINUX if ACPI >> select ACPI_REDUCED_HARDWARE_ONLY if ACPI >> select ARCH_BINFMT_ELF_RANDOMIZE_PIE >> select ARCH_HAS_ATOMIC64_DEC_IF_POSITIVE >> diff --git a/drivers/acpi/Kconfig b/drivers/acpi/Kconfig >> index 3e3bd35..024c5f0 100644 >> --- a/drivers/acpi/Kconfig >> +++ b/drivers/acpi/Kconfig >> @@ -369,6 +369,14 @@ config ACPI_REDUCED_HARDWARE_ONLY >> >> If you are unsure what to do, do not enable this option. >> >> +config ACPI_OS_NAME_LINUX >> + bool "Using Linux for _OS method" if EXPERT >> + def_bool n >> + When used, this option makes sure the _OS method returns >> + the value "Linux". >> + >> + If you are unsure what to do, do not enable this option. >> + > > -> you don't need this at all ->> > >> source "drivers/acpi/apei/Kconfig" >> >> config ACPI_EXTLOG >> diff --git a/include/acpi/acconfig.h b/include/acpi/acconfig.h >> index 5a0a3e5..1980bf4 100644 >> --- a/include/acpi/acconfig.h >> +++ b/include/acpi/acconfig.h >> @@ -69,7 +69,9 @@ >> * code that will not execute the _OSI method unless _OS matches the string >> * below. Therefore, change this string at your own risk. >> */ >> +#ifndef ACPI_OS_NAME >> #define ACPI_OS_NAME "Microsoft Windows NT" >> +#endif >> >> /* Maximum objects in the various object caches */ >> >> diff --git a/include/acpi/platform/aclinux.h b/include/acpi/platform/aclinux.h >> index 1ba7c19..a8a7ee3 100644 >> --- a/include/acpi/platform/aclinux.h >> +++ b/include/acpi/platform/aclinux.h >> @@ -69,6 +69,10 @@ >> #define ACPI_REDUCED_HARDWARE 1 >> #endif >> >> +#ifdef CONFIG_ACPI_OS_NAME_LINUX >> +#define ACPI_OS_NAME "Linux" >> +#endif > > ->> and here you can do > > +#ifdef CONFIG_ACPI_OS_NAME > +#define ACPI_OS_NAME CONFIG_ACPI_OS_NAME > +#endif > > unless I'm missing something. > >> + >> #include >> #include >> #include >> > D'oh. Yup. I'll correct it. -- ciao, al ----------------------------------- Al Stone Software Engineer Red Hat, Inc. ahs3@redhat.com ----------------------------------- From mboxrd@z Thu Jan 1 00:00:00 1970 From: ahs3@redhat.com (Al Stone) Date: Fri, 23 Jan 2015 09:34:50 -0700 Subject: [PATCH 7/7] ACPI: use Linux as ACPI_OS_NAME for _OS on ARM64 In-Reply-To: <3375613.hf7nCUoE0P@vostro.rjw.lan> References: <1421973884-13029-1-git-send-email-al.stone@linaro.org> <1421973884-13029-8-git-send-email-al.stone@linaro.org> <3375613.hf7nCUoE0P@vostro.rjw.lan> Message-ID: <54C2782A.8090200@redhat.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 01/23/2015 08:55 AM, Rafael J. Wysocki wrote: > On Thursday, January 22, 2015 05:44:44 PM al.stone at linaro.org wrote: >> From: Hanjun Guo >> >> ACPI_OS_NAME is globally defined as "Microsoft Windows NT" for now. >> That doesn't make much sense in the ARM context, so set it to "Linux" >> when CONFIG_ARM64. >> >> If it is necessary to change the return value from \_OS_ (that is, return >> some value other than the default in ACPI_OS_NAME), use the kernel parameter >> "acpi_os_name=". >> >> Signed-off-by: Hanjun Guo >> --- >> arch/arm64/Kconfig | 1 + >> drivers/acpi/Kconfig | 8 ++++++++ >> include/acpi/acconfig.h | 2 ++ >> include/acpi/platform/aclinux.h | 4 ++++ >> 4 files changed, 15 insertions(+) >> >> diff --git a/arch/arm64/Kconfig b/arch/arm64/Kconfig >> index 915aa16..20677d1 100644 >> --- a/arch/arm64/Kconfig >> +++ b/arch/arm64/Kconfig >> @@ -1,5 +1,6 @@ > > If you added > > config ACPI_OS_NAME > string > default "Linux" > depends on ACPI > > to this file, then you don't need to add the new select below and -> > >> config ARM64 >> def_bool y >> + select ACPI_OS_NAME_LINUX if ACPI >> select ACPI_REDUCED_HARDWARE_ONLY if ACPI >> select ARCH_BINFMT_ELF_RANDOMIZE_PIE >> select ARCH_HAS_ATOMIC64_DEC_IF_POSITIVE >> diff --git a/drivers/acpi/Kconfig b/drivers/acpi/Kconfig >> index 3e3bd35..024c5f0 100644 >> --- a/drivers/acpi/Kconfig >> +++ b/drivers/acpi/Kconfig >> @@ -369,6 +369,14 @@ config ACPI_REDUCED_HARDWARE_ONLY >> >> If you are unsure what to do, do not enable this option. >> >> +config ACPI_OS_NAME_LINUX >> + bool "Using Linux for _OS method" if EXPERT >> + def_bool n >> + When used, this option makes sure the _OS method returns >> + the value "Linux". >> + >> + If you are unsure what to do, do not enable this option. >> + > > -> you don't need this at all ->> > >> source "drivers/acpi/apei/Kconfig" >> >> config ACPI_EXTLOG >> diff --git a/include/acpi/acconfig.h b/include/acpi/acconfig.h >> index 5a0a3e5..1980bf4 100644 >> --- a/include/acpi/acconfig.h >> +++ b/include/acpi/acconfig.h >> @@ -69,7 +69,9 @@ >> * code that will not execute the _OSI method unless _OS matches the string >> * below. Therefore, change this string at your own risk. >> */ >> +#ifndef ACPI_OS_NAME >> #define ACPI_OS_NAME "Microsoft Windows NT" >> +#endif >> >> /* Maximum objects in the various object caches */ >> >> diff --git a/include/acpi/platform/aclinux.h b/include/acpi/platform/aclinux.h >> index 1ba7c19..a8a7ee3 100644 >> --- a/include/acpi/platform/aclinux.h >> +++ b/include/acpi/platform/aclinux.h >> @@ -69,6 +69,10 @@ >> #define ACPI_REDUCED_HARDWARE 1 >> #endif >> >> +#ifdef CONFIG_ACPI_OS_NAME_LINUX >> +#define ACPI_OS_NAME "Linux" >> +#endif > > ->> and here you can do > > +#ifdef CONFIG_ACPI_OS_NAME > +#define ACPI_OS_NAME CONFIG_ACPI_OS_NAME > +#endif > > unless I'm missing something. > >> + >> #include >> #include >> #include >> > D'oh. Yup. I'll correct it. -- ciao, al ----------------------------------- Al Stone Software Engineer Red Hat, Inc. ahs3 at redhat.com -----------------------------------