Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1567046 > unrolled thread
| Started by | Stephen Boyd <sboyd@codeaurora.org> |
|---|---|
| First post | 2017-01-26 00:50 +0100 |
| Last post | 2017-01-26 13:30 +0100 |
| Articles | 5 — 3 participants |
Back to article view | Back to linux.kernel
[PATCH] perf: xgene: Include module.h Stephen Boyd <sboyd@codeaurora.org> - 2017-01-26 00:50 +0100
Re: [PATCH] perf: xgene: Include module.h Tai Tri Nguyen <ttnguyen@apm.com> - 2017-01-26 03:20 +0100
Re: [PATCH] perf: xgene: Include module.h Tai Tri Nguyen <ttnguyen@apm.com> - 2017-01-26 03:20 +0100
Re: [PATCH] perf: xgene: Include module.h Tai Tri Nguyen <ttnguyen@apm.com> - 2017-01-26 04:30 +0100
Re: [PATCH] perf: xgene: Include module.h Will Deacon <will.deacon@arm.com> - 2017-01-26 13:30 +0100
| From | Stephen Boyd <sboyd@codeaurora.org> |
|---|---|
| Date | 2017-01-26 00:50 +0100 |
| Subject | [PATCH] perf: xgene: Include module.h |
| Message-ID | <t3Fua-8os-15@gated-at.bofh.it> |
I ran into a build error when I disabled CONFIG_ACPI and tried to compile this driver: drivers/perf/xgene_pmu.c:1242:1: warning: data definition has no type or storage class MODULE_DEVICE_TABLE(of, xgene_pmu_of_match); ^ drivers/perf/xgene_pmu.c:1242:1: error: type defaults to 'int' in declaration of 'MODULE_DEVICE_TABLE' [-Werror=implicit-int] Include module.h for the MODULE_DEVICE_TABLE macro that's implicitly included through ACPI. Cc: Tai Nguyen <ttnguyen@apm.com> Signed-off-by: Stephen Boyd <sboyd@codeaurora.org> --- drivers/perf/xgene_pmu.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/perf/xgene_pmu.c b/drivers/perf/xgene_pmu.c index a8ac4bcef2c0..fb3281962a6d 100644 --- a/drivers/perf/xgene_pmu.c +++ b/drivers/perf/xgene_pmu.c @@ -20,6 +20,7 @@ */ #include <linux/acpi.h> +#include <linux/module.h> #include <linux/clk.h> #include <linux/cpumask.h> #include <linux/interrupt.h> -- The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project
[toc] | [next] | [standalone]
| From | Tai Tri Nguyen <ttnguyen@apm.com> |
|---|---|
| Date | 2017-01-26 03:20 +0100 |
| Message-ID | <t3HPj-1vR-3@gated-at.bofh.it> |
| In reply to | #1567046 |
My bad, I can reproduce the issue now. Thanks Tai On Wed, Jan 25, 2017 at 6:13 PM, Tai Tri Nguyen <ttnguyen@apm.com> wrote: > Hi Stephen, > > On Wed, Jan 25, 2017 at 3:46 PM, Stephen Boyd <sboyd@codeaurora.org> wrote: >> I ran into a build error when I disabled CONFIG_ACPI and tried to >> compile this driver: > > I tried kernel 4.9 and couldn't reproduce the issue. > What is the kernel version you are seeing the compilation issue? > > Regards > Tai > >> >> drivers/perf/xgene_pmu.c:1242:1: warning: data definition has no type or storage class >> MODULE_DEVICE_TABLE(of, xgene_pmu_of_match); >> ^ >> drivers/perf/xgene_pmu.c:1242:1: error: type defaults to 'int' in declaration of 'MODULE_DEVICE_TABLE' [-Werror=implicit-int] >> >> Include module.h for the MODULE_DEVICE_TABLE macro that's >> implicitly included through ACPI. >> >> Cc: Tai Nguyen <ttnguyen@apm.com> >> Signed-off-by: Stephen Boyd <sboyd@codeaurora.org> >> --- >> drivers/perf/xgene_pmu.c | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/drivers/perf/xgene_pmu.c b/drivers/perf/xgene_pmu.c >> index a8ac4bcef2c0..fb3281962a6d 100644 >> --- a/drivers/perf/xgene_pmu.c >> +++ b/drivers/perf/xgene_pmu.c >> @@ -20,6 +20,7 @@ >> */ >> >> #include <linux/acpi.h> >> +#include <linux/module.h> >> #include <linux/clk.h> >> #include <linux/cpumask.h> >> #include <linux/interrupt.h> >> -- >> The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, >> a Linux Foundation Collaborative Project >> > > > > -- > Tai -- Tai
[toc] | [prev] | [next] | [standalone]
| From | Tai Tri Nguyen <ttnguyen@apm.com> |
|---|---|
| Date | 2017-01-26 03:20 +0100 |
| Message-ID | <t3HPj-1vR-1@gated-at.bofh.it> |
| In reply to | #1567046 |
Hi Stephen, On Wed, Jan 25, 2017 at 3:46 PM, Stephen Boyd <sboyd@codeaurora.org> wrote: > I ran into a build error when I disabled CONFIG_ACPI and tried to > compile this driver: I tried kernel 4.9 and couldn't reproduce the issue. What is the kernel version you are seeing the compilation issue? Regards Tai > > drivers/perf/xgene_pmu.c:1242:1: warning: data definition has no type or storage class > MODULE_DEVICE_TABLE(of, xgene_pmu_of_match); > ^ > drivers/perf/xgene_pmu.c:1242:1: error: type defaults to 'int' in declaration of 'MODULE_DEVICE_TABLE' [-Werror=implicit-int] > > Include module.h for the MODULE_DEVICE_TABLE macro that's > implicitly included through ACPI. > > Cc: Tai Nguyen <ttnguyen@apm.com> > Signed-off-by: Stephen Boyd <sboyd@codeaurora.org> > --- > drivers/perf/xgene_pmu.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/perf/xgene_pmu.c b/drivers/perf/xgene_pmu.c > index a8ac4bcef2c0..fb3281962a6d 100644 > --- a/drivers/perf/xgene_pmu.c > +++ b/drivers/perf/xgene_pmu.c > @@ -20,6 +20,7 @@ > */ > > #include <linux/acpi.h> > +#include <linux/module.h> > #include <linux/clk.h> > #include <linux/cpumask.h> > #include <linux/interrupt.h> > -- > The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, > a Linux Foundation Collaborative Project > -- Tai
[toc] | [prev] | [next] | [standalone]
| From | Tai Tri Nguyen <ttnguyen@apm.com> |
|---|---|
| Date | 2017-01-26 04:30 +0100 |
| Message-ID | <t3IV4-2dX-3@gated-at.bofh.it> |
| In reply to | #1567046 |
Hi Stephen, On Wed, Jan 25, 2017 at 3:46 PM, Stephen Boyd <sboyd@codeaurora.org> wrote: > I ran into a build error when I disabled CONFIG_ACPI and tried to > compile this driver: > > drivers/perf/xgene_pmu.c:1242:1: warning: data definition has no type or storage class > MODULE_DEVICE_TABLE(of, xgene_pmu_of_match); > ^ > drivers/perf/xgene_pmu.c:1242:1: error: type defaults to 'int' in declaration of 'MODULE_DEVICE_TABLE' [-Werror=implicit-int] > > Include module.h for the MODULE_DEVICE_TABLE macro that's > implicitly included through ACPI. > > Cc: Tai Nguyen <ttnguyen@apm.com> > Signed-off-by: Stephen Boyd <sboyd@codeaurora.org> > --- > drivers/perf/xgene_pmu.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/perf/xgene_pmu.c b/drivers/perf/xgene_pmu.c > index a8ac4bcef2c0..fb3281962a6d 100644 > --- a/drivers/perf/xgene_pmu.c > +++ b/drivers/perf/xgene_pmu.c > @@ -20,6 +20,7 @@ > */ > > #include <linux/acpi.h> > +#include <linux/module.h> Please fix alphabetic order of the include file. Tested-by: Tai Nguyen <ttnguyen@apm.com> > #include <linux/clk.h> > #include <linux/cpumask.h> > #include <linux/interrupt.h> > -- > The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, > a Linux Foundation Collaborative Project > -- Tai
[toc] | [prev] | [next] | [standalone]
| From | Will Deacon <will.deacon@arm.com> |
|---|---|
| Date | 2017-01-26 13:30 +0100 |
| Message-ID | <t3RlE-7fL-15@gated-at.bofh.it> |
| In reply to | #1567095 |
On Wed, Jan 25, 2017 at 07:29:30PM -0800, Tai Tri Nguyen wrote: > On Wed, Jan 25, 2017 at 3:46 PM, Stephen Boyd <sboyd@codeaurora.org> wrote: > > I ran into a build error when I disabled CONFIG_ACPI and tried to > > compile this driver: > > > > drivers/perf/xgene_pmu.c:1242:1: warning: data definition has no type or storage class > > MODULE_DEVICE_TABLE(of, xgene_pmu_of_match); > > ^ > > drivers/perf/xgene_pmu.c:1242:1: error: type defaults to 'int' in declaration of 'MODULE_DEVICE_TABLE' [-Werror=implicit-int] > > > > Include module.h for the MODULE_DEVICE_TABLE macro that's > > implicitly included through ACPI. > > > > Cc: Tai Nguyen <ttnguyen@apm.com> > > Signed-off-by: Stephen Boyd <sboyd@codeaurora.org> > > --- > > drivers/perf/xgene_pmu.c | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/drivers/perf/xgene_pmu.c b/drivers/perf/xgene_pmu.c > > index a8ac4bcef2c0..fb3281962a6d 100644 > > --- a/drivers/perf/xgene_pmu.c > > +++ b/drivers/perf/xgene_pmu.c > > @@ -20,6 +20,7 @@ > > */ > > > > #include <linux/acpi.h> > > +#include <linux/module.h> > > Please fix alphabetic order of the include file. > > Tested-by: Tai Nguyen <ttnguyen@apm.com> I've picked this up with the includes put in the correct order. Thanks, Will
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web