[RFC] Introduce HAVE_IDE to support flexible IDE per arch configuration

Previous thread: [PATCH 1/2] kmemcheck v3 by Vegard Nossum on Thursday, February 7, 2008 - 2:36 pm. (25 messages)

Next thread: ACPI_WMI: worst config description of all times by Pavel Machek on Thursday, February 7, 2008 - 2:47 pm. (15 messages)
From: Sam Ravnborg
Date: Thursday, February 7, 2008 - 2:42 pm

Following patch introduce HAVE_IDE to support flexible per
arch or even per. sub-arch configuration of IDE support.
This patch is needed to allow arm to use the generic
drivers/Kconfig file.

Introducing HAVE_IDE so each arch explicit select HAVE_IDE
if supported allowed us to get rid of HAS_IOMEM which
is anyway overloaded.
And doing it this way is a much better way to document which
architectures that supports IDE.
Furthermore the decision if IDE is supported or not is
distributed.
Consider seeing this all over:

-if PCMCIA || ARCH_CLPS7500 || ARCH_IOP32X || ARCH_IOP33X || ARCH_IXP4XX \
-       || ARCH_L7200 || ARCH_LH7A40X || ARCH_PXA || ARCH_RPC \
-       || ARCH_S3C2410 || ARCH_SA1100 || ARCH_SHARK || FOOTBRIDGE \
-       || ARCH_IXP23XX
 source "drivers/ide/Kconfig"
-endif

Only s390 and um does not support IDE from my quick
investigation, if there are others let me know.
[Added linux-arch to catch all arch maintainers].

Comments?

	Sam

From 085cc6a6b7781961ce0f5df9876e4b0a40ee2eb8 Mon Sep 17 00:00:00 2001
From: Sam Ravnborg <sam@ravnborg.org>
Date: Thu, 7 Feb 2008 22:27:10 +0100
Subject: [PATCH] ide: introduce HAVE_IDE

To allow flexible configuration of IDE introduce HAVE_IDE.
All archs except arm, um and s390 unconditionally select it.
For arm the actual configuration determine if IDE is supported.

Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
Cc: Russell King - ARM Linux <linux@arm.linux.org.uk>
Cc: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
---
 arch/alpha/Kconfig     |    1 +
 arch/arm/Kconfig       |   19 ++++++++++++++-----
 arch/avr32/Kconfig     |    1 +
 arch/blackfin/Kconfig  |    1 +
 arch/cris/Kconfig      |    1 +
 arch/frv/Kconfig       |    1 +
 arch/h8300/Kconfig     |    1 +
 arch/ia64/Kconfig      |    1 +
 arch/m32r/Kconfig      |    1 +
 arch/m68k/Kconfig      |    1 +
 arch/m68knommu/Kconfig |    1 +
 arch/mips/Kconfig      |    1 +
 arch/parisc/Kconfig    |    1 +
 arch/powerpc/Kconfig   |    1 +
 arch/ppc/Kconfig    ...
From: Russell King - ARM Linux
Date: Thursday, February 7, 2008 - 2:54 pm

Acked-by: Russell King <rmk+kernel@arm.linux.org.uk>
--

From: Sam Ravnborg
Date: Friday, February 8, 2008 - 1:04 pm

Hi Bartlomiej.

Do you have any opinion on this patch?
I can push it via kbuild.git if you prefer but not without an
acknowledge from you.

	Sam
--

From: Bartlomiej Zolnierkiewicz
Date: Friday, February 8, 2008 - 1:26 pm

Hi,




Thanks,
Bart
--

Previous thread: [PATCH 1/2] kmemcheck v3 by Vegard Nossum on Thursday, February 7, 2008 - 2:36 pm. (25 messages)

Next thread: ACPI_WMI: worst config description of all times by Pavel Machek on Thursday, February 7, 2008 - 2:47 pm. (15 messages)