diff mbox

[03/21] sparc: drop unused Kconfig symbol

Message ID 1318595257.6132.59.camel@x61.thuisdomein
State Not Applicable
Delegated to: David Miller
Headers show

Commit Message

Paul Bolle Oct. 14, 2011, 12:27 p.m. UTC
Signed-off-by: Paul Bolle <pebolle@tiscali.nl>
---
 arch/sparc/Kconfig |    7 +------
 1 files changed, 1 insertions(+), 6 deletions(-)

Comments

Chris Newport Oct. 15, 2011, 12:14 a.m. UTC | #1
How does this affect Javastations, ISTR that they need PCI and one
(rare) variant has a PCI slot. ??

On Fri, 14 Oct 2011, Paul Bolle wrote:

> Signed-off-by: Paul Bolle <pebolle@tiscali.nl>
> ---
> arch/sparc/Kconfig |    7 +------
> 1 files changed, 1 insertions(+), 6 deletions(-)
>
> diff --git a/arch/sparc/Kconfig b/arch/sparc/Kconfig
> index 1a6f20d..f92602e 100644
> --- a/arch/sparc/Kconfig
> +++ b/arch/sparc/Kconfig
> @@ -560,12 +560,7 @@ config SUN_OPENPROMFS
> 	  Only choose N if you know in advance that you will not need to modify
> 	  OpenPROM settings on the running system.
>
> -# Makefile helpers
> -config SPARC32_PCI
> -	bool
> -	default y
> -	depends on SPARC32 && PCI
> -
> +# Makefile helper
> config SPARC64_PCI
> 	bool
> 	default y
> -- 
> 1.7.4.4
>
>
>
>
> --
> To unsubscribe from this list: send the line "unsubscribe sparclinux" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>
--
To unsubscribe from this list: send the line "unsubscribe sparclinux" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
David Miller Oct. 15, 2011, 12:34 a.m. UTC | #2
From: crn@netunix.com
Date: Sat, 15 Oct 2011 01:14:02 +0100 (BST)

> 
> How does this affect Javastations, ISTR that they need PCI and one
> (rare) variant has a PCI slot. ??

How about you take even the briefest look at the code and find out for
yourself?  This is what drives me crazy about the questions you tend
to ask, you never put forth any effort of investigation on your own
and just toss arbitrary questions onto the list.

Meanwhile those of us with initiative are burdoned to do the work for
you.

The config symbol is completely unreferenced anywhere, so this
change can't possibly make any functional difference whatsoever.
--
To unsubscribe from this list: send the line "unsubscribe sparclinux" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Sam Ravnborg Oct. 16, 2011, 8:22 p.m. UTC | #3
On Sat, Oct 15, 2011 at 01:14:02AM +0100, crn@netunix.com wrote:
>
> How does this affect Javastations, ISTR that they need PCI and one
> (rare) variant has a PCI slot. ??

It was replaced by a more specific CONFIG_SPARC_PCIC symbol
in commit: cfe3af5dde34a019c1bd2485b1dc0f2d7a326ccd

	Sam
--
To unsubscribe from this list: send the line "unsubscribe sparclinux" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/arch/sparc/Kconfig b/arch/sparc/Kconfig
index 1a6f20d..f92602e 100644
--- a/arch/sparc/Kconfig
+++ b/arch/sparc/Kconfig
@@ -560,12 +560,7 @@  config SUN_OPENPROMFS
 	  Only choose N if you know in advance that you will not need to modify
 	  OpenPROM settings on the running system.
 
-# Makefile helpers
-config SPARC32_PCI
-	bool
-	default y
-	depends on SPARC32 && PCI
-
+# Makefile helper
 config SPARC64_PCI
 	bool
 	default y