RE: [PATCH v2] SFI: fix compilation warnings

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

> From: linux-acpi-owner@xxxxxxxxxxxxxxx [mailto:linux-acpi-owner@xxxxxxxxxxxxxxx] 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@xxxxxxxxxxxxxxx]
> > > 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@xxxxxxxxxxxxxxx>
> > > ---
> > > 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@xxxxxxxxxxxxxxx>
> Intel Finland Oy
> 
> --
> 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
��.n��������+%������w��{.n�����{�����ܨ}���Ơz�j:+v�����w����ޙ��&�)ߡ�a����z�ޗ���ݢj��w�f





[Index of Archives]     [Linux IBM ACPI]     [Linux Power Management]     [Linux Kernel]     [Linux Laptop]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux