2018-02-05 01:24:15

by Ulf Magnusson

[permalink] [raw]
Subject: [PATCH 00/20] Remove references to undefined symbols and fix two potential bugs

Hello,

This patchset fixes most references to globally undefined symbols in Kconfig
files, as reported by the
https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
script. I will send emails in reply to the patchset for some cases where I'm
not sure what the proper fix is.

All of the fixes are independent. I just created a patchset to track them more
easily.

Summary of changes:

- Commits 1 and 2 fix what looks like potential bugs: A broken (intended)
select of PL310_ERRATA_753970 for ARM/mvebu, and a misspelled dependency on
PPC_PSERIES in KVM/PPC.

- Commits 3-5 remove leftover AVR32 stuff.

- Commits 6 and 7 remove some selects of removed symbols from riscv.

- Commits 8-17 remove various references to removed symbols.

- Commits 18-20 fix some (harmless) malformed defaults.

Ulf Magnusson (20):
ARM: mvebu: Fix broken PL310_ERRATA_753970 selects
KVM: PPC: Book3S PR: Fix broken select due to misspelling
kconfig: Remove leftover references to AVR32 symbol
misc: atmel: Remove CPU_AT32AP700X (AVR32) reference
rtc: at32ap700x: Remove PLATFORM_AT32AP dependency
riscv: Remove ARCH_HAS_ATOMIC64_DEC_IF_POSITIVE select
riscv: Remove ARCH_WANT_OPTIONAL_GPIOLIB select
s390/kconfig: Remove ARCH_WANTS_PROT_NUMA_PROT_NONE select
clk: sunxi-ng: Remove SUNXI_CCU_* selects
lib/Kconfig: Remove leftover select of GENERIC_IO
usb: gadget: udc: Remove USB_GADGET_DUALSPEED select
ARM: debug: Remove ARCH_MSM dep. from UNCOMPRESS_INCLUDE
ore: Remove PNFS_OBJLAYOUT dependency
iio: gyro: mpu3050: Remove INPUT_MPU3050 dep. from I2C
phy: Remove SOC_EXYNOS4212 dep. from PHY_EXYNOS4X12_USB
auxdisplay: img-ascii-lcd: Remove MIPS_SEAD3 dep.
media: sec: Remove PLAT_S5P dependency
iwlwifi: fix malformed CONFIG_IWLWIFI_PCIE_RTPM default
ARC: Fix malformed ARC_EMUL_UNALIGNED default
x86/PCI: VMD: Fix malformed default

arch/arc/Kconfig | 1 -
arch/arm/Kconfig.debug | 2 +-
arch/arm/mach-mvebu/Kconfig | 4 ++--
arch/powerpc/kvm/Kconfig | 2 +-
arch/riscv/Kconfig | 2 --
arch/s390/Kconfig | 1 -
drivers/auxdisplay/Kconfig | 2 +-
drivers/clk/sunxi-ng/Kconfig | 7 -------
drivers/cpufreq/Kconfig | 10 ----------
drivers/iio/gyro/Kconfig | 1 -
drivers/media/platform/Kconfig | 2 +-
drivers/misc/Kconfig | 1 -
drivers/net/wireless/intel/iwlwifi/Kconfig | 1 -
drivers/pci/host/Kconfig | 1 -
drivers/phy/samsung/Kconfig | 2 +-
drivers/pwm/Kconfig | 2 +-
drivers/rtc/Kconfig | 2 +-
drivers/spi/Kconfig | 2 +-
drivers/usb/gadget/udc/Kconfig | 1 -
drivers/video/console/Kconfig | 2 +-
fs/exofs/Kconfig.ore | 2 +-
lib/Kconfig | 1 -
22 files changed, 12 insertions(+), 39 deletions(-)

--
2.14.1



2018-02-05 01:24:18

by Ulf Magnusson

[permalink] [raw]
Subject: [PATCH 01/20] ARM: mvebu: Fix broken PL310_ERRATA_753970 selects

The MACH_ARMADA_375 and MACH_ARMADA_38X boards select ARM_ERRATA_753970,
but it was renamed to PL310_ERRATA_753970 by commit fa0ce4035d48 ("ARM:
7162/1: errata: tidy up Kconfig options for PL310 errata workarounds").

Fix the selects to use the new name.

Discovered with the
https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
script.

Signed-off-by: Ulf Magnusson <[email protected]>
---
arch/arm/mach-mvebu/Kconfig | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/mach-mvebu/Kconfig b/arch/arm/mach-mvebu/Kconfig
index 6b32dc527edc..2c20599cc350 100644
--- a/arch/arm/mach-mvebu/Kconfig
+++ b/arch/arm/mach-mvebu/Kconfig
@@ -41,7 +41,7 @@ config MACH_ARMADA_375
depends on ARCH_MULTI_V7
select ARMADA_370_XP_IRQ
select ARM_ERRATA_720789
- select ARM_ERRATA_753970
+ select PL310_ERRATA_753970
select ARM_GIC
select ARMADA_375_CLK
select HAVE_ARM_SCU
@@ -57,7 +57,7 @@ config MACH_ARMADA_38X
bool "Marvell Armada 380/385 boards"
depends on ARCH_MULTI_V7
select ARM_ERRATA_720789
- select ARM_ERRATA_753970
+ select PL310_ERRATA_753970
select ARM_GIC
select ARM_GLOBAL_TIMER
select CLKSRC_ARM_GLOBAL_TIMER_SCHED_CLOCK
--
2.14.1


2018-02-05 01:24:31

by Ulf Magnusson

[permalink] [raw]
Subject: [PATCH 02/20] KVM: PPC: Book3S PR: Fix broken select due to misspelling

Commit 76d837a4c0f9 ("KVM: PPC: Book3S PR: Don't include SPAPR TCE code
on non-pseries platforms") added a reference to the globally undefined
symbol PPC_SERIES. Looking at the rest of the commit, PPC_PSERIES was
probably intended.

Change PPC_SERIES to PPC_PSERIES.

Discovered with the
https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
script.

Signed-off-by: Ulf Magnusson <[email protected]>
---
arch/powerpc/kvm/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/powerpc/kvm/Kconfig b/arch/powerpc/kvm/Kconfig
index f884a0529dfe..68a0e9d5b440 100644
--- a/arch/powerpc/kvm/Kconfig
+++ b/arch/powerpc/kvm/Kconfig
@@ -69,7 +69,7 @@ config KVM_BOOK3S_64
select KVM_BOOK3S_64_HANDLER
select KVM
select KVM_BOOK3S_PR_POSSIBLE if !KVM_BOOK3S_HV_POSSIBLE
- select SPAPR_TCE_IOMMU if IOMMU_SUPPORT && (PPC_SERIES || PPC_POWERNV)
+ select SPAPR_TCE_IOMMU if IOMMU_SUPPORT && (PPC_PSERIES || PPC_POWERNV)
---help---
Support running unmodified book3s_64 and book3s_32 guest kernels
in virtual machines on book3s_64 host processors.
--
2.14.1


2018-02-05 01:25:32

by Ulf Magnusson

[permalink] [raw]
Subject: [PATCH 05/20] rtc: at32ap700x: Remove PLATFORM_AT32AP dependency

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

Remove the RTC_DRV_AT32AP700X dependency from RTC_DRV_AT32AP700X.

Discovered with the
https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
script.

Signed-off-by: Ulf Magnusson <[email protected]>
---
drivers/rtc/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/rtc/Kconfig b/drivers/rtc/Kconfig
index b71cc7b5fbc9..1f5f3bd82ab6 100644
--- a/drivers/rtc/Kconfig
+++ b/drivers/rtc/Kconfig
@@ -1404,7 +1404,7 @@ config RTC_DRV_PL031

config RTC_DRV_AT32AP700X
tristate "AT32AP700X series RTC"
- depends on PLATFORM_AT32AP || COMPILE_TEST
+ depends on COMPILE_TEST
help
Driver for the internal RTC (Realtime Clock) on Atmel AVR32
AT32AP700x family processors.
--
2.14.1


2018-02-05 01:26:00

by Ulf Magnusson

[permalink] [raw]
Subject: [PATCH 03/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.

No references remain to the removed AVR32_AT32AP_CPUFREQ symbol either,
so remove that one too.

Discovered with the
https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
script.

Signed-off-by: Ulf Magnusson <[email protected]>
---
drivers/cpufreq/Kconfig | 10 ----------
drivers/pwm/Kconfig | 2 +-
drivers/spi/Kconfig | 2 +-
drivers/video/console/Kconfig | 2 +-
4 files changed, 3 insertions(+), 13 deletions(-)

diff --git a/drivers/cpufreq/Kconfig b/drivers/cpufreq/Kconfig
index d8addbce40bc..608af20a3494 100644
--- a/drivers/cpufreq/Kconfig
+++ b/drivers/cpufreq/Kconfig
@@ -239,16 +239,6 @@ if PPC32 || PPC64
source "drivers/cpufreq/Kconfig.powerpc"
endif

-if AVR32
-config AVR32_AT32AP_CPUFREQ
- bool "CPU frequency driver for AT32AP"
- depends on PLATFORM_AT32AP
- default n
- help
- This enables the CPU frequency driver for AT32AP processors.
- If in doubt, say N.
-endif
-
if IA64
config IA64_ACPI_CPUFREQ
tristate "ACPI Processor P-States driver"
diff --git a/drivers/pwm/Kconfig b/drivers/pwm/Kconfig
index 763ee50ea57d..f16aad3bf5d6 100644
--- a/drivers/pwm/Kconfig
+++ b/drivers/pwm/Kconfig
@@ -43,7 +43,7 @@ config PWM_AB8500

config PWM_ATMEL
tristate "Atmel PWM support"
- depends on ARCH_AT91 || AVR32
+ depends on ARCH_AT91
help
Generic PWM framework driver for Atmel SoC.

diff --git a/drivers/spi/Kconfig b/drivers/spi/Kconfig
index 603783976b81..c1e455d46c7f 100644
--- a/drivers/spi/Kconfig
+++ b/drivers/spi/Kconfig
@@ -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.
diff --git a/drivers/video/console/Kconfig b/drivers/video/console/Kconfig
index 7f1f1fbcef9e..be63759b6027 100644
--- a/drivers/video/console/Kconfig
+++ b/drivers/video/console/Kconfig
@@ -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
--
2.14.1


2018-02-05 01:26:05

by Ulf Magnusson

[permalink] [raw]
Subject: [PATCH 10/20] lib/Kconfig: Remove leftover select of GENERIC_IO

GENERIC_IO was removed by commit 9de8da47742b ("kconfig: kill off
GENERIC_IO option"), but HAS_IOMEM still selects it.

Remove the select of GENERIC_IO from HAS_IOMEM.

Discovered with the
https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
script.

Signed-off-by: Ulf Magnusson <[email protected]>
---
lib/Kconfig | 1 -
1 file changed, 1 deletion(-)

diff --git a/lib/Kconfig b/lib/Kconfig
index e96089499371..8ce8bcad6eff 100644
--- a/lib/Kconfig
+++ b/lib/Kconfig
@@ -396,7 +396,6 @@ config ASSOCIATIVE_ARRAY
config HAS_IOMEM
bool
depends on !NO_IOMEM
- select GENERIC_IO
default y

config HAS_IOPORT_MAP
--
2.14.1


2018-02-05 01:26:06

by Ulf Magnusson

[permalink] [raw]
Subject: [PATCH 06/20] riscv: Remove ARCH_HAS_ATOMIC64_DEC_IF_POSITIVE select

The ARCH_HAS_ATOMIC64_DEC_IF_POSITIVE symbol was removed in
commit 51a021244b9d ("atomic64: no need for
CONFIG_ARCH_HAS_ATOMIC64_DEC_IF_POSITIVE").

Remove the ARCH_HAS_ATOMIC64_DEC_IS_POSITIVE select from RISCV.

Discovered with the
https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
script.

Signed-off-by: Ulf Magnusson <[email protected]>
---
arch/riscv/Kconfig | 1 -
1 file changed, 1 deletion(-)

diff --git a/arch/riscv/Kconfig b/arch/riscv/Kconfig
index b6722c246d9c..ff69c77b9e78 100644
--- a/arch/riscv/Kconfig
+++ b/arch/riscv/Kconfig
@@ -8,7 +8,6 @@ config RISCV
select OF
select OF_EARLY_FLATTREE
select OF_IRQ
- select ARCH_HAS_ATOMIC64_DEC_IF_POSITIVE
select ARCH_WANT_FRAME_POINTERS
select CLONE_BACKWARDS
select COMMON_CLK
--
2.14.1


2018-02-05 01:26:17

by Ulf Magnusson

[permalink] [raw]
Subject: [PATCH 12/20] ARM: debug: Remove ARCH_MSM dep. from UNCOMPRESS_INCLUDE

The ARCH_MSM symbol was removed in commit c0c89fafa289 ("ARM: Remove
mach-msm and associated ARM architecture code"), but a default in
UNCOMPRESS_INCLUDE still depends on it.

Remove the ARCH_MSM dependency from the default in UNCOMPRESS_INCLUDE.

Discovered with the
https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
script.

Signed-off-by: Ulf Magnusson <[email protected]>
---
arch/arm/Kconfig.debug | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/Kconfig.debug b/arch/arm/Kconfig.debug
index 78a647080ebc..20c601a045d2 100644
--- a/arch/arm/Kconfig.debug
+++ b/arch/arm/Kconfig.debug
@@ -1825,7 +1825,7 @@ config DEBUG_UNCOMPRESS

config UNCOMPRESS_INCLUDE
string
- default "debug/uncompress.h" if ARCH_MULTIPLATFORM || ARCH_MSM || \
+ default "debug/uncompress.h" if ARCH_MULTIPLATFORM || \
PLAT_SAMSUNG || ARM_SINGLE_ARMV7M
default "mach/uncompress.h"

--
2.14.1


2018-02-05 01:26:20

by Ulf Magnusson

[permalink] [raw]
Subject: [PATCH 04/20] misc: atmel: Remove CPU_AT32AP700X (AVR32) reference

The CPU_AT32AP700X symbol symbol went away when when AVR32 was removed
in commit 26202873bb51 ("avr32: remove support for AVR32 architecture")

Remove the prompt from ATMEL_TCB_CLKSRC_BLOCK. The prompt condition
could never be satisfied now.

Discovered with the
https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
script.

Signed-off-by: Ulf Magnusson <[email protected]>
---
drivers/misc/Kconfig | 1 -
1 file changed, 1 deletion(-)

diff --git a/drivers/misc/Kconfig b/drivers/misc/Kconfig
index 03605f8fc0dc..5d713008749b 100644
--- a/drivers/misc/Kconfig
+++ b/drivers/misc/Kconfig
@@ -75,7 +75,6 @@ config ATMEL_TCB_CLKSRC
config ATMEL_TCB_CLKSRC_BLOCK
int
depends on ATMEL_TCB_CLKSRC
- prompt "TC Block" if CPU_AT32AP700X
default 0
range 0 1
help
--
2.14.1


2018-02-05 01:26:41

by Ulf Magnusson

[permalink] [raw]
Subject: [PATCH 11/20] usb: gadget: udc: Remove USB_GADGET_DUALSPEED select

USB_GADGET_DUALSPEED was removed by commit 85b8614d7223 ("usb: gadget:
get rid of USB_GADGET_{DUAL,SUPER}SPEED"), but the USB_SNP_UDC_PLAT
symbol still selects it.

Remove the USB_GADGET_DUALSPEED select from USB_SNP_UDC_PLAT.

Discovered with the
https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
script.

Signed-off-by: Ulf Magnusson <[email protected]>
---
drivers/usb/gadget/udc/Kconfig | 1 -
1 file changed, 1 deletion(-)

diff --git a/drivers/usb/gadget/udc/Kconfig b/drivers/usb/gadget/udc/Kconfig
index 1e9567091d86..0875d38476ee 100644
--- a/drivers/usb/gadget/udc/Kconfig
+++ b/drivers/usb/gadget/udc/Kconfig
@@ -274,7 +274,6 @@ config USB_SNP_UDC_PLAT
tristate "Synopsys USB 2.0 Device controller"
depends on USB_GADGET && OF && HAS_DMA
depends on EXTCON || EXTCON=n
- select USB_GADGET_DUALSPEED
select USB_SNP_CORE
default ARCH_BCM_IPROC
help
--
2.14.1


2018-02-05 01:26:51

by Ulf Magnusson

[permalink] [raw]
Subject: [PATCH 15/20] phy: Remove SOC_EXYNOS4212 dep. from PHY_EXYNOS4X12_USB

Exynos4212 support was removed by commit bca9085e0ae9 ("ARM: dts:
exynos: remove Exynos4212 support (dead code)").

Remove the SOC_EXYNOS4212 dependency from PHY_EXYNOS4X12_USB.

Discovered with the
https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
script.

Signed-off-by: Ulf Magnusson <[email protected]>
---
drivers/phy/samsung/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/phy/samsung/Kconfig b/drivers/phy/samsung/Kconfig
index b7e0645a7bd9..2a5d33cb0e7e 100644
--- a/drivers/phy/samsung/Kconfig
+++ b/drivers/phy/samsung/Kconfig
@@ -49,7 +49,7 @@ config PHY_EXYNOS4210_USB2
config PHY_EXYNOS4X12_USB2
bool
depends on PHY_SAMSUNG_USB2
- default SOC_EXYNOS3250 || SOC_EXYNOS4212 || SOC_EXYNOS4412
+ default SOC_EXYNOS3250 || SOC_EXYNOS4412

config PHY_EXYNOS5250_USB2
bool
--
2.14.1


2018-02-05 01:27:01

by Ulf Magnusson

[permalink] [raw]
Subject: [PATCH 08/20] s390/kconfig: Remove ARCH_WANTS_PROT_NUMA_PROT_NONE select

The ARCH_WANTS_PROT_NUMA_PROT_NONE symbol was removed by
commit 6a33979d5bd7 ("mm: remove misleading ARCH_USES_NUMA_PROT_NONE"),
but S390 still selects it.

Remove the ARCH_WANTS_PROT_NUMA_PROT_NONE select from the S390 symbol.

Discovered with the
https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
script.

Signed-off-by: Ulf Magnusson <[email protected]>
---
arch/s390/Kconfig | 1 -
1 file changed, 1 deletion(-)

diff --git a/arch/s390/Kconfig b/arch/s390/Kconfig
index 0105ce28e246..bb090b7992e4 100644
--- a/arch/s390/Kconfig
+++ b/arch/s390/Kconfig
@@ -112,7 +112,6 @@ config S390
select ARCH_USE_BUILTIN_BSWAP
select ARCH_USE_CMPXCHG_LOCKREF
select ARCH_WANTS_DYNAMIC_TASK_STRUCT
- select ARCH_WANTS_PROT_NUMA_PROT_NONE
select ARCH_WANTS_UBSAN_NO_NULL
select ARCH_WANT_IPC_PARSE_VERSION
select BUILDTIME_EXTABLE_SORT
--
2.14.1


2018-02-05 01:27:11

by Ulf Magnusson

[permalink] [raw]
Subject: [PATCH 09/20] clk: sunxi-ng: Remove SUNXI_CCU_* selects

The selected symbols were removed in commit 06e226c7fb23 ("clk:
sunxi-ng: Move all clock types to a library").

Remove the selects from SUN4I_A10_CCU.

Discovered with the
https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
script.

Signed-off-by: Ulf Magnusson <[email protected]>
---
drivers/clk/sunxi-ng/Kconfig | 7 -------
1 file changed, 7 deletions(-)

diff --git a/drivers/clk/sunxi-ng/Kconfig b/drivers/clk/sunxi-ng/Kconfig
index 6427d0ebe2de..33168f94ee39 100644
--- a/drivers/clk/sunxi-ng/Kconfig
+++ b/drivers/clk/sunxi-ng/Kconfig
@@ -13,13 +13,6 @@ config SUN50I_A64_CCU

config SUN4I_A10_CCU
bool "Support for the Allwinner A10/A20 CCU"
- select SUNXI_CCU_DIV
- select SUNXI_CCU_MULT
- select SUNXI_CCU_NK
- select SUNXI_CCU_NKM
- select SUNXI_CCU_NM
- select SUNXI_CCU_MP
- select SUNXI_CCU_PHASE
default MACH_SUN4I
default MACH_SUN7I
depends on MACH_SUN4I || MACH_SUN7I || COMPILE_TEST
--
2.14.1


2018-02-05 01:27:36

by Ulf Magnusson

[permalink] [raw]
Subject: [PATCH 13/20] ore: Remove PNFS_OBJLAYOUT dependency

The PNFS_OBJLAYOUT symbol was removed in commit 6d22323b2e9f ("nfs:
remove the objlayout driver"), but the ORE symbol still depends on it.

Remove the PNFS_OBJLAYOUT dependency from ORE.

Discovered with the
https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
script.

Signed-off-by: Ulf Magnusson <[email protected]>
---
fs/exofs/Kconfig.ore | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/fs/exofs/Kconfig.ore b/fs/exofs/Kconfig.ore
index 2daf2329c28d..73cab8d92dfd 100644
--- a/fs/exofs/Kconfig.ore
+++ b/fs/exofs/Kconfig.ore
@@ -7,7 +7,7 @@
# selected by any of the users.
config ORE
tristate
- depends on EXOFS_FS || PNFS_OBJLAYOUT
+ depends on EXOFS_FS
select ASYNC_XOR
select RAID6_PQ
select ASYNC_PQ
--
2.14.1


2018-02-05 01:27:38

by Ulf Magnusson

[permalink] [raw]
Subject: [PATCH 17/20] media: sec: Remove PLAT_S5P dependency

The PLAT_S5P symbol was removed in commit d78c16ccde96 ("ARM: SAMSUNG:
Remove remaining legacy code").

Remove the PLAT_S5P dependency from VIDEO_SAMSUNG_S5P_CEC.

Discovered with the
https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
script.

Signed-off-by: Ulf Magnusson <[email protected]>
---
drivers/media/platform/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/media/platform/Kconfig b/drivers/media/platform/Kconfig
index 614fbef08ddc..8b4cd02e2938 100644
--- a/drivers/media/platform/Kconfig
+++ b/drivers/media/platform/Kconfig
@@ -582,7 +582,7 @@ config CEC_GPIO

config VIDEO_SAMSUNG_S5P_CEC
tristate "Samsung S5P CEC driver"
- depends on PLAT_S5P || ARCH_EXYNOS || COMPILE_TEST
+ depends on ARCH_EXYNOS || COMPILE_TEST
select CEC_CORE
select CEC_NOTIFIER
---help---
--
2.14.1


2018-02-05 01:27:46

by Ulf Magnusson

[permalink] [raw]
Subject: [PATCH 19/20] ARC: Fix malformed ARC_EMUL_UNALIGNED default

'default N' should be 'default n', though they happen to have the same
effect here, due to undefined symbols (N in this case) evaluating to n
in a tristate sense.

Remove the default from ARC_EMUL_UNALIGNED instead of changing it. bool
and tristate symbols implicitly default to n.

Discovered with the
https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
script.

Signed-off-by: Ulf Magnusson <[email protected]>
---
arch/arc/Kconfig | 1 -
1 file changed, 1 deletion(-)

diff --git a/arch/arc/Kconfig b/arch/arc/Kconfig
index f3a80cf164cc..d76bf4a83740 100644
--- a/arch/arc/Kconfig
+++ b/arch/arc/Kconfig
@@ -484,7 +484,6 @@ config ARC_CURR_IN_REG

config ARC_EMUL_UNALIGNED
bool "Emulate unaligned memory access (userspace only)"
- default N
select SYSCTL_ARCH_UNALIGN_NO_WARN
select SYSCTL_ARCH_UNALIGN_ALLOW
depends on ISA_ARCOMPACT
--
2.14.1


2018-02-05 01:28:07

by Ulf Magnusson

[permalink] [raw]
Subject: [PATCH 16/20] auxdisplay: img-ascii-lcd: Remove MIPS_SEAD3 dep.

The MIPS_SEAD3 symbol was removed in commit 64601cb1343f ("leds: Remove SEAD-3
driver").

Remove the MIPS_SEAD3 dependency from IMG_ASCII_LCD.

Discovered with the
https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
script.

Signed-off-by: Ulf Magnusson <[email protected]>
---
drivers/auxdisplay/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/auxdisplay/Kconfig b/drivers/auxdisplay/Kconfig
index 2c2ed9cf8796..3cba78d36eec 100644
--- a/drivers/auxdisplay/Kconfig
+++ b/drivers/auxdisplay/Kconfig
@@ -137,7 +137,7 @@ config CFAG12864B_RATE
config IMG_ASCII_LCD
tristate "Imagination Technologies ASCII LCD Display"
depends on HAS_IOMEM
- default y if MIPS_MALTA || MIPS_SEAD3
+ default y if MIPS_MALTA
select SYSCON
help
Enable this to support the simple ASCII LCD displays found on
--
2.14.1


2018-02-05 01:28:22

by Ulf Magnusson

[permalink] [raw]
Subject: [PATCH 07/20] riscv: Remove ARCH_WANT_OPTIONAL_GPIOLIB select

The ARCH_WANT_OPTIONAL_GPIOLIB symbol was removed in commit 65053e1a7743
("gpio: delete ARCH_[WANTS_OPTIONAL|REQUIRE]_GPIOLIB"). GPIOLIB should
just be selected explicitly if needed.

Remove the ARCH_WANT_OPTIONAL_GPIOLIB select from RISCV.

See commit 0145071b3314 ("x86: Do away with
ARCH_[WANT_OPTIONAL|REQUIRE]_GPIOLIB") and commit da9a1c6767 ("arm64: do
away with ARCH_[WANT_OPTIONAL|REQUIRE]_GPIOLIB") as well.

Discovered with the
https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
script.

Signed-off-by: Ulf Magnusson <[email protected]>
---
arch/riscv/Kconfig | 1 -
1 file changed, 1 deletion(-)

diff --git a/arch/riscv/Kconfig b/arch/riscv/Kconfig
index ff69c77b9e78..716e90e60e5c 100644
--- a/arch/riscv/Kconfig
+++ b/arch/riscv/Kconfig
@@ -19,7 +19,6 @@ config RISCV
select GENERIC_STRNLEN_USER
select GENERIC_SMP_IDLE_THREAD
select GENERIC_ATOMIC64 if !64BIT || !RISCV_ISA_A
- select ARCH_WANT_OPTIONAL_GPIOLIB
select HAVE_MEMBLOCK
select HAVE_MEMBLOCK_NODE_MAP
select HAVE_DMA_API_DEBUG
--
2.14.1


2018-02-05 01:28:36

by Ulf Magnusson

[permalink] [raw]
Subject: [PATCH 14/20] iio: gyro: mpu3050: Remove INPUT_MPU3050 dep. from I2C

The INPUT_MPU3050 symbol was removed in commit b1fe0cf06f92 ("Input:
delete MPU3050 driver").

Remove the negative dependency on it from the MPU3050_I2C symbol. It is
always satisfied now.

Discovered with the
https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
script.

Signed-off-by: Ulf Magnusson <[email protected]>
---
drivers/iio/gyro/Kconfig | 1 -
1 file changed, 1 deletion(-)

diff --git a/drivers/iio/gyro/Kconfig b/drivers/iio/gyro/Kconfig
index 3126cf05e6b9..b702b9176006 100644
--- a/drivers/iio/gyro/Kconfig
+++ b/drivers/iio/gyro/Kconfig
@@ -92,7 +92,6 @@ config MPU3050

config MPU3050_I2C
tristate "Invensense MPU3050 devices on I2C"
- depends on !(INPUT_MPU3050=y || INPUT_MPU3050=m)
depends on I2C
select MPU3050
select REGMAP_I2C
--
2.14.1


2018-02-05 01:28:38

by Ulf Magnusson

[permalink] [raw]
Subject: [PATCH 20/20] x86/PCI: VMD: Fix malformed default

'default N' should be 'default n', though they happen to have the same
effect here, due to undefined symbols (N in this case) evaluating to n
in a tristate sense.

Remove the default from VMD instead of changing it. bool and tristate
symbols implicitly default to n.

Discovered with the
https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
script.

Signed-off-by: Ulf Magnusson <[email protected]>
---
drivers/pci/host/Kconfig | 1 -
1 file changed, 1 deletion(-)

diff --git a/drivers/pci/host/Kconfig b/drivers/pci/host/Kconfig
index a4ed7484d127..dc8a2a175f19 100644
--- a/drivers/pci/host/Kconfig
+++ b/drivers/pci/host/Kconfig
@@ -215,7 +215,6 @@ config PCIE_TANGO_SMP8759
config VMD
depends on PCI_MSI && X86_64 && SRCU
tristate "Intel Volume Management Device Driver"
- default N
---help---
Adds support for the Intel Volume Management Device (VMD). VMD is a
secondary PCI host bridge that allows PCI Express root ports,
--
2.14.1


2018-02-05 01:29:15

by Ulf Magnusson

[permalink] [raw]
Subject: [PATCH 18/20] iwlwifi: fix malformed CONFIG_IWLWIFI_PCIE_RTPM default

'default false' should be 'default n', though they happen to have the
same effect here, due to undefined symbols ('false' in this case)
evaluating to n in a tristate sense.

Remove the default instead of changing it. bool and tristate symbols
implicitly default to n.

Discovered with the
https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
script.

Signed-off-by: Ulf Magnusson <[email protected]>
---
drivers/net/wireless/intel/iwlwifi/Kconfig | 1 -
1 file changed, 1 deletion(-)

diff --git a/drivers/net/wireless/intel/iwlwifi/Kconfig b/drivers/net/wireless/intel/iwlwifi/Kconfig
index c5f2ddf9b0fe..e5a2fc738ac3 100644
--- a/drivers/net/wireless/intel/iwlwifi/Kconfig
+++ b/drivers/net/wireless/intel/iwlwifi/Kconfig
@@ -91,7 +91,6 @@ config IWLWIFI_BCAST_FILTERING
config IWLWIFI_PCIE_RTPM
bool "Enable runtime power management mode for PCIe devices"
depends on IWLMVM && PM && EXPERT
- default false
help
Say Y here to enable runtime power management for PCIe
devices. If enabled, the device will go into low power mode
--
2.14.1


2018-02-05 01:31:49

by Ulf Magnusson

[permalink] [raw]
Subject: Re: [PATCH 07/20] riscv: Remove ARCH_WANT_OPTIONAL_GPIOLIB select

On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <[email protected]> wrote:
> The ARCH_WANT_OPTIONAL_GPIOLIB symbol was removed in commit 65053e1a7743
> ("gpio: delete ARCH_[WANTS_OPTIONAL|REQUIRE]_GPIOLIB"). GPIOLIB should
> just be selected explicitly if needed.
>
> Remove the ARCH_WANT_OPTIONAL_GPIOLIB select from RISCV.
>
> See commit 0145071b3314 ("x86: Do away with
> ARCH_[WANT_OPTIONAL|REQUIRE]_GPIOLIB") and commit da9a1c6767 ("arm64: do
> away with ARCH_[WANT_OPTIONAL|REQUIRE]_GPIOLIB") as well.
>
> Discovered with the
> https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
> script.
>
> Signed-off-by: Ulf Magnusson <[email protected]>
> ---
> arch/riscv/Kconfig | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/arch/riscv/Kconfig b/arch/riscv/Kconfig
> index ff69c77b9e78..716e90e60e5c 100644
> --- a/arch/riscv/Kconfig
> +++ b/arch/riscv/Kconfig
> @@ -19,7 +19,6 @@ config RISCV
> select GENERIC_STRNLEN_USER
> select GENERIC_SMP_IDLE_THREAD
> select GENERIC_ATOMIC64 if !64BIT || !RISCV_ISA_A
> - select ARCH_WANT_OPTIONAL_GPIOLIB
> select HAVE_MEMBLOCK
> select HAVE_MEMBLOCK_NODE_MAP
> select HAVE_DMA_API_DEBUG
> --
> 2.14.1
>

There's also a select of a globally undefined symbol RISCV_IRQ_INTC
here. I couldn't dig up any historical or unmerged definition for it
either. Googling just gets you the select.

Cheers,
Ulf

2018-02-05 01:41:17

by Ulf Magnusson

[permalink] [raw]
Subject: Re: [PATCH 00/20] Remove references to undefined symbols and fix two potential bugs

On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <[email protected]> wrote:
> Hello,
>
> This patchset fixes most references to globally undefined symbols in Kconfig
> files, as reported by the
> https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
> script. I will send emails in reply to the patchset for some cases where I'm
> not sure what the proper fix is.
>
> All of the fixes are independent. I just created a patchset to track them more
> easily.
>
> Summary of changes:
>
> - Commits 1 and 2 fix what looks like potential bugs: A broken (intended)
> select of PL310_ERRATA_753970 for ARM/mvebu, and a misspelled dependency on
> PPC_PSERIES in KVM/PPC.
>
> - Commits 3-5 remove leftover AVR32 stuff.
>
> - Commits 6 and 7 remove some selects of removed symbols from riscv.
>
> - Commits 8-17 remove various references to removed symbols.
>
> - Commits 18-20 fix some (harmless) malformed defaults.
>
> Ulf Magnusson (20):
> ARM: mvebu: Fix broken PL310_ERRATA_753970 selects
> KVM: PPC: Book3S PR: Fix broken select due to misspelling
> kconfig: Remove leftover references to AVR32 symbol
> misc: atmel: Remove CPU_AT32AP700X (AVR32) reference
> rtc: at32ap700x: Remove PLATFORM_AT32AP dependency
> riscv: Remove ARCH_HAS_ATOMIC64_DEC_IF_POSITIVE select
> riscv: Remove ARCH_WANT_OPTIONAL_GPIOLIB select
> s390/kconfig: Remove ARCH_WANTS_PROT_NUMA_PROT_NONE select
> clk: sunxi-ng: Remove SUNXI_CCU_* selects
> lib/Kconfig: Remove leftover select of GENERIC_IO
> usb: gadget: udc: Remove USB_GADGET_DUALSPEED select
> ARM: debug: Remove ARCH_MSM dep. from UNCOMPRESS_INCLUDE
> ore: Remove PNFS_OBJLAYOUT dependency
> iio: gyro: mpu3050: Remove INPUT_MPU3050 dep. from I2C
> phy: Remove SOC_EXYNOS4212 dep. from PHY_EXYNOS4X12_USB
> auxdisplay: img-ascii-lcd: Remove MIPS_SEAD3 dep.
> media: sec: Remove PLAT_S5P dependency
> iwlwifi: fix malformed CONFIG_IWLWIFI_PCIE_RTPM default
> ARC: Fix malformed ARC_EMUL_UNALIGNED default
> x86/PCI: VMD: Fix malformed default
>
> arch/arc/Kconfig | 1 -
> arch/arm/Kconfig.debug | 2 +-
> arch/arm/mach-mvebu/Kconfig | 4 ++--
> arch/powerpc/kvm/Kconfig | 2 +-
> arch/riscv/Kconfig | 2 --
> arch/s390/Kconfig | 1 -
> drivers/auxdisplay/Kconfig | 2 +-
> drivers/clk/sunxi-ng/Kconfig | 7 -------
> drivers/cpufreq/Kconfig | 10 ----------
> drivers/iio/gyro/Kconfig | 1 -
> drivers/media/platform/Kconfig | 2 +-
> drivers/misc/Kconfig | 1 -
> drivers/net/wireless/intel/iwlwifi/Kconfig | 1 -
> drivers/pci/host/Kconfig | 1 -
> drivers/phy/samsung/Kconfig | 2 +-
> drivers/pwm/Kconfig | 2 +-
> drivers/rtc/Kconfig | 2 +-
> drivers/spi/Kconfig | 2 +-
> drivers/usb/gadget/udc/Kconfig | 1 -
> drivers/video/console/Kconfig | 2 +-
> fs/exofs/Kconfig.ore | 2 +-
> lib/Kconfig | 1 -
> 22 files changed, 12 insertions(+), 39 deletions(-)
>
> --
> 2.14.1
>

Since 05a4a9527931 ("kernel/watchdog: split up config options"),
lib/Kconfig.debug references a globally undefined symbol
HARDLOCK_DETECTOR_ARCH. My guess is that it was supposed to be defined
alongside HARDLOCK_DETECTOR_PERF in the same file.

Cheers,
Ulf

2018-02-05 01:45:27

by Ulf Magnusson

[permalink] [raw]
Subject: Re: [PATCH 05/20] rtc: at32ap700x: Remove PLATFORM_AT32AP dependency

On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <[email protected]> wrote:
> The PLATFORM_AT32AP symbol went away when AVR32 was removed in
> commit 26202873bb51 ("avr32: remove support for AVR32 architecture").
>
> Remove the RTC_DRV_AT32AP700X dependency from RTC_DRV_AT32AP700X.
>
> Discovered with the
> https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
> script.
>
> Signed-off-by: Ulf Magnusson <[email protected]>
> ---
> drivers/rtc/Kconfig | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/rtc/Kconfig b/drivers/rtc/Kconfig
> index b71cc7b5fbc9..1f5f3bd82ab6 100644
> --- a/drivers/rtc/Kconfig
> +++ b/drivers/rtc/Kconfig
> @@ -1404,7 +1404,7 @@ config RTC_DRV_PL031
>
> config RTC_DRV_AT32AP700X
> tristate "AT32AP700X series RTC"
> - depends on PLATFORM_AT32AP || COMPILE_TEST
> + depends on COMPILE_TEST
> help
> Driver for the internal RTC (Realtime Clock) on Atmel AVR32
> AT32AP700x family processors.
> --
> 2.14.1
>

RTC_DRV_AT32AP700X could probably be removed along with
./drivers/rtc/rtc-at32ap700x.c as well.

Cheers,
Ulf

2018-02-05 01:47:11

by Ulf Magnusson

[permalink] [raw]
Subject: Re: [PATCH 12/20] ARM: debug: Remove ARCH_MSM dep. from UNCOMPRESS_INCLUDE

On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <[email protected]> wrote:
> The ARCH_MSM symbol was removed in commit c0c89fafa289 ("ARM: Remove
> mach-msm and associated ARM architecture code"), but a default in
> UNCOMPRESS_INCLUDE still depends on it.
>
> Remove the ARCH_MSM dependency from the default in UNCOMPRESS_INCLUDE.
>
> Discovered with the
> https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
> script.
>
> Signed-off-by: Ulf Magnusson <[email protected]>
> ---
> arch/arm/Kconfig.debug | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/Kconfig.debug b/arch/arm/Kconfig.debug
> index 78a647080ebc..20c601a045d2 100644
> --- a/arch/arm/Kconfig.debug
> +++ b/arch/arm/Kconfig.debug
> @@ -1825,7 +1825,7 @@ config DEBUG_UNCOMPRESS
>
> config UNCOMPRESS_INCLUDE
> string
> - default "debug/uncompress.h" if ARCH_MULTIPLATFORM || ARCH_MSM || \
> + default "debug/uncompress.h" if ARCH_MULTIPLATFORM || \
> PLAT_SAMSUNG || ARM_SINGLE_ARMV7M
> default "mach/uncompress.h"
>
> --
> 2.14.1
>

These globally undefined symbols (with no historical definitions that
I could find) are also referenced in this file:

DEBUG_OMAP4UART2
DEBUG_OMAP4UART1
DEBUG_OMAP5UART1
DEBUG_OMAP5UART2
DEBUG_OMAP3UART2
DEBUG_OMAP3UART1

Cheers,
Ulf

2018-02-05 01:55:09

by Ulf Magnusson

[permalink] [raw]
Subject: Re: [PATCH 00/20] Remove references to undefined symbols and fix two potential bugs

On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <[email protected]> wrote:
> Hello,
>
> This patchset fixes most references to globally undefined symbols in Kconfig
> files, as reported by the
> https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
> script. I will send emails in reply to the patchset for some cases where I'm
> not sure what the proper fix is.
>
> All of the fixes are independent. I just created a patchset to track them more
> easily.
>
> Summary of changes:
>
> - Commits 1 and 2 fix what looks like potential bugs: A broken (intended)
> select of PL310_ERRATA_753970 for ARM/mvebu, and a misspelled dependency on
> PPC_PSERIES in KVM/PPC.
>
> - Commits 3-5 remove leftover AVR32 stuff.
>
> - Commits 6 and 7 remove some selects of removed symbols from riscv.
>
> - Commits 8-17 remove various references to removed symbols.
>
> - Commits 18-20 fix some (harmless) malformed defaults.
>
> Ulf Magnusson (20):
> ARM: mvebu: Fix broken PL310_ERRATA_753970 selects
> KVM: PPC: Book3S PR: Fix broken select due to misspelling
> kconfig: Remove leftover references to AVR32 symbol
> misc: atmel: Remove CPU_AT32AP700X (AVR32) reference
> rtc: at32ap700x: Remove PLATFORM_AT32AP dependency
> riscv: Remove ARCH_HAS_ATOMIC64_DEC_IF_POSITIVE select
> riscv: Remove ARCH_WANT_OPTIONAL_GPIOLIB select
> s390/kconfig: Remove ARCH_WANTS_PROT_NUMA_PROT_NONE select
> clk: sunxi-ng: Remove SUNXI_CCU_* selects
> lib/Kconfig: Remove leftover select of GENERIC_IO
> usb: gadget: udc: Remove USB_GADGET_DUALSPEED select
> ARM: debug: Remove ARCH_MSM dep. from UNCOMPRESS_INCLUDE
> ore: Remove PNFS_OBJLAYOUT dependency
> iio: gyro: mpu3050: Remove INPUT_MPU3050 dep. from I2C
> phy: Remove SOC_EXYNOS4212 dep. from PHY_EXYNOS4X12_USB
> auxdisplay: img-ascii-lcd: Remove MIPS_SEAD3 dep.
> media: sec: Remove PLAT_S5P dependency
> iwlwifi: fix malformed CONFIG_IWLWIFI_PCIE_RTPM default
> ARC: Fix malformed ARC_EMUL_UNALIGNED default
> x86/PCI: VMD: Fix malformed default
>
> arch/arc/Kconfig | 1 -
> arch/arm/Kconfig.debug | 2 +-
> arch/arm/mach-mvebu/Kconfig | 4 ++--
> arch/powerpc/kvm/Kconfig | 2 +-
> arch/riscv/Kconfig | 2 --
> arch/s390/Kconfig | 1 -
> drivers/auxdisplay/Kconfig | 2 +-
> drivers/clk/sunxi-ng/Kconfig | 7 -------
> drivers/cpufreq/Kconfig | 10 ----------
> drivers/iio/gyro/Kconfig | 1 -
> drivers/media/platform/Kconfig | 2 +-
> drivers/misc/Kconfig | 1 -
> drivers/net/wireless/intel/iwlwifi/Kconfig | 1 -
> drivers/pci/host/Kconfig | 1 -
> drivers/phy/samsung/Kconfig | 2 +-
> drivers/pwm/Kconfig | 2 +-
> drivers/rtc/Kconfig | 2 +-
> drivers/spi/Kconfig | 2 +-
> drivers/usb/gadget/udc/Kconfig | 1 -
> drivers/video/console/Kconfig | 2 +-
> fs/exofs/Kconfig.ore | 2 +-
> lib/Kconfig | 1 -
> 22 files changed, 12 insertions(+), 39 deletions(-)
>
> --
> 2.14.1
>

GDBSTUB in arch/mn10300/Kconfig.debug depends on the globally
undefined symbol DEPRECATED, which seems to have been removed a long
time ago. Here's an old reference I found:
https://lwn.net/Articles/113867/

Cheers,
Ulf

2018-02-05 02:00:40

by Ulf Magnusson

[permalink] [raw]
Subject: Re: [PATCH 16/20] auxdisplay: img-ascii-lcd: Remove MIPS_SEAD3 dep.

On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <[email protected]> wrote:
> The MIPS_SEAD3 symbol was removed in commit 64601cb1343f ("leds: Remove SEAD-3
> driver").
>
> Remove the MIPS_SEAD3 dependency from IMG_ASCII_LCD.
>
> Discovered with the
> https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
> script.
>
> Signed-off-by: Ulf Magnusson <[email protected]>
> ---
> drivers/auxdisplay/Kconfig | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/auxdisplay/Kconfig b/drivers/auxdisplay/Kconfig
> index 2c2ed9cf8796..3cba78d36eec 100644
> --- a/drivers/auxdisplay/Kconfig
> +++ b/drivers/auxdisplay/Kconfig
> @@ -137,7 +137,7 @@ config CFAG12864B_RATE
> config IMG_ASCII_LCD
> tristate "Imagination Technologies ASCII LCD Display"
> depends on HAS_IOMEM
> - default y if MIPS_MALTA || MIPS_SEAD3
> + default y if MIPS_MALTA
> select SYSCON
> help
> Enable this to support the simple ASCII LCD displays found on
> --
> 2.14.1
>

The SYSCON symbol selected here is also globally undefined. Maybe
MFD_SYSCON was intended.

Cheers,
Ulf

2018-02-05 02:06:15

by Ulf Magnusson

[permalink] [raw]
Subject: Re: [PATCH 20/20] x86/PCI: VMD: Fix malformed default

On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <[email protected]> wrote:
> 'default N' should be 'default n', though they happen to have the same
> effect here, due to undefined symbols (N in this case) evaluating to n
> in a tristate sense.
>
> Remove the default from VMD instead of changing it. bool and tristate
> symbols implicitly default to n.
>
> Discovered with the
> https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
> script.
>
> Signed-off-by: Ulf Magnusson <[email protected]>
> ---
> drivers/pci/host/Kconfig | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/drivers/pci/host/Kconfig b/drivers/pci/host/Kconfig
> index a4ed7484d127..dc8a2a175f19 100644
> --- a/drivers/pci/host/Kconfig
> +++ b/drivers/pci/host/Kconfig
> @@ -215,7 +215,6 @@ config PCIE_TANGO_SMP8759
> config VMD
> depends on PCI_MSI && X86_64 && SRCU
> tristate "Intel Volume Management Device Driver"
> - default N
> ---help---
> Adds support for the Intel Volume Management Device (VMD). VMD is a
> secondary PCI host bridge that allows PCI Express root ports,
> --
> 2.14.1
>

The PCIE_ALTERA symbol in this file also depends on the globally
undefined symbol OF_PCI, which was removed in 4670d610d592 ("PCI: Move
OF-related PCI functions into PCI core").

Cheers,
Ulf

2018-02-05 02:10:54

by Ulf Magnusson

[permalink] [raw]
Subject: Re: [PATCH 00/20] Remove references to undefined symbols and fix two potential bugs

On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <[email protected]> wrote:
> Hello,
>
> This patchset fixes most references to globally undefined symbols in Kconfig
> files, as reported by the
> https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
> script. I will send emails in reply to the patchset for some cases where I'm
> not sure what the proper fix is.
>
> All of the fixes are independent. I just created a patchset to track them more
> easily.
>
> Summary of changes:
>
> - Commits 1 and 2 fix what looks like potential bugs: A broken (intended)
> select of PL310_ERRATA_753970 for ARM/mvebu, and a misspelled dependency on
> PPC_PSERIES in KVM/PPC.
>
> - Commits 3-5 remove leftover AVR32 stuff.
>
> - Commits 6 and 7 remove some selects of removed symbols from riscv.
>
> - Commits 8-17 remove various references to removed symbols.
>
> - Commits 18-20 fix some (harmless) malformed defaults.
>
> Ulf Magnusson (20):
> ARM: mvebu: Fix broken PL310_ERRATA_753970 selects
> KVM: PPC: Book3S PR: Fix broken select due to misspelling
> kconfig: Remove leftover references to AVR32 symbol
> misc: atmel: Remove CPU_AT32AP700X (AVR32) reference
> rtc: at32ap700x: Remove PLATFORM_AT32AP dependency
> riscv: Remove ARCH_HAS_ATOMIC64_DEC_IF_POSITIVE select
> riscv: Remove ARCH_WANT_OPTIONAL_GPIOLIB select
> s390/kconfig: Remove ARCH_WANTS_PROT_NUMA_PROT_NONE select
> clk: sunxi-ng: Remove SUNXI_CCU_* selects
> lib/Kconfig: Remove leftover select of GENERIC_IO
> usb: gadget: udc: Remove USB_GADGET_DUALSPEED select
> ARM: debug: Remove ARCH_MSM dep. from UNCOMPRESS_INCLUDE
> ore: Remove PNFS_OBJLAYOUT dependency
> iio: gyro: mpu3050: Remove INPUT_MPU3050 dep. from I2C
> phy: Remove SOC_EXYNOS4212 dep. from PHY_EXYNOS4X12_USB
> auxdisplay: img-ascii-lcd: Remove MIPS_SEAD3 dep.
> media: sec: Remove PLAT_S5P dependency
> iwlwifi: fix malformed CONFIG_IWLWIFI_PCIE_RTPM default
> ARC: Fix malformed ARC_EMUL_UNALIGNED default
> x86/PCI: VMD: Fix malformed default
>
> arch/arc/Kconfig | 1 -
> arch/arm/Kconfig.debug | 2 +-
> arch/arm/mach-mvebu/Kconfig | 4 ++--
> arch/powerpc/kvm/Kconfig | 2 +-
> arch/riscv/Kconfig | 2 --
> arch/s390/Kconfig | 1 -
> drivers/auxdisplay/Kconfig | 2 +-
> drivers/clk/sunxi-ng/Kconfig | 7 -------
> drivers/cpufreq/Kconfig | 10 ----------
> drivers/iio/gyro/Kconfig | 1 -
> drivers/media/platform/Kconfig | 2 +-
> drivers/misc/Kconfig | 1 -
> drivers/net/wireless/intel/iwlwifi/Kconfig | 1 -
> drivers/pci/host/Kconfig | 1 -
> drivers/phy/samsung/Kconfig | 2 +-
> drivers/pwm/Kconfig | 2 +-
> drivers/rtc/Kconfig | 2 +-
> drivers/spi/Kconfig | 2 +-
> drivers/usb/gadget/udc/Kconfig | 1 -
> drivers/video/console/Kconfig | 2 +-
> fs/exofs/Kconfig.ore | 2 +-
> lib/Kconfig | 1 -
> 22 files changed, 12 insertions(+), 39 deletions(-)
>
> --
> 2.14.1
>

The symbol AC97_BUS_NEW in sound/ac97/Kconfig selects the globally
undefined symbol AC97.

It doesn't seem to simply be a misspelling of AC97_BUS, because then
the dependencies of AC97_BUS_COMPAT would be impossible to satisfy.

Cheers,
Ulf

2018-02-05 04:06:25

by Viresh Kumar

[permalink] [raw]
Subject: Re: [PATCH 03/20] kconfig: Remove leftover references to AVR32 symbol

On 05-02-18, 02:21, Ulf Magnusson wrote:
> The AVR32 symbol was removed in commit 26202873bb51 ("avr32: remove
> support for AVR32 architecture"). Remove the remaining references to it
> from the Kconfig files.
>
> No references remain to the removed AVR32_AT32AP_CPUFREQ symbol either,
> so remove that one too.
>
> Discovered with the
> https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
> script.
>
> Signed-off-by: Ulf Magnusson <[email protected]>
> ---
> drivers/cpufreq/Kconfig | 10 ----------
> drivers/pwm/Kconfig | 2 +-
> drivers/spi/Kconfig | 2 +-
> drivers/video/console/Kconfig | 2 +-
> 4 files changed, 3 insertions(+), 13 deletions(-)
>
> diff --git a/drivers/cpufreq/Kconfig b/drivers/cpufreq/Kconfig
> index d8addbce40bc..608af20a3494 100644
> --- a/drivers/cpufreq/Kconfig
> +++ b/drivers/cpufreq/Kconfig
> @@ -239,16 +239,6 @@ if PPC32 || PPC64
> source "drivers/cpufreq/Kconfig.powerpc"
> endif
>
> -if AVR32
> -config AVR32_AT32AP_CPUFREQ
> - bool "CPU frequency driver for AT32AP"
> - depends on PLATFORM_AT32AP
> - default n
> - help
> - This enables the CPU frequency driver for AT32AP processors.
> - If in doubt, say N.
> -endif
> -

https://lkml.kernel.org/r/[email protected]

--
viresh

2018-02-05 04:16:22

by Ulf Magnusson

[permalink] [raw]
Subject: Re: [PATCH 00/20] Remove references to undefined symbols and fix two potential bugs

On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <[email protected]> wrote:
> Hello,
>
> This patchset fixes most references to globally undefined symbols in Kconfig
> files, as reported by the
> https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
> script. I will send emails in reply to the patchset for some cases where I'm
> not sure what the proper fix is.
>
> All of the fixes are independent. I just created a patchset to track them more
> easily.
>
> Summary of changes:
>
> - Commits 1 and 2 fix what looks like potential bugs: A broken (intended)
> select of PL310_ERRATA_753970 for ARM/mvebu, and a misspelled dependency on
> PPC_PSERIES in KVM/PPC.
>
> - Commits 3-5 remove leftover AVR32 stuff.
>
> - Commits 6 and 7 remove some selects of removed symbols from riscv.
>
> - Commits 8-17 remove various references to removed symbols.
>
> - Commits 18-20 fix some (harmless) malformed defaults.
>
> Ulf Magnusson (20):
> ARM: mvebu: Fix broken PL310_ERRATA_753970 selects
> KVM: PPC: Book3S PR: Fix broken select due to misspelling
> kconfig: Remove leftover references to AVR32 symbol
> misc: atmel: Remove CPU_AT32AP700X (AVR32) reference
> rtc: at32ap700x: Remove PLATFORM_AT32AP dependency
> riscv: Remove ARCH_HAS_ATOMIC64_DEC_IF_POSITIVE select
> riscv: Remove ARCH_WANT_OPTIONAL_GPIOLIB select
> s390/kconfig: Remove ARCH_WANTS_PROT_NUMA_PROT_NONE select
> clk: sunxi-ng: Remove SUNXI_CCU_* selects
> lib/Kconfig: Remove leftover select of GENERIC_IO
> usb: gadget: udc: Remove USB_GADGET_DUALSPEED select
> ARM: debug: Remove ARCH_MSM dep. from UNCOMPRESS_INCLUDE
> ore: Remove PNFS_OBJLAYOUT dependency
> iio: gyro: mpu3050: Remove INPUT_MPU3050 dep. from I2C
> phy: Remove SOC_EXYNOS4212 dep. from PHY_EXYNOS4X12_USB
> auxdisplay: img-ascii-lcd: Remove MIPS_SEAD3 dep.
> media: sec: Remove PLAT_S5P dependency
> iwlwifi: fix malformed CONFIG_IWLWIFI_PCIE_RTPM default
> ARC: Fix malformed ARC_EMUL_UNALIGNED default
> x86/PCI: VMD: Fix malformed default
>
> arch/arc/Kconfig | 1 -
> arch/arm/Kconfig.debug | 2 +-
> arch/arm/mach-mvebu/Kconfig | 4 ++--
> arch/powerpc/kvm/Kconfig | 2 +-
> arch/riscv/Kconfig | 2 --
> arch/s390/Kconfig | 1 -
> drivers/auxdisplay/Kconfig | 2 +-
> drivers/clk/sunxi-ng/Kconfig | 7 -------
> drivers/cpufreq/Kconfig | 10 ----------
> drivers/iio/gyro/Kconfig | 1 -
> drivers/media/platform/Kconfig | 2 +-
> drivers/misc/Kconfig | 1 -
> drivers/net/wireless/intel/iwlwifi/Kconfig | 1 -
> drivers/pci/host/Kconfig | 1 -
> drivers/phy/samsung/Kconfig | 2 +-
> drivers/pwm/Kconfig | 2 +-
> drivers/rtc/Kconfig | 2 +-
> drivers/spi/Kconfig | 2 +-
> drivers/usb/gadget/udc/Kconfig | 1 -
> drivers/video/console/Kconfig | 2 +-
> fs/exofs/Kconfig.ore | 2 +-
> lib/Kconfig | 1 -
> 22 files changed, 12 insertions(+), 39 deletions(-)
>
> --
> 2.14.1
>

Another one that looks like a potential bug:

b49efd762482 ("dma-mapping: move dma_mark_clean to dma-direct.h")
selects ARCH_HAS_DMA_MARK_CLEAN for ia64 but never defines it. It's
used in include/linux/dma-direct.h:

#ifdef CONFIG_ARCH_HAS_DMA_MARK_CLEAN
void dma_mark_clean(void *addr, size_t size);
#else
static inline void dma_mark_clean(void *addr, size_t size)
{
}
#endif /* CONFIG_ARCH_HAS_DMA_MARK_CLEAN */

Cheers,
Ulf

2018-02-05 04:50:09

by Ulf Magnusson

[permalink] [raw]
Subject: [PATCH v2 03/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 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/pwm/Kconfig | 2 +-
drivers/spi/Kconfig | 2 +-
drivers/video/console/Kconfig | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/pwm/Kconfig b/drivers/pwm/Kconfig
index 763ee50ea57d..f16aad3bf5d6 100644
--- a/drivers/pwm/Kconfig
+++ b/drivers/pwm/Kconfig
@@ -43,7 +43,7 @@ config PWM_AB8500

config PWM_ATMEL
tristate "Atmel PWM support"
- depends on ARCH_AT91 || AVR32
+ depends on ARCH_AT91
help
Generic PWM framework driver for Atmel SoC.

diff --git a/drivers/spi/Kconfig b/drivers/spi/Kconfig
index 603783976b81..c1e455d46c7f 100644
--- a/drivers/spi/Kconfig
+++ b/drivers/spi/Kconfig
@@ -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.
diff --git a/drivers/video/console/Kconfig b/drivers/video/console/Kconfig
index 7f1f1fbcef9e..be63759b6027 100644
--- a/drivers/video/console/Kconfig
+++ b/drivers/video/console/Kconfig
@@ -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
--
2.14.1


2018-02-05 04:52:42

by Paul Mackerras

[permalink] [raw]
Subject: Re: [PATCH 02/20] KVM: PPC: Book3S PR: Fix broken select due to misspelling

On Mon, Feb 05, 2018 at 02:21:14AM +0100, Ulf Magnusson wrote:
> Commit 76d837a4c0f9 ("KVM: PPC: Book3S PR: Don't include SPAPR TCE code
> on non-pseries platforms") added a reference to the globally undefined
> symbol PPC_SERIES. Looking at the rest of the commit, PPC_PSERIES was
> probably intended.
>
> Change PPC_SERIES to PPC_PSERIES.
>
> Discovered with the
> https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
> script.
>
> Signed-off-by: Ulf Magnusson <[email protected]>

Acked-by: Paul Mackerras <[email protected]>

Which tree is this series going into?

Paul.

2018-02-05 05:01:05

by Ulf Magnusson

[permalink] [raw]
Subject: Re: [PATCH 02/20] KVM: PPC: Book3S PR: Fix broken select due to misspelling

On Mon, Feb 5, 2018 at 5:48 AM, Paul Mackerras <[email protected]> wrote:
> On Mon, Feb 05, 2018 at 02:21:14AM +0100, Ulf Magnusson wrote:
>> Commit 76d837a4c0f9 ("KVM: PPC: Book3S PR: Don't include SPAPR TCE code
>> on non-pseries platforms") added a reference to the globally undefined
>> symbol PPC_SERIES. Looking at the rest of the commit, PPC_PSERIES was
>> probably intended.
>>
>> Change PPC_SERIES to PPC_PSERIES.
>>
>> Discovered with the
>> https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
>> script.
>>
>> Signed-off-by: Ulf Magnusson <[email protected]>
>
> Acked-by: Paul Mackerras <[email protected]>
>
> Which tree is this series going into?
>
> Paul.

I didn't have a particular one in mind. I imagined people would pick
up individual patches into the trees that make the most sense. It was
easiest to do the undefined symbol removal as a kernel-wide batch job.

All patches are against linux-next.

Cheers,
Ulf

2018-02-05 06:57:59

by Heiko Carstens

[permalink] [raw]
Subject: Re: [PATCH 08/20] s390/kconfig: Remove ARCH_WANTS_PROT_NUMA_PROT_NONE select

On Mon, Feb 05, 2018 at 02:21:20AM +0100, Ulf Magnusson wrote:
> The ARCH_WANTS_PROT_NUMA_PROT_NONE symbol was removed by
> commit 6a33979d5bd7 ("mm: remove misleading ARCH_USES_NUMA_PROT_NONE"),
> but S390 still selects it.
>
> Remove the ARCH_WANTS_PROT_NUMA_PROT_NONE select from the S390 symbol.
>
> Discovered with the
> https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
> script.
>
> Signed-off-by: Ulf Magnusson <[email protected]>
> ---
> arch/s390/Kconfig | 1 -
> 1 file changed, 1 deletion(-)

Applied, thanks!


2018-02-05 07:57:39

by Paul Mackerras

[permalink] [raw]
Subject: Re: [PATCH 02/20] KVM: PPC: Book3S PR: Fix broken select due to misspelling

On Mon, Feb 05, 2018 at 05:58:59AM +0100, Ulf Magnusson wrote:
> On Mon, Feb 5, 2018 at 5:48 AM, Paul Mackerras <[email protected]> wrote:
> > On Mon, Feb 05, 2018 at 02:21:14AM +0100, Ulf Magnusson wrote:
> >> Commit 76d837a4c0f9 ("KVM: PPC: Book3S PR: Don't include SPAPR TCE code
> >> on non-pseries platforms") added a reference to the globally undefined
> >> symbol PPC_SERIES. Looking at the rest of the commit, PPC_PSERIES was
> >> probably intended.
> >>
> >> Change PPC_SERIES to PPC_PSERIES.
> >>
> >> Discovered with the
> >> https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
> >> script.
> >>
> >> Signed-off-by: Ulf Magnusson <[email protected]>
> >
> > Acked-by: Paul Mackerras <[email protected]>
> >
> > Which tree is this series going into?
> >
> > Paul.
>
> I didn't have a particular one in mind. I imagined people would pick
> up individual patches into the trees that make the most sense. It was
> easiest to do the undefined symbol removal as a kernel-wide batch job.

OK, I'll take this one then.

Paul.

2018-02-05 08:03:10

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH 15/20] phy: Remove SOC_EXYNOS4212 dep. from PHY_EXYNOS4X12_USB

On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <[email protected]> wrote:
> Exynos4212 support was removed by commit bca9085e0ae9 ("ARM: dts:
> exynos: remove Exynos4212 support (dead code)").
>
> Remove the SOC_EXYNOS4212 dependency from PHY_EXYNOS4X12_USB.
>
> Discovered with the
> https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
> script.
>
> Signed-off-by: Ulf Magnusson <[email protected]>
> ---
> drivers/phy/samsung/Kconfig | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)

Acked-by: Krzysztof Kozlowski <[email protected]>

Best regards,
Krzysztof

2018-02-05 10:48:13

by J. Neuschäfer

[permalink] [raw]
Subject: Re: [PATCH 06/20] riscv: Remove ARCH_HAS_ATOMIC64_DEC_IF_POSITIVE select

On Mon, Feb 05, 2018 at 02:21:18AM +0100, Ulf Magnusson wrote:
> The ARCH_HAS_ATOMIC64_DEC_IF_POSITIVE symbol was removed in
> commit 51a021244b9d ("atomic64: no need for
> CONFIG_ARCH_HAS_ATOMIC64_DEC_IF_POSITIVE").
>
> Remove the ARCH_HAS_ATOMIC64_DEC_IS_POSITIVE select from RISCV.
>
> Discovered with the
> https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
> script.
>
> Signed-off-by: Ulf Magnusson <[email protected]>
> ---
> arch/riscv/Kconfig | 1 -
> 1 file changed, 1 deletion(-)

Looks good.

Reviewed-by: Jonathan Neuschäfer <[email protected]>

>
> diff --git a/arch/riscv/Kconfig b/arch/riscv/Kconfig
> index b6722c246d9c..ff69c77b9e78 100644
> --- a/arch/riscv/Kconfig
> +++ b/arch/riscv/Kconfig
> @@ -8,7 +8,6 @@ config RISCV
> select OF
> select OF_EARLY_FLATTREE
> select OF_IRQ
> - select ARCH_HAS_ATOMIC64_DEC_IF_POSITIVE
> select ARCH_WANT_FRAME_POINTERS
> select CLONE_BACKWARDS
> select COMMON_CLK


Attachments:
(No filename) (0.98 kB)
signature.asc (836.00 B)
Download all attachments

2018-02-05 13:47:57

by Maxime Ripard

[permalink] [raw]
Subject: Re: [PATCH 09/20] clk: sunxi-ng: Remove SUNXI_CCU_* selects

Hi Ulf,

On Mon, Feb 05, 2018 at 02:21:21AM +0100, Ulf Magnusson wrote:
> The selected symbols were removed in commit 06e226c7fb23 ("clk:
> sunxi-ng: Move all clock types to a library").
>
> Remove the selects from SUN4I_A10_CCU.
>
> Discovered with the
> https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
> script.
>
> Signed-off-by: Ulf Magnusson <[email protected]>

This has already been submitted by Corentin:
https://patchwork.kernel.org/patch/10175435/

Thanks!
Maxime

--
Maxime Ripard, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
http://bootlin.com


Attachments:
(No filename) (645.00 B)
signature.asc (849.00 B)
Download all attachments

2018-02-05 14:03:57

by Alexandre Belloni

[permalink] [raw]
Subject: Re: [PATCH 05/20] rtc: at32ap700x: Remove PLATFORM_AT32AP dependency

Hi,

On 05/02/2018 at 02:42:38 +0100, Ulf Magnusson wrote:
> On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <[email protected]> wrote:
> > The PLATFORM_AT32AP symbol went away when AVR32 was removed in
> > commit 26202873bb51 ("avr32: remove support for AVR32 architecture").
> >
> > Remove the RTC_DRV_AT32AP700X dependency from RTC_DRV_AT32AP700X.
> >
> > Discovered with the
> > https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
> > script.
> >
> > Signed-off-by: Ulf Magnusson <[email protected]>
> > ---
> > drivers/rtc/Kconfig | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/rtc/Kconfig b/drivers/rtc/Kconfig
> > index b71cc7b5fbc9..1f5f3bd82ab6 100644
> > --- a/drivers/rtc/Kconfig
> > +++ b/drivers/rtc/Kconfig
> > @@ -1404,7 +1404,7 @@ config RTC_DRV_PL031
> >
> > config RTC_DRV_AT32AP700X
> > tristate "AT32AP700X series RTC"
> > - depends on PLATFORM_AT32AP || COMPILE_TEST
> > + depends on COMPILE_TEST
> > help
> > Driver for the internal RTC (Realtime Clock) on Atmel AVR32
> > AT32AP700x family processors.
> > --
> > 2.14.1
> >
>
> RTC_DRV_AT32AP700X could probably be removed along with
> ./drivers/rtc/rtc-at32ap700x.c as well.
>

Note that the driver is already removed in Linus' tree.


--
Alexandre Belloni, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
http://bootlin.com

2018-02-05 17:19:09

by Vineet Gupta

[permalink] [raw]
Subject: Re: [PATCH 19/20] ARC: Fix malformed ARC_EMUL_UNALIGNED default

On 02/04/2018 05:21 PM, Ulf Magnusson wrote:
> 'default N' should be 'default n', though they happen to have the same
> effect here, due to undefined symbols (N in this case) evaluating to n
> in a tristate sense.
>
> Remove the default from ARC_EMUL_UNALIGNED instead of changing it. bool
> and tristate symbols implicitly default to n.
>
> Discovered with the
> https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
> script.
>
> Signed-off-by: Ulf Magnusson <[email protected]>


Added to ARC for-curr.

Thx,
-Vineet

> ---
> arch/arc/Kconfig | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/arch/arc/Kconfig b/arch/arc/Kconfig
> index f3a80cf164cc..d76bf4a83740 100644
> --- a/arch/arc/Kconfig
> +++ b/arch/arc/Kconfig
> @@ -484,7 +484,6 @@ config ARC_CURR_IN_REG
>
> config ARC_EMUL_UNALIGNED
> bool "Emulate unaligned memory access (userspace only)"
> - default N
> select SYSCTL_ARCH_UNALIGN_NO_WARN
> select SYSCTL_ARCH_UNALIGN_ALLOW
> depends on ISA_ARCOMPACT
>


2018-02-05 19:07:26

by Ulf Magnusson

[permalink] [raw]
Subject: [PATCH v3 03/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(-)

diff --git a/drivers/spi/Kconfig b/drivers/spi/Kconfig
index 603783976b81..c1e455d46c7f 100644
--- a/drivers/spi/Kconfig
+++ b/drivers/spi/Kconfig
@@ -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.
diff --git a/drivers/video/console/Kconfig b/drivers/video/console/Kconfig
index 7f1f1fbcef9e..be63759b6027 100644
--- a/drivers/video/console/Kconfig
+++ b/drivers/video/console/Kconfig
@@ -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
--
2.14.1


2018-02-05 20:33:50

by Ulf Magnusson

[permalink] [raw]
Subject: Re: [PATCH 05/20] rtc: at32ap700x: Remove PLATFORM_AT32AP dependency

On Mon, Feb 5, 2018 at 3:01 PM, Alexandre Belloni
<[email protected]> wrote:
> Hi,
>
> On 05/02/2018 at 02:42:38 +0100, Ulf Magnusson wrote:
>> On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <[email protected]> wrote:
>> > The PLATFORM_AT32AP symbol went away when AVR32 was removed in
>> > commit 26202873bb51 ("avr32: remove support for AVR32 architecture").
>> >
>> > Remove the RTC_DRV_AT32AP700X dependency from RTC_DRV_AT32AP700X.
>> >
>> > Discovered with the
>> > https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
>> > script.
>> >
>> > Signed-off-by: Ulf Magnusson <[email protected]>
>> > ---
>> > drivers/rtc/Kconfig | 2 +-
>> > 1 file changed, 1 insertion(+), 1 deletion(-)
>> >
>> > diff --git a/drivers/rtc/Kconfig b/drivers/rtc/Kconfig
>> > index b71cc7b5fbc9..1f5f3bd82ab6 100644
>> > --- a/drivers/rtc/Kconfig
>> > +++ b/drivers/rtc/Kconfig
>> > @@ -1404,7 +1404,7 @@ config RTC_DRV_PL031
>> >
>> > config RTC_DRV_AT32AP700X
>> > tristate "AT32AP700X series RTC"
>> > - depends on PLATFORM_AT32AP || COMPILE_TEST
>> > + depends on COMPILE_TEST
>> > help
>> > Driver for the internal RTC (Realtime Clock) on Atmel AVR32
>> > AT32AP700x family processors.
>> > --
>> > 2.14.1
>> >
>>
>> RTC_DRV_AT32AP700X could probably be removed along with
>> ./drivers/rtc/rtc-at32ap700x.c as well.
>>
>
> Note that the driver is already removed in Linus' tree.
>
>
> --
> Alexandre Belloni, Bootlin (formerly Free Electrons)
> Embedded Linux and Kernel engineering
> http://bootlin.com

That got rid of the config symbol too. This patch is moot then.

Cheers,
Ulf

2018-02-05 23:26:48

by Linus Walleij

[permalink] [raw]
Subject: Re: [PATCH 07/20] riscv: Remove ARCH_WANT_OPTIONAL_GPIOLIB select

On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <[email protected]> wrote:

> The ARCH_WANT_OPTIONAL_GPIOLIB symbol was removed in commit 65053e1a7743
> ("gpio: delete ARCH_[WANTS_OPTIONAL|REQUIRE]_GPIOLIB"). GPIOLIB should
> just be selected explicitly if needed.
>
> Remove the ARCH_WANT_OPTIONAL_GPIOLIB select from RISCV.
>
> See commit 0145071b3314 ("x86: Do away with
> ARCH_[WANT_OPTIONAL|REQUIRE]_GPIOLIB") and commit da9a1c6767 ("arm64: do
> away with ARCH_[WANT_OPTIONAL|REQUIRE]_GPIOLIB") as well.
>
> Discovered with the
> https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
> script.
>
> Signed-off-by: Ulf Magnusson <[email protected]>

This should probaly be replaced with
select GPIOLIB

Yours,
Linus Walleij

2018-02-05 23:40:42

by Ulf Magnusson

[permalink] [raw]
Subject: Re: [PATCH 07/20] riscv: Remove ARCH_WANT_OPTIONAL_GPIOLIB select

On Tue, Feb 6, 2018 at 12:25 AM, Linus Walleij <[email protected]> wrote:
> On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <[email protected]> wrote:
>
>> The ARCH_WANT_OPTIONAL_GPIOLIB symbol was removed in commit 65053e1a7743
>> ("gpio: delete ARCH_[WANTS_OPTIONAL|REQUIRE]_GPIOLIB"). GPIOLIB should
>> just be selected explicitly if needed.
>>
>> Remove the ARCH_WANT_OPTIONAL_GPIOLIB select from RISCV.
>>
>> See commit 0145071b3314 ("x86: Do away with
>> ARCH_[WANT_OPTIONAL|REQUIRE]_GPIOLIB") and commit da9a1c6767 ("arm64: do
>> away with ARCH_[WANT_OPTIONAL|REQUIRE]_GPIOLIB") as well.
>>
>> Discovered with the
>> https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
>> script.
>>
>> Signed-off-by: Ulf Magnusson <[email protected]>
>
> This should probaly be replaced with
> select GPIOLIB
>
> Yours,
> Linus Walleij

That'd make more sense if GPIOLIB is actually used. The select has
been a no-op since mid-2016 at least, but maybe something else selects
GPIOLIB.

Cheers,
Ulf

2018-02-05 23:51:35

by Ulf Magnusson

[permalink] [raw]
Subject: Re: [PATCH 07/20] riscv: Remove ARCH_WANT_OPTIONAL_GPIOLIB select

On Tue, Feb 6, 2018 at 12:38 AM, Ulf Magnusson <[email protected]> wrote:
> On Tue, Feb 6, 2018 at 12:25 AM, Linus Walleij <[email protected]> wrote:
>> On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <[email protected]> wrote:
>>
>>> The ARCH_WANT_OPTIONAL_GPIOLIB symbol was removed in commit 65053e1a7743
>>> ("gpio: delete ARCH_[WANTS_OPTIONAL|REQUIRE]_GPIOLIB"). GPIOLIB should
>>> just be selected explicitly if needed.
>>>
>>> Remove the ARCH_WANT_OPTIONAL_GPIOLIB select from RISCV.
>>>
>>> See commit 0145071b3314 ("x86: Do away with
>>> ARCH_[WANT_OPTIONAL|REQUIRE]_GPIOLIB") and commit da9a1c6767 ("arm64: do
>>> away with ARCH_[WANT_OPTIONAL|REQUIRE]_GPIOLIB") as well.
>>>
>>> Discovered with the
>>> https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
>>> script.
>>>
>>> Signed-off-by: Ulf Magnusson <[email protected]>
>>
>> This should probaly be replaced with
>> select GPIOLIB
>>
>> Yours,
>> Linus Walleij
>
> That'd make more sense if GPIOLIB is actually used. The select has
> been a no-op since mid-2016 at least, but maybe something else selects
> GPIOLIB.
>
> Cheers,
> Ulf

Just replace it with your own patch once you've figured out the proper
fix. More efficient than having a middle man making (sometimes poor)
guesses.

Cheers,
Ulf

2018-02-06 09:47:44

by Linus Walleij

[permalink] [raw]
Subject: Re: [PATCH 07/20] riscv: Remove ARCH_WANT_OPTIONAL_GPIOLIB select

On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <[email protected]> wrote:

> The ARCH_WANT_OPTIONAL_GPIOLIB symbol was removed in commit 65053e1a7743
> ("gpio: delete ARCH_[WANTS_OPTIONAL|REQUIRE]_GPIOLIB"). GPIOLIB should
> just be selected explicitly if needed.
>
> Remove the ARCH_WANT_OPTIONAL_GPIOLIB select from RISCV.
>
> See commit 0145071b3314 ("x86: Do away with
> ARCH_[WANT_OPTIONAL|REQUIRE]_GPIOLIB") and commit da9a1c6767 ("arm64: do
> away with ARCH_[WANT_OPTIONAL|REQUIRE]_GPIOLIB") as well.
>
> Discovered with the
> https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
> script.
>
> Signed-off-by: Ulf Magnusson <[email protected]>

Reviewed-by: Linus Walleij <[email protected]>

RISCV maintainers: please apply and carry this patch.

Yours,
Linus Walleij

2018-02-06 09:50:52

by Linus Walleij

[permalink] [raw]
Subject: Re: [PATCH 07/20] riscv: Remove ARCH_WANT_OPTIONAL_GPIOLIB select

On Tue, Feb 6, 2018 at 12:49 AM, Ulf Magnusson <[email protected]> wrote:
> On Tue, Feb 6, 2018 at 12:38 AM, Ulf Magnusson <[email protected]> wrote:
>> On Tue, Feb 6, 2018 at 12:25 AM, Linus Walleij <[email protected]> wrote:
>>> On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <[email protected]> wrote:
>>>
>>>> The ARCH_WANT_OPTIONAL_GPIOLIB symbol was removed in commit 65053e1a7743
>>>> ("gpio: delete ARCH_[WANTS_OPTIONAL|REQUIRE]_GPIOLIB"). GPIOLIB should
>>>> just be selected explicitly if needed.
>>>>
>>>> Remove the ARCH_WANT_OPTIONAL_GPIOLIB select from RISCV.
>>>>
>>>> See commit 0145071b3314 ("x86: Do away with
>>>> ARCH_[WANT_OPTIONAL|REQUIRE]_GPIOLIB") and commit da9a1c6767 ("arm64: do
>>>> away with ARCH_[WANT_OPTIONAL|REQUIRE]_GPIOLIB") as well.
>>>>
>>>> Discovered with the
>>>> https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
>>>> script.
>>>>
>>>> Signed-off-by: Ulf Magnusson <[email protected]>
>>>
>>> This should probaly be replaced with
>>> select GPIOLIB
>>>
>>> Yours,
>>> Linus Walleij
>>
>> That'd make more sense if GPIOLIB is actually used. The select has
>> been a no-op since mid-2016 at least, but maybe something else selects
>> GPIOLIB.
>>
>> Cheers,
>> Ulf
>
> Just replace it with your own patch once you've figured out the proper
> fix. More efficient than having a middle man making (sometimes poor)
> guesses.

I think it's just a side effect of the new RISCV arch being developed
outside of the mainline kernel. And they may have had a gpio driver
outside of the mainline as well.

Normally we don't encourage out-of-tree development, but new archs
are a bit of an exception since they have to be merged as a big
piece of upfront code at some point...

Anyways, if they don't have any GPIO in their machine yet, let's
let the RISCV maintainers apply this.

Yours,
Linus Walleij

2018-02-06 12:09:25

by Mark Brown

[permalink] [raw]
Subject: Re: [PATCH v3 03/20] kconfig: Remove leftover references to AVR32 symbol

On Mon, Feb 05, 2018 at 08:05:39PM +0100, Ulf Magnusson wrote:
> 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]>


Attachments:
(No filename) (682.00 B)
signature.asc (499.00 B)
Download all attachments

2018-02-06 16:10:55

by Rob Herring (Arm)

[permalink] [raw]
Subject: Re: [PATCH 10/20] lib/Kconfig: Remove leftover select of GENERIC_IO

On Sun, Feb 4, 2018 at 7:21 PM, Ulf Magnusson <[email protected]> wrote:
> GENERIC_IO was removed by commit 9de8da47742b ("kconfig: kill off
> GENERIC_IO option"), but HAS_IOMEM still selects it.
>
> Remove the select of GENERIC_IO from HAS_IOMEM.
>
> Discovered with the
> https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
> script.
>
> Signed-off-by: Ulf Magnusson <[email protected]>
> ---
> lib/Kconfig | 1 -
> 1 file changed, 1 deletion(-)

There was another fix[1] for this but it seems to have gotten lost.

Acked-by: Rob Herring <[email protected]>

Rob

[1] https://lkml.org/lkml/2018/1/17/924

2018-02-06 16:18:28

by Robin Murphy

[permalink] [raw]
Subject: Re: [PATCH 10/20] lib/Kconfig: Remove leftover select of GENERIC_IO

On 05/02/18 01:21, Ulf Magnusson wrote:
> GENERIC_IO was removed by commit 9de8da47742b ("kconfig: kill off
> GENERIC_IO option"), but HAS_IOMEM still selects it.
>
> Remove the select of GENERIC_IO from HAS_IOMEM.
>
> Discovered with the
> https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
> script.

Since I also stumbled over this and tried to fix it,

Acked-by: Robin Murphy <[email protected]>

> Signed-off-by: Ulf Magnusson <[email protected]>
> ---
> lib/Kconfig | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/lib/Kconfig b/lib/Kconfig
> index e96089499371..8ce8bcad6eff 100644
> --- a/lib/Kconfig
> +++ b/lib/Kconfig
> @@ -396,7 +396,6 @@ config ASSOCIATIVE_ARRAY
> config HAS_IOMEM
> bool
> depends on !NO_IOMEM
> - select GENERIC_IO
> default y
>
> config HAS_IOPORT_MAP
>

2018-02-06 18:11:42

by Randy Dunlap

[permalink] [raw]
Subject: Re: [PATCH 16/20] auxdisplay: img-ascii-lcd: Remove MIPS_SEAD3 dep.

On 02/04/2018 05:58 PM, Ulf Magnusson wrote:
> On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <[email protected]> wrote:
>> The MIPS_SEAD3 symbol was removed in commit 64601cb1343f ("leds: Remove SEAD-3
>> driver").
>>
>> Remove the MIPS_SEAD3 dependency from IMG_ASCII_LCD.
>>
>> Discovered with the
>> https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
>> script.
>>
>> Signed-off-by: Ulf Magnusson <[email protected]>
>> ---
>> drivers/auxdisplay/Kconfig | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/auxdisplay/Kconfig b/drivers/auxdisplay/Kconfig
>> index 2c2ed9cf8796..3cba78d36eec 100644
>> --- a/drivers/auxdisplay/Kconfig
>> +++ b/drivers/auxdisplay/Kconfig
>> @@ -137,7 +137,7 @@ config CFAG12864B_RATE
>> config IMG_ASCII_LCD
>> tristate "Imagination Technologies ASCII LCD Display"
>> depends on HAS_IOMEM
>> - default y if MIPS_MALTA || MIPS_SEAD3
>> + default y if MIPS_MALTA
>> select SYSCON
>> help
>> Enable this to support the simple ASCII LCD displays found on
>> --
>> 2.14.1
>>
>
> The SYSCON symbol selected here is also globally undefined. Maybe
> MFD_SYSCON was intended.

Yes.

from 2018-JAN-18, Corentin Labbe:
https://marc.info/?l=linux-kernel&m=151630656607526&w=2

--
~Randy

2018-02-06 19:13:49

by Bjorn Helgaas

[permalink] [raw]
Subject: Re: [PATCH 20/20] x86/PCI: VMD: Fix malformed default

> x86/PCI: VMD: Fix malformed default

In the title, please include a clue about what default you're talking
about, e.g., mention Kconfig somehow.

On Mon, Feb 05, 2018 at 02:21:32AM +0100, Ulf Magnusson wrote:
> 'default N' should be 'default n', though they happen to have the same
> effect here, due to undefined symbols (N in this case) evaluating to n
> in a tristate sense.
>
> Remove the default from VMD instead of changing it. bool and tristate
> symbols implicitly default to n.
>
> Discovered with the
> https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
> script.
>
> Signed-off-by: Ulf Magnusson <[email protected]>
> ---
> drivers/pci/host/Kconfig | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/drivers/pci/host/Kconfig b/drivers/pci/host/Kconfig
> index a4ed7484d127..dc8a2a175f19 100644
> --- a/drivers/pci/host/Kconfig
> +++ b/drivers/pci/host/Kconfig
> @@ -215,7 +215,6 @@ config PCIE_TANGO_SMP8759
> config VMD
> depends on PCI_MSI && X86_64 && SRCU
> tristate "Intel Volume Management Device Driver"
> - default N
> ---help---
> Adds support for the Intel Volume Management Device (VMD). VMD is a
> secondary PCI host bridge that allows PCI Express root ports,
> --
> 2.14.1
>

2018-02-06 22:19:27

by Ulf Magnusson

[permalink] [raw]
Subject: Re: [PATCH 20/20] x86/PCI: VMD: Fix malformed default

On Tue, Feb 6, 2018 at 8:11 PM, Bjorn Helgaas <[email protected]> wrote:
>> x86/PCI: VMD: Fix malformed default
>
> In the title, please include a clue about what default you're talking
> about, e.g., mention Kconfig somehow.
>
> On Mon, Feb 05, 2018 at 02:21:32AM +0100, Ulf Magnusson wrote:
>> 'default N' should be 'default n', though they happen to have the same
>> effect here, due to undefined symbols (N in this case) evaluating to n
>> in a tristate sense.
>>
>> Remove the default from VMD instead of changing it. bool and tristate
>> symbols implicitly default to n.
>>
>> Discovered with the
>> https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
>> script.
>>
>> Signed-off-by: Ulf Magnusson <[email protected]>
>> ---
>> drivers/pci/host/Kconfig | 1 -
>> 1 file changed, 1 deletion(-)
>>
>> diff --git a/drivers/pci/host/Kconfig b/drivers/pci/host/Kconfig
>> index a4ed7484d127..dc8a2a175f19 100644
>> --- a/drivers/pci/host/Kconfig
>> +++ b/drivers/pci/host/Kconfig
>> @@ -215,7 +215,6 @@ config PCIE_TANGO_SMP8759
>> config VMD
>> depends on PCI_MSI && X86_64 && SRCU
>> tristate "Intel Volume Management Device Driver"
>> - default N
>> ---help---
>> Adds support for the Intel Volume Management Device (VMD). VMD is a
>> secondary PCI host bridge that allows PCI Express root ports,
>> --
>> 2.14.1
>>

Typoed the --in-reply-to. v2 is at https://lkml.org/lkml/2018/2/6/770

Cheers,
Ulf

2018-02-06 22:54:41

by Ulf Magnusson

[permalink] [raw]
Subject: [PATCH] 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(-)

diff --git a/drivers/video/console/Kconfig b/drivers/video/console/Kconfig
index 7f1f1fbcef9e..be63759b6027 100644
--- a/drivers/video/console/Kconfig
+++ b/drivers/video/console/Kconfig
@@ -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
--
2.14.1


2018-02-06 22:57:15

by Ulf Magnusson

[permalink] [raw]
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]>
---
drivers/spi/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/spi/Kconfig b/drivers/spi/Kconfig
index 603783976b81..c1e455d46c7f 100644
--- a/drivers/spi/Kconfig
+++ b/drivers/spi/Kconfig
@@ -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.
--
2.14.1


2018-02-06 22:58:48

by Ulf Magnusson

[permalink] [raw]
Subject: Re: [PATCH v3 03/20] kconfig: Remove leftover references to AVR32 symbol

On Mon, Feb 5, 2018 at 8:05 PM, Ulf Magnusson <[email protected]> wrote:
> 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(-)
>
> diff --git a/drivers/spi/Kconfig b/drivers/spi/Kconfig
> index 603783976b81..c1e455d46c7f 100644
> --- a/drivers/spi/Kconfig
> +++ b/drivers/spi/Kconfig
> @@ -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.
> diff --git a/drivers/video/console/Kconfig b/drivers/video/console/Kconfig
> index 7f1f1fbcef9e..be63759b6027 100644
> --- a/drivers/video/console/Kconfig
> +++ b/drivers/video/console/Kconfig
> @@ -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
> --
> 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

2018-02-06 23:00:48

by Ulf Magnusson

[permalink] [raw]
Subject: Re: [PATCH v3 03/20] kconfig: Remove leftover references to AVR32 symbol

On Tue, Feb 6, 2018 at 1:08 PM, Mark Brown <[email protected]> wrote:
> On Mon, Feb 05, 2018 at 08:05:39PM +0100, Ulf Magnusson wrote:
>> 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]>

Done

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

Cheers,
Ulf

2018-02-07 08:59:53

by Matthias Brugger

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



On 02/06/2018 11:55 PM, Ulf Magnusson wrote:
> 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]>

> drivers/spi/Kconfig | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/spi/Kconfig b/drivers/spi/Kconfig
> index 603783976b81..c1e455d46c7f 100644
> --- a/drivers/spi/Kconfig
> +++ b/drivers/spi/Kconfig
> @@ -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.
>

2018-02-07 09:00:55

by Matthias Brugger

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



On 02/07/2018 09:58 AM, Matthias Brugger wrote:
>
>
> On 02/06/2018 11:55 PM, Ulf Magnusson wrote:
>> 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.

>> drivers/spi/Kconfig | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/spi/Kconfig b/drivers/spi/Kconfig
>> index 603783976b81..c1e455d46c7f 100644
>> --- a/drivers/spi/Kconfig
>> +++ b/drivers/spi/Kconfig
>> @@ -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

2018-02-07 09:29:52

by Robert Jarzmik

[permalink] [raw]
Subject: Re: [PATCH 00/20] Remove references to undefined symbols and fix two potential bugs

Ulf Magnusson <[email protected]> writes:

> The symbol AC97_BUS_NEW in sound/ac97/Kconfig selects the globally
> undefined symbol AC97.
>
> It doesn't seem to simply be a misspelling of AC97_BUS, because then
> the dependencies of AC97_BUS_COMPAT would be impossible to satisfy.
You're correct.

I don't know what happened to me when coding this, but you can safely remove
this useless select statement.

Cheers.

--
Robert

2018-02-07 12:12:14

by Linus Walleij

[permalink] [raw]
Subject: Re: [PATCH 14/20] iio: gyro: mpu3050: Remove INPUT_MPU3050 dep. from I2C

On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <[email protected]> wrote:

> The INPUT_MPU3050 symbol was removed in commit b1fe0cf06f92 ("Input:
> delete MPU3050 driver").
>
> Remove the negative dependency on it from the MPU3050_I2C symbol. It is
> always satisfied now.
>
> Discovered with the
> https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
> script.
>
> Signed-off-by: Ulf Magnusson <[email protected]>

Reviewed-by: Linus Walleij <[email protected]>

Jonathan, please pick this up to IIO!

Yours,
Linus Walleij

2018-02-08 02:46:03

by Ulf Magnusson

[permalink] [raw]
Subject: [PATCH v2] 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(-)

diff --git a/drivers/spi/Kconfig b/drivers/spi/Kconfig
index 603783976b81..6fb0347a24f2 100644
--- a/drivers/spi/Kconfig
+++ b/drivers/spi/Kconfig
@@ -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"
--
2.14.1


2018-02-08 15:38:25

by Miguel Ojeda

[permalink] [raw]
Subject: Re: [PATCH 16/20] auxdisplay: img-ascii-lcd: Remove MIPS_SEAD3 dep.

On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <[email protected]> wrote:
> The MIPS_SEAD3 symbol was removed in commit 64601cb1343f ("leds: Remove SEAD-3
> driver").

Commit 64601cb1343f did not remove the MIPS_SEAD3 symbol, it seemed to
a few months before, in 3f5f0a4475e1 ("MIPS: generic: Convert SEAD-3
to a generic board"). Can you please amend the description?

Thanks!
Miguel

>
> Remove the MIPS_SEAD3 dependency from IMG_ASCII_LCD.
>
> Discovered with the
> https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
> script.
>
> Signed-off-by: Ulf Magnusson <[email protected]>
> ---
> drivers/auxdisplay/Kconfig | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/auxdisplay/Kconfig b/drivers/auxdisplay/Kconfig
> index 2c2ed9cf8796..3cba78d36eec 100644
> --- a/drivers/auxdisplay/Kconfig
> +++ b/drivers/auxdisplay/Kconfig
> @@ -137,7 +137,7 @@ config CFAG12864B_RATE
> config IMG_ASCII_LCD
> tristate "Imagination Technologies ASCII LCD Display"
> depends on HAS_IOMEM
> - default y if MIPS_MALTA || MIPS_SEAD3
> + default y if MIPS_MALTA
> select SYSCON
> help
> Enable this to support the simple ASCII LCD displays found on
> --
> 2.14.1
>

2018-02-08 16:34:55

by Matthias Brugger

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



On 02/08/2018 03:44 AM, Ulf Magnusson wrote:
> The AVR32 symbol was removed in commit 26202873bb51 ("avr32: remove
> support for AVR32 architecture").

You forgot the 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(-)
>
> diff --git a/drivers/spi/Kconfig b/drivers/spi/Kconfig
> index 603783976b81..6fb0347a24f2 100644
> --- a/drivers/spi/Kconfig
> +++ b/drivers/spi/Kconfig
> @@ -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"
>

2018-02-08 17:36:59

by James Hogan

[permalink] [raw]
Subject: Re: [PATCH 16/20] auxdisplay: img-ascii-lcd: Remove MIPS_SEAD3 dep.

On 5 February 2018 at 01:21, Ulf Magnusson <[email protected]> wrote:
> The MIPS_SEAD3 symbol was removed in commit 64601cb1343f ("leds: Remove SEAD-3
> driver").
>
> Remove the MIPS_SEAD3 dependency from IMG_ASCII_LCD.

Its not a dependency, just a default (presumably so that existing
kernel configs get updated automatically).

Note that sead3 now uses MIPS_GENERIC (or more specifically
CONFIG_LEGACY_BOARD_SEAD3), but this driver is enabled in the config
snippets, e.g.:
arch/mips/configs/generic/board-sead-3.config
arch/mips/configs/generic/board-boston.config

so indeed there is no real need to replace MIPS_SEAD3 with anything here.

Cheers
James

>
> Discovered with the
> https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
> script.
>
> Signed-off-by: Ulf Magnusson <[email protected]>
> ---
> drivers/auxdisplay/Kconfig | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/auxdisplay/Kconfig b/drivers/auxdisplay/Kconfig
> index 2c2ed9cf8796..3cba78d36eec 100644
> --- a/drivers/auxdisplay/Kconfig
> +++ b/drivers/auxdisplay/Kconfig
> @@ -137,7 +137,7 @@ config CFAG12864B_RATE
> config IMG_ASCII_LCD
> tristate "Imagination Technologies ASCII LCD Display"
> depends on HAS_IOMEM
> - default y if MIPS_MALTA || MIPS_SEAD3
> + default y if MIPS_MALTA
> select SYSCON
> help
> Enable this to support the simple ASCII LCD displays found on
> --
> 2.14.1
>



--
James Hogan

2018-02-08 18:35:20

by Palmer Dabbelt

[permalink] [raw]
Subject: Re: [PATCH 07/20] riscv: Remove ARCH_WANT_OPTIONAL_GPIOLIB select

On Sun, 04 Feb 2018 17:21:19 PST (-0800), [email protected] wrote:
> The ARCH_WANT_OPTIONAL_GPIOLIB symbol was removed in commit 65053e1a7743
> ("gpio: delete ARCH_[WANTS_OPTIONAL|REQUIRE]_GPIOLIB"). GPIOLIB should
> just be selected explicitly if needed.
>
> Remove the ARCH_WANT_OPTIONAL_GPIOLIB select from RISCV.
>
> See commit 0145071b3314 ("x86: Do away with
> ARCH_[WANT_OPTIONAL|REQUIRE]_GPIOLIB") and commit da9a1c6767 ("arm64: do
> away with ARCH_[WANT_OPTIONAL|REQUIRE]_GPIOLIB") as well.
>
> Discovered with the
> https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
> script.
>
> Signed-off-by: Ulf Magnusson <[email protected]>
> ---
> arch/riscv/Kconfig | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/arch/riscv/Kconfig b/arch/riscv/Kconfig
> index ff69c77b9e78..716e90e60e5c 100644
> --- a/arch/riscv/Kconfig
> +++ b/arch/riscv/Kconfig
> @@ -19,7 +19,6 @@ config RISCV
> select GENERIC_STRNLEN_USER
> select GENERIC_SMP_IDLE_THREAD
> select GENERIC_ATOMIC64 if !64BIT || !RISCV_ISA_A
> - select ARCH_WANT_OPTIONAL_GPIOLIB
> select HAVE_MEMBLOCK
> select HAVE_MEMBLOCK_NODE_MAP
> select HAVE_DMA_API_DEBUG

Thanks!

(If you want these through my tree, just say something.)

Reviewed-by: Palmer Dabbelt <[email protected]>

2018-02-08 18:35:28

by Palmer Dabbelt

[permalink] [raw]
Subject: Re: [PATCH 06/20] riscv: Remove ARCH_HAS_ATOMIC64_DEC_IF_POSITIVE select

On Sun, 04 Feb 2018 17:21:18 PST (-0800), [email protected] wrote:
> The ARCH_HAS_ATOMIC64_DEC_IF_POSITIVE symbol was removed in
> commit 51a021244b9d ("atomic64: no need for
> CONFIG_ARCH_HAS_ATOMIC64_DEC_IF_POSITIVE").
>
> Remove the ARCH_HAS_ATOMIC64_DEC_IS_POSITIVE select from RISCV.
>
> Discovered with the
> https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
> script.
>
> Signed-off-by: Ulf Magnusson <[email protected]>
> ---
> arch/riscv/Kconfig | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/arch/riscv/Kconfig b/arch/riscv/Kconfig
> index b6722c246d9c..ff69c77b9e78 100644
> --- a/arch/riscv/Kconfig
> +++ b/arch/riscv/Kconfig
> @@ -8,7 +8,6 @@ config RISCV
> select OF
> select OF_EARLY_FLATTREE
> select OF_IRQ
> - select ARCH_HAS_ATOMIC64_DEC_IF_POSITIVE
> select ARCH_WANT_FRAME_POINTERS
> select CLONE_BACKWARDS
> select COMMON_CLK

Thanks!

Reviewed-by: Palmer Dabbelt <[email protected]>

2018-02-08 20:52:41

by Palmer Dabbelt

[permalink] [raw]
Subject: Re: [PATCH 07/20] riscv: Remove ARCH_WANT_OPTIONAL_GPIOLIB select

On Sun, 04 Feb 2018 17:30:57 PST (-0800), [email protected] wrote:
> On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <[email protected]> wrote:
>> The ARCH_WANT_OPTIONAL_GPIOLIB symbol was removed in commit 65053e1a7743
>> ("gpio: delete ARCH_[WANTS_OPTIONAL|REQUIRE]_GPIOLIB"). GPIOLIB should
>> just be selected explicitly if needed.
>>
>> Remove the ARCH_WANT_OPTIONAL_GPIOLIB select from RISCV.
>>
>> See commit 0145071b3314 ("x86: Do away with
>> ARCH_[WANT_OPTIONAL|REQUIRE]_GPIOLIB") and commit da9a1c6767 ("arm64: do
>> away with ARCH_[WANT_OPTIONAL|REQUIRE]_GPIOLIB") as well.
>>
>> Discovered with the
>> https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
>> script.
>>
>> Signed-off-by: Ulf Magnusson <[email protected]>
>> ---
>> arch/riscv/Kconfig | 1 -
>> 1 file changed, 1 deletion(-)
>>
>> diff --git a/arch/riscv/Kconfig b/arch/riscv/Kconfig
>> index ff69c77b9e78..716e90e60e5c 100644
>> --- a/arch/riscv/Kconfig
>> +++ b/arch/riscv/Kconfig
>> @@ -19,7 +19,6 @@ config RISCV
>> select GENERIC_STRNLEN_USER
>> select GENERIC_SMP_IDLE_THREAD
>> select GENERIC_ATOMIC64 if !64BIT || !RISCV_ISA_A
>> - select ARCH_WANT_OPTIONAL_GPIOLIB
>> select HAVE_MEMBLOCK
>> select HAVE_MEMBLOCK_NODE_MAP
>> select HAVE_DMA_API_DEBUG
>> --
>> 2.14.1
>>
>
> There's also a select of a globally undefined symbol RISCV_IRQ_INTC
> here. I couldn't dig up any historical or unmerged definition for it
> either. Googling just gets you the select.

It looks like this slipped through, the symbol has been renamed RISCV_INTC. A
patch to remove it is

Reviewed-by: Palmer Dabbelt <[email protected]>

(and like the other ones, if you want it through my tree just say something)

2018-02-08 22:04:03

by Ulf Magnusson

[permalink] [raw]
Subject: [PATCH v3] 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(-)

diff --git a/drivers/spi/Kconfig b/drivers/spi/Kconfig
index 603783976b81..6fb0347a24f2 100644
--- a/drivers/spi/Kconfig
+++ b/drivers/spi/Kconfig
@@ -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"
--
2.14.1


2018-02-08 22:11:13

by Ulf Magnusson

[permalink] [raw]
Subject: Re: [PATCH 06/20] riscv: Remove ARCH_HAS_ATOMIC64_DEC_IF_POSITIVE select

On Thu, Feb 8, 2018 at 7:34 PM, Palmer Dabbelt <[email protected]> wrote:
> On Sun, 04 Feb 2018 17:21:18 PST (-0800), [email protected] wrote:
>>
>> The ARCH_HAS_ATOMIC64_DEC_IF_POSITIVE symbol was removed in
>> commit 51a021244b9d ("atomic64: no need for
>> CONFIG_ARCH_HAS_ATOMIC64_DEC_IF_POSITIVE").
>>
>> Remove the ARCH_HAS_ATOMIC64_DEC_IS_POSITIVE select from RISCV.
>>
>> Discovered with the
>>
>> https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
>> script.
>>
>> Signed-off-by: Ulf Magnusson <[email protected]>
>> ---
>> arch/riscv/Kconfig | 1 -
>> 1 file changed, 1 deletion(-)
>>
>> diff --git a/arch/riscv/Kconfig b/arch/riscv/Kconfig
>> index b6722c246d9c..ff69c77b9e78 100644
>> --- a/arch/riscv/Kconfig
>> +++ b/arch/riscv/Kconfig
>> @@ -8,7 +8,6 @@ config RISCV
>> select OF
>> select OF_EARLY_FLATTREE
>> select OF_IRQ
>> - select ARCH_HAS_ATOMIC64_DEC_IF_POSITIVE
>> select ARCH_WANT_FRAME_POINTERS
>> select CLONE_BACKWARDS
>> select COMMON_CLK
>
>
> Thanks!
>
> Reviewed-by: Palmer Dabbelt <[email protected]>

I didn't have a particular tree in mind for these patches by the way,
so feel free to pull it in yourselves.

Cheers,
Ulf

2018-02-08 22:56:52

by Ulf Magnusson

[permalink] [raw]
Subject: [PATCH] riscv: kconfig: Remove RISCV_IRQ_INTC select

The RISCV_IRQ_INTC configuration symbol is undefined, but RISCV selects
it. Quoting Palmer Dabbelt:

It looks like this slipped through, the symbol has been renamed
RISCV_INTC.

No RISCV_INTC configuration symbol has been merged either. Just remove
the RISCV_IRQ_INTC select for now.

Signed-off-by: Ulf Magnusson <[email protected]>
---
arch/riscv/Kconfig | 1 -
1 file changed, 1 deletion(-)

diff --git a/arch/riscv/Kconfig b/arch/riscv/Kconfig
index b6722c246d9c..2be44b6651c5 100644
--- a/arch/riscv/Kconfig
+++ b/arch/riscv/Kconfig
@@ -34,7 +34,6 @@ config RISCV
select HAVE_ARCH_TRACEHOOK
select MODULES_USE_ELF_RELA if MODULES
select THREAD_INFO_IN_TASK
- select RISCV_IRQ_INTC
select RISCV_TIMER

config MMU
--
2.14.1


2018-02-08 22:58:11

by Ulf Magnusson

[permalink] [raw]
Subject: Re: [PATCH 07/20] riscv: Remove ARCH_WANT_OPTIONAL_GPIOLIB select

On Thu, Feb 08, 2018 at 10:34:19AM -0800, Palmer Dabbelt wrote:
> On Sun, 04 Feb 2018 17:21:19 PST (-0800), [email protected] wrote:
> > The ARCH_WANT_OPTIONAL_GPIOLIB symbol was removed in commit 65053e1a7743
> > ("gpio: delete ARCH_[WANTS_OPTIONAL|REQUIRE]_GPIOLIB"). GPIOLIB should
> > just be selected explicitly if needed.
> >
> > Remove the ARCH_WANT_OPTIONAL_GPIOLIB select from RISCV.
> >
> > See commit 0145071b3314 ("x86: Do away with
> > ARCH_[WANT_OPTIONAL|REQUIRE]_GPIOLIB") and commit da9a1c6767 ("arm64: do
> > away with ARCH_[WANT_OPTIONAL|REQUIRE]_GPIOLIB") as well.
> >
> > Discovered with the
> > https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
> > script.
> >
> > Signed-off-by: Ulf Magnusson <[email protected]>
> > ---
> > arch/riscv/Kconfig | 1 -
> > 1 file changed, 1 deletion(-)
> >
> > diff --git a/arch/riscv/Kconfig b/arch/riscv/Kconfig
> > index ff69c77b9e78..716e90e60e5c 100644
> > --- a/arch/riscv/Kconfig
> > +++ b/arch/riscv/Kconfig
> > @@ -19,7 +19,6 @@ config RISCV
> > select GENERIC_STRNLEN_USER
> > select GENERIC_SMP_IDLE_THREAD
> > select GENERIC_ATOMIC64 if !64BIT || !RISCV_ISA_A
> > - select ARCH_WANT_OPTIONAL_GPIOLIB
> > select HAVE_MEMBLOCK
> > select HAVE_MEMBLOCK_NODE_MAP
> > select HAVE_DMA_API_DEBUG
>
> Thanks!
>
> (If you want these through my tree, just say something.)

I didn't have a particular tree in mind for these patches, so feel free
to take it.

>
> Reviewed-by: Palmer Dabbelt <[email protected]>

Cheers,
Ulf

2018-02-08 23:00:23

by Palmer Dabbelt

[permalink] [raw]
Subject: Re: [PATCH 07/20] riscv: Remove ARCH_WANT_OPTIONAL_GPIOLIB select

On Tue, 06 Feb 2018 01:47:00 PST (-0800), [email protected] wrote:
> On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <[email protected]> wrote:
>
>> The ARCH_WANT_OPTIONAL_GPIOLIB symbol was removed in commit 65053e1a7743
>> ("gpio: delete ARCH_[WANTS_OPTIONAL|REQUIRE]_GPIOLIB"). GPIOLIB should
>> just be selected explicitly if needed.
>>
>> Remove the ARCH_WANT_OPTIONAL_GPIOLIB select from RISCV.
>>
>> See commit 0145071b3314 ("x86: Do away with
>> ARCH_[WANT_OPTIONAL|REQUIRE]_GPIOLIB") and commit da9a1c6767 ("arm64: do
>> away with ARCH_[WANT_OPTIONAL|REQUIRE]_GPIOLIB") as well.
>>
>> Discovered with the
>> https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
>> script.
>>
>> Signed-off-by: Ulf Magnusson <[email protected]>
>
> Reviewed-by: Linus Walleij <[email protected]>
>
> RISCV maintainers: please apply and carry this patch.

OK, I'll atke it into my tree. Thanks!

2018-02-08 23:16:48

by Ulf Magnusson

[permalink] [raw]
Subject: [PATCH] ALSA: ac97: kconfig: Remove select of undefined symbol AC97

The AC97_BUS_NEW Kconfig symbol selects the globally undefined symbol
AC97.

Robert Jarzmik confirmed in https://lkml.org/lkml/2018/2/7/96 that the
select was put in by mistake and can be safely removed, with no other
changes required. Remove it.

Signed-off-by: Ulf Magnusson <[email protected]>
---
sound/ac97/Kconfig | 1 -
1 file changed, 1 deletion(-)

diff --git a/sound/ac97/Kconfig b/sound/ac97/Kconfig
index f8a64e15e5bf..baa5f8ef89d2 100644
--- a/sound/ac97/Kconfig
+++ b/sound/ac97/Kconfig
@@ -5,7 +5,6 @@

config AC97_BUS_NEW
tristate
- select AC97
help
This is the new AC97 bus type, successor of AC97_BUS. The ported
drivers which benefit from the AC97 automatic probing should "select"
--
2.14.1


2018-02-08 23:22:20

by Ulf Magnusson

[permalink] [raw]
Subject: Re: [PATCH] ALSA: ac97: kconfig: Remove select of undefined symbol AC97

On Fri, Feb 09, 2018 at 12:15:36AM +0100, Ulf Magnusson wrote:
> The AC97_BUS_NEW Kconfig symbol selects the globally undefined symbol
> AC97.
>
> Robert Jarzmik confirmed in https://lkml.org/lkml/2018/2/7/96 that the
> select was put in by mistake and can be safely removed, with no other
> changes required. Remove it.
>
> Signed-off-by: Ulf Magnusson <[email protected]>
> ---
> sound/ac97/Kconfig | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/sound/ac97/Kconfig b/sound/ac97/Kconfig
> index f8a64e15e5bf..baa5f8ef89d2 100644
> --- a/sound/ac97/Kconfig
> +++ b/sound/ac97/Kconfig
> @@ -5,7 +5,6 @@
>
> config AC97_BUS_NEW
> tristate
> - select AC97
> help
> This is the new AC97 bus type, successor of AC97_BUS. The ported
> drivers which benefit from the AC97 automatic probing should "select"
> --
> 2.14.1
>

I didn't have a particular tree in mind for this patchset by the way.
Feel free to take the patch in wherever it makes sense.

Cheers,
Ulf

2018-02-10 14:15:21

by Ulf Magnusson

[permalink] [raw]
Subject: [PATCH v2] auxdisplay: img-ascii-lcd: kconfig: Remove MIPS_SEAD3 reference

Commit 3f5f0a4475e1 ("MIPS: generic: Convert SEAD-3 to a generic board")
removed the MIPS_SEAD3 symbol and moved the setting of IMG_ASCII_LCD to
the board-sead-3.config defconfig file, but IMG_ASCII_LCD still
references the removed MIPS_SEAD3 symbol.

Remove the reference.

Discovered with the
https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
script.

Signed-off-by: Ulf Magnusson <[email protected]>
---
Changes in v2:

- The wrong commit was referenced in the commit message.

- Mention that IMG_ASCII_LCD is now set via the board-sead-3.config defconfig
file.

- Clarify that Kconfig is involved. The previous commit title was
"auxdisplay: img-ascii-lcd: Remove MIPS_SEAD3 reference".


drivers/auxdisplay/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/auxdisplay/Kconfig b/drivers/auxdisplay/Kconfig
index 2c2ed9cf8796..3cba78d36eec 100644
--- a/drivers/auxdisplay/Kconfig
+++ b/drivers/auxdisplay/Kconfig
@@ -137,7 +137,7 @@ config CFAG12864B_RATE
config IMG_ASCII_LCD
tristate "Imagination Technologies ASCII LCD Display"
depends on HAS_IOMEM
- default y if MIPS_MALTA || MIPS_SEAD3
+ default y if MIPS_MALTA
select SYSCON
help
Enable this to support the simple ASCII LCD displays found on
--
2.14.1


2018-02-10 14:20:37

by Ulf Magnusson

[permalink] [raw]
Subject: Re: [PATCH 16/20] auxdisplay: img-ascii-lcd: Remove MIPS_SEAD3 dep.

On Thu, Feb 08, 2018 at 05:35:30PM +0000, James Hogan wrote:
> On 5 February 2018 at 01:21, Ulf Magnusson <[email protected]> wrote:
> > The MIPS_SEAD3 symbol was removed in commit 64601cb1343f ("leds: Remove SEAD-3
> > driver").
> >
> > Remove the MIPS_SEAD3 dependency from IMG_ASCII_LCD.
>
> Its not a dependency, just a default (presumably so that existing
> kernel configs get updated automatically).

Note that it appears in the default's condition rather than as part of
the default value.

Undefined Kconfig symbols always evaluate to n in a tristate state,
regardless of whether they're given a user value in a .config file or
not, so this is a no-op as far as Kconfig in concerned.

The motivation in the original commit message was a bit lackluster
though. I've submitted a v2.

>
> Note that sead3 now uses MIPS_GENERIC (or more specifically
> CONFIG_LEGACY_BOARD_SEAD3), but this driver is enabled in the config
> snippets, e.g.:
> arch/mips/configs/generic/board-sead-3.config
> arch/mips/configs/generic/board-boston.config
>
> so indeed there is no real need to replace MIPS_SEAD3 with anything here.
>
> Cheers
> James
>
> >
> > Discovered with the
> > https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
> > script.
> >
> > Signed-off-by: Ulf Magnusson <[email protected]>
> > ---
> > drivers/auxdisplay/Kconfig | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/auxdisplay/Kconfig b/drivers/auxdisplay/Kconfig
> > index 2c2ed9cf8796..3cba78d36eec 100644
> > --- a/drivers/auxdisplay/Kconfig
> > +++ b/drivers/auxdisplay/Kconfig
> > @@ -137,7 +137,7 @@ config CFAG12864B_RATE
> > config IMG_ASCII_LCD
> > tristate "Imagination Technologies ASCII LCD Display"
> > depends on HAS_IOMEM
> > - default y if MIPS_MALTA || MIPS_SEAD3
> > + default y if MIPS_MALTA
> > select SYSCON
> > help
> > Enable this to support the simple ASCII LCD displays found on
> > --
> > 2.14.1
> >
>
>
>
> --
> James Hogan

Cheers,
Ulf

2018-02-11 16:42:45

by Miguel Ojeda

[permalink] [raw]
Subject: Re: [PATCH v2] auxdisplay: img-ascii-lcd: kconfig: Remove MIPS_SEAD3 reference

On Sat, Feb 10, 2018 at 3:14 PM, Ulf Magnusson <[email protected]> wrote:
> Commit 3f5f0a4475e1 ("MIPS: generic: Convert SEAD-3 to a generic board")
> removed the MIPS_SEAD3 symbol and moved the setting of IMG_ASCII_LCD to
> the board-sead-3.config defconfig file, but IMG_ASCII_LCD still
> references the removed MIPS_SEAD3 symbol.
>
> Remove the reference.
>
> Discovered with the
> https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
> script.
>
> Signed-off-by: Ulf Magnusson <[email protected]>
> ---
> Changes in v2:
>
> - The wrong commit was referenced in the commit message.
>
> - Mention that IMG_ASCII_LCD is now set via the board-sead-3.config defconfig
> file.
>
> - Clarify that Kconfig is involved. The previous commit title was
> "auxdisplay: img-ascii-lcd: Remove MIPS_SEAD3 reference".
>

Thanks for the new version!

Signed-off-by: Miguel Ojeda <[email protected]>

>
> drivers/auxdisplay/Kconfig | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/auxdisplay/Kconfig b/drivers/auxdisplay/Kconfig
> index 2c2ed9cf8796..3cba78d36eec 100644
> --- a/drivers/auxdisplay/Kconfig
> +++ b/drivers/auxdisplay/Kconfig
> @@ -137,7 +137,7 @@ config CFAG12864B_RATE
> config IMG_ASCII_LCD
> tristate "Imagination Technologies ASCII LCD Display"
> depends on HAS_IOMEM
> - default y if MIPS_MALTA || MIPS_SEAD3
> + default y if MIPS_MALTA
> select SYSCON
> help
> Enable this to support the simple ASCII LCD displays found on
> --
> 2.14.1
>

2018-02-12 09:20:13

by Takashi Iwai

[permalink] [raw]
Subject: Re: [PATCH] ALSA: ac97: kconfig: Remove select of undefined symbol AC97

On Fri, 09 Feb 2018 00:20:21 +0100,
Ulf Magnusson wrote:
>
> On Fri, Feb 09, 2018 at 12:15:36AM +0100, Ulf Magnusson wrote:
> > The AC97_BUS_NEW Kconfig symbol selects the globally undefined symbol
> > AC97.
> >
> > Robert Jarzmik confirmed in https://lkml.org/lkml/2018/2/7/96 that the
> > select was put in by mistake and can be safely removed, with no other
> > changes required. Remove it.
> >
> > Signed-off-by: Ulf Magnusson <[email protected]>
> > ---
> > sound/ac97/Kconfig | 1 -
> > 1 file changed, 1 deletion(-)
> >
> > diff --git a/sound/ac97/Kconfig b/sound/ac97/Kconfig
> > index f8a64e15e5bf..baa5f8ef89d2 100644
> > --- a/sound/ac97/Kconfig
> > +++ b/sound/ac97/Kconfig
> > @@ -5,7 +5,6 @@
> >
> > config AC97_BUS_NEW
> > tristate
> > - select AC97
> > help
> > This is the new AC97 bus type, successor of AC97_BUS. The ported
> > drivers which benefit from the AC97 automatic probing should "select"
> > --
> > 2.14.1
> >
>
> I didn't have a particular tree in mind for this patchset by the way.
> Feel free to take the patch in wherever it makes sense.

Applied now, thanks.


Takashi

2018-02-12 14:49:30

by Miguel Ojeda

[permalink] [raw]
Subject: Re: [PATCH v2] auxdisplay: img-ascii-lcd: kconfig: Remove MIPS_SEAD3 reference

On Sun, Feb 11, 2018 at 5:40 PM, Miguel Ojeda
<[email protected]> wrote:
> On Sat, Feb 10, 2018 at 3:14 PM, Ulf Magnusson <[email protected]> wrote:
>> Commit 3f5f0a4475e1 ("MIPS: generic: Convert SEAD-3 to a generic board")
>> removed the MIPS_SEAD3 symbol and moved the setting of IMG_ASCII_LCD to
>> the board-sead-3.config defconfig file, but IMG_ASCII_LCD still
>> references the removed MIPS_SEAD3 symbol.
>>
>> Remove the reference.
>>
>> Discovered with the
>> https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
>> script.
>>
>> Signed-off-by: Ulf Magnusson <[email protected]>
>> ---
>> Changes in v2:
>>
>> - The wrong commit was referenced in the commit message.
>>
>> - Mention that IMG_ASCII_LCD is now set via the board-sead-3.config defconfig
>> file.
>>
>> - Clarify that Kconfig is involved. The previous commit title was
>> "auxdisplay: img-ascii-lcd: Remove MIPS_SEAD3 reference".
>>
>
> Thanks for the new version!
>
> Signed-off-by: Miguel Ojeda <[email protected]>

To clarify, I am picking this patch up for my queue (sorry for any confusion!).

>
>>
>> drivers/auxdisplay/Kconfig | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/auxdisplay/Kconfig b/drivers/auxdisplay/Kconfig
>> index 2c2ed9cf8796..3cba78d36eec 100644
>> --- a/drivers/auxdisplay/Kconfig
>> +++ b/drivers/auxdisplay/Kconfig
>> @@ -137,7 +137,7 @@ config CFAG12864B_RATE
>> config IMG_ASCII_LCD
>> tristate "Imagination Technologies ASCII LCD Display"
>> depends on HAS_IOMEM
>> - default y if MIPS_MALTA || MIPS_SEAD3
>> + default y if MIPS_MALTA
>> select SYSCON
>> help
>> Enable this to support the simple ASCII LCD displays found on
>> --
>> 2.14.1
>>

2018-02-12 16:49:30

by Mark Brown

[permalink] [raw]
Subject: Applied "spi: kconfig: Remove AVR32 dep. from SPI_ATMEL" to the spi tree

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

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(-)

diff --git a/drivers/spi/Kconfig b/drivers/spi/Kconfig
index 603783976b81..6fb0347a24f2 100644
--- a/drivers/spi/Kconfig
+++ b/drivers/spi/Kconfig
@@ -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"
--
2.16.1


2018-02-13 02:09:30

by Palmer Dabbelt

[permalink] [raw]
Subject: Re: [PATCH] riscv: kconfig: Remove RISCV_IRQ_INTC select

On Thu, 08 Feb 2018 14:54:46 PST (-0800), [email protected] wrote:
> The RISCV_IRQ_INTC configuration symbol is undefined, but RISCV selects
> it. Quoting Palmer Dabbelt:
>
> It looks like this slipped through, the symbol has been renamed
> RISCV_INTC.
>
> No RISCV_INTC configuration symbol has been merged either. Just remove
> the RISCV_IRQ_INTC select for now.
>
> Signed-off-by: Ulf Magnusson <[email protected]>
> ---
> arch/riscv/Kconfig | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/arch/riscv/Kconfig b/arch/riscv/Kconfig
> index b6722c246d9c..2be44b6651c5 100644
> --- a/arch/riscv/Kconfig
> +++ b/arch/riscv/Kconfig
> @@ -34,7 +34,6 @@ config RISCV
> select HAVE_ARCH_TRACEHOOK
> select MODULES_USE_ELF_RELA if MODULES
> select THREAD_INFO_IN_TASK
> - select RISCV_IRQ_INTC
> select RISCV_TIMER
>
> config MMU

Thanks! I'll take this (and the other relevant ones) into the RISC-V tree.

2018-02-13 15:45:32

by Gregory CLEMENT

[permalink] [raw]
Subject: Re: [PATCH 01/20] ARM: mvebu: Fix broken PL310_ERRATA_753970 selects

Hi Ulf,

On lun., févr. 05 2018, Ulf Magnusson <[email protected]> wrote:

> The MACH_ARMADA_375 and MACH_ARMADA_38X boards select ARM_ERRATA_753970,
> but it was renamed to PL310_ERRATA_753970 by commit fa0ce4035d48 ("ARM:
> 7162/1: errata: tidy up Kconfig options for PL310 errata workarounds").
>
> Fix the selects to use the new name.
>
> Discovered with the
> https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
> script.

Applied on mvebu/fixes with the "Fixes": and "CC:stable" flags

Thanks,

Gregory

>
> Signed-off-by: Ulf Magnusson <[email protected]>
> ---
> arch/arm/mach-mvebu/Kconfig | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm/mach-mvebu/Kconfig b/arch/arm/mach-mvebu/Kconfig
> index 6b32dc527edc..2c20599cc350 100644
> --- a/arch/arm/mach-mvebu/Kconfig
> +++ b/arch/arm/mach-mvebu/Kconfig
> @@ -41,7 +41,7 @@ config MACH_ARMADA_375
> depends on ARCH_MULTI_V7
> select ARMADA_370_XP_IRQ
> select ARM_ERRATA_720789
> - select ARM_ERRATA_753970
> + select PL310_ERRATA_753970
> select ARM_GIC
> select ARMADA_375_CLK
> select HAVE_ARM_SCU
> @@ -57,7 +57,7 @@ config MACH_ARMADA_38X
> bool "Marvell Armada 380/385 boards"
> depends on ARCH_MULTI_V7
> select ARM_ERRATA_720789
> - select ARM_ERRATA_753970
> + select PL310_ERRATA_753970
> select ARM_GIC
> select ARM_GLOBAL_TIMER
> select CLKSRC_ARM_GLOBAL_TIMER_SCHED_CLOCK
> --
> 2.14.1
>
>
> _______________________________________________
> linux-arm-kernel mailing list
> [email protected]
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

--
Gregory Clement, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
http://bootlin.com

2018-02-15 07:37:31

by Luciano Coelho

[permalink] [raw]
Subject: Re: [PATCH 18/20] iwlwifi: fix malformed CONFIG_IWLWIFI_PCIE_RTPM default

On Mon, 2018-02-05 at 02:21 +0100, Ulf Magnusson wrote:
> 'default false' should be 'default n', though they happen to have the
> same effect here, due to undefined symbols ('false' in this case)
> evaluating to n in a tristate sense.
>
> Remove the default instead of changing it. bool and tristate symbols
> implicitly default to n.
>
> Discovered with the
> https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_und
> efined.py
> script.
>
> Signed-off-by: Ulf Magnusson <[email protected]>
> ---

Thanks, Ulf!

This is applied in our internal tree and will eventually reach the
mainline, following our usual upstreaming process.

--
Cheers,
Luca.

2018-02-16 19:05:49

by Kishon Vijay Abraham I

[permalink] [raw]
Subject: Re: [PATCH 15/20] phy: Remove SOC_EXYNOS4212 dep. from PHY_EXYNOS4X12_USB



On Monday 05 February 2018 01:30 PM, Krzysztof Kozlowski wrote:
> On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <[email protected]> wrote:
>> Exynos4212 support was removed by commit bca9085e0ae9 ("ARM: dts:
>> exynos: remove Exynos4212 support (dead code)").
>>
>> Remove the SOC_EXYNOS4212 dependency from PHY_EXYNOS4X12_USB.
>>
>> Discovered with the
>> https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
>> script.
>>
>> Signed-off-by: Ulf Magnusson <[email protected]>
>> ---
>> drivers/phy/samsung/Kconfig | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> Acked-by: Krzysztof Kozlowski <[email protected]>

merged, thanks!

-Kishon
>
> Best regards,
> Krzysztof
>

2018-02-19 15:50:19

by Rob Herring (Arm)

[permalink] [raw]
Subject: Re: [PATCH 10/20] lib/Kconfig: Remove leftover select of GENERIC_IO

On Sun, Feb 4, 2018 at 7:21 PM, Ulf Magnusson <[email protected]> wrote:
> GENERIC_IO was removed by commit 9de8da47742b ("kconfig: kill off
> GENERIC_IO option"), but HAS_IOMEM still selects it.
>
> Remove the select of GENERIC_IO from HAS_IOMEM.
>
> Discovered with the
> https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
> script.
>
> Signed-off-by: Ulf Magnusson <[email protected]>
> ---
> lib/Kconfig | 1 -
> 1 file changed, 1 deletion(-)

Who do you intend to apply this because people keep sending the same fix?

Rob

2018-02-19 17:05:34

by Ulf Magnusson

[permalink] [raw]
Subject: Re: [PATCH 10/20] lib/Kconfig: Remove leftover select of GENERIC_IO

On Mon, Feb 19, 2018 at 4:48 PM, Rob Herring <[email protected]> wrote:
> On Sun, Feb 4, 2018 at 7:21 PM, Ulf Magnusson <[email protected]> wrote:
>> GENERIC_IO was removed by commit 9de8da47742b ("kconfig: kill off
>> GENERIC_IO option"), but HAS_IOMEM still selects it.
>>
>> Remove the select of GENERIC_IO from HAS_IOMEM.
>>
>> Discovered with the
>> https://github.com/ulfalizer/Kconfiglib/blob/master/examples/list_undefined.py
>> script.
>>
>> Signed-off-by: Ulf Magnusson <[email protected]>
>> ---
>> lib/Kconfig | 1 -
>> 1 file changed, 1 deletion(-)
>
> Who do you intend to apply this because people keep sending the same fix?
>
> Rob

I didn't have a particular tree in mind. I imagined people would pick
up patches into the trees where they make the most sense.

I just turned it into a patchset to keep track of it easier myself.
Looks like that might have caused some confusion in retrospect.

Cheers,
Ulf

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

On Tuesday, February 06, 2018 11:52:46 PM Ulf Magnusson wrote:
> 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