HI, > From: linux-acpi-owner@xxxxxxxxxxxxxxx [mailto:linux-acpi-owner@xxxxxxxxxxxxxxx] On Behalf Of Hanjun Guo > Sent: Friday, November 22, 2013 5:57 PM > > On 2013-11-22 14:14, Zheng, Lv wrote: > [...] > >> endif # ACPI > >> diff --git a/include/acpi/platform/aclinux.h b/include/acpi/platform/aclinux.h > >> index 28f4f4d..ae93a91 100644 > >> --- a/include/acpi/platform/aclinux.h > >> +++ b/include/acpi/platform/aclinux.h > >> @@ -67,6 +67,10 @@ > >> > >> /* Host-dependent types and defines for in-kernel ACPICA */ > >> > >> +#ifdef CONFIG_ACPI_REDUCED_HARDWARE > >> +#define ACPI_REDUCED_HARDWARE TRUE > >> +#endif > >> + > > > > Maybe you put this here because of my previous wrong comment. > > > > For ACPICA environments that work like Kconfigs for Linux, it is good to define them before including any ACPICA files. > > While putting things here cannot cover <asm/acpi.h>. > > Good catch! thanks for the reminding. > > > > > Normally, I will do: > > > > ... > > > > #ifdef __KERNEL__ > > > > /* some comment */ > > (one empty line as ACPICA enforces 1 empty line after 1 line comment and no empty lines after a block of comments) > > #ifdef CONFIG_ACPI_REDUCED_HARDWARE > > #define ACPI_REDUCED_HARDWARE(spaces not tabs here according to ACPICA's coding style)TRUE > > #endif > > > > #include <linux/string.h> > > There is a problem when I try yours suggestion, it is a compiling warning: > > warning: "TRUE" is not defined > > And I find that "TRUE" is defined in include/acpi/actypes. > > So, is this ok to you? ... How ugly the TRUE is. OK, you can keep your original code as this is really not a real issue, just a tricky point. I'll try to offer a cleanup after another ACPICA cleanup that tries to modify all "#if" condition related TRUE into "1" in ACPICA. Thanks and best regards -Lv > > --- a/include/acpi/platform/aclinux.h > +++ b/include/acpi/platform/aclinux.h > @@ -63,6 +63,13 @@ > #ifdef EXPORT_ACPI_INTERFACES > #include <linux/export.h> > #endif > + > +#define TRUE (1 == 1) > + > +#ifdef CONFIG_ACPI_REDUCED_HARDWARE > +#define ACPI_REDUCED_HARDWARE TRUE > +#endif > + > #include <asm/acpi.h> > > > > > The coding style can help ACPICA release process to generate correct Linuxized patches. > > It would be good to Linux developers to follow this currently for ACPICA internal code or we may see a small useless divergences > commit generated from a back ported Linux commit :-( . > > I'm sorry for the inconvenience. > > ok, will update in next version. > > > > > Thanks > > -Lv > > -- > 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 -- 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
