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;
Date:   Mon, 3 Jul 2023 12:42:33 -0500
From:   Bjorn Helgaas <helgaas@kernel.org>
To:     Niklas Schnelle <schnelle@linux.ibm.com>
Cc:     Bjorn Helgaas <bhelgaas@google.com>, Arnd Bergmann <arnd@arndb.de>,
        linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v6 0/2] PCI: Handle HAS_IOPORT dependencies
Message-ID: <20230703174233.GA529479@bhelgaas>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20230703135255.2202721-1-schnelle@linux.ibm.com>
Precedence: bulk

On Mon, Jul 03, 2023 at 03:52:53PM +0200, Niklas Schnelle wrote:
> Hi Bjorn,
> 
> 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 this as
> a complete treewide series titled "treewide: Remove I/O port accessors for
> HAS_IOPORT=n" with the latest being its 5th version[0]. Now about half 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 merged
> via the relevant tree such that with all subsystems complete we can follow
> this up with the last patch[1] that will make the I/O port accessors
> compile-time optional.

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.

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