All of lore.kernel.org
 help / color / mirror / Atom feed
From: Arnd Bergmann <arnd@kernel.org>
To: linux-arm-kernel@lists.infradead.org
Cc: Arnd Bergmann <arnd@arndb.de>,
	Russell King <linux@armlinux.org.uk>,
	Christoph Hellwig <hch@lst.de>,
	Linus Walleij <linus.walleij@linaro.org>,
	Marc Zyngier <maz@kernel.org>
Subject: [PATCH v3 4/5] ARM: footbridge: limit CONFIG_ISA to CATS
Date: Fri,  9 Sep 2022 22:57:48 +0200	[thread overview]
Message-ID: <20220909205749.1793677-5-arnd@kernel.org> (raw)
In-Reply-To: <20220909205749.1793677-1-arnd@kernel.org>

From: Arnd Bergmann <arnd@arndb.de>

The CONFIG_ISA option is only used for actual add-in cards, not
on-board devices. NetWinder has no hardware support for those,
and ebsa285 appears to not support them in Linux, as the XBUS
is not set up that way and there is no support for a PCI-ISA
bridge.

CATS on the other hand has four ISA slots connected to a PCI-ISA
bridge device.

Russell mentioned that it is no longer possible to select the sound
drivers for Netwinder after this, as the WaveArtist sound chip is
connected to the ISA bus. However, the WaveArtist module was removed
years ago in linux-4.15 and did not actually depend on CONFIG_ISA.
While the SoundBlaster compatibility mode of the SoundChip might work
with the ALSA sb16-isa driver with a little help, there is no mixer
support without the waveartist driver, and the default settings for
sb16 never probe IRQ 3.

Link: https://lore.kernel.org/linux-arm-kernel/87v8q3unlv.wl-maz@kernel.org/
Tested-by: Marc Zyngier <maz@kernel.org>
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---
 arch/arm/mach-footbridge/Kconfig | 2 --
 1 file changed, 2 deletions(-)

diff --git a/arch/arm/mach-footbridge/Kconfig b/arch/arm/mach-footbridge/Kconfig
index 1a3676109654..cd18871fd03f 100644
--- a/arch/arm/mach-footbridge/Kconfig
+++ b/arch/arm/mach-footbridge/Kconfig
@@ -18,7 +18,6 @@ config ARCH_CATS
 config ARCH_EBSA285_HOST
 	bool "EBSA285 (host mode)"
 	select ARCH_EBSA285
-	select ISA
 	select ARCH_MAY_HAVE_PC_FDC
 	select FORCE_PCI
 	help
@@ -31,7 +30,6 @@ config ARCH_NETWINDER
 	bool "NetWinder"
 	select CLKEVT_I8253
 	select CLKSRC_I8253
-	select ISA
 	select FORCE_PCI
 	help
 	  Say Y here if you intend to run this kernel on the Rebel.COM
-- 
2.29.2


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

  parent reply	other threads:[~2022-09-09 20:59 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-09-09 20:57 [PATCH v3 0/5] ARM: footbridge: cleanups Arnd Bergmann
2022-09-09 20:57 ` [PATCH v3 1/5] ARM: footbridge: remove addin mode Arnd Bergmann
2022-09-09 20:57 ` [PATCH v3 2/5] ARM: footbridge: remove leftover from personal-server Arnd Bergmann
2022-09-09 20:57 ` [PATCH v3 3/5] ARM: footbridge: move isa-dma support into footbridge Arnd Bergmann
2022-09-09 20:57 ` Arnd Bergmann [this message]
2022-09-11 19:53   ` [PATCH v3 4/5] ARM: footbridge: limit CONFIG_ISA to CATS Russell King (Oracle)
2022-09-15 14:00     ` Arnd Bergmann
2022-09-09 20:57 ` [PATCH v3 5/5] ARM: footbridge: remove custom DMA address handling Arnd Bergmann

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20220909205749.1793677-5-arnd@kernel.org \
    --to=arnd@kernel.org \
    --cc=arnd@arndb.de \
    --cc=hch@lst.de \
    --cc=linus.walleij@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux@armlinux.org.uk \
    --cc=maz@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.