* [PATCH] drivers/acpi: make evged.c explicitly non-modular
@ 2016-05-09 18:40 ` Paul Gortmaker
0 siblings, 0 replies; 6+ messages in thread
From: Paul Gortmaker @ 2016-05-09 18:40 UTC (permalink / raw)
To: linux-kernel; +Cc: Paul Gortmaker, Sinan Kaya, Len Brown, linux-acpi
The Makefile/Kconfig currently controlling compilation of this code is:
Makefile:acpi-$(CONFIG_ACPI_REDUCED_HARDWARE_ONLY) += evged.o
drivers/acpi/Kconfig:config ACPI_REDUCED_HARDWARE_ONLY
drivers/acpi/Kconfig: bool "Hardware-reduced ACPI support only" if EXPERT
...meaning that it currently is not being built as a module by anyone.
Lets remove the couple traces of modularity so that when reading the
code there is no doubt it is builtin-only.
Since module_platform_driver() uses the same init level priority as
builtin_platform_driver() the init ordering remains unchanged with
this commit.
The file wasn't explicitly including the module.h file but it did
already have init.h so, unlike similar changes, this one has no
header changes at all.
We also delete the MODULE_LICENSE tag since all that information
is already contained at the top of the file in the comments.
Cc: Sinan Kaya <okaya@codeaurora.org>
c: "Rafael J. Wysocki" <rjw@rjwysocki.net>
Cc: Len Brown <lenb@kernel.org>
Cc: linux-acpi@vger.kernel.org
Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com>
---
drivers/acpi/evged.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/acpi/evged.c b/drivers/acpi/evged.c
index 9c0a868d7b57..46f060356a22 100644
--- a/drivers/acpi/evged.c
+++ b/drivers/acpi/evged.c
@@ -151,6 +151,4 @@ static struct platform_driver ged_driver = {
.acpi_match_table = ACPI_PTR(ged_acpi_ids),
},
};
-
-module_platform_driver(ged_driver);
-MODULE_LICENSE("GPL v2");
+builtin_platform_driver(ged_driver);
--
2.8.0
^ permalink raw reply related [flat|nested] 6+ messages in thread* [PATCH] drivers/acpi: make evged.c explicitly non-modular
@ 2016-05-09 18:40 ` Paul Gortmaker
0 siblings, 0 replies; 6+ messages in thread
From: Paul Gortmaker @ 2016-05-09 18:40 UTC (permalink / raw)
To: linux-kernel; +Cc: Paul Gortmaker, Sinan Kaya, Len Brown, linux-acpi
The Makefile/Kconfig currently controlling compilation of this code is:
Makefile:acpi-$(CONFIG_ACPI_REDUCED_HARDWARE_ONLY) += evged.o
drivers/acpi/Kconfig:config ACPI_REDUCED_HARDWARE_ONLY
drivers/acpi/Kconfig: bool "Hardware-reduced ACPI support only" if EXPERT
...meaning that it currently is not being built as a module by anyone.
Lets remove the couple traces of modularity so that when reading the
code there is no doubt it is builtin-only.
Since module_platform_driver() uses the same init level priority as
builtin_platform_driver() the init ordering remains unchanged with
this commit.
The file wasn't explicitly including the module.h file but it did
already have init.h so, unlike similar changes, this one has no
header changes at all.
We also delete the MODULE_LICENSE tag since all that information
is already contained at the top of the file in the comments.
Cc: Sinan Kaya <okaya@codeaurora.org>
c: "Rafael J. Wysocki" <rjw@rjwysocki.net>
Cc: Len Brown <lenb@kernel.org>
Cc: linux-acpi@vger.kernel.org
Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com>
---
drivers/acpi/evged.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/acpi/evged.c b/drivers/acpi/evged.c
index 9c0a868d7b57..46f060356a22 100644
--- a/drivers/acpi/evged.c
+++ b/drivers/acpi/evged.c
@@ -151,6 +151,4 @@ static struct platform_driver ged_driver = {
.acpi_match_table = ACPI_PTR(ged_acpi_ids),
},
};
-
-module_platform_driver(ged_driver);
-MODULE_LICENSE("GPL v2");
+builtin_platform_driver(ged_driver);
--
2.8.0
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [PATCH] drivers/acpi: make evged.c explicitly non-modular
2016-05-09 18:40 ` Paul Gortmaker
(?)
@ 2016-05-09 19:35 ` Sinan Kaya
2016-05-09 19:59 ` Paul Gortmaker
-1 siblings, 1 reply; 6+ messages in thread
From: Sinan Kaya @ 2016-05-09 19:35 UTC (permalink / raw)
To: Paul Gortmaker, linux-kernel; +Cc: Len Brown, linux-acpi, Rafael J. Wysocki
+Rafael,
On 5/9/2016 2:40 PM, Paul Gortmaker wrote:
> The Makefile/Kconfig currently controlling compilation of this code is:
>
> Makefile:acpi-$(CONFIG_ACPI_REDUCED_HARDWARE_ONLY) += evged.o
>
> -
> -module_platform_driver(ged_driver);
> -MODULE_LICENSE("GPL v2");
> +builtin_platform_driver(ged_driver);
>
Thanks, it looks good to me. I realized Rafael was missing in your post.
I added him here.
--
Sinan Kaya
Qualcomm Technologies, Inc. on behalf of Qualcomm Innovation Center, Inc.
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH] drivers/acpi: make evged.c explicitly non-modular
2016-05-09 19:35 ` Sinan Kaya
@ 2016-05-09 19:59 ` Paul Gortmaker
0 siblings, 0 replies; 6+ messages in thread
From: Paul Gortmaker @ 2016-05-09 19:59 UTC (permalink / raw)
To: Sinan Kaya; +Cc: linux-kernel, Len Brown, linux-acpi, Rafael J. Wysocki
[Re: [PATCH] drivers/acpi: make evged.c explicitly non-modular] On 09/05/2016 (Mon 15:35) Sinan Kaya wrote:
> +Rafael,
>
> On 5/9/2016 2:40 PM, Paul Gortmaker wrote:
> > The Makefile/Kconfig currently controlling compilation of this code is:
> >
> > Makefile:acpi-$(CONFIG_ACPI_REDUCED_HARDWARE_ONLY) += evged.o
> >
> > -
> > -module_platform_driver(ged_driver);
> > -MODULE_LICENSE("GPL v2");
> > +builtin_platform_driver(ged_driver);
> >
>
> Thanks, it looks good to me. I realized Rafael was missing in your post.
> I added him here.
Thanks for spotting that. It looks like I fat fingered his "Cc:" line
into a "c:" at some point and I didn't notice his name was absent when
git gave me the summary of recipients before hitting send. :-(
Would have been nice if git send-email had given me a warning, but I
guess there are so many different non-stanadard headers out there now,
it would be impossible to assume "c:" wasn't valid for someone....
P.
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH] drivers/acpi: make evged.c explicitly non-modular
@ 2016-05-09 19:59 ` Paul Gortmaker
0 siblings, 0 replies; 6+ messages in thread
From: Paul Gortmaker @ 2016-05-09 19:59 UTC (permalink / raw)
To: Sinan Kaya; +Cc: linux-kernel, Len Brown, linux-acpi, Rafael J. Wysocki
[Re: [PATCH] drivers/acpi: make evged.c explicitly non-modular] On 09/05/2016 (Mon 15:35) Sinan Kaya wrote:
> +Rafael,
>
> On 5/9/2016 2:40 PM, Paul Gortmaker wrote:
> > The Makefile/Kconfig currently controlling compilation of this code is:
> >
> > Makefile:acpi-$(CONFIG_ACPI_REDUCED_HARDWARE_ONLY) += evged.o
> >
> > -
> > -module_platform_driver(ged_driver);
> > -MODULE_LICENSE("GPL v2");
> > +builtin_platform_driver(ged_driver);
> >
>
> Thanks, it looks good to me. I realized Rafael was missing in your post.
> I added him here.
Thanks for spotting that. It looks like I fat fingered his "Cc:" line
into a "c:" at some point and I didn't notice his name was absent when
git gave me the summary of recipients before hitting send. :-(
Would have been nice if git send-email had given me a warning, but I
guess there are so many different non-stanadard headers out there now,
it would be impossible to assume "c:" wasn't valid for someone....
P.
--
>
> --
> Sinan Kaya
> Qualcomm Technologies, Inc. on behalf of Qualcomm Innovation Center, Inc.
> Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH] drivers/acpi: make evged.c explicitly non-modular
2016-05-09 19:59 ` Paul Gortmaker
(?)
@ 2016-05-11 0:11 ` Rafael J. Wysocki
-1 siblings, 0 replies; 6+ messages in thread
From: Rafael J. Wysocki @ 2016-05-11 0:11 UTC (permalink / raw)
To: Paul Gortmaker; +Cc: Sinan Kaya, linux-kernel, Len Brown, linux-acpi
On Monday, May 09, 2016 03:59:29 PM Paul Gortmaker wrote:
> [Re: [PATCH] drivers/acpi: make evged.c explicitly non-modular] On 09/05/2016 (Mon 15:35) Sinan Kaya wrote:
>
> > +Rafael,
> >
> > On 5/9/2016 2:40 PM, Paul Gortmaker wrote:
> > > The Makefile/Kconfig currently controlling compilation of this code is:
> > >
> > > Makefile:acpi-$(CONFIG_ACPI_REDUCED_HARDWARE_ONLY) += evged.o
> > >
> > > -
> > > -module_platform_driver(ged_driver);
> > > -MODULE_LICENSE("GPL v2");
> > > +builtin_platform_driver(ged_driver);
> > >
> >
> > Thanks, it looks good to me. I realized Rafael was missing in your post.
> > I added him here.
>
> Thanks for spotting that. It looks like I fat fingered his "Cc:" line
> into a "c:" at some point and I didn't notice his name was absent when
> git gave me the summary of recipients before hitting send. :-(
>
> Would have been nice if git send-email had given me a warning, but I
> guess there are so many different non-stanadard headers out there now,
> it would be impossible to assume "c:" wasn't valid for someone....
No worries.
If it goes to linux-acpi@vger.kernel.org, I can find it.
Patch applied, thanks!
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2016-05-11 0:08 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-05-09 18:40 [PATCH] drivers/acpi: make evged.c explicitly non-modular Paul Gortmaker
2016-05-09 18:40 ` Paul Gortmaker
2016-05-09 19:35 ` Sinan Kaya
2016-05-09 19:59 ` Paul Gortmaker
2016-05-09 19:59 ` Paul Gortmaker
2016-05-11 0:11 ` Rafael J. Wysocki
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.