[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <866973b7-1f54-21a3-79aa-992ed0594c1a@lwfinger.net>
Date: Tue, 28 Feb 2023 19:13:48 -0600
From: Larry Finger <Larry.Finger@...inger.net>
To: Arnd Bergmann <arnd@...db.de>, Arnd Bergmann <arnd@...nel.org>,
Dominik Brodowski <linux@...inikbrodowski.net>,
linux-kernel@...r.kernel.org
Cc: Bjorn Helgaas <bhelgaas@...gle.com>,
Florian Fainelli <f.fainelli@...il.com>,
Hartley Sweeten <hsweeten@...ionengravers.com>,
Ian Abbott <abbotti@....co.uk>,
Jakub Kicinski <kuba@...nel.org>,
Kevin Cernekee <cernekee@...il.com>,
Lukas Wunner <lukas@...ner.de>,
Manuel Lauss <manuel.lauss@...il.com>,
Oliver Hartkopp <socketcan@...tkopp.net>,
Olof Johansson <olof@...om.net>,
Robert Jarzmik <robert.jarzmik@...e.fr>,
YOKOTA Hiroshi <yokota@...lab.is.tsukuba.ac.jp>,
bcm-kernel-feedback-list@...adcom.com,
linux-arm-kernel@...ts.infradead.org, linux-can@...r.kernel.org,
linux-mips@...r.kernel.org, linux-pci@...r.kernel.org,
linux-wireless@...r.kernel.org, Netdev <netdev@...r.kernel.org>
Subject: Re: [RFC 0/6] pcmcia: separate 16-bit support from cardbus
On 2/28/23 02:37, Arnd Bergmann wrote:
> My intention was to keep Cardbus support working with old defconfig files,
> and I've not moved CONFIG_CARDBUS into a separate submenu between
> CONFIG_PCI_HOTPLUG and CONFIG_PCI_CONTROLLER but left the driver in
> drivers/pci/hotplug. I think that's the best compromise here, but maybe
> the PCI maintainers have a better idea.
Arnd,
I did a bit more investigation. My original .config had CONFIG_PCI_HOTPLUG not
defined, but did have CONFIG_CARDBUS and the various yenta modules turned on.
With your changes, the CONFIG_PCI_HOTPLUG overrode CARDBUS.
I thought mine was a corner case, but now I am not sure. As stated above, the
Debian 12 factory configuration for ppc32 does not turn on PCI hotplug, but the
x86_64 configuration for openSUSE Tumbleweed does. The x86_64 configuration in
Fedora 37 does not contain CONFIG_PCI_HOTPLUG, but does have CARDBUS set.
It seems that several distros may get the wrong result with this change,
Larry
Powered by blists - more mailing lists