Patchwork [03/21] sparc: drop unused Kconfig symbol

login
register
mail settings
Submitter Paul Bolle
Date Oct. 14, 2011, 12:27 p.m.
Message ID <1318595257.6132.59.camel@x61.thuisdomein>
Download mbox | patch
Permalink /patch/119779/
State Not Applicable
Delegated to: David Miller
Headers show

Comments

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

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