Date:   Thu, 16 Feb 2023 13:34:30 +0000
From:   Nick Alcock <nick.alcock@...cle.com>
To:     Bjorn Helgaas <helgaas@...nel.org>
Cc:     mcgrof@...nel.org, linux-modules@...r.kernel.org,
        linux-kernel@...r.kernel.org, Rob Herring <robh@...nel.org>,
        linux-pci@...r.kernel.org, linux-arm-kernel@...ts.infradead.org
Subject: Re: [PATCH 1/8] kbuild, PCI: generic,versatile: comment out
 MODULE_LICENSE in non-modules

On 14 Feb 2023, Bjorn Helgaas uttered the following:

> On Tue, Feb 14, 2023 at 03:41:32PM +0000, Nick Alcock wrote:
>> On 13 Feb 2023, Bjorn Helgaas spake thusly:
>> > On Fri, Feb 10, 2023 at 04:47:42PM +0000, Nick Alcock wrote:
>
>> >   - Please follow the subject line convention for each file.  They're
>> >     mostly there after dropping "kbuild", but do capitalize the
>> >     sentence that follows the prefix.  The prefix should always be
>> >     "PCI/<driver-tag>: "
>
> I misspoke about "PCI/<driver-tag>".  I use "PCI/<feature>" for things
> like MSI, AER, ASPM, etc.  "PCI: <driver-tag>" is the usual pattern
> for things specific to a driver, and it sounds like this is what
> you've already done:
>
>   $ git log --oneline drivers/pci/controller/pci-versatile.c
>   6086987bdeb5 PCI: versatile: Remove redundant variable retval
>   b64aa11eb2dd PCI: Set bridge map_irq and swizzle_irq to default functions
>   669cbc708122 PCI: Move DT resource setup into devm_pci_alloc_host_bridge()
>   d3bb94d06aae PCI: Drop unnecessary zeroing of bridge fields
>   6a589900d050 PCI: Set default bridge parent device
>   79cbde56f98b PCI: versatile: Drop flag PCI_ENABLE_PROC_DOMAINS
>   3cf0eead9fb8 PCI: controller: Convert to devm_platform_ioremap_resource()
>   a4b21b858b56 PCI: versatile: Use pci_host_probe() to register host
>   331f63457165 PCI: of: Add inbound resource parsing to helpers
>   2999dea8e94a PCI: versatile: Remove usage of PHYS_OFFSET
>   f9f4fdaa3509 PCI: versatile: Use pci_parse_request_of_pci_ranges()
>   0018b265adf7 PCI: versatile: Fix I/O space page leak

Oh good, that's what I was hoping.

After a bunch more bugfixing It's coming out as 'kbuild, PCI: versatile"
now. (This seems better than 'kbuild: PCI: versatile' because 'PCI:
versatile' isn't a subsystem of 'kbuild'.)

>> ... If the rules for generating prefixes vary by subsystem this means
>> I'll have to fight through God knows how many annoyed maintainers to get
>> this incredibly trivial change in.)
>
> I think your script generally does the right thing, and it's already
> far more than most folks do, so thank you for that!

Heh, I knew it wouldn't be time totally wasted: I've saved time already
on account of having had to redo the individual patches in some way
eight or nine times by now, and being able to make the changes in one go
and split it out into separate commits after that was definitely easier
than the alternative.

FYI: the splitting script's improved a bit. it's still an undocumented,
uncommented horror, but it now supports arbitrary regex-replacements of
"bad prefixes" in a file prefix-transforms.yaml, for this run containing

treewide: ''
fix spelling mistake: ''
task_get_unused_fd_flags: ''
x86/mm/dump_pagetables: 'mm'

I stuffed the splitting script in
https://github.com/nickalcock/linux.git mass-split (as
scripts/kernel-mass-split) just in case it's useful to someone else.
Definitely absolutely not for upstreaming!

(Next tranche coming soon, the previous set again -- improved as
suggested, removing MODULE_LICENSE instead of commenting out, etc -- and
then another, bigger tranche inflicted on a different subset of
maintainers.)

-- 
NULL && (void)

Powered by blists - more mailing lists