Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <9778740b9e07b9bb92b86bf13b8f6329485ae418.camel@linux.ibm.com>
Subject: Re: [PATCH v6 0/2] PCI: Handle HAS_IOPORT dependencies
From:   Niklas Schnelle <schnelle@linux.ibm.com>
To:     Bjorn Helgaas <helgaas@kernel.org>
Cc:     Bjorn Helgaas <bhelgaas@google.com>, Arnd Bergmann <arnd@arndb.de>,
        linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org
Date:   Mon, 03 Jul 2023 20:02:40 +0200
In-Reply-To: <20230703174233.GA529479@bhelgaas>
References: <20230703174233.GA529479@bhelgaas>
Content-Type: text/plain; charset="UTF-8"
User-Agent: Evolution 3.48.3 (3.48.3-1.fc38) 
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
Precedence: bulk

On Mon, 2023-07-03 at 12:42 -0500, Bjorn Helgaas wrote:
> On Mon, Jul 03, 2023 at 03:52:53PM +0200, Niklas Schnelle wrote:
> > Hi Bjorn,
> >=20
> > This is a follow up to my ongoing effort of making the inb()/outb() and
> > similar I/O port accessors compile-time optional. Previously I sent thi=
s as
> > a complete treewide series titled "treewide: Remove I/O port accessors =
for
> > HAS_IOPORT=3Dn" with the latest being its 5th version[0]. Now about hal=
f of
> > the per-subsystem patches have been merged so I'm changing over to stand
> > alone subsystem patches. These series are stand alone and should be mer=
ged
> > via the relevant tree such that with all subsystems complete we can fol=
low
> > this up with the last patch[1] that will make the I/O port accessors
> > compile-time optional.
>=20
> Is the merge plan for each subsystem to merge this separately?  I
> acked these so they could be merged along with all the tree-wide
> changes.

Hi Bj=C3=B6rn,

Yeah this went back and forth a little, sorry about that. With the
Kconfig introduction of HAS_IOPORT merged about half of the original
patches have been merged via subsystem maintainers or are at least
sitting in linux-next already.  Arnd was anticipating that he'll pick
up some left overs but at the moment the simplest is for subsystems to
pick the patches up themselves ideally and if small enough for v6.5
still.

Thanks,
Niklas

>=20
> > As for compile-time vs runtime see Linus' reply to my first attempt[2].
> >=20
> > Other than rebasing on current master there are no changes to these
> > two patches for the PCI subsystem.
> >=20
> > Thanks,
> > Niklas
> >=20
> > [0] https://lore.kernel.org/all/20230516110038.2413224-1-schnelle@linux=
.ibm.com/
> > [1] https://lore.kernel.org/all/20230516110038.2413224-42-schnelle@linu=
x.ibm.com/
> > [2] https://lore.kernel.org/lkml/CAHk-=3Dwg80je=3DK7madF4e7WrRNp37e3qh6=
y10Svhdc7O8SZ_-8g@mail.gmail.com/
> >=20
> > Niklas Schnelle (2):
> >   PCI: Make quirk using inw() depend on HAS_IOPORT
> >   PCI/sysfs: Make I/O resource depend on HAS_IOPORT
> >=20
> >  drivers/pci/pci-sysfs.c | 4 ++++
> >  drivers/pci/quirks.c    | 2 ++
> >  2 files changed, 6 insertions(+)
> >=20
> >=20
> > base-commit: a901a3568fd26ca9c4a82d8bc5ed5b3ed844d451
> > --=20
> > 2.39.2
> >=20