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

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

[Ulf Magnusson]

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

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

[Ulf Magnusson]

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

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

[Ulf Magnusson]

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 <ulfalizer@gmail.com>
---
 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

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

[Ulf Magnusson]

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 <ulfalizer@gmail.com>
---
 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

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

[Gregory CLEMENT]

Hi Ulf,
 
 On lun., févr. 05 2018, Ulf Magnusson <ulfalizer@gmail.com> 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 <ulfalizer@gmail.com>
> ---
>  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
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

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

[Gregory CLEMENT]

Hi Ulf,
 
 On lun., f?vr. 05 2018, Ulf Magnusson <ulfalizer@gmail.com> 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 <ulfalizer@gmail.com>
> ---
>  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
> linux-arm-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

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

[Ulf Magnusson]

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 <ulfalizer@gmail.com>
---
 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

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

[Ulf Magnusson]

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 <ulfalizer@gmail.com>
---
 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

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

[Paul Mackerras]

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 <ulfalizer@gmail.com>

Acked-by: Paul Mackerras <paulus@ozlabs.org>

Which tree is this series going into?

Paul.

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

[Paul Mackerras]

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 <ulfalizer@gmail.com>

Acked-by: Paul Mackerras <paulus@ozlabs.org>

Which tree is this series going into?

Paul.

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

[Ulf Magnusson]

On Mon, Feb 5, 2018 at 5:48 AM, Paul Mackerras <paulus@ozlabs.org> 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 <ulfalizer@gmail.com>
>
> Acked-by: Paul Mackerras <paulus@ozlabs.org>
>
> 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

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

[Ulf Magnusson]

On Mon, Feb 5, 2018 at 5:48 AM, Paul Mackerras <paulus@ozlabs.org> 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 <ulfalizer@gmail.com>
>
> Acked-by: Paul Mackerras <paulus@ozlabs.org>
>
> 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

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

[Paul Mackerras]

On Mon, Feb 05, 2018 at 05:58:59AM +0100, Ulf Magnusson wrote:
> On Mon, Feb 5, 2018 at 5:48 AM, Paul Mackerras <paulus@ozlabs.org> 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 <ulfalizer@gmail.com>
> >
> > Acked-by: Paul Mackerras <paulus@ozlabs.org>
> >
> > 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.

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

[Paul Mackerras]

On Mon, Feb 05, 2018 at 05:58:59AM +0100, Ulf Magnusson wrote:
> On Mon, Feb 5, 2018 at 5:48 AM, Paul Mackerras <paulus@ozlabs.org> 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 <ulfalizer@gmail.com>
> >
> > Acked-by: Paul Mackerras <paulus@ozlabs.org>
> >
> > 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.

[PATCH 03/20] kconfig: Remove leftover references to AVR32 symbol

[Ulf Magnusson]

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

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 <ulfalizer@gmail.com>
---
 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

[PATCH 03/20] kconfig: Remove leftover references to AVR32 symbol

[Ulf Magnusson]

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

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 <ulfalizer@gmail.com>
---
 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

[PATCH 03/20] kconfig: Remove leftover references to AVR32 symbol

[Ulf Magnusson]

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

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 <ulfalizer@gmail.com>
---
 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

[PATCH 03/20] kconfig: Remove leftover references to AVR32 symbol

[Ulf Magnusson]

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

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 <ulfalizer@gmail.com>
---
 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

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

[Viresh Kumar]

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 <ulfalizer@gmail.com>
> ---
>  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/20180118200202.11883-1-clabbe.montjoie@gmail.com

-- 
viresh

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

[Viresh Kumar]

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 <ulfalizer@gmail.com>
> ---
>  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/20180118200202.11883-1-clabbe.montjoie@gmail.com

-- 
viresh

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

[Viresh Kumar]

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 <ulfalizer@gmail.com>
> ---
>  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/20180118200202.11883-1-clabbe.montjoie@gmail.com

-- 
viresh

[PATCH v2 03/20] kconfig: Remove leftover references to AVR32 symbol

[Ulf Magnusson]

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

Signed-off-by: Ulf Magnusson <ulfalizer@gmail.com>
---
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

[PATCH v2 03/20] kconfig: Remove leftover references to AVR32 symbol

[Ulf Magnusson]

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

Signed-off-by: Ulf Magnusson <ulfalizer@gmail.com>
---
Changes in v2:
Removal of AVR32_AT32AP_CPUFREQ is already pending in
https://marc.info/?i 180118200202.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

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

[Ulf Magnusson]

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 <ulfalizer@gmail.com>
---
 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

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

[Ulf Magnusson]

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 <ulfalizer@gmail.com>
---
 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

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

[Ulf Magnusson]

On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <ulfalizer@gmail.com> 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 <ulfalizer@gmail.com>
> ---
>  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

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

[Alexandre Belloni]

Hi,

On 05/02/2018 at 02:42:38 +0100, Ulf Magnusson wrote:
> On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <ulfalizer@gmail.com> 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 <ulfalizer@gmail.com>
> > ---
> >  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

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

[Ulf Magnusson]

On Mon, Feb 5, 2018 at 3:01 PM, Alexandre Belloni
<alexandre.belloni@free-electrons.com> 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 <ulfalizer@gmail.com> 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 <ulfalizer@gmail.com>
>> > ---
>> >  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

[PATCH 06/20] riscv: Remove ARCH_HAS_ATOMIC64_DEC_IF_POSITIVE select

[Ulf Magnusson]

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 <ulfalizer@gmail.com>
---
 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

[PATCH 06/20] riscv: Remove ARCH_HAS_ATOMIC64_DEC_IF_POSITIVE select

[Ulf Magnusson]

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 <ulfalizer@gmail.com>
---
 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

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

[Jonathan Neuschäfer]

[-- Attachment #1: Type: text/plain, Size: 1007 bytes --]

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 <ulfalizer@gmail.com>
> ---
>  arch/riscv/Kconfig | 1 -
>  1 file changed, 1 deletion(-)

Looks good.

Reviewed-by: Jonathan Neuschäfer <j.neuschaefer@gmx.net>

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

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

[PATCH 06/20] riscv: Remove ARCH_HAS_ATOMIC64_DEC_IF_POSITIVE select

[Jonathan Neuschäfer]

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 <ulfalizer@gmail.com>
> ---
>  arch/riscv/Kconfig | 1 -
>  1 file changed, 1 deletion(-)

Looks good.

Reviewed-by: Jonathan Neusch?fer <j.neuschaefer@gmx.net>

> 
> 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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-riscv/attachments/20180205/3c543c9a/attachment.sig>

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

[Palmer Dabbelt]

On Sun, 04 Feb 2018 17:21:18 PST (-0800), ulfalizer@gmail.com 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 <ulfalizer@gmail.com>
> ---
>  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 <palmer@sifive.com>

[PATCH 06/20] riscv: Remove ARCH_HAS_ATOMIC64_DEC_IF_POSITIVE select

[Palmer Dabbelt]

On Sun, 04 Feb 2018 17:21:18 PST (-0800), ulfalizer at gmail.com 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 <ulfalizer@gmail.com>
> ---
>  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 <palmer@sifive.com>

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

[Palmer Dabbelt]

On Sun, 04 Feb 2018 17:21:18 PST (-0800), ulfalizer@gmail.com 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 <ulfalizer@gmail.com>
> ---
>  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 <palmer@sifive.com>

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

[Ulf Magnusson]

On Thu, Feb 8, 2018 at 7:34 PM, Palmer Dabbelt <palmer@sifive.com> wrote:
> On Sun, 04 Feb 2018 17:21:18 PST (-0800), ulfalizer@gmail.com 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 <ulfalizer@gmail.com>
>> ---
>>  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 <palmer@sifive.com>

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

[PATCH 06/20] riscv: Remove ARCH_HAS_ATOMIC64_DEC_IF_POSITIVE select

[Ulf Magnusson]

On Thu, Feb 8, 2018 at 7:34 PM, Palmer Dabbelt <palmer@sifive.com> wrote:
> On Sun, 04 Feb 2018 17:21:18 PST (-0800), ulfalizer at gmail.com 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 <ulfalizer@gmail.com>
>> ---
>>  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 <palmer@sifive.com>

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

[PATCH 07/20] riscv: Remove ARCH_WANT_OPTIONAL_GPIOLIB select

[Ulf Magnusson]

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 <ulfalizer@gmail.com>
---
 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

[PATCH 07/20] riscv: Remove ARCH_WANT_OPTIONAL_GPIOLIB select

[Ulf Magnusson]

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 <ulfalizer@gmail.com>
---
 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

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

[Ulf Magnusson]

On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <ulfalizer@gmail.com> 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 <ulfalizer@gmail.com>
> ---
>  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

[PATCH 07/20] riscv: Remove ARCH_WANT_OPTIONAL_GPIOLIB select

[Ulf Magnusson]

On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <ulfalizer@gmail.com> 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 <ulfalizer@gmail.com>
> ---
>  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

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

[Palmer Dabbelt]

On Sun, 04 Feb 2018 17:30:57 PST (-0800), ulfalizer@gmail.com wrote:
> On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <ulfalizer@gmail.com> 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 <ulfalizer@gmail.com>
>> ---
>>  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 <palmer@sifive.com>

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

[PATCH 07/20] riscv: Remove ARCH_WANT_OPTIONAL_GPIOLIB select

[Palmer Dabbelt]

On Sun, 04 Feb 2018 17:30:57 PST (-0800), ulfalizer at gmail.com wrote:
> On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <ulfalizer@gmail.com> 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 <ulfalizer@gmail.com>
>> ---
>>  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 <palmer@sifive.com>

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

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

[Palmer Dabbelt]

On Sun, 04 Feb 2018 17:30:57 PST (-0800), ulfalizer@gmail.com wrote:
> On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <ulfalizer@gmail.com> 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 <ulfalizer@gmail.com>
>> ---
>>  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 <palmer@sifive.com>

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

[PATCH] riscv: kconfig: Remove RISCV_IRQ_INTC select

[Ulf Magnusson]

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 <ulfalizer@gmail.com>
---
 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

[PATCH] riscv: kconfig: Remove RISCV_IRQ_INTC select

[Ulf Magnusson]

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 <ulfalizer@gmail.com>
---
 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

Re: [PATCH] riscv: kconfig: Remove RISCV_IRQ_INTC select

[Palmer Dabbelt]

On Thu, 08 Feb 2018 14:54:46 PST (-0800), ulfalizer@gmail.com 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 <ulfalizer@gmail.com>
> ---
>  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.

[PATCH] riscv: kconfig: Remove RISCV_IRQ_INTC select

[Palmer Dabbelt]

On Thu, 08 Feb 2018 14:54:46 PST (-0800), ulfalizer at gmail.com 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 <ulfalizer@gmail.com>
> ---
>  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.

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

[Linus Walleij]

On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <ulfalizer@gmail.com> 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 <ulfalizer@gmail.com>

This should probaly be replaced with
select GPIOLIB

Yours,
Linus Walleij

[PATCH 07/20] riscv: Remove ARCH_WANT_OPTIONAL_GPIOLIB select

[Linus Walleij]

On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <ulfalizer@gmail.com> 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 <ulfalizer@gmail.com>

This should probaly be replaced with
select GPIOLIB

Yours,
Linus Walleij

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

[Ulf Magnusson]

On Tue, Feb 6, 2018 at 12:25 AM, Linus Walleij <linus.walleij@linaro.org> wrote:
> On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <ulfalizer@gmail.com> 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 <ulfalizer@gmail.com>
>
> 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

[PATCH 07/20] riscv: Remove ARCH_WANT_OPTIONAL_GPIOLIB select

[Ulf Magnusson]

On Tue, Feb 6, 2018 at 12:25 AM, Linus Walleij <linus.walleij@linaro.org> wrote:
> On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <ulfalizer@gmail.com> 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 <ulfalizer@gmail.com>
>
> 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

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

[Ulf Magnusson]

On Tue, Feb 6, 2018 at 12:38 AM, Ulf Magnusson <ulfalizer@gmail.com> wrote:
> On Tue, Feb 6, 2018 at 12:25 AM, Linus Walleij <linus.walleij@linaro.org> wrote:
>> On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <ulfalizer@gmail.com> 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 <ulfalizer@gmail.com>
>>
>> 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

[PATCH 07/20] riscv: Remove ARCH_WANT_OPTIONAL_GPIOLIB select

[Ulf Magnusson]

On Tue, Feb 6, 2018 at 12:38 AM, Ulf Magnusson <ulfalizer@gmail.com> wrote:
> On Tue, Feb 6, 2018 at 12:25 AM, Linus Walleij <linus.walleij@linaro.org> wrote:
>> On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <ulfalizer@gmail.com> 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 <ulfalizer@gmail.com>
>>
>> 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

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

[Linus Walleij]

On Tue, Feb 6, 2018 at 12:49 AM, Ulf Magnusson <ulfalizer@gmail.com> wrote:
> On Tue, Feb 6, 2018 at 12:38 AM, Ulf Magnusson <ulfalizer@gmail.com> wrote:
>> On Tue, Feb 6, 2018 at 12:25 AM, Linus Walleij <linus.walleij@linaro.org> wrote:
>>> On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <ulfalizer@gmail.com> 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 <ulfalizer@gmail.com>
>>>
>>> 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

[PATCH 07/20] riscv: Remove ARCH_WANT_OPTIONAL_GPIOLIB select

[Linus Walleij]

On Tue, Feb 6, 2018 at 12:49 AM, Ulf Magnusson <ulfalizer@gmail.com> wrote:
> On Tue, Feb 6, 2018 at 12:38 AM, Ulf Magnusson <ulfalizer@gmail.com> wrote:
>> On Tue, Feb 6, 2018 at 12:25 AM, Linus Walleij <linus.walleij@linaro.org> wrote:
>>> On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <ulfalizer@gmail.com> 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 <ulfalizer@gmail.com>
>>>
>>> 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

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

[Linus Walleij]

On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <ulfalizer@gmail.com> 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 <ulfalizer@gmail.com>

Reviewed-by: Linus Walleij <linus.walleij@linaro.org>

RISCV maintainers: please apply and carry this patch.

Yours,
Linus Walleij

[PATCH 07/20] riscv: Remove ARCH_WANT_OPTIONAL_GPIOLIB select

[Linus Walleij]

On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <ulfalizer@gmail.com> 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 <ulfalizer@gmail.com>

Reviewed-by: Linus Walleij <linus.walleij@linaro.org>

RISCV maintainers: please apply and carry this patch.

Yours,
Linus Walleij

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

[Palmer Dabbelt]

On Tue, 06 Feb 2018 01:47:00 PST (-0800), linus.walleij@linaro.org wrote:
> On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <ulfalizer@gmail.com> 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 <ulfalizer@gmail.com>
>
> Reviewed-by: Linus Walleij <linus.walleij@linaro.org>
>
> RISCV maintainers: please apply and carry this patch.

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

[PATCH 07/20] riscv: Remove ARCH_WANT_OPTIONAL_GPIOLIB select

[Palmer Dabbelt]

On Tue, 06 Feb 2018 01:47:00 PST (-0800), linus.walleij at linaro.org wrote:
> On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <ulfalizer@gmail.com> 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 <ulfalizer@gmail.com>
>
> Reviewed-by: Linus Walleij <linus.walleij@linaro.org>
>
> RISCV maintainers: please apply and carry this patch.

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

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

[Palmer Dabbelt]

On Sun, 04 Feb 2018 17:21:19 PST (-0800), ulfalizer@gmail.com 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 <ulfalizer@gmail.com>
> ---
>  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 <palmer@sifive.com>

[PATCH 07/20] riscv: Remove ARCH_WANT_OPTIONAL_GPIOLIB select

[Palmer Dabbelt]

On Sun, 04 Feb 2018 17:21:19 PST (-0800), ulfalizer at gmail.com 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 <ulfalizer@gmail.com>
> ---
>  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 <palmer@sifive.com>

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

[Palmer Dabbelt]

On Sun, 04 Feb 2018 17:21:19 PST (-0800), ulfalizer@gmail.com 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 <ulfalizer@gmail.com>
> ---
>  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 <palmer@sifive.com>

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

[Ulf Magnusson]

On Thu, Feb 08, 2018 at 10:34:19AM -0800, Palmer Dabbelt wrote:
> On Sun, 04 Feb 2018 17:21:19 PST (-0800), ulfalizer@gmail.com 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 <ulfalizer@gmail.com>
> > ---
> >  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 <palmer@sifive.com>

Cheers,
Ulf

[PATCH 07/20] riscv: Remove ARCH_WANT_OPTIONAL_GPIOLIB select

[Ulf Magnusson]

On Thu, Feb 08, 2018 at 10:34:19AM -0800, Palmer Dabbelt wrote:
> On Sun, 04 Feb 2018 17:21:19 PST (-0800), ulfalizer at gmail.com 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 <ulfalizer@gmail.com>
> > ---
> >  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 <palmer@sifive.com>

Cheers,
Ulf

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

[Ulf Magnusson]

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 <ulfalizer@gmail.com>
---
 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

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

[Heiko Carstens]

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 <ulfalizer@gmail.com>
> ---
>  arch/s390/Kconfig | 1 -
>  1 file changed, 1 deletion(-)

Applied, thanks!

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

[Ulf Magnusson]

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 <ulfalizer@gmail.com>
---
 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

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

[Ulf Magnusson]

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 <ulfalizer@gmail.com>
---
 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

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

[Maxime Ripard]

[-- Attachment #1: Type: text/plain, Size: 645 bytes --]

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 <ulfalizer@gmail.com>

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

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

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

[Maxime Ripard]

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 <ulfalizer@gmail.com>

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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20180205/5dadb51f/attachment.sig>

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

[Ulf Magnusson]

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 <ulfalizer@gmail.com>
---
 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

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

[Rob Herring]

On Sun, Feb 4, 2018 at 7:21 PM, Ulf Magnusson <ulfalizer@gmail.com> 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 <ulfalizer@gmail.com>
> ---
>  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 <robh@kernel.org>

Rob

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

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

[Robin Murphy]

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 <robin.murphy@arm.com>

> Signed-off-by: Ulf Magnusson <ulfalizer@gmail.com>
> ---
>   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
> 

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

[Rob Herring]

On Sun, Feb 4, 2018 at 7:21 PM, Ulf Magnusson <ulfalizer@gmail.com> 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 <ulfalizer@gmail.com>
> ---
>  lib/Kconfig | 1 -
>  1 file changed, 1 deletion(-)

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

Rob

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

[Ulf Magnusson]

On Mon, Feb 19, 2018 at 4:48 PM, Rob Herring <robh@kernel.org> wrote:
> On Sun, Feb 4, 2018 at 7:21 PM, Ulf Magnusson <ulfalizer@gmail.com> 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 <ulfalizer@gmail.com>
>> ---
>>  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

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

[Ulf Magnusson]

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 <ulfalizer@gmail.com>
---
 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

[11/20] usb: gadget: udc: Remove USB_GADGET_DUALSPEED select

[Ulf Magnusson]

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 <ulfalizer@gmail.com>
---
 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

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

[Ulf Magnusson]

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 <ulfalizer@gmail.com>
---
 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

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

[Ulf Magnusson]

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 <ulfalizer@gmail.com>
---
 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

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

[Ulf Magnusson]

On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <ulfalizer@gmail.com> 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 <ulfalizer@gmail.com>
> ---
>  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

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

[Ulf Magnusson]

On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <ulfalizer@gmail.com> 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 <ulfalizer@gmail.com>
> ---
>  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

[PATCH 13/20] ore: Remove PNFS_OBJLAYOUT dependency

[Ulf Magnusson]

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 <ulfalizer@gmail.com>
---
 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

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

[Ulf Magnusson]

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 <ulfalizer@gmail.com>
---
 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

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

[Linus Walleij]

On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <ulfalizer@gmail.com> 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 <ulfalizer@gmail.com>

Reviewed-by: Linus Walleij <linus.walleij@linaro.org>

Jonathan, please pick this up to IIO!

Yours,
Linus Walleij

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

[Ulf Magnusson]

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 <ulfalizer@gmail.com>
---
 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

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

[Krzysztof Kozlowski]

On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <ulfalizer@gmail.com> 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 <ulfalizer@gmail.com>
> ---
>  drivers/phy/samsung/Kconfig | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Acked-by: Krzysztof Kozlowski <krzk@kernel.org>

Best regards,
Krzysztof

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

[Kishon Vijay Abraham I]

On Monday 05 February 2018 01:30 PM, Krzysztof Kozlowski wrote:
> On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <ulfalizer@gmail.com> 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 <ulfalizer@gmail.com>
>> ---
>>  drivers/phy/samsung/Kconfig | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> Acked-by: Krzysztof Kozlowski <krzk@kernel.org>

merged, thanks!

-Kishon
> 
> Best regards,
> Krzysztof
> 

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

[Kishon Vijay Abraham I]

On Monday 05 February 2018 01:30 PM, Krzysztof Kozlowski wrote:
> On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <ulfalizer@gmail.com> 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 <ulfalizer@gmail.com>
>> ---
>>  drivers/phy/samsung/Kconfig | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> Acked-by: Krzysztof Kozlowski <krzk@kernel.org>

merged, thanks!

-Kishon
> 
> Best regards,
> Krzysztof
> 

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

[Ulf Magnusson]

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 <ulfalizer@gmail.com>
---
 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

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

[Ulf Magnusson]

On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <ulfalizer@gmail.com> 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 <ulfalizer@gmail.com>
> ---
>  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

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

[Randy Dunlap]

On 02/04/2018 05:58 PM, Ulf Magnusson wrote:
> On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <ulfalizer@gmail.com> 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 <ulfalizer@gmail.com>
>> ---
>>  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

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

[Miguel Ojeda]

On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <ulfalizer@gmail.com> 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 <ulfalizer@gmail.com>
> ---
>  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
>

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

[Ulf Magnusson]

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 <ulfalizer@gmail.com>
---
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

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

[Miguel Ojeda]

On Sat, Feb 10, 2018 at 3:14 PM, Ulf Magnusson <ulfalizer@gmail.com> 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 <ulfalizer@gmail.com>
> ---
> 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 <miguel.ojeda.sandonis@gmail.com>

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

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

[Miguel Ojeda]

On Sun, Feb 11, 2018 at 5:40 PM, Miguel Ojeda
<miguel.ojeda.sandonis@gmail.com> wrote:
> On Sat, Feb 10, 2018 at 3:14 PM, Ulf Magnusson <ulfalizer@gmail.com> 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 <ulfalizer@gmail.com>
>> ---
>> 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 <miguel.ojeda.sandonis@gmail.com>

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

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

[James Hogan]

On 5 February 2018 at 01:21, Ulf Magnusson <ulfalizer@gmail.com> 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 <ulfalizer@gmail.com>
> ---
>  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

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

[Ulf Magnusson]

On Thu, Feb 08, 2018 at 05:35:30PM +0000, James Hogan wrote:
> On 5 February 2018 at 01:21, Ulf Magnusson <ulfalizer@gmail.com> 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 <ulfalizer@gmail.com>
> > ---
> >  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

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

[Ulf Magnusson]

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 <ulfalizer@gmail.com>
---
 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

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

[Ulf Magnusson]

'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 <ulfalizer@gmail.com>
---
 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

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

[Luciano Coelho]

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 <ulfalizer@gmail.com>
> ---

Thanks, Ulf!

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

--
Cheers,
Luca.

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

[Ulf Magnusson]

'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 <ulfalizer@gmail.com>
---
 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

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

[Ulf Magnusson]

'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 <ulfalizer at gmail.com>
---
 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

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

[Vineet Gupta]

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 <ulfalizer@gmail.com>


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
> 

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

[Vineet Gupta]

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 <ulfalizer at gmail.com>


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
> 

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

[Vineet Gupta]

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 <ulfalizer@gmail.com>


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
> 

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

[Ulf Magnusson]

'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 <ulfalizer@gmail.com>
---
 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

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

[Ulf Magnusson]

On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <ulfalizer@gmail.com> 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 <ulfalizer@gmail.com>
> ---
>  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

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

[Bjorn Helgaas]

> 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 <ulfalizer@gmail.com>
> ---
>  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
> 

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

[Ulf Magnusson]

On Tue, Feb 6, 2018 at 8:11 PM, Bjorn Helgaas <helgaas@kernel.org> 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 <ulfalizer@gmail.com>
>> ---
>>  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

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

[Ulf Magnusson]

On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <ulfalizer@gmail.com> 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

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

[Ulf Magnusson]

On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <ulfalizer@gmail.com> 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

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

[Ulf Magnusson]

On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <ulfalizer@gmail.com> 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

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

[Ulf Magnusson]

On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <ulfalizer@gmail.com> 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

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

[Ulf Magnusson]

On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <ulfalizer@gmail.com> 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

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

[Ulf Magnusson]

On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <ulfalizer@gmail.com> 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

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

[Robert Jarzmik]

Ulf Magnusson <ulfalizer@gmail.com> 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

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

[Robert Jarzmik]

Ulf Magnusson <ulfalizer@gmail.com> 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

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

[Ulf Magnusson]

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 <ulfalizer@gmail.com>
---
 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

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

[Ulf Magnusson]

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 <ulfalizer@gmail.com>
---
 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

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

[Ulf Magnusson]

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 <ulfalizer@gmail.com>
> ---
>  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

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

[Ulf Magnusson]

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 <ulfalizer@gmail.com>
> ---
>  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

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

[Takashi Iwai]

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 <ulfalizer@gmail.com>
> > ---
> >  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

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

[Takashi Iwai]

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 <ulfalizer@gmail.com>
> > ---
> >  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

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

[Takashi Iwai]

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 <ulfalizer@gmail.com>
> > ---
> >  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

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

[Takashi Iwai]

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 <ulfalizer@gmail.com>
> > ---
> >  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

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

[Ulf Magnusson]

On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <ulfalizer@gmail.com> 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

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

[Ulf Magnusson]

On Mon, Feb 5, 2018 at 2:21 AM, Ulf Magnusson <ulfalizer@gmail.com> 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