* [PATCH] bcma: keep *config menu together
@ 2017-09-27 20:01 Randy Dunlap
2017-10-07 21:04 ` Randy Dunlap
2017-10-13 9:37 ` Kalle Valo
0 siblings, 2 replies; 3+ messages in thread
From: Randy Dunlap @ 2017-09-27 20:01 UTC (permalink / raw)
To: LKML, linux-wireless; +Cc: Andrew Morton, Rafał Miłecki
From: Randy Dunlap <rdunlap@infradead.org>
Use "if BCMA"/"endif" around all Kconfig symbols so that they are
kept together in *config menus instead of showing up in unexpected
places. Also remove "depends on BCMA" since this is handled by the
"if BCMA" addition.
Tested with ARCH={x86_64,MIPS} using make {n,menu,g,x}config.
Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
Cc: Rafał Miłecki <zajec5@gmail.com>
---
drivers/bcma/Kconfig | 18 +++++++++---------
1 file changed, 9 insertions(+), 9 deletions(-)
--- lnx-414-rc2.orig/drivers/bcma/Kconfig
+++ lnx-414-rc2/drivers/bcma/Kconfig
@@ -10,14 +10,15 @@ menuconfig BCMA
Bus driver for Broadcom specific Advanced Microcontroller Bus
Architecture.
+if BCMA
+
# Support for Block-I/O. SELECT this from the driver that needs it.
config BCMA_BLOCKIO
bool
- depends on BCMA
config BCMA_HOST_PCI_POSSIBLE
bool
- depends on BCMA && PCI = y
+ depends on PCI = y
default y
config BCMA_HOST_PCI
@@ -28,7 +29,6 @@ config BCMA_HOST_PCI
config BCMA_HOST_SOC
bool "Support for BCMA in a SoC"
- depends on BCMA
help
Host interface for a Broadcom AIX bus directly mapped into
the memory. This only works with the Broadcom SoCs from the
@@ -38,7 +38,7 @@ config BCMA_HOST_SOC
config BCMA_DRIVER_PCI
bool "BCMA Broadcom PCI core driver"
- depends on BCMA && PCI
+ depends on PCI
default y
help
BCMA bus may have many versions of PCIe core. This driver
@@ -54,13 +54,13 @@ config BCMA_DRIVER_PCI
config BCMA_DRIVER_PCI_HOSTMODE
bool "Driver for PCI core working in hostmode"
- depends on BCMA && MIPS && BCMA_DRIVER_PCI
+ depends on MIPS && BCMA_DRIVER_PCI
help
PCI core hostmode operation (external PCI bus).
config BCMA_DRIVER_MIPS
bool "BCMA Broadcom MIPS core driver"
- depends on BCMA && MIPS
+ depends on MIPS
help
Driver for the Broadcom MIPS core attached to Broadcom specific
Advanced Microcontroller Bus.
@@ -91,7 +91,6 @@ config BCMA_NFLASH
config BCMA_DRIVER_GMAC_CMN
bool "BCMA Broadcom GBIT MAC COMMON core driver"
- depends on BCMA
help
Driver for the Broadcom GBIT MAC COMMON core attached to Broadcom
specific Advanced Microcontroller Bus.
@@ -100,7 +99,7 @@ config BCMA_DRIVER_GMAC_CMN
config BCMA_DRIVER_GPIO
bool "BCMA GPIO driver"
- depends on BCMA && GPIOLIB
+ depends on GPIOLIB
select GPIOLIB_IRQCHIP if BCMA_HOST_SOC
help
Driver to provide access to the GPIO pins of the bcma bus.
@@ -109,8 +108,9 @@ config BCMA_DRIVER_GPIO
config BCMA_DEBUG
bool "BCMA debugging"
- depends on BCMA
help
This turns on additional debugging messages.
If unsure, say N
+
+endif # BCMA
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] bcma: keep *config menu together
2017-09-27 20:01 [PATCH] bcma: keep *config menu together Randy Dunlap
@ 2017-10-07 21:04 ` Randy Dunlap
2017-10-13 9:37 ` Kalle Valo
1 sibling, 0 replies; 3+ messages in thread
From: Randy Dunlap @ 2017-10-07 21:04 UTC (permalink / raw)
To: LKML, linux-wireless; +Cc: Andrew Morton, Rafał Miłecki
On 09/27/17 13:01, Randy Dunlap wrote:
> From: Randy Dunlap <rdunlap@infradead.org>
>
> Use "if BCMA"/"endif" around all Kconfig symbols so that they are
> kept together in *config menus instead of showing up in unexpected
> places. Also remove "depends on BCMA" since this is handled by the
> "if BCMA" addition.
>
> Tested with ARCH={x86_64,MIPS} using make {n,menu,g,x}config.
>
> Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
> Cc: Rafał Miłecki <zajec5@gmail.com>
Rafał -- ping?
> ---
> drivers/bcma/Kconfig | 18 +++++++++---------
> 1 file changed, 9 insertions(+), 9 deletions(-)
>
> --- lnx-414-rc2.orig/drivers/bcma/Kconfig
> +++ lnx-414-rc2/drivers/bcma/Kconfig
> @@ -10,14 +10,15 @@ menuconfig BCMA
> Bus driver for Broadcom specific Advanced Microcontroller Bus
> Architecture.
>
> +if BCMA
> +
> # Support for Block-I/O. SELECT this from the driver that needs it.
> config BCMA_BLOCKIO
> bool
> - depends on BCMA
>
> config BCMA_HOST_PCI_POSSIBLE
> bool
> - depends on BCMA && PCI = y
> + depends on PCI = y
> default y
>
> config BCMA_HOST_PCI
> @@ -28,7 +29,6 @@ config BCMA_HOST_PCI
>
> config BCMA_HOST_SOC
> bool "Support for BCMA in a SoC"
> - depends on BCMA
> help
> Host interface for a Broadcom AIX bus directly mapped into
> the memory. This only works with the Broadcom SoCs from the
> @@ -38,7 +38,7 @@ config BCMA_HOST_SOC
>
> config BCMA_DRIVER_PCI
> bool "BCMA Broadcom PCI core driver"
> - depends on BCMA && PCI
> + depends on PCI
> default y
> help
> BCMA bus may have many versions of PCIe core. This driver
> @@ -54,13 +54,13 @@ config BCMA_DRIVER_PCI
>
> config BCMA_DRIVER_PCI_HOSTMODE
> bool "Driver for PCI core working in hostmode"
> - depends on BCMA && MIPS && BCMA_DRIVER_PCI
> + depends on MIPS && BCMA_DRIVER_PCI
> help
> PCI core hostmode operation (external PCI bus).
>
> config BCMA_DRIVER_MIPS
> bool "BCMA Broadcom MIPS core driver"
> - depends on BCMA && MIPS
> + depends on MIPS
> help
> Driver for the Broadcom MIPS core attached to Broadcom specific
> Advanced Microcontroller Bus.
> @@ -91,7 +91,6 @@ config BCMA_NFLASH
>
> config BCMA_DRIVER_GMAC_CMN
> bool "BCMA Broadcom GBIT MAC COMMON core driver"
> - depends on BCMA
> help
> Driver for the Broadcom GBIT MAC COMMON core attached to Broadcom
> specific Advanced Microcontroller Bus.
> @@ -100,7 +99,7 @@ config BCMA_DRIVER_GMAC_CMN
>
> config BCMA_DRIVER_GPIO
> bool "BCMA GPIO driver"
> - depends on BCMA && GPIOLIB
> + depends on GPIOLIB
> select GPIOLIB_IRQCHIP if BCMA_HOST_SOC
> help
> Driver to provide access to the GPIO pins of the bcma bus.
> @@ -109,8 +108,9 @@ config BCMA_DRIVER_GPIO
>
> config BCMA_DEBUG
> bool "BCMA debugging"
> - depends on BCMA
> help
> This turns on additional debugging messages.
>
> If unsure, say N
> +
> +endif # BCMA
>
>
>
--
~Randy
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: bcma: keep *config menu together
2017-09-27 20:01 [PATCH] bcma: keep *config menu together Randy Dunlap
2017-10-07 21:04 ` Randy Dunlap
@ 2017-10-13 9:37 ` Kalle Valo
1 sibling, 0 replies; 3+ messages in thread
From: Kalle Valo @ 2017-10-13 9:37 UTC (permalink / raw)
To: Randy Dunlap; +Cc: LKML, linux-wireless, Andrew Morton, Rafał Miłecki
Randy Dunlap <rdunlap@infradead.org> wrote:
> From: Randy Dunlap <rdunlap@infradead.org>
>
> Use "if BCMA"/"endif" around all Kconfig symbols so that they are
> kept together in *config menus instead of showing up in unexpected
> places. Also remove "depends on BCMA" since this is handled by the
> "if BCMA" addition.
>
> Tested with ARCH={x86_64,MIPS} using make {n,menu,g,x}config.
>
> Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
> Cc: Rafał Miłecki <zajec5@gmail.com>
Patch applied to wireless-drivers-next.git, thanks.
0f0a0af82626 bcma: keep *config menu together
--
https://patchwork.kernel.org/patch/9974679/
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2017-10-13 9:37 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-09-27 20:01 [PATCH] bcma: keep *config menu together Randy Dunlap
2017-10-07 21:04 ` Randy Dunlap
2017-10-13 9:37 ` Kalle Valo
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).