Skip to contentSkip to conversation

kconfig: Remove leftover references to AVR32 symbol

PATCH In review Corentin Labbe 5 participants 14 messages Feb 5, 2018 – Mar 12, 2018

Subsystems (4)
Framebuffer layer SPI PWM Console
lore
Share & tools
lore
#1 lore raw

PATCH v3 3/20 kconfig: Remove leftover references to AVR32 symbol

The AVR32 symbol was removed in commit 26202873bb51 ("avr32: remove support for AVR32 architecture"). Remove the remaining references to it from the Kconfig files.

Signed-off-by Ulf Magnusson <[email protected]>

---
Changes in v3:
Removal of the AVR32 dependency in PWM_ATMEL is already pending:
https://lkml.org/lkml/2018/1/18/699

Changes in v2:
Removal of AVR32_AT32AP_CPUFREQ is already pending in
https://marc.info/?i=20180118200202.11883-1-clabbe.montjoie%40gmail.com (which also does it properly -- I forgot to search for related files here). Just remove the other references.

 drivers/spi/Kconfig           | 2 +-
 drivers/video/console/Kconfig | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)
drivers/spi/Kconfig +1 -1
index 603783976b81..c1e455d46c7f 100644
@@ -72,7 +72,7 @@ config SPI_ARMADA_3700
 config SPI_ATMEL
 	tristate "Atmel SPI Controller"
 	depends on HAS_DMA
-	depends on (ARCH_AT91 || AVR32 || COMPILE_TEST)
+	depends on (ARCH_AT91 || COMPILE_TEST)
 	help
 	  This selects a driver for the Atmel SPI Controller, present on
 	  many AT32 (AVR32) and AT91 (ARM) chips.
drivers/video/console/Kconfig +1 -1
index 7f1f1fbcef9e..be63759b6027 100644
@@ -7,7 +7,7 @@ menu "Console display driver support"
 config VGA_CONSOLE
 	bool "VGA text console" if EXPERT || !X86
 	depends on !4xx && !PPC_8xx && !SPARC && !M68K && !PARISC && !FRV && \
-		!SUPERH && !BLACKFIN && !AVR32 && !MN10300 && !CRIS && \
+		!SUPERH && !BLACKFIN && !MN10300 && !CRIS && \
 		(!ARM || ARCH_FOOTBRIDGE || ARCH_INTEGRATOR || ARCH_NETWINDER) && \
 		!ARM64 && !ARC && !MICROBLAZE && !OPENRISC
 	default y
Signature · 1 line
2.14.1
maintainer Acked-by on v3 3/20
#2 lore raw
Quoted 7 lines · Ulf Magnusson

The AVR32 symbol was removed in commit 26202873bb51 ("avr32: remove support for AVR32 architecture"). Remove the remaining references to it from the Kconfig files.

 drivers/spi/Kconfig           | 2 +-
 drivers/video/console/Kconfig | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

It'd probably be easier to just submit these as separate patches to the individual subsystems, that'd make it more likely that reviewers will see them. As I've no context for this I can't tell if there's any dependencies, I'm guessing not but just in case

Acked-by Mark Brown <[email protected]>
#3 lore raw

PATCH 1/1 video: console: kconfig: Remove AVR32 dep. from VGA_CONSOLE

The AVR32 symbol was removed in commit 26202873bb51 ("avr32: remove support for AVR32 architecture").

Signed-off-by Ulf Magnusson <[email protected]>
---
 drivers/video/console/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/video/console/Kconfig +1 -1
index 7f1f1fbcef9e..be63759b6027 100644
@@ -7,7 +7,7 @@ menu "Console display driver support"
 config VGA_CONSOLE
 	bool "VGA text console" if EXPERT || !X86
 	depends on !4xx && !PPC_8xx && !SPARC && !M68K && !PARISC && !FRV && \
-		!SUPERH && !BLACKFIN && !AVR32 && !MN10300 && !CRIS && \
+		!SUPERH && !BLACKFIN && !MN10300 && !CRIS && \
 		(!ARM || ARCH_FOOTBRIDGE || ARCH_INTEGRATOR || ARCH_NETWINDER) && \
 		!ARM64 && !ARC && !MICROBLAZE && !OPENRISC
 	default y
Signature · 6 lines
2.14.1

--
To unsubscribe from this list: send the line "unsubscribe linux-spi" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
#4 lore raw

Re: [PATCH] video: console: kconfig: Remove AVR32 dep. from VGA_CONSOLE

Quoted 4 lines · Ulf Magnusson

The AVR32 symbol was removed in commit 26202873bb51 ("avr32: remove support for AVR32 architecture").

Signed-off-by Ulf Magnusson <[email protected]>

Patch queued for 4.17, thanks.

Best regards,
--
Bartlomiej Zolnierkiewicz
Samsung R&D Institute Poland
Samsung Electronics

#5 lore raw

PATCH 1/1 spi: kconfig: Remove AVR32 dep. from SPI_ATMEL

In reply to Mark Brown

The AVR32 symbol was removed in commit 26202873bb51 ("avr32: remove support for AVR32 architecture").

Signed-off-by Ulf Magnusson <[email protected]>
---
 drivers/spi/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/spi/Kconfig +1 -1
index 603783976b81..c1e455d46c7f 100644
@@ -72,7 +72,7 @@ config SPI_ARMADA_3700
 config SPI_ATMEL
 	tristate "Atmel SPI Controller"
 	depends on HAS_DMA
-	depends on (ARCH_AT91 || AVR32 || COMPILE_TEST)
+	depends on (ARCH_AT91 || COMPILE_TEST)
 	help
 	  This selects a driver for the Atmel SPI Controller, present on
 	  many AT32 (AVR32) and AT91 (ARM) chips.
Signature · 1 line
2.14.1
Reviewed-by on v1 1/1
#6 lore raw

Re: [PATCH] spi: kconfig: Remove AVR32 dep. from SPI_ATMEL

Quoted 5 lines · Ulf Magnusson

The AVR32 symbol was removed in commit 26202873bb51 ("avr32: remove support for AVR32 architecture").

Signed-off-by Ulf Magnusson <[email protected]>

---

Reviewed-by Matthias Brugger <[email protected]>
Quoted 17 lines
 drivers/spi/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/spi/Kconfig +1 -1
index 603783976b81..c1e455d46c7f 100644
@@ -72,7 +72,7 @@ config SPI_ARMADA_3700
 config SPI_ATMEL
 	tristate "Atmel SPI Controller"
 	depends on HAS_DMA
-	depends on (ARCH_AT91 || AVR32 || COMPILE_TEST)
+	depends on (ARCH_AT91 || COMPILE_TEST)
 	help
 	  This selects a driver for the Atmel SPI Controller, present on
 	  many AT32 (AVR32) and AT91 (ARM) chips.
#7 lore raw

Re: [PATCH] spi: kconfig: Remove AVR32 dep. from SPI_ATMEL

Quoted 11 lines · Matthias Brugger
Quoted 5 lines · Ulf Magnusson

The AVR32 symbol was removed in commit 26202873bb51 ("avr32: remove support for AVR32 architecture").

Signed-off-by Ulf Magnusson <[email protected]>

---

Reviewed-by Matthias Brugger <[email protected]>

Sigh, that was too fast, see comment below.

Quoted 16 lines
Quoted 8 lines
 drivers/spi/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/spi/Kconfig +0 -0
index 603783976b81..c1e455d46c7f 100644

drivers/spi/Kconfig config SPI_ARMADA_3700 · line 78

Quoted 8 lines
drivers/spi/Kconfig +1 -1
@@ -72,7 +72,7 @@ config SPI_ARMADA_3700
 config SPI_ATMEL
 	tristate "Atmel SPI Controller"
 	depends on HAS_DMA
-	depends on (ARCH_AT91 || AVR32 || COMPILE_TEST)
+	depends on (ARCH_AT91 || COMPILE_TEST)
 	help
 	  This selects a driver for the Atmel SPI Controller, present on
 	  many AT32 (AVR32) and AT91 (ARM) chips.

AVR32 is no longer supported, please update the description as well.

Thanks,
Matthias

#8 lore raw

PATCH v2 1/1 spi: kconfig: Remove AVR32 dep. from SPI_ATMEL

The AVR32 symbol was removed in commit 26202873bb51 ("avr32: remove support for AVR32 architecture").
---
Changes in v2:
Remove the AVR32 reference from the help text too.

 drivers/spi/Kconfig | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
drivers/spi/Kconfig +2 -2
index 603783976b81..6fb0347a24f2 100644
@@ -72,10 +72,10 @@ config SPI_ARMADA_3700
 config SPI_ATMEL
 	tristate "Atmel SPI Controller"
 	depends on HAS_DMA
-	depends on (ARCH_AT91 || AVR32 || COMPILE_TEST)
+	depends on (ARCH_AT91 || COMPILE_TEST)
 	help
 	  This selects a driver for the Atmel SPI Controller, present on
-	  many AT32 (AVR32) and AT91 (ARM) chips.
+	  many AT91 (ARM) chips.
 
 config SPI_AU1550
 	tristate "Au1550/Au1200/Au1300 SPI Controller"
Signature · 6 lines
2.14.1

--
To unsubscribe from this list: send the line "unsubscribe linux-spi" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
#9 lore raw

Re: [PATCH v2] spi: kconfig: Remove AVR32 dep. from SPI_ATMEL

Quoted 2 lines · Ulf Magnusson

The AVR32 symbol was removed in commit 26202873bb51 ("avr32: remove support for AVR32 architecture").

You forgot the Signed-off-by tag.

Quoted 25 lines

---
Changes in v2:
Remove the AVR32 reference from the help text too.

 drivers/spi/Kconfig | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
drivers/spi/Kconfig +2 -2
index 603783976b81..6fb0347a24f2 100644
@@ -72,10 +72,10 @@ config SPI_ARMADA_3700
 config SPI_ATMEL
 	tristate "Atmel SPI Controller"
 	depends on HAS_DMA
-	depends on (ARCH_AT91 || AVR32 || COMPILE_TEST)
+	depends on (ARCH_AT91 || COMPILE_TEST)
 	help
 	  This selects a driver for the Atmel SPI Controller, present on
-	  many AT32 (AVR32) and AT91 (ARM) chips.
+	  many AT91 (ARM) chips.
 
 config SPI_AU1550
 	tristate "Au1550/Au1200/Au1300 SPI Controller"
#10 lore raw

PATCH v3 1/1 spi: kconfig: Remove AVR32 dep. from SPI_ATMEL

The AVR32 symbol was removed in commit 26202873bb51 ("avr32: remove support for AVR32 architecture").

Signed-off-by Ulf Magnusson <[email protected]>

---
Changes in v3:
Add Signed-off-by tag

Changes in v2:
Remove the AVR32 reference from the help text too.

 drivers/spi/Kconfig | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
drivers/spi/Kconfig +2 -2
index 603783976b81..6fb0347a24f2 100644
@@ -72,10 +72,10 @@ config SPI_ARMADA_3700
 config SPI_ATMEL
 	tristate "Atmel SPI Controller"
 	depends on HAS_DMA
-	depends on (ARCH_AT91 || AVR32 || COMPILE_TEST)
+	depends on (ARCH_AT91 || COMPILE_TEST)
 	help
 	  This selects a driver for the Atmel SPI Controller, present on
-	  many AT32 (AVR32) and AT91 (ARM) chips.
+	  many AT91 (ARM) chips.
 
 config SPI_AU1550
 	tristate "Au1550/Au1200/Au1300 SPI Controller"
Signature · 1 line
2.14.1
maintainer
#11 lore raw

Applied "spi: kconfig: Remove AVR32 dep. from SPI_ATMEL" to the spi tree

In reply to Ulf Magnusson

The patch

spi: kconfig: Remove AVR32 dep. from SPI_ATMEL

has been applied to the spi tree at

https://git.kernel.org/pub/scm/linux/kernel/git/broonie/spi.git

All being well this means that it will be integrated into the linux-next tree (usually sometime in the next 24 hours) and sent to Linus during the next merge window (or sooner if it is a bug fix), however if problems are discovered then the patch may be dropped or reverted.

You may get further e-mails resulting from automated or manual testing and review of the tree, please engage with people reporting problems and send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they should be sent as incremental updates against current git, existing patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying to this mail.

Thanks,
Mark

Quoted 1 line

From e3d001a0c29d6c9a87dd9e86bcb84fc66c2ddbe8 Mon Sep 17 00:00:00 2001

From: Ulf Magnusson <[email protected]>
Date: Thu, 8 Feb 2018 23:00:16 +0100
Subject: [PATCH] spi: kconfig: Remove AVR32 dep. from SPI_ATMEL

The AVR32 symbol was removed in commit 26202873bb51 ("avr32: remove support for AVR32 architecture").

Signed-off-by Ulf Magnusson <[email protected]>
Signed-off-by Mark Brown <[email protected]>
---
 drivers/spi/Kconfig | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
drivers/spi/Kconfig +2 -2
index 603783976b81..6fb0347a24f2 100644
@@ -72,10 +72,10 @@ config SPI_ARMADA_3700
 config SPI_ATMEL
 	tristate "Atmel SPI Controller"
 	depends on HAS_DMA
-	depends on (ARCH_AT91 || AVR32 || COMPILE_TEST)
+	depends on (ARCH_AT91 || COMPILE_TEST)
 	help
 	  This selects a driver for the Atmel SPI Controller, present on
-	  many AT32 (AVR32) and AT91 (ARM) chips.
+	  many AT91 (ARM) chips.
 
 config SPI_AU1550
 	tristate "Au1550/Au1200/Au1300 SPI Controller"
Signature · 6 lines
2.16.1

--
To unsubscribe from this list: send the line "unsubscribe linux-spi" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
#12 lore raw
In reply to Mark Brown
Quoted 9 lines · Mark Brown
Quoted 7 lines · Ulf Magnusson

The AVR32 symbol was removed in commit 26202873bb51 ("avr32: remove support for AVR32 architecture"). Remove the remaining references to it from the Kconfig files.

 drivers/spi/Kconfig           | 2 +-
 drivers/video/console/Kconfig | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)
Quoted 6 lines

It'd probably be easier to just submit these as separate patches to the individual subsystems, that'd make it more likely that reviewers will see them. As I've no context for this I can't tell if there's any dependencies, I'm guessing not but just in case

Acked-by Mark Brown <[email protected]>

Done

https://lkml.org/lkml/2018/2/6/805
https://lkml.org/lkml/2018/2/6/808

Cheers,
Ulf

#13 lore raw
In reply to Ulf Magnusson
Quoted 47 lines · Ulf Magnusson

The AVR32 symbol was removed in commit 26202873bb51 ("avr32: remove support for AVR32 architecture"). Remove the remaining references to it from the Kconfig files.

Signed-off-by Ulf Magnusson <[email protected]>

---
Changes in v3:
Removal of the AVR32 dependency in PWM_ATMEL is already pending:
https://lkml.org/lkml/2018/1/18/699

Changes in v2:
Removal of AVR32_AT32AP_CPUFREQ is already pending in
https://marc.info/?i=20180118200202.11883-1-clabbe.montjoie%40gmail.com (which also does it properly -- I forgot to search for related files here). Just remove the other references.

 drivers/spi/Kconfig           | 2 +-
 drivers/video/console/Kconfig | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)
drivers/spi/Kconfig +1 -1
index 603783976b81..c1e455d46c7f 100644
@@ -72,7 +72,7 @@ config SPI_ARMADA_3700
 config SPI_ATMEL
        tristate "Atmel SPI Controller"
        depends on HAS_DMA
-       depends on (ARCH_AT91 || AVR32 || COMPILE_TEST)
+       depends on (ARCH_AT91 || COMPILE_TEST)
        help
          This selects a driver for the Atmel SPI Controller, present on
          many AT32 (AVR32) and AT91 (ARM) chips.
drivers/video/console/Kconfig +1 -2
index 7f1f1fbcef9e..be63759b6027 100644
@@ -7,7 +7,7 @@ menu "Console display driver support"
 config VGA_CONSOLE
        bool "VGA text console" if EXPERT || !X86
        depends on !4xx && !PPC_8xx && !SPARC && !M68K && !PARISC && !FRV && \
-               !SUPERH && !BLACKFIN && !AVR32 && !MN10300 && !CRIS && \
+               !SUPERH && !BLACKFIN && !MN10300 && !CRIS && \
                (!ARM || ARCH_FOOTBRIDGE || ARCH_INTEGRATOR || ARCH_NETWINDER) && \
                !ARM64 && !ARC && !MICROBLAZE && !OPENRISC
        default y
--
Quoted 2 lines

2.14.1

Superseded by two patches, split up per subsystem:

https://lkml.org/lkml/2018/2/6/805
https://lkml.org/lkml/2018/2/6/808

Cheers,
Ulf