Re: [PATCH v2] cris: drop unused Kconfig symbols

2013-04-16 Thread Jesper Nilsson 
On Tue, Apr 16, 2013 at 10:34:04AM +0200, Paul Bolle wrote:
> Signed-off-by: Paul Bolle 
> ---
> 0) The first version had the subject "[PATCH 21/21] cris: drop unused
> Kconfig symbols".
> 
> 1) This version was redone on top of v3.9-rc7. The changes since the
> first version are:
> - dropped OOM_REBOOT (I sent a separate patch for that symbol because I
> didn't realize it was part of the first version, as it is the only
> symbol without the EXTRAX_ prefix);
> - added ETRAX_ETHERNET_IFACE0, ETRAX_ETHERNET_IFACE1, and
> ETRAX_SERIAL_PORT4 (my scripts got smarter).
> 
> 2) This patch can be tested (after applying) with this one-liner:
> for symbol in $(git log -1 -p | grep "^-config" | awk '{ print $2 }'); do 
> git grep -n "$symbol\b"; done
> 
> That should show no output.

Thanks, applied to the CRIS-tree.

/^JN - Jesper Nilsson
-- 
   Jesper Nilsson -- jesper.nils...@axis.com
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH v2] cris: drop unused Kconfig symbols

2013-04-16 Thread Paul Bolle 
Signed-off-by: Paul Bolle 
---
0) The first version had the subject "[PATCH 21/21] cris: drop unused
Kconfig symbols".

1) This version was redone on top of v3.9-rc7. The changes since the
first version are:
- dropped OOM_REBOOT (I sent a separate patch for that symbol because I
didn't realize it was part of the first version, as it is the only
symbol without the EXTRAX_ prefix);
- added ETRAX_ETHERNET_IFACE0, ETRAX_ETHERNET_IFACE1, and
ETRAX_SERIAL_PORT4 (my scripts got smarter).

2) This patch can be tested (after applying) with this one-liner:
for symbol in $(git log -1 -p | grep "^-config" | awk '{ print $2 }'); do 
git grep -n "$symbol\b"; done

That should show no output.

 arch/cris/Kconfig  |  46 -
 arch/cris/arch-v10/drivers/Kconfig |  23 ---
 arch/cris/arch-v32/drivers/Kconfig | 394 -
 arch/cris/arch-v32/mach-a3/Kconfig |   4 -
 4 files changed, 467 deletions(-)

diff --git a/arch/cris/Kconfig b/arch/cris/Kconfig
index 64c590f..e209be0 100644
--- a/arch/cris/Kconfig
+++ b/arch/cris/Kconfig
@@ -170,12 +170,6 @@ config ETRAX_FLASH_BUSWIDTH
help
  Width in bytes of the NOR Flash bus (1, 2 or 4). Is usually 2.
 
-config ETRAX_NANDFLASH_BUSWIDTH
-   int "Buswidth of NAND flash in bytes"
-   default "1"
-   help
- Width in bytes of the NAND flash (1 or 2).
-
 config ETRAX_FLASH1_SIZE
int "FLASH1 size (dec, in MB. 0 = Unknown)"
default "0"
@@ -542,26 +536,6 @@ config ETRAX_SERIAL_PORT3_DMA5_IN
depends on ETRAX_ARCH_V10
bool "DMA 5"
 
-config ETRAX_SERIAL_PORT3_DMA9_IN
-   bool "Ser3 uses DMA9 for input"
-   depends on ETRAXFS
-   help
- Enables the DMA9 input channel for ser3 (ttyS3).
- If you do not enable DMA, an interrupt for each character will be
- used when receiving data.
- Normally you want to use DMA, unless you use the DMA channel for
- something else.
-
-config ETRAX_SERIAL_PORT3_DMA3_IN
-   bool "Ser3 uses DMA3 for input"
-   depends on CRIS_MACH_ARTPEC3
-   help
- Enables the DMA3 input channel for ser3 (ttyS3).
- If you do not enable DMA, an interrupt for each character will be
- used when receiving data.
- Normally you want to use DMA, unless you use the DMA channel for
- something else.
-
 endchoice
 
 choice
@@ -579,26 +553,6 @@ config ETRAX_SERIAL_PORT3_DMA4_OUT
depends on ETRAX_ARCH_V10
bool "DMA 4"
 
-config ETRAX_SERIAL_PORT3_DMA8_OUT
-   bool "Ser3 uses DMA8 for output"
-   depends on ETRAXFS
-   help
- Enables the DMA8 output channel for ser3 (ttyS3).
- If you do not enable DMA, an interrupt for each character will be
- used when transmitting data.
- Normally you want to use DMA, unless you use the DMA channel for
- something else.
-
-config ETRAX_SERIAL_PORT3_DMA2_OUT
-   bool "Ser3 uses DMA2 for output"
-   depends on CRIS_MACH_ARTPEC3
-   help
- Enables the DMA2 output channel for ser3 (ttyS3).
- If you do not enable DMA, an interrupt for each character will be
- used when transmitting data.
- Normally you want to use DMA, unless you use the DMA channel for
- something else.
-
 endchoice
 
 endmenu
diff --git a/arch/cris/arch-v10/drivers/Kconfig 
b/arch/cris/arch-v10/drivers/Kconfig
index e5be544..dd65ec5 100644
--- a/arch/cris/arch-v10/drivers/Kconfig
+++ b/arch/cris/arch-v10/drivers/Kconfig
@@ -419,16 +419,6 @@ config ETRAX_USB_HOST
   for CTRL and BULK traffic only, INTR traffic may work as well
   however (depending on the requirements of timeliness).
 
-config ETRAX_USB_HOST_PORT1
-   bool "USB port 1 enabled"
-   depends on ETRAX_USB_HOST
-   default n
-
-config ETRAX_USB_HOST_PORT2
-   bool "USB port 2 enabled"
-   depends on ETRAX_USB_HOST
-   default n
-
 config ETRAX_PTABLE_SECTOR
int "Byte-offset of partition table sector"
depends on ETRAX_AXISFLASHMAP
@@ -529,19 +519,6 @@ config ETRAX_GPIO
  Remember that you need to setup the port directions appropriately in
  the General configuration.
 
-config ETRAX_PA_BUTTON_BITMASK
-   hex "PA-buttons bitmask"
-   depends on ETRAX_GPIO
-   default "02"
-   help
- This is a bitmask with information about what bits on PA that
- are used for buttons.
- Most products has a so called TEST button on PA1, if that's true
- use 02 here.
- Use 00 if there are no buttons on PA.
- If the bitmask is <> 00 a button driver will be included in the gpio
- driver. ETRAX general I/O support must be enabled.
-
 config ETRAX_PA_CHANGEABLE_DIR
hex "PA user changeable dir mask"
depends on ETRAX_GPIO
diff --git a/arch/cris/arch-v32/drivers/Kconfig 
b/arch/cris/arch-v32/drivers/Kconfig
index af4a486..a67819b 100644
--- a/arch/cris/arch-v32/drivers/K