Since commit 8b41fc4454e ("kbuild: create modules.builtin without
Makefile.modbuiltin or tristate.conf"), MODULE_LICENSE declarations
are used to identify modules. As a consequence, uses of the macro
in non-modules will cause modprobe to misidentify their containing
object file as a module when it is not (false positives), and modprobe
might succeed rather than failing with a suitable error message.
So remove it in the files in this commit, none of which can be built as
modules.
Signed-off-by: Nick Alcock <[email protected]>
Suggested-by: Luis Chamberlain <[email protected]>
Cc: Luis Chamberlain <[email protected]>
Cc: [email protected]
Cc: [email protected]
Cc: Hitomi Hasegawa <[email protected]>
Cc: Tony Lindgren <[email protected]>
Cc: Lee Jones <[email protected]>
Cc: [email protected]
---
drivers/mfd/omap-usb-host.c | 1 -
drivers/mfd/omap-usb-tll.c | 1 -
drivers/mfd/twl4030-audio.c | 1 -
drivers/mfd/twl6040.c | 1 -
4 files changed, 4 deletions(-)
diff --git a/drivers/mfd/omap-usb-host.c b/drivers/mfd/omap-usb-host.c
index 787d2ae86375..7f5775109593 100644
--- a/drivers/mfd/omap-usb-host.c
+++ b/drivers/mfd/omap-usb-host.c
@@ -853,7 +853,6 @@ static struct platform_driver usbhs_omap_driver = {
MODULE_AUTHOR("Keshava Munegowda <[email protected]>");
MODULE_AUTHOR("Roger Quadros <[email protected]>");
MODULE_ALIAS("platform:" USBHS_DRIVER_NAME);
-MODULE_LICENSE("GPL v2");
MODULE_DESCRIPTION("usb host common core driver for omap EHCI and OHCI");
static int omap_usbhs_drvinit(void)
diff --git a/drivers/mfd/omap-usb-tll.c b/drivers/mfd/omap-usb-tll.c
index 080d7970a377..8ca4067da6cd 100644
--- a/drivers/mfd/omap-usb-tll.c
+++ b/drivers/mfd/omap-usb-tll.c
@@ -450,7 +450,6 @@ EXPORT_SYMBOL_GPL(omap_tll_disable);
MODULE_AUTHOR("Keshava Munegowda <[email protected]>");
MODULE_AUTHOR("Roger Quadros <[email protected]>");
-MODULE_LICENSE("GPL v2");
MODULE_DESCRIPTION("usb tll driver for TI OMAP EHCI and OHCI controllers");
static int __init omap_usbtll_drvinit(void)
diff --git a/drivers/mfd/twl4030-audio.c b/drivers/mfd/twl4030-audio.c
index 4536d829b43e..88002f8941e5 100644
--- a/drivers/mfd/twl4030-audio.c
+++ b/drivers/mfd/twl4030-audio.c
@@ -285,5 +285,4 @@ module_platform_driver(twl4030_audio_driver);
MODULE_AUTHOR("Peter Ujfalusi <[email protected]>");
MODULE_DESCRIPTION("TWL4030 audio block MFD driver");
-MODULE_LICENSE("GPL");
MODULE_ALIAS("platform:twl4030-audio");
diff --git a/drivers/mfd/twl6040.c b/drivers/mfd/twl6040.c
index fc97fa5a2d0c..e982119bbefa 100644
--- a/drivers/mfd/twl6040.c
+++ b/drivers/mfd/twl6040.c
@@ -839,4 +839,3 @@ module_i2c_driver(twl6040_driver);
MODULE_DESCRIPTION("TWL6040 MFD");
MODULE_AUTHOR("Misael Lopez Cruz <[email protected]>");
MODULE_AUTHOR("Jorge Eduardo Candelaria <[email protected]>");
-MODULE_LICENSE("GPL");
--
2.39.1.268.g9de2f9a303
On Fri, 24 Feb 2023, Nick Alcock wrote:
> Since commit 8b41fc4454e ("kbuild: create modules.builtin without
> Makefile.modbuiltin or tristate.conf"), MODULE_LICENSE declarations
> are used to identify modules. As a consequence, uses of the macro
> in non-modules will cause modprobe to misidentify their containing
> object file as a module when it is not (false positives), and modprobe
> might succeed rather than failing with a suitable error message.
>
> So remove it in the files in this commit, none of which can be built as
> modules.
>
> Signed-off-by: Nick Alcock <[email protected]>
> Suggested-by: Luis Chamberlain <[email protected]>
> Cc: Luis Chamberlain <[email protected]>
> Cc: [email protected]
> Cc: [email protected]
> Cc: Hitomi Hasegawa <[email protected]>
> Cc: Tony Lindgren <[email protected]>
> Cc: Lee Jones <[email protected]>
> Cc: [email protected]
> ---
> drivers/mfd/omap-usb-host.c | 1 -
> drivers/mfd/omap-usb-tll.c | 1 -
> drivers/mfd/twl4030-audio.c | 1 -
> drivers/mfd/twl6040.c | 1 -
> 4 files changed, 4 deletions(-)
Please adapt the subject line(s) to include the drivers changed. It might
also make sense to separate out changes to cover one driver per patch.
mfd: <device>: Succinct subject-line describing changes
> diff --git a/drivers/mfd/omap-usb-host.c b/drivers/mfd/omap-usb-host.c
> index 787d2ae86375..7f5775109593 100644
> --- a/drivers/mfd/omap-usb-host.c
> +++ b/drivers/mfd/omap-usb-host.c
> @@ -853,7 +853,6 @@ static struct platform_driver usbhs_omap_driver = {
> MODULE_AUTHOR("Keshava Munegowda <[email protected]>");
> MODULE_AUTHOR("Roger Quadros <[email protected]>");
> MODULE_ALIAS("platform:" USBHS_DRIVER_NAME);
> -MODULE_LICENSE("GPL v2");
> MODULE_DESCRIPTION("usb host common core driver for omap EHCI and OHCI");
>
> static int omap_usbhs_drvinit(void)
> diff --git a/drivers/mfd/omap-usb-tll.c b/drivers/mfd/omap-usb-tll.c
> index 080d7970a377..8ca4067da6cd 100644
> --- a/drivers/mfd/omap-usb-tll.c
> +++ b/drivers/mfd/omap-usb-tll.c
> @@ -450,7 +450,6 @@ EXPORT_SYMBOL_GPL(omap_tll_disable);
>
> MODULE_AUTHOR("Keshava Munegowda <[email protected]>");
> MODULE_AUTHOR("Roger Quadros <[email protected]>");
> -MODULE_LICENSE("GPL v2");
> MODULE_DESCRIPTION("usb tll driver for TI OMAP EHCI and OHCI controllers");
>
> static int __init omap_usbtll_drvinit(void)
> diff --git a/drivers/mfd/twl4030-audio.c b/drivers/mfd/twl4030-audio.c
> index 4536d829b43e..88002f8941e5 100644
> --- a/drivers/mfd/twl4030-audio.c
> +++ b/drivers/mfd/twl4030-audio.c
> @@ -285,5 +285,4 @@ module_platform_driver(twl4030_audio_driver);
>
> MODULE_AUTHOR("Peter Ujfalusi <[email protected]>");
> MODULE_DESCRIPTION("TWL4030 audio block MFD driver");
> -MODULE_LICENSE("GPL");
> MODULE_ALIAS("platform:twl4030-audio");
> diff --git a/drivers/mfd/twl6040.c b/drivers/mfd/twl6040.c
> index fc97fa5a2d0c..e982119bbefa 100644
> --- a/drivers/mfd/twl6040.c
> +++ b/drivers/mfd/twl6040.c
> @@ -839,4 +839,3 @@ module_i2c_driver(twl6040_driver);
> MODULE_DESCRIPTION("TWL6040 MFD");
> MODULE_AUTHOR("Misael Lopez Cruz <[email protected]>");
> MODULE_AUTHOR("Jorge Eduardo Candelaria <[email protected]>");
> -MODULE_LICENSE("GPL");
> --
> 2.39.1.268.g9de2f9a303
>
--
Lee Jones [李琼斯]
On 3 Mar 2023, Lee Jones verbalised:
> On Fri, 24 Feb 2023, Nick Alcock wrote:
>
>> Since commit 8b41fc4454e ("kbuild: create modules.builtin without
>> Makefile.modbuiltin or tristate.conf"), MODULE_LICENSE declarations
>> are used to identify modules. As a consequence, uses of the macro
>> in non-modules will cause modprobe to misidentify their containing
>> object file as a module when it is not (false positives), and modprobe
>> might succeed rather than failing with a suitable error message.
>>
>> So remove it in the files in this commit, none of which can be built as
>> modules.
>>
>> Signed-off-by: Nick Alcock <[email protected]>
>> Suggested-by: Luis Chamberlain <[email protected]>
>> Cc: Luis Chamberlain <[email protected]>
>> Cc: [email protected]
>> Cc: [email protected]
>> Cc: Hitomi Hasegawa <[email protected]>
>> Cc: Tony Lindgren <[email protected]>
>> Cc: Lee Jones <[email protected]>
>> Cc: [email protected]
>> ---
>> drivers/mfd/omap-usb-host.c | 1 -
>> drivers/mfd/omap-usb-tll.c | 1 -
>> drivers/mfd/twl4030-audio.c | 1 -
>> drivers/mfd/twl6040.c | 1 -
>> 4 files changed, 4 deletions(-)
>
> Please adapt the subject line(s) to include the drivers changed. It might
> also make sense to separate out changes to cover one driver per patch.
The subject line is automatically generated, and uses whatever
subsystem prefix was most commonly used for all files touched in that
subsystem, while not containing any prefixes *not* so touched.
It's also automatically split up per-subsystem from a single big source
commit that changes everything at once. I can split this bit more
finely, but that means automated regeneration will be impossible, so
it'll probably backslide if I ever have to regenerate it -- and I've had
to regenerate this series a *lot*.
In fact if I'd split this bit per driver, I'd probably have
automatically got the subject line right as well -- but if the series as
a whole were split up like that, it'd have had about 400 patches rather
than 120, and got a lot of maintainers even more annoyed than they are
now.
For now, I'll split this one into four and mail it out again: coming
shortly.
On 3 Mar 2023, Lee Jones told this:
> Please adapt the subject line(s) to include the drivers changed. It might
> also make sense to separate out changes to cover one driver per patch.
There are a bunch of other commits in this series covering mfd: do you
want the same thing done to all of them as well? (They are all in subtly
subsystems.)
On Wed, 08 Mar 2023, Nick Alcock wrote:
> On 3 Mar 2023, Lee Jones verbalised:
>
> > On Fri, 24 Feb 2023, Nick Alcock wrote:
> >
> >> Since commit 8b41fc4454e ("kbuild: create modules.builtin without
> >> Makefile.modbuiltin or tristate.conf"), MODULE_LICENSE declarations
> >> are used to identify modules. As a consequence, uses of the macro
> >> in non-modules will cause modprobe to misidentify their containing
> >> object file as a module when it is not (false positives), and modprobe
> >> might succeed rather than failing with a suitable error message.
> >>
> >> So remove it in the files in this commit, none of which can be built as
> >> modules.
> >>
> >> Signed-off-by: Nick Alcock <[email protected]>
> >> Suggested-by: Luis Chamberlain <[email protected]>
> >> Cc: Luis Chamberlain <[email protected]>
> >> Cc: [email protected]
> >> Cc: [email protected]
> >> Cc: Hitomi Hasegawa <[email protected]>
> >> Cc: Tony Lindgren <[email protected]>
> >> Cc: Lee Jones <[email protected]>
> >> Cc: [email protected]
> >> ---
> >> drivers/mfd/omap-usb-host.c | 1 -
> >> drivers/mfd/omap-usb-tll.c | 1 -
> >> drivers/mfd/twl4030-audio.c | 1 -
> >> drivers/mfd/twl6040.c | 1 -
> >> 4 files changed, 4 deletions(-)
> >
> > Please adapt the subject line(s) to include the drivers changed. It might
> > also make sense to separate out changes to cover one driver per patch.
>
> The subject line is automatically generated, and uses whatever
> subsystem prefix was most commonly used for all files touched in that
> subsystem, while not containing any prefixes *not* so touched.
>
> It's also automatically split up per-subsystem from a single big source
> commit that changes everything at once. I can split this bit more
> finely, but that means automated regeneration will be impossible, so
> it'll probably backslide if I ever have to regenerate it -- and I've had
> to regenerate this series a *lot*.
>
> In fact if I'd split this bit per driver, I'd probably have
> automatically got the subject line right as well -- but if the series as
> a whole were split up like that, it'd have had about 400 patches rather
> than 120, and got a lot of maintainers even more annoyed than they are
> now.
>
> For now, I'll split this one into four and mail it out again: coming
> shortly.
If you haven't sent it out already, 2 will be fine.
mfd: omap: ...
mfd: twl: ...
--
Lee Jones [李琼斯]
On 8 Mar 2023, Lee Jones uttered the following:
> On Wed, 08 Mar 2023, Nick Alcock wrote:
>
>> >> drivers/mfd/omap-usb-host.c | 1 -
>> >> drivers/mfd/omap-usb-tll.c | 1 -
>> >> drivers/mfd/twl4030-audio.c | 1 -
>> >> drivers/mfd/twl6040.c | 1 -
>> >> 4 files changed, 4 deletions(-)
[...]
>> For now, I'll split this one into four and mail it out again: coming
>> shortly.
>
> If you haven't sent it out already, 2 will be fine.
>
> mfd: omap: ...
> mfd: twl: ...
I'm trying to automate this whole thing (since I've had to resplit it a
dozen times already), and even making mfd a special case that wants
splitting finer than by subsystem, I honestly have no idea how to
automatically determine a split like *that*. I can't split on file, on
subsystem, on dashes in the names... I think I'd have to literally parse
the makefile to figure out which things belonged together in commits,
and that's getting ridiculous even for me.
So, if you don't mind, I'd rather do it by subsystem or by file :)
--
NULL && (void)
On Wed, 08 Mar 2023, Nick Alcock wrote:
> On 8 Mar 2023, Lee Jones uttered the following:
>
> > On Wed, 08 Mar 2023, Nick Alcock wrote:
> >
> >> >> drivers/mfd/omap-usb-host.c | 1 -
> >> >> drivers/mfd/omap-usb-tll.c | 1 -
> >> >> drivers/mfd/twl4030-audio.c | 1 -
> >> >> drivers/mfd/twl6040.c | 1 -
> >> >> 4 files changed, 4 deletions(-)
> [...]
> >> For now, I'll split this one into four and mail it out again: coming
> >> shortly.
> >
> > If you haven't sent it out already, 2 will be fine.
> >
> > mfd: omap: ...
> > mfd: twl: ...
>
> I'm trying to automate this whole thing (since I've had to resplit it a
> dozen times already), and even making mfd a special case that wants
> splitting finer than by subsystem, I honestly have no idea how to
> automatically determine a split like *that*. I can't split on file, on
> subsystem, on dashes in the names... I think I'd have to literally parse
> the makefile to figure out which things belonged together in commits,
> and that's getting ridiculous even for me.
>
> So, if you don't mind, I'd rather do it by subsystem or by file :)
File is fine.
--
Lee Jones [李琼斯]