Commit 51e68d05 authored by Linus Torvalds's avatar Linus Torvalds

x86 isa: add back X86_32 dependency on CONFIG_ISA

Commit b3c1be1b ("base: isa: Remove X86_32 dependency") made ISA
support available on x86-64 too.  That's not right - while there are
some LPC-style devices that might be useful still and be based on
ISA-like IP blocks, that is *not* an excuse to try to enable any random
legacy drivers.

Such drivers should be individually enabled and made to perhaps depend
on ISA_DMA_API instead (which we have continued to support on x86-64).
Or we could add another "ISA_XYZ_API" that we support that doesn't
enable random old drivers that aren't even 64-bit clean nor do we have
any test coverage for.

Turning off ISA will now also turn off some drivers that have been
marked as depending on it as part of this series, and that used to work
on modern platforms.

See for example commits ad7afc38..cc736607, which may also need
to be reverted.

This commit means that the warnings that came in due to enabling ISA
widely are now gone again.
Acked-by: default avatarWilliam Breathitt Gray <vilhelm.gray@gmail.com>
Cc: Linus Walleij <linus.walleij@linaro.org>
Cc: Guenter Roeck <linux@roeck-us.net>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 5469dc27
...@@ -2447,6 +2447,8 @@ config ISA_DMA_API ...@@ -2447,6 +2447,8 @@ config ISA_DMA_API
Enables ISA-style DMA support for devices requiring such controllers. Enables ISA-style DMA support for devices requiring such controllers.
If unsure, say Y. If unsure, say Y.
if X86_32
config ISA config ISA
bool "ISA support" bool "ISA support"
---help--- ---help---
...@@ -2456,8 +2458,6 @@ config ISA ...@@ -2456,8 +2458,6 @@ config ISA
(MCA) or VESA. ISA is an older system, now being displaced by PCI; (MCA) or VESA. ISA is an older system, now being displaced by PCI;
newer boards don't support it. If you have ISA, say Y, otherwise N. newer boards don't support it. If you have ISA, say Y, otherwise N.
if X86_32
config EISA config EISA
bool "EISA support" bool "EISA support"
depends on ISA depends on ISA
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment