* [PATCH v2] SFI: fix compilation warnings @ 2013-12-09 9:09 Andy Shevchenko 2013-12-10 1:30 ` Zheng, Lv 0 siblings, 1 reply; 5+ messages in thread From: Andy Shevchenko @ 2013-12-09 9:09 UTC (permalink / raw) To: Wysocki, Rafael J, linux-acpi @ vger . kernel . org, LKML, Zheng, Lv Cc: Andy Shevchenko When build kernel with make W=1 we get the following compiler error. In file included from drivers/sfi/sfi_acpi.c:66:0: include/linux/sfi_acpi.h: In function ‘acpi_sfi_table_parse’: include/linux/sfi_acpi.h:72:2: error: implicit declaration of function ‘acpi_table_parse’ [-Werror=implicit-function-declaration] if (!acpi_table_parse(signature, handler)) ^ cc1: some warnings being treated as errors The patch add linux/acpi.h to the top of file. Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> --- Changes to v2: - fix for build when ACPI=n, SFI=y drivers/sfi/sfi_acpi.c | 2 +- include/linux/sfi_acpi.h | 4 +++- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/sfi/sfi_acpi.c b/drivers/sfi/sfi_acpi.c index 5e753d7..5589ec5 100644 --- a/drivers/sfi/sfi_acpi.c +++ b/drivers/sfi/sfi_acpi.c @@ -60,7 +60,7 @@ #define pr_fmt(fmt) KMSG_COMPONENT ": " fmt #include <linux/kernel.h> -#include <acpi/acpi.h> /* FIXME: inclusion should be removed */ +#include <linux/acpi.h> #include <linux/sfi.h> #include "sfi_core.h" diff --git a/include/linux/sfi_acpi.h b/include/linux/sfi_acpi.h index 2cfcb79..ecc2344 100644 --- a/include/linux/sfi_acpi.h +++ b/include/linux/sfi_acpi.h @@ -60,7 +60,9 @@ #define _LINUX_SFI_ACPI_H #ifdef CONFIG_SFI -#include <acpi/acpi.h> /* FIXME: inclusion should be removed */ + +#include <acpi/acpi.h> +#include <linux/acpi.h> extern int sfi_acpi_table_parse(char *signature, char *oem_id, char *oem_table_id, -- 1.8.4.4 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* RE: [PATCH v2] SFI: fix compilation warnings 2013-12-09 9:09 [PATCH v2] SFI: fix compilation warnings Andy Shevchenko @ 2013-12-10 1:30 ` Zheng, Lv 2013-12-10 12:07 ` Andy Shevchenko 2013-12-10 12:45 ` Andy Shevchenko 0 siblings, 2 replies; 5+ messages in thread From: Zheng, Lv @ 2013-12-10 1:30 UTC (permalink / raw) To: Andy Shevchenko, Wysocki, Rafael J, linux-acpi @ vger . kernel . org, LKML Hi, Andy IMO: 1. sfi_acpi.h should include both <linux/acpi.h> and <linux/sfi.h> and it should include <acpi/acpi.h> for !CONFIG_ACPI builds. 2. then we should include sfi_acpi.h instead of <linux/acpi.h>, <linux/sfi.h> and <acpi/acpi.h> for all of its users. I think the patch in the linux-pm/bleeding-edge queue can fix the issue you've reported: https://git.kernel.org/cgit/linux/kernel/git/rafael/linux-pm.git/commit/?h=bleeding-edge&id=c099eacbcaec4475936fbf73e499507728ce47e1 I made it a part of my header cleanup series. The story can be found in this mailing list thread: http://www.spinics.net/lists/linux-acpi/msg47510.html Thanks and best regards -Lv > -----Original Message----- > From: Andy Shevchenko [mailto:andriy.shevchenko@linux.intel.com] > Sent: Monday, December 09, 2013 5:09 PM > To: Wysocki, Rafael J; linux-acpi @ vger . kernel . org; LKML; Zheng, Lv > Cc: Andy Shevchenko > Subject: [PATCH v2] SFI: fix compilation warnings > > When build kernel with make W=1 we get the following compiler error. > > In file included from drivers/sfi/sfi_acpi.c:66:0: > include/linux/sfi_acpi.h: In function ‘acpi_sfi_table_parse’: > include/linux/sfi_acpi.h:72:2: error: implicit declaration of function ‘acpi_table_parse’ [-Werror=implicit-function-declaration] > if (!acpi_table_parse(signature, handler)) > ^ > cc1: some warnings being treated as errors > > The patch add linux/acpi.h to the top of file. > > Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> > --- > Changes to v2: > - fix for build when ACPI=n, SFI=y > drivers/sfi/sfi_acpi.c | 2 +- > include/linux/sfi_acpi.h | 4 +++- > 2 files changed, 4 insertions(+), 2 deletions(-) > > diff --git a/drivers/sfi/sfi_acpi.c b/drivers/sfi/sfi_acpi.c > index 5e753d7..5589ec5 100644 > --- a/drivers/sfi/sfi_acpi.c > +++ b/drivers/sfi/sfi_acpi.c > @@ -60,7 +60,7 @@ > #define pr_fmt(fmt) KMSG_COMPONENT ": " fmt > > #include <linux/kernel.h> > -#include <acpi/acpi.h> /* FIXME: inclusion should be removed */ > +#include <linux/acpi.h> > > #include <linux/sfi.h> > #include "sfi_core.h" > diff --git a/include/linux/sfi_acpi.h b/include/linux/sfi_acpi.h > index 2cfcb79..ecc2344 100644 > --- a/include/linux/sfi_acpi.h > +++ b/include/linux/sfi_acpi.h > @@ -60,7 +60,9 @@ > #define _LINUX_SFI_ACPI_H > > #ifdef CONFIG_SFI > -#include <acpi/acpi.h> /* FIXME: inclusion should be removed */ > + > +#include <acpi/acpi.h> > +#include <linux/acpi.h> > > extern int sfi_acpi_table_parse(char *signature, char *oem_id, > char *oem_table_id, > -- > 1.8.4.4 ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2] SFI: fix compilation warnings 2013-12-10 1:30 ` Zheng, Lv @ 2013-12-10 12:07 ` Andy Shevchenko 2013-12-11 0:22 ` Zheng, Lv 2013-12-10 12:45 ` Andy Shevchenko 1 sibling, 1 reply; 5+ messages in thread From: Andy Shevchenko @ 2013-12-10 12:07 UTC (permalink / raw) To: Zheng, Lv; +Cc: Wysocki, Rafael J, linux-acpi @ vger . kernel . org, LKML On Tue, 2013-12-10 at 01:30 +0000, Zheng, Lv wrote: > Hi, Andy > > IMO: > 1. sfi_acpi.h should include both <linux/acpi.h> and <linux/sfi.h> and it should include <acpi/acpi.h> for !CONFIG_ACPI builds. > 2. then we should include sfi_acpi.h instead of <linux/acpi.h>, <linux/sfi.h> and <acpi/acpi.h> for all of its users. > > I think the patch in the linux-pm/bleeding-edge queue can fix the issue you've reported: > https://git.kernel.org/cgit/linux/kernel/git/rafael/linux-pm.git/commit/?h=bleeding-edge&id=c099eacbcaec4475936fbf73e499507728ce47e1 I think so, I'll test this later. Only one thing I don't like is the FIXME stuff in the sfi_acpi.h. Could it be fixed as well? > > I made it a part of my header cleanup series. > The story can be found in this mailing list thread: > http://www.spinics.net/lists/linux-acpi/msg47510.html > > Thanks and best regards > -Lv > > > > -----Original Message----- > > From: Andy Shevchenko [mailto:andriy.shevchenko@linux.intel.com] > > Sent: Monday, December 09, 2013 5:09 PM > > To: Wysocki, Rafael J; linux-acpi @ vger . kernel . org; LKML; Zheng, Lv > > Cc: Andy Shevchenko > > Subject: [PATCH v2] SFI: fix compilation warnings > > > > When build kernel with make W=1 we get the following compiler error. > > > > In file included from drivers/sfi/sfi_acpi.c:66:0: > > include/linux/sfi_acpi.h: In function ‘acpi_sfi_table_parse’: > > include/linux/sfi_acpi.h:72:2: error: implicit declaration of function ‘acpi_table_parse’ [-Werror=implicit-function-declaration] > > if (!acpi_table_parse(signature, handler)) > > ^ > > cc1: some warnings being treated as errors > > > > The patch add linux/acpi.h to the top of file. > > > > Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> > > --- > > Changes to v2: > > - fix for build when ACPI=n, SFI=y > > drivers/sfi/sfi_acpi.c | 2 +- > > include/linux/sfi_acpi.h | 4 +++- > > 2 files changed, 4 insertions(+), 2 deletions(-) > > > > diff --git a/drivers/sfi/sfi_acpi.c b/drivers/sfi/sfi_acpi.c > > index 5e753d7..5589ec5 100644 > > --- a/drivers/sfi/sfi_acpi.c > > +++ b/drivers/sfi/sfi_acpi.c > > @@ -60,7 +60,7 @@ > > #define pr_fmt(fmt) KMSG_COMPONENT ": " fmt > > > > #include <linux/kernel.h> > > -#include <acpi/acpi.h> /* FIXME: inclusion should be removed */ > > +#include <linux/acpi.h> > > > > #include <linux/sfi.h> > > #include "sfi_core.h" > > diff --git a/include/linux/sfi_acpi.h b/include/linux/sfi_acpi.h > > index 2cfcb79..ecc2344 100644 > > --- a/include/linux/sfi_acpi.h > > +++ b/include/linux/sfi_acpi.h > > @@ -60,7 +60,9 @@ > > #define _LINUX_SFI_ACPI_H > > > > #ifdef CONFIG_SFI > > -#include <acpi/acpi.h> /* FIXME: inclusion should be removed */ > > + > > +#include <acpi/acpi.h> > > +#include <linux/acpi.h> > > > > extern int sfi_acpi_table_parse(char *signature, char *oem_id, > > char *oem_table_id, > > -- > > 1.8.4.4 > -- Andy Shevchenko <andriy.shevchenko@linux.intel.com> Intel Finland Oy -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 5+ messages in thread
* RE: [PATCH v2] SFI: fix compilation warnings 2013-12-10 12:07 ` Andy Shevchenko @ 2013-12-11 0:22 ` Zheng, Lv 0 siblings, 0 replies; 5+ messages in thread From: Zheng, Lv @ 2013-12-11 0:22 UTC (permalink / raw) To: Andy Shevchenko; +Cc: Wysocki, Rafael J, linux-acpi @ vger . kernel . org, LKML Hi, > From: linux-acpi-owner@vger.kernel.org [mailto:linux-acpi-owner@vger.kernel.org] On Behalf Of Andy Shevchenko > Sent: Tuesday, December 10, 2013 8:08 PM > > On Tue, 2013-12-10 at 01:30 +0000, Zheng, Lv wrote: > > Hi, Andy > > > > IMO: > > 1. sfi_acpi.h should include both <linux/acpi.h> and <linux/sfi.h> and it should include <acpi/acpi.h> for !CONFIG_ACPI builds. > > 2. then we should include sfi_acpi.h instead of <linux/acpi.h>, <linux/sfi.h> and <acpi/acpi.h> for all of its users. > > > > I think the patch in the linux-pm/bleeding-edge queue can fix the issue you've reported: > > https://git.kernel.org/cgit/linux/kernel/git/rafael/linux-pm.git/commit/?h=bleeding- > edge&id=c099eacbcaec4475936fbf73e499507728ce47e1 > > I think so, I'll test this later. > > Only one thing I don't like is the FIXME stuff in the sfi_acpi.h. > Could it be fixed as well? It is just a reminder to tell people <acpi/acpi.h> inclusion here cannot be a reference for those who want to include it directly. I have offered 2 solutions to remove this FIXME: 1. Implementing stubs for <acpi/acpi.h>, then we can move <acpi/acpi.h> inclusion out of the "#ifdef CONFIG_ACPI" block. To achieve this, I drafted more than 10 commits. Someone might think it was too complicated to address such an issue, but we might utilize this to kill more "#ifdef CONFIG_ACPI" blocks in the entire kernel source tree. 2. Re-defining necessary table structures for !CONFIG_ACPI builds. 1 commit was ready for this. But having things defined twice looks ugly. They are still under discussion. If the decision was determined, you might see further patches sent from me to the above mentioned thread or during the ACPICA release process to remove it. Thanks and best regards -Lv > > > > > > I made it a part of my header cleanup series. > > The story can be found in this mailing list thread: > > http://www.spinics.net/lists/linux-acpi/msg47510.html > > > > Thanks and best regards > > -Lv > > > > > > > -----Original Message----- > > > From: Andy Shevchenko [mailto:andriy.shevchenko@linux.intel.com] > > > Sent: Monday, December 09, 2013 5:09 PM > > > To: Wysocki, Rafael J; linux-acpi @ vger . kernel . org; LKML; Zheng, Lv > > > Cc: Andy Shevchenko > > > Subject: [PATCH v2] SFI: fix compilation warnings > > > > > > When build kernel with make W=1 we get the following compiler error. > > > > > > In file included from drivers/sfi/sfi_acpi.c:66:0: > > > include/linux/sfi_acpi.h: In function ‘acpi_sfi_table_parse’: > > > include/linux/sfi_acpi.h:72:2: error: implicit declaration of function ‘acpi_table_parse’ [-Werror=implicit-function-declaration] > > > if (!acpi_table_parse(signature, handler)) > > > ^ > > > cc1: some warnings being treated as errors > > > > > > The patch add linux/acpi.h to the top of file. > > > > > > Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> > > > --- > > > Changes to v2: > > > - fix for build when ACPI=n, SFI=y > > > drivers/sfi/sfi_acpi.c | 2 +- > > > include/linux/sfi_acpi.h | 4 +++- > > > 2 files changed, 4 insertions(+), 2 deletions(-) > > > > > > diff --git a/drivers/sfi/sfi_acpi.c b/drivers/sfi/sfi_acpi.c > > > index 5e753d7..5589ec5 100644 > > > --- a/drivers/sfi/sfi_acpi.c > > > +++ b/drivers/sfi/sfi_acpi.c > > > @@ -60,7 +60,7 @@ > > > #define pr_fmt(fmt) KMSG_COMPONENT ": " fmt > > > > > > #include <linux/kernel.h> > > > -#include <acpi/acpi.h> /* FIXME: inclusion should be removed */ > > > +#include <linux/acpi.h> > > > > > > #include <linux/sfi.h> > > > #include "sfi_core.h" > > > diff --git a/include/linux/sfi_acpi.h b/include/linux/sfi_acpi.h > > > index 2cfcb79..ecc2344 100644 > > > --- a/include/linux/sfi_acpi.h > > > +++ b/include/linux/sfi_acpi.h > > > @@ -60,7 +60,9 @@ > > > #define _LINUX_SFI_ACPI_H > > > > > > #ifdef CONFIG_SFI > > > -#include <acpi/acpi.h> /* FIXME: inclusion should be removed */ > > > + > > > +#include <acpi/acpi.h> > > > +#include <linux/acpi.h> > > > > > > extern int sfi_acpi_table_parse(char *signature, char *oem_id, > > > char *oem_table_id, > > > -- > > > 1.8.4.4 > > > > -- > Andy Shevchenko <andriy.shevchenko@linux.intel.com> > Intel Finland Oy > > -- > To unsubscribe from this list: send the line "unsubscribe linux-acpi" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2] SFI: fix compilation warnings 2013-12-10 1:30 ` Zheng, Lv 2013-12-10 12:07 ` Andy Shevchenko @ 2013-12-10 12:45 ` Andy Shevchenko 1 sibling, 0 replies; 5+ messages in thread From: Andy Shevchenko @ 2013-12-10 12:45 UTC (permalink / raw) To: Zheng, Lv; +Cc: Wysocki, Rafael J, linux-acpi @ vger . kernel . org, LKML On Tue, 2013-12-10 at 01:30 +0000, Zheng, Lv wrote: > Hi, Andy > > IMO: > 1. sfi_acpi.h should include both <linux/acpi.h> and <linux/sfi.h> and it should include <acpi/acpi.h> for !CONFIG_ACPI builds. > 2. then we should include sfi_acpi.h instead of <linux/acpi.h>, <linux/sfi.h> and <acpi/acpi.h> for all of its users. > > I think the patch in the linux-pm/bleeding-edge queue can fix the issue you've reported: > https://git.kernel.org/cgit/linux/kernel/git/rafael/linux-pm.git/commit/?h=bleeding-edge&id=c099eacbcaec4475936fbf73e499507728ce47e1 Seems okay for me: Tested-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> > > I made it a part of my header cleanup series. > The story can be found in this mailing list thread: > http://www.spinics.net/lists/linux-acpi/msg47510.html > > Thanks and best regards > -Lv > > > > -----Original Message----- > > From: Andy Shevchenko [mailto:andriy.shevchenko@linux.intel.com] > > Sent: Monday, December 09, 2013 5:09 PM > > To: Wysocki, Rafael J; linux-acpi @ vger . kernel . org; LKML; Zheng, Lv > > Cc: Andy Shevchenko > > Subject: [PATCH v2] SFI: fix compilation warnings > > > > When build kernel with make W=1 we get the following compiler error. > > > > In file included from drivers/sfi/sfi_acpi.c:66:0: > > include/linux/sfi_acpi.h: In function ‘acpi_sfi_table_parse’: > > include/linux/sfi_acpi.h:72:2: error: implicit declaration of function ‘acpi_table_parse’ [-Werror=implicit-function-declaration] > > if (!acpi_table_parse(signature, handler)) > > ^ > > cc1: some warnings being treated as errors > > > > The patch add linux/acpi.h to the top of file. > > > > Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> > > --- > > Changes to v2: > > - fix for build when ACPI=n, SFI=y > > drivers/sfi/sfi_acpi.c | 2 +- > > include/linux/sfi_acpi.h | 4 +++- > > 2 files changed, 4 insertions(+), 2 deletions(-) > > > > diff --git a/drivers/sfi/sfi_acpi.c b/drivers/sfi/sfi_acpi.c > > index 5e753d7..5589ec5 100644 > > --- a/drivers/sfi/sfi_acpi.c > > +++ b/drivers/sfi/sfi_acpi.c > > @@ -60,7 +60,7 @@ > > #define pr_fmt(fmt) KMSG_COMPONENT ": " fmt > > > > #include <linux/kernel.h> > > -#include <acpi/acpi.h> /* FIXME: inclusion should be removed */ > > +#include <linux/acpi.h> > > > > #include <linux/sfi.h> > > #include "sfi_core.h" > > diff --git a/include/linux/sfi_acpi.h b/include/linux/sfi_acpi.h > > index 2cfcb79..ecc2344 100644 > > --- a/include/linux/sfi_acpi.h > > +++ b/include/linux/sfi_acpi.h > > @@ -60,7 +60,9 @@ > > #define _LINUX_SFI_ACPI_H > > > > #ifdef CONFIG_SFI > > -#include <acpi/acpi.h> /* FIXME: inclusion should be removed */ > > + > > +#include <acpi/acpi.h> > > +#include <linux/acpi.h> > > > > extern int sfi_acpi_table_parse(char *signature, char *oem_id, > > char *oem_table_id, > > -- > > 1.8.4.4 > -- Andy Shevchenko <andriy.shevchenko@linux.intel.com> Intel Finland Oy -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2013-12-11 0:25 UTC | newest] Thread overview: 5+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2013-12-09 9:09 [PATCH v2] SFI: fix compilation warnings Andy Shevchenko 2013-12-10 1:30 ` Zheng, Lv 2013-12-10 12:07 ` Andy Shevchenko 2013-12-11 0:22 ` Zheng, Lv 2013-12-10 12:45 ` Andy Shevchenko
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox