Re: [PATCH v2 0/7] MODULE_LICENSE removals, first tranche

Conor Dooley
2023-02-16T18:01:35Z

[-- Attachment #1: Type: text/plain, Size: 3388 bytes --]

On Thu, Feb 16, 2023 at 11:43:33AM -0600, Bjorn Helgaas wrote:
> [+cc PCI folks, who got the patches but not this cover letter, and
> participants from the v1 posting]

Thanks Bjorn..

> On Thu, Feb 16, 2023 at 03:24:03PM +0000, Nick Alcock wrote:
> > This series, based on current modules-next, is part of a treewide cleanup
> > suggested by Luis Chamberlain, to remove the LICENSE_MODULE usage from
> > files/objects that are not tristate. Due to recent changes to kbuild, these
> > uses are now problematic. See the commit logs for more details.
> >
> > (The commit log prefixes and Cc lists are automatically determined using
> > the script mentioned below. I've eyeballed them, and they seem reasonable:
> > my apologies if they are not.)
> >
> > This is a small initial tranche to see if the general approach is valid:
> > larger tranches can follow if desired and if these patches seem OK.
> > I'll be sending a second 24-patch tranche in a few hours if this
> > one meets with general approval.
> >
> > (In total, there are 121 patches in this series.)
> >
> >
> > Differences from v1:
> > - just remove MODULE_LICENSE, don't comment it out
> > - Cc: paid maintainers as well as unpaid ones
> > - improvements to the prefix determination code (notable in
> > PCI: versatile in particular)
> >
> > The series at a whole can be found here:
> > https://github.com/nickalcock/linux module-license
> >
> > The patch splitter (not for upstreaming!) used to prepare these
> > commits can be found here:
> > https://github.com/nickalcock/linux mass-split
> >
> > Cc: Luis Chamberlain <mcgrof@kernel.org>
> > Cc: linux-modules@vger.kernel.org
> > Cc: linux-kernel@vger.kernel.org
> >
> > Nick Alcock (7):
> > kbuild, PCI: versatile: remove MODULE_LICENSE in non-modules
> > kbuild, PCI: mobiveil: remove MODULE_LICENSE in non-modules
> > kbuild, PCI: tegra: remove MODULE_LICENSE in non-modules
> > kbuild, PCI: endpoint: remove MODULE_LICENSE in non-modules
> > kbuild, PCI: hip: remove MODULE_LICENSE in non-modules
> > kbuild, shpchp: remove MODULE_LICENSE in non-modules
> > kbuild, PCI: dwc: remove MODULE_LICENSE in non-modules
> >
> > drivers/pci/controller/dwc/pcie-histb.c | 1 -
> > drivers/pci/controller/mobiveil/pcie-mobiveil-plat.c | 1 -
> > drivers/pci/controller/pci-tegra.c | 1 -
> > drivers/pci/controller/pci-versatile.c | 1 -
> > drivers/pci/controller/pcie-hisi-error.c | 1 -
> > drivers/pci/endpoint/pci-ep-cfs.c | 1 -
> > drivers/pci/endpoint/pci-epc-core.c | 1 -
> > drivers/pci/endpoint/pci-epc-mem.c | 1 -
> > drivers/pci/endpoint/pci-epf-core.c | 1 -
> > drivers/pci/hotplug/shpchp_core.c | 1 -
> > 10 files changed, 10 deletions(-)
>
> I squashed these and applied to pci/kbuild for v6.3, thanks!
>
> I added examples from the v1 posting to the commit log; let me know if
> I got anything wrong:

I noticed that (as requested!) Nick dropped the change to the microchip
driver.
Would you like me to resend patch 3/11 of [1] to clear the decks of
"incorrect" MODULE_LICENSE usage?

Thanks,
Conor.

1 - https://lore.kernel.org/linux-pci/20230111125323.1911373-4-daire.mcnamara@microchip.com/

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]