Patchwork powerpc: make Freescale QE support a selectable Kconfig option

login
register
mail settings
Submitter Anton Vorontsov
Date Oct. 3, 2008, 4:23 p.m.
Message ID <20081003162317.GA31740@oksana.dev.rtsoft.ru>
Download mbox | patch
Permalink /patch/2595/
State Changes Requested
Delegated to: Kumar Gala
Headers show

Comments

Anton Vorontsov - Oct. 3, 2008, 4:23 p.m.
Otherwise user-selectable options appears at the top level.

Signed-off-by: Anton Vorontsov <avorontsov@ru.mvista.com>
---

On Fri, Oct 03, 2008 at 11:09:09AM -0500, Timur Tabi wrote:
> Anton Vorontsov wrote:
> 
> > Yup. David said that platform GPIO controllers should not go into the
> > drivers/gpio/, their place is in the arch/.
> 
> Then when I do "make menuconfig", why doesn't that option appear under "Platform
> support"?

Because the whole arch/powerpc/sysdev/qe_lib/Kconfig includes into
the top level Kconfig.

> This is what I see now:
> 
> [*] Networking support  --->
>     Device Drivers  --->
>     File systems  --->
> [*] QE GPIO support
>     Library routines  --->
>     Kernel hacking  --->

Damn, this is bad indeed (I don't use the menuconfig, never saw this).

I think arch/powerpc/sysdev/qe_lib/Kconfig inclusion should be fixed.

How about this patch? Or we can move QUICC_ENGINE into the
qe_lib/Kconfig and include it in the platform/Kconfig... either
way would work.

Thanks for noticing.

 arch/powerpc/Kconfig               |    2 --
 arch/powerpc/platforms/Kconfig     |   36 ++++++++++++++++++++++++++++++++++++
 arch/powerpc/sysdev/qe_lib/Kconfig |   35 -----------------------------------
 3 files changed, 36 insertions(+), 37 deletions(-)
 delete mode 100644 arch/powerpc/sysdev/qe_lib/Kconfig
Timur Tabi - Oct. 3, 2008, 4:54 p.m.
On Fri, Oct 3, 2008 at 11:23 AM, Anton Vorontsov
<avorontsov@ru.mvista.com> wrote:
> How about this patch? Or we can move QUICC_ENGINE into the
> qe_lib/Kconfig and include it in the platform/Kconfig... either
> way would work.
>
> Thanks for noticing.
>
>  arch/powerpc/Kconfig               |    2 --
>  arch/powerpc/platforms/Kconfig     |   36 ++++++++++++++++++++++++++++++++++++
>  arch/powerpc/sysdev/qe_lib/Kconfig |   35 -----------------------------------
>  3 files changed, 36 insertions(+), 37 deletions(-)
>  delete mode 100644 arch/powerpc/sysdev/qe_lib/Kconfig

Acked-by: Timur Tabi <timur@freescale.com>
Kumar Gala - Oct. 3, 2008, 5:48 p.m.
> I think arch/powerpc/sysdev/qe_lib/Kconfig inclusion should be fixed.
>
> How about this patch? Or we can move QUICC_ENGINE into the
> qe_lib/Kconfig and include it in the platform/Kconfig... either
> way would work.
>
> Thanks for noticing.

I want to leave the "hidden" options in qe_lib/Kconfig.. just move the  
user selectable one (QE_GPIO) in to platforms/Kconfig

- k

Patch

diff --git a/arch/powerpc/Kconfig b/arch/powerpc/Kconfig
index 587da5e..dd976b6 100644
--- a/arch/powerpc/Kconfig
+++ b/arch/powerpc/Kconfig
@@ -829,8 +829,6 @@  source "fs/Kconfig"
 
 # XXX source "arch/ppc/8260_io/Kconfig"
 
-source "arch/powerpc/sysdev/qe_lib/Kconfig"
-
 source "lib/Kconfig"
 
 source "arch/powerpc/Kconfig.debug"
diff --git a/arch/powerpc/platforms/Kconfig b/arch/powerpc/platforms/Kconfig
index 4c900ef..da6da51 100644
--- a/arch/powerpc/platforms/Kconfig
+++ b/arch/powerpc/platforms/Kconfig
@@ -248,6 +248,42 @@  config QUICC_ENGINE
 	  Selecting this option means that you wish to build a kernel
 	  for a machine with a QE coprocessor.
 
+if QUICC_ENGINE
+
+config UCC_SLOW
+	bool
+	default y if SERIAL_QE
+	help
+	  This option provides qe_lib support to UCC slow
+	  protocols: UART, BISYNC, QMC
+
+config UCC_FAST
+	bool
+	default y if UCC_GETH
+	help
+	  This option provides qe_lib support to UCC fast
+	  protocols: HDLC, Ethernet, ATM, transparent
+
+config UCC
+	bool
+	default y if UCC_FAST || UCC_SLOW
+
+config QE_USB
+	bool
+	help
+	  QE USB Host Controller support
+
+config QE_GPIO
+	bool "QE GPIO support"
+	depends on QUICC_ENGINE
+	select GENERIC_GPIO
+	select ARCH_REQUIRE_GPIOLIB
+	help
+	  Say Y here if you're going to use hardware that connects to the
+	  QE GPIOs.
+
+endif
+
 config CPM2
 	bool "Enable support for the CPM2 (Communications Processor Module)"
 	depends on MPC85xx || 8260
diff --git a/arch/powerpc/sysdev/qe_lib/Kconfig b/arch/powerpc/sysdev/qe_lib/Kconfig
deleted file mode 100644
index 1ce5464..0000000
--- a/arch/powerpc/sysdev/qe_lib/Kconfig
+++ /dev/null
@@ -1,35 +0,0 @@ 
-#
-# QE Communication options
-#
-
-config UCC_SLOW
-	bool
-	default y if SERIAL_QE
-	help
-	  This option provides qe_lib support to UCC slow
-	  protocols: UART, BISYNC, QMC
-
-config UCC_FAST
-	bool
-	default y if UCC_GETH
-	help
-	  This option provides qe_lib support to UCC fast
-	  protocols: HDLC, Ethernet, ATM, transparent
-
-config UCC
-	bool
-	default y if UCC_FAST || UCC_SLOW
-
-config QE_USB
-	bool
-	help
-	  QE USB Host Controller support
-
-config QE_GPIO
-	bool "QE GPIO support"
-	depends on QUICC_ENGINE
-	select GENERIC_GPIO
-	select ARCH_REQUIRE_GPIOLIB
-	help
-	  Say Y here if you're going to use hardware that connects to the
-	  QE GPIOs.