2013-10-08 05:13:14

by Mark Salter

[permalink] [raw]
Subject: [PATCH v2 00/14] Kconfig cleanup (PARPORT_PC dependencies)

This patch series removes the messy dependencies from PARPORT_PC
by having it depend on one symbol (ARCH_MAY_HAVE_PC_PARPORT) and
having architectures which need it, select ARCH_MAY_HAVE_PC_PARPORT
in arch/*/Kconfig.

New architectures are unlikely to need PARPORT_PC, so this avoids
having an ever growing list of architectures to exclude. Those
architectures which do select ARCH_MAY_HAVE_PC_PARPORT in this
patch are the ones which have an asm/parport.h (or use the generic
version).

V1 or this patch can be found here:

http://www.spinics.net/lists/linux-arch/msg23167.html

V2 changes:

* Use select instead of adding config option to arch/*/Kconfig
* Split into multiple patches for individual architectures
* Dropped tile architecture

Mark Salter (14):
alpha: select ARCH_MAY_HAVE_PC_PARPORT
arc: select ARCH_MAY_HAVE_PC_PARPORT
arm: select ARCH_MAY_HAVE_PC_PARPORT
ia64: select ARCH_MAY_HAVE_PC_PARPORT
m68k: select ARCH_MAY_HAVE_PC_PARPORT
microblaze: select ARCH_MAY_HAVE_PC_PARPORT
mips: select ARCH_MAY_HAVE_PC_PARPORT
parisc: select ARCH_MAY_HAVE_PC_PARPORT
powerpc: select ARCH_MAY_HAVE_PC_PARPORT
sh: select ARCH_MAY_HAVE_PC_PARPORT
sparc: select ARCH_MAY_HAVE_PC_PARPORT
unicore32: select ARCH_MAY_HAVE_PC_PARPORT
x86: select ARCH_MAY_HAVE_PC_PARPORT
Kconfig cleanup (PARPORT_PC dependencies)

arch/alpha/Kconfig | 1 +
arch/arc/Kconfig | 1 +
arch/arm/Kconfig | 1 +
arch/ia64/Kconfig | 1 +
arch/m68k/Kconfig | 1 +
arch/microblaze/Kconfig | 1 +
arch/mips/Kconfig | 1 +
arch/parisc/Kconfig | 1 +
arch/powerpc/Kconfig | 1 +
arch/sh/Kconfig | 1 +
arch/sparc/Kconfig | 1 +
arch/unicore32/Kconfig | 1 +
arch/x86/Kconfig | 1 +
drivers/parport/Kconfig | 10 +++++++---
14 files changed, 20 insertions(+), 3 deletions(-)

--
1.8.3.1


2013-10-08 05:12:34

by Mark Salter

[permalink] [raw]
Subject: [PATCH v2 01/14] alpha: select ARCH_MAY_HAVE_PC_PARPORT

Architectures which support CONFIG_PARPORT_PC should select
ARCH_MAY_HAVE_PC_PARPORT.

Signed-off-by: Mark Salter <[email protected]>
Acked-by: Richard Henderson <[email protected]>
CC: [email protected]
---
arch/alpha/Kconfig | 1 +
1 file changed, 1 insertion(+)

diff --git a/arch/alpha/Kconfig b/arch/alpha/Kconfig
index 35a300d..626949c 100644
--- a/arch/alpha/Kconfig
+++ b/arch/alpha/Kconfig
@@ -1,6 +1,7 @@
config ALPHA
bool
default y
+ select ARCH_MAY_HAVE_PC_PARPORT
select HAVE_AOUT
select HAVE_IDE
select HAVE_OPROFILE
--
1.8.3.1

2013-10-08 05:13:06

by Mark Salter

[permalink] [raw]
Subject: [PATCH v2 04/14] ia64: select ARCH_MAY_HAVE_PC_PARPORT

Architectures which support CONFIG_PARPORT_PC should select
ARCH_MAY_HAVE_PC_PARPORT.

Signed-off-by: Mark Salter <[email protected]>
CC: Tony Luck <[email protected]>
CC: Fenghua Yu <[email protected]>
CC: [email protected]
---
arch/ia64/Kconfig | 1 +
1 file changed, 1 insertion(+)

diff --git a/arch/ia64/Kconfig b/arch/ia64/Kconfig
index 7740ab1..f88117a 100644
--- a/arch/ia64/Kconfig
+++ b/arch/ia64/Kconfig
@@ -6,6 +6,7 @@ menu "Processor type and features"

config IA64
bool
+ select ARCH_MAY_HAVE_PC_PARPORT
select PCI if (!IA64_HP_SIM)
select ACPI if (!IA64_HP_SIM)
select PM if (!IA64_HP_SIM)
--
1.8.3.1

2013-10-08 05:13:10

by Mark Salter

[permalink] [raw]
Subject: [PATCH v2 02/14] arc: select ARCH_MAY_HAVE_PC_PARPORT

Architectures which support CONFIG_PARPORT_PC should select
ARCH_MAY_HAVE_PC_PARPORT.

Signed-off-by: Mark Salter <[email protected]>
CC: Vineet Gupta <[email protected]>
---
arch/arc/Kconfig | 1 +
1 file changed, 1 insertion(+)

diff --git a/arch/arc/Kconfig b/arch/arc/Kconfig
index 91dbb27..484b1a7 100644
--- a/arch/arc/Kconfig
+++ b/arch/arc/Kconfig
@@ -8,6 +8,7 @@

config ARC
def_bool y
+ select ARCH_MAY_HAVE_PC_PARPORT
select CLONE_BACKWARDS
# ARC Busybox based initramfs absolutely relies on DEVTMPFS for /dev
select DEVTMPFS if !INITRAMFS_SOURCE=""
--
1.8.3.1

2013-10-08 05:13:16

by Mark Salter

[permalink] [raw]
Subject: [PATCH v2 08/14] parisc: select ARCH_MAY_HAVE_PC_PARPORT

Architectures which support CONFIG_PARPORT_PC should select
ARCH_MAY_HAVE_PC_PARPORT.

Signed-off-by: Mark Salter <[email protected]>
CC: "James E.J. Bottomley" <[email protected]>
CC: Helge Deller <[email protected]>
CC: [email protected]
---
arch/parisc/Kconfig | 1 +
1 file changed, 1 insertion(+)

diff --git a/arch/parisc/Kconfig b/arch/parisc/Kconfig
index ad2ce8d..9af24ac 100644
--- a/arch/parisc/Kconfig
+++ b/arch/parisc/Kconfig
@@ -1,6 +1,7 @@
config PARISC
def_bool y
select ARCH_HAS_DEBUG_STRICT_USER_COPY_CHECKS
+ select ARCH_MAY_HAVE_PC_PARPORT
select HAVE_IDE
select HAVE_OPROFILE
select HAVE_FUNCTION_TRACER if 64BIT
--
1.8.3.1

2013-10-08 05:13:22

by Mark Salter

[permalink] [raw]
Subject: [PATCH v2 06/14] microblaze: select ARCH_MAY_HAVE_PC_PARPORT

Architectures which support CONFIG_PARPORT_PC should select
ARCH_MAY_HAVE_PC_PARPORT.

Signed-off-by: Mark Salter <[email protected]>
CC: Michal Simek <[email protected]>
CC: [email protected]
---
arch/microblaze/Kconfig | 1 +
1 file changed, 1 insertion(+)

diff --git a/arch/microblaze/Kconfig b/arch/microblaze/Kconfig
index b82f82b..19caa61 100644
--- a/arch/microblaze/Kconfig
+++ b/arch/microblaze/Kconfig
@@ -1,5 +1,6 @@
config MICROBLAZE
def_bool y
+ select ARCH_MAY_HAVE_PC_PARPORT
select HAVE_MEMBLOCK
select HAVE_MEMBLOCK_NODE_MAP
select HAVE_FUNCTION_TRACER
--
1.8.3.1

2013-10-08 05:13:31

by Mark Salter

[permalink] [raw]
Subject: [PATCH v2 13/14] x86: select ARCH_MAY_HAVE_PC_PARPORT

Architectures which support CONFIG_PARPORT_PC should select
ARCH_MAY_HAVE_PC_PARPORT.

Signed-off-by: Mark Salter <[email protected]>
CC: Thomas Gleixner <[email protected]>
CC: Ingo Molnar <[email protected]>
CC: "H. Peter Anvin" <[email protected]>
CC: [email protected]
---
arch/x86/Kconfig | 1 +
1 file changed, 1 insertion(+)

diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig
index ee2fb9d..72fca40 100644
--- a/arch/x86/Kconfig
+++ b/arch/x86/Kconfig
@@ -22,6 +22,7 @@ config X86_64
config X86
def_bool y
select ARCH_HAS_DEBUG_STRICT_USER_COPY_CHECKS
+ select ARCH_MAY_HAVE_PC_PARPORT
select HAVE_AOUT if X86_32
select HAVE_UNSTABLE_SCHED_CLOCK
select ARCH_SUPPORTS_NUMA_BALANCING
--
1.8.3.1

2013-10-08 05:13:25

by Mark Salter

[permalink] [raw]
Subject: [PATCH v2 09/14] powerpc: select ARCH_MAY_HAVE_PC_PARPORT

Architectures which support CONFIG_PARPORT_PC should select
ARCH_MAY_HAVE_PC_PARPORT.

Signed-off-by: Mark Salter <[email protected]>
CC: Benjamin Herrenschmidt <[email protected]>
CC: Paul Mackerras <[email protected]>
CC: [email protected]
---
arch/powerpc/Kconfig | 1 +
1 file changed, 1 insertion(+)

diff --git a/arch/powerpc/Kconfig b/arch/powerpc/Kconfig
index 38f3b7e..9211207 100644
--- a/arch/powerpc/Kconfig
+++ b/arch/powerpc/Kconfig
@@ -85,6 +85,7 @@ config GENERIC_HWEIGHT
config PPC
bool
default y
+ select ARCH_MAY_HAVE_PC_PARPORT
select BINFMT_ELF
select OF
select OF_EARLY_FLATTREE
--
1.8.3.1

2013-10-08 05:13:30

by Mark Salter

[permalink] [raw]
Subject: [PATCH v2 12/14] unicore32: select ARCH_MAY_HAVE_PC_PARPORT

Architectures which support CONFIG_PARPORT_PC should select
ARCH_MAY_HAVE_PC_PARPORT.

Signed-off-by: Mark Salter <[email protected]>
CC: Guan Xuetao <[email protected]>
---
arch/unicore32/Kconfig | 1 +
1 file changed, 1 insertion(+)

diff --git a/arch/unicore32/Kconfig b/arch/unicore32/Kconfig
index 82cdd89..9e0b8bd 100644
--- a/arch/unicore32/Kconfig
+++ b/arch/unicore32/Kconfig
@@ -1,5 +1,6 @@
config UNICORE32
def_bool y
+ select ARCH_MAY_HAVE_PC_PARPORT
select HAVE_MEMBLOCK
select HAVE_GENERIC_DMA_COHERENT
select HAVE_DMA_ATTRS
--
1.8.3.1

2013-10-08 05:13:27

by Mark Salter

[permalink] [raw]
Subject: [PATCH v2 10/14] sh: select ARCH_MAY_HAVE_PC_PARPORT

Architectures which support CONFIG_PARPORT_PC should select
ARCH_MAY_HAVE_PC_PARPORT.

Signed-off-by: Mark Salter <[email protected]>
CC: Paul Mundt <[email protected]>
CC: [email protected]
---
arch/sh/Kconfig | 1 +
1 file changed, 1 insertion(+)

diff --git a/arch/sh/Kconfig b/arch/sh/Kconfig
index 224f4bc..a5d1d2e 100644
--- a/arch/sh/Kconfig
+++ b/arch/sh/Kconfig
@@ -1,5 +1,6 @@
config SUPERH
def_bool y
+ select ARCH_MAY_HAVE_PC_PARPORT
select EXPERT
select CLKDEV_LOOKUP
select HAVE_IDE if HAS_IOPORT
--
1.8.3.1

2013-10-08 05:13:20

by Mark Salter

[permalink] [raw]
Subject: [PATCH v2 11/14] sparc: select ARCH_MAY_HAVE_PC_PARPORT

Architectures which support CONFIG_PARPORT_PC should select
ARCH_MAY_HAVE_PC_PARPORT.

Signed-off-by: Mark Salter <[email protected]>
CC: "David S. Miller" <[email protected]>
CC: [email protected]
---
arch/sparc/Kconfig | 1 +
1 file changed, 1 insertion(+)

diff --git a/arch/sparc/Kconfig b/arch/sparc/Kconfig
index 78c4fdb..362229a 100644
--- a/arch/sparc/Kconfig
+++ b/arch/sparc/Kconfig
@@ -12,6 +12,7 @@ config 64BIT
config SPARC
bool
default y
+ select ARCH_MAY_HAVE_PC_PARPORT if SPARC64 && PCI
select OF
select OF_PROMTREE
select HAVE_IDE
--
1.8.3.1

2013-10-08 05:13:12

by Mark Salter

[permalink] [raw]
Subject: [PATCH v2 07/14] mips: select ARCH_MAY_HAVE_PC_PARPORT

Architectures which support CONFIG_PARPORT_PC should select
ARCH_MAY_HAVE_PC_PARPORT.

Signed-off-by: Mark Salter <[email protected]>
CC: Ralf Baechle <[email protected]>
CC: [email protected]
---
arch/mips/Kconfig | 1 +
1 file changed, 1 insertion(+)

diff --git a/arch/mips/Kconfig b/arch/mips/Kconfig
index f75ab4a..199fde67 100644
--- a/arch/mips/Kconfig
+++ b/arch/mips/Kconfig
@@ -1,6 +1,7 @@
config MIPS
bool
default y
+ select ARCH_MAY_HAVE_PC_PARPORT
select HAVE_CONTEXT_TRACKING
select HAVE_GENERIC_DMA_COHERENT
select HAVE_IDE
--
1.8.3.1

2013-10-08 05:13:05

by Mark Salter

[permalink] [raw]
Subject: [PATCH v2 05/14] m68k: select ARCH_MAY_HAVE_PC_PARPORT

Architectures which support CONFIG_PARPORT_PC should select
ARCH_MAY_HAVE_PC_PARPORT.

Signed-off-by: Mark Salter <[email protected]>
Acked by: Geert Uytterhoeven <[email protected]>
CC: [email protected]
---
arch/m68k/Kconfig | 1 +
1 file changed, 1 insertion(+)

diff --git a/arch/m68k/Kconfig b/arch/m68k/Kconfig
index 311a300..0d88a0b 100644
--- a/arch/m68k/Kconfig
+++ b/arch/m68k/Kconfig
@@ -1,6 +1,7 @@
config M68K
bool
default y
+ select ARCH_MAY_HAVE_PC_PARPORT if ISA
select HAVE_IDE
select HAVE_AOUT if MMU
select HAVE_DEBUG_BUGVERBOSE
--
1.8.3.1

2013-10-08 05:15:52

by Mark Salter

[permalink] [raw]
Subject: [PATCH v2 03/14] arm: select ARCH_MAY_HAVE_PC_PARPORT

Architectures which support CONFIG_PARPORT_PC should select
ARCH_MAY_HAVE_PC_PARPORT.

Signed-off-by: Mark Salter <[email protected]>
CC: Russell King <[email protected]>
CC: [email protected]
---
arch/arm/Kconfig | 1 +
1 file changed, 1 insertion(+)

diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
index 1ad6fb6..2734ec1 100644
--- a/arch/arm/Kconfig
+++ b/arch/arm/Kconfig
@@ -5,6 +5,7 @@ config ARM
select ARCH_HAS_ATOMIC64_DEC_IF_POSITIVE
select ARCH_HAS_TICK_BROADCAST if GENERIC_CLOCKEVENTS_BROADCAST
select ARCH_HAVE_CUSTOM_GPIO_H
+ select ARCH_MAY_HAVE_PC_PARPORT
select ARCH_WANT_IPC_PARSE_VERSION
select BUILDTIME_EXTABLE_SORT if MMU
select CLONE_BACKWARDS
--
1.8.3.1

2013-10-08 05:43:21

by Ingo Molnar

[permalink] [raw]
Subject: Re: [PATCH v2 13/14] x86: select ARCH_MAY_HAVE_PC_PARPORT


* Mark Salter <[email protected]> wrote:

> Architectures which support CONFIG_PARPORT_PC should select
> ARCH_MAY_HAVE_PC_PARPORT.
>
> Signed-off-by: Mark Salter <[email protected]>
> CC: Thomas Gleixner <[email protected]>
> CC: Ingo Molnar <[email protected]>
> CC: "H. Peter Anvin" <[email protected]>
> CC: [email protected]
> ---
> arch/x86/Kconfig | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig
> index ee2fb9d..72fca40 100644
> --- a/arch/x86/Kconfig
> +++ b/arch/x86/Kconfig
> @@ -22,6 +22,7 @@ config X86_64
> config X86
> def_bool y
> select ARCH_HAS_DEBUG_STRICT_USER_COPY_CHECKS
> + select ARCH_MAY_HAVE_PC_PARPORT
> select HAVE_AOUT if X86_32
> select HAVE_UNSTABLE_SCHED_CLOCK
> select ARCH_SUPPORTS_NUMA_BALANCING

Acked-by: Ingo Molnar <[email protected]>

Thanks,

Ingo

2013-10-08 05:44:38

by Vineet Gupta

[permalink] [raw]
Subject: Re: [PATCH v2 02/14] arc: select ARCH_MAY_HAVE_PC_PARPORT

Hi Mark,

On 10/08/2013 10:40 AM, Mark Salter wrote:
> Architectures which support CONFIG_PARPORT_PC should select
> ARCH_MAY_HAVE_PC_PARPORT.
>
> Signed-off-by: Mark Salter <[email protected]>
> CC: Vineet Gupta <[email protected]>
> ---
> arch/arc/Kconfig | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/arch/arc/Kconfig b/arch/arc/Kconfig
> index 91dbb27..484b1a7 100644
> --- a/arch/arc/Kconfig
> +++ b/arch/arc/Kconfig
> @@ -8,6 +8,7 @@
>
> config ARC
> def_bool y
> + select ARCH_MAY_HAVE_PC_PARPORT
> select CLONE_BACKWARDS
> # ARC Busybox based initramfs absolutely relies on DEVTMPFS for /dev
> select DEVTMPFS if !INITRAMFS_SOURCE=""
>


You can drop ARC from the series, we don't care for this deivce.

Thx,
-Vineet

2013-10-08 07:02:13

by Ralf Baechle

[permalink] [raw]
Subject: Re: [PATCH v2 07/14] mips: select ARCH_MAY_HAVE_PC_PARPORT

On Tue, Oct 08, 2013 at 01:10:23AM -0400, Mark Salter wrote:

> Architectures which support CONFIG_PARPORT_PC should select
> ARCH_MAY_HAVE_PC_PARPORT.

Acked-by: Ralf Baechle <[email protected]>

Ralf