* [PATCH] platform/x86: never 'select DMI' from a driver
@ 2016-02-29 12:13 Arnd Bergmann
2016-02-29 15:46 ` Andy Lutomirski
2016-02-29 23:10 ` Darren Hart
0 siblings, 2 replies; 6+ messages in thread
From: Arnd Bergmann @ 2016-02-29 12:13 UTC (permalink / raw)
To: Darren Hart
Cc: linux-arm-kernel, Andy Lutomirski, Pali Rohár, Arnd Bergmann,
platform-driver-x86, linux-kernel
CONFIG_DMI is a user-selectable Kconfig symbol that some drivers
depend on. As part of a recent patch, other drivers started
adding a 'select' for the same symbol, which now causes
a recursive dependency:
drivers/gpio/Kconfig:34:error: recursive dependency detected!
subsection "Kconfig recursive dependency limitations"
drivers/gpio/Kconfig:34: symbol GPIOLIB is selected by GEOS
For a resolution refer to Documentation/kbuild/kconfig-language.txt
subsection "Kconfig recursive dependency limitations"
arch/x86/Kconfig:2591: symbol GEOS depends on DMI
For a resolution refer to Documentation/kbuild/kconfig-language.txt
subsection "Kconfig recursive dependency limitations"
arch/x86/Kconfig:815: symbol DMI is selected by DELL_LAPTOP
For a resolution refer to Documentation/kbuild/kconfig-language.txt
subsection "Kconfig recursive dependency limitations"
drivers/platform/x86/Kconfig:104: symbol DELL_LAPTOP depends on BACKLIGHT_CLASS_DEVICE
For a resolution refer to Documentation/kbuild/kconfig-language.txt
subsection "Kconfig recursive dependency limitations"
drivers/video/backlight/Kconfig:158: symbol BACKLIGHT_CLASS_DEVICE is selected by FB_BACKLIGHT
For a resolution refer to Documentation/kbuild/kconfig-language.txt
subsection "Kconfig recursive dependency limitations"
drivers/video/fbdev/Kconfig:192: symbol FB_BACKLIGHT is selected by FB_SSD1307
For a resolution refer to Documentation/kbuild/kconfig-language.txt
subsection "Kconfig recursive dependency limitations"
drivers/video/fbdev/Kconfig:2462: symbol FB_SSD1307 depends on GPIOLIB
Basically we should either always use 'depends on' or always use 'select'
to avoid this kind of loop. Using 'depends on' is more useful here,
as it still allows users to turn off the symbol of they really
want to, without having to track down every driver selecting it.
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Fixes: cbd9d95b2b27 ("dell-wmi, dell-laptop: select DMI")
---
drivers/platform/x86/Kconfig | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/platform/x86/Kconfig b/drivers/platform/x86/Kconfig
index 659e13b1e6f0..a65d974f387a 100644
--- a/drivers/platform/x86/Kconfig
+++ b/drivers/platform/x86/Kconfig
@@ -105,6 +105,7 @@ config DELL_LAPTOP
tristate "Dell Laptop Extras"
depends on X86
depends on DELL_SMBIOS
+ depends on DMI
depends on BACKLIGHT_CLASS_DEVICE
depends on ACPI_VIDEO || ACPI_VIDEO = n
depends on RFKILL || RFKILL = n
@@ -112,7 +113,6 @@ config DELL_LAPTOP
select POWER_SUPPLY
select LEDS_CLASS
select NEW_LEDS
- select DMI
default n
---help---
This driver adds support for rfkill and backlight control to Dell
@@ -121,10 +121,10 @@ config DELL_LAPTOP
config DELL_WMI
tristate "Dell WMI extras"
depends on ACPI_WMI
+ depends on DMI
depends on INPUT
depends on ACPI_VIDEO || ACPI_VIDEO = n
select INPUT_SPARSEKMAP
- select DMI
---help---
Say Y here if you want to support WMI-based hotkeys on Dell laptops.
--
2.7.0
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [PATCH] platform/x86: never 'select DMI' from a driver
2016-02-29 12:13 [PATCH] platform/x86: never 'select DMI' from a driver Arnd Bergmann
@ 2016-02-29 15:46 ` Andy Lutomirski
2016-02-29 23:07 ` Darren Hart
2016-02-29 23:10 ` Darren Hart
1 sibling, 1 reply; 6+ messages in thread
From: Andy Lutomirski @ 2016-02-29 15:46 UTC (permalink / raw)
To: Arnd Bergmann
Cc: Darren Hart, linux-arm-kernel@lists.infradead.org,
Andy Lutomirski, Pali Rohár, platform-driver-x86,
linux-kernel@vger.kernel.org
On Mon, Feb 29, 2016 at 4:13 AM, Arnd Bergmann <arnd@arndb.de> wrote:
> CONFIG_DMI is a user-selectable Kconfig symbol that some drivers
> depend on. As part of a recent patch, other drivers started
> adding a 'select' for the same symbol, which now causes
> a recursive dependency:
Darren, it may make sense for you to fold the DELL_LAPTOP change in.
--Andy
>
> drivers/gpio/Kconfig:34:error: recursive dependency detected!
> subsection "Kconfig recursive dependency limitations"
> drivers/gpio/Kconfig:34: symbol GPIOLIB is selected by GEOS
> For a resolution refer to Documentation/kbuild/kconfig-language.txt
> subsection "Kconfig recursive dependency limitations"
> arch/x86/Kconfig:2591: symbol GEOS depends on DMI
> For a resolution refer to Documentation/kbuild/kconfig-language.txt
> subsection "Kconfig recursive dependency limitations"
> arch/x86/Kconfig:815: symbol DMI is selected by DELL_LAPTOP
> For a resolution refer to Documentation/kbuild/kconfig-language.txt
> subsection "Kconfig recursive dependency limitations"
> drivers/platform/x86/Kconfig:104: symbol DELL_LAPTOP depends on BACKLIGHT_CLASS_DEVICE
> For a resolution refer to Documentation/kbuild/kconfig-language.txt
> subsection "Kconfig recursive dependency limitations"
> drivers/video/backlight/Kconfig:158: symbol BACKLIGHT_CLASS_DEVICE is selected by FB_BACKLIGHT
> For a resolution refer to Documentation/kbuild/kconfig-language.txt
> subsection "Kconfig recursive dependency limitations"
> drivers/video/fbdev/Kconfig:192: symbol FB_BACKLIGHT is selected by FB_SSD1307
> For a resolution refer to Documentation/kbuild/kconfig-language.txt
> subsection "Kconfig recursive dependency limitations"
> drivers/video/fbdev/Kconfig:2462: symbol FB_SSD1307 depends on GPIOLIB
>
> Basically we should either always use 'depends on' or always use 'select'
> to avoid this kind of loop. Using 'depends on' is more useful here,
> as it still allows users to turn off the symbol of they really
> want to, without having to track down every driver selecting it.
>
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> Fixes: cbd9d95b2b27 ("dell-wmi, dell-laptop: select DMI")
> ---
> drivers/platform/x86/Kconfig | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/platform/x86/Kconfig b/drivers/platform/x86/Kconfig
> index 659e13b1e6f0..a65d974f387a 100644
> --- a/drivers/platform/x86/Kconfig
> +++ b/drivers/platform/x86/Kconfig
> @@ -105,6 +105,7 @@ config DELL_LAPTOP
> tristate "Dell Laptop Extras"
> depends on X86
> depends on DELL_SMBIOS
> + depends on DMI
> depends on BACKLIGHT_CLASS_DEVICE
> depends on ACPI_VIDEO || ACPI_VIDEO = n
> depends on RFKILL || RFKILL = n
> @@ -112,7 +113,6 @@ config DELL_LAPTOP
> select POWER_SUPPLY
> select LEDS_CLASS
> select NEW_LEDS
> - select DMI
> default n
> ---help---
> This driver adds support for rfkill and backlight control to Dell
> @@ -121,10 +121,10 @@ config DELL_LAPTOP
> config DELL_WMI
> tristate "Dell WMI extras"
> depends on ACPI_WMI
> + depends on DMI
> depends on INPUT
> depends on ACPI_VIDEO || ACPI_VIDEO = n
> select INPUT_SPARSEKMAP
> - select DMI
> ---help---
> Say Y here if you want to support WMI-based hotkeys on Dell laptops.
>
> --
> 2.7.0
>
--
Andy Lutomirski
AMA Capital Management, LLC
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH] platform/x86: never 'select DMI' from a driver
2016-02-29 15:46 ` Andy Lutomirski
@ 2016-02-29 23:07 ` Darren Hart
2016-02-29 23:13 ` Andy Lutomirski
0 siblings, 1 reply; 6+ messages in thread
From: Darren Hart @ 2016-02-29 23:07 UTC (permalink / raw)
To: Andy Lutomirski
Cc: Arnd Bergmann, linux-arm-kernel@lists.infradead.org,
Andy Lutomirski, Pali Rohár, platform-driver-x86,
linux-kernel@vger.kernel.org
On Mon, Feb 29, 2016 at 07:46:58AM -0800, Andy Lutomirski wrote:
> On Mon, Feb 29, 2016 at 4:13 AM, Arnd Bergmann <arnd@arndb.de> wrote:
> > CONFIG_DMI is a user-selectable Kconfig symbol that some drivers
> > depend on. As part of a recent patch, other drivers started
> > adding a 'select' for the same symbol, which now causes
> > a recursive dependency:
>
> Darren, it may make sense for you to fold the DELL_LAPTOP change in.
Andy, I didn't quite follow you. Are you referring to this patch from Arnd, or
from the dell-smbios series from Michał?
I'm planning on merging both (Michał's is pending a v5).
--
Darren Hart
Intel Open Source Technology Center
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] platform/x86: never 'select DMI' from a driver
2016-02-29 23:07 ` Darren Hart
@ 2016-02-29 23:13 ` Andy Lutomirski
2016-02-29 23:19 ` Darren Hart
0 siblings, 1 reply; 6+ messages in thread
From: Andy Lutomirski @ 2016-02-29 23:13 UTC (permalink / raw)
To: Darren Hart
Cc: Arnd Bergmann, linux-arm-kernel@lists.infradead.org,
Andy Lutomirski, Pali Rohár, platform-driver-x86,
linux-kernel@vger.kernel.org
On Mon, Feb 29, 2016 at 3:07 PM, Darren Hart <dvhart@infradead.org> wrote:
> On Mon, Feb 29, 2016 at 07:46:58AM -0800, Andy Lutomirski wrote:
>> On Mon, Feb 29, 2016 at 4:13 AM, Arnd Bergmann <arnd@arndb.de> wrote:
>> > CONFIG_DMI is a user-selectable Kconfig symbol that some drivers
>> > depend on. As part of a recent patch, other drivers started
>> > adding a 'select' for the same symbol, which now causes
>> > a recursive dependency:
>>
>> Darren, it may make sense for you to fold the DELL_LAPTOP change in.
>
> Andy, I didn't quite follow you. Are you referring to this patch from Arnd, or
> from the dell-smbios series from Michał?
>
> I'm planning on merging both (Michał's is pending a v5).
>
I meant to fold the "select DMI" -> "depends on DMI" for DELL_WMI into my patch.
--Andy
--
Andy Lutomirski
AMA Capital Management, LLC
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] platform/x86: never 'select DMI' from a driver
2016-02-29 23:13 ` Andy Lutomirski
@ 2016-02-29 23:19 ` Darren Hart
0 siblings, 0 replies; 6+ messages in thread
From: Darren Hart @ 2016-02-29 23:19 UTC (permalink / raw)
To: Andy Lutomirski
Cc: Arnd Bergmann, linux-arm-kernel@lists.infradead.org,
Andy Lutomirski, Pali Rohár, platform-driver-x86,
linux-kernel@vger.kernel.org
On Mon, Feb 29, 2016 at 03:13:54PM -0800, Andy Lutomirski wrote:
> On Mon, Feb 29, 2016 at 3:07 PM, Darren Hart <dvhart@infradead.org> wrote:
> > On Mon, Feb 29, 2016 at 07:46:58AM -0800, Andy Lutomirski wrote:
> >> On Mon, Feb 29, 2016 at 4:13 AM, Arnd Bergmann <arnd@arndb.de> wrote:
> >> > CONFIG_DMI is a user-selectable Kconfig symbol that some drivers
> >> > depend on. As part of a recent patch, other drivers started
> >> > adding a 'select' for the same symbol, which now causes
> >> > a recursive dependency:
> >>
> >> Darren, it may make sense for you to fold the DELL_LAPTOP change in.
> >
> > Andy, I didn't quite follow you. Are you referring to this patch from Arnd, or
> > from the dell-smbios series from Michał?
> >
> > I'm planning on merging both (Michał's is pending a v5).
> >
>
> I meant to fold the "select DMI" -> "depends on DMI" for DELL_WMI into my patch.
Of course. Obvious in retrospect. :-)
--
Darren Hart
Intel Open Source Technology Center
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] platform/x86: never 'select DMI' from a driver
2016-02-29 12:13 [PATCH] platform/x86: never 'select DMI' from a driver Arnd Bergmann
2016-02-29 15:46 ` Andy Lutomirski
@ 2016-02-29 23:10 ` Darren Hart
1 sibling, 0 replies; 6+ messages in thread
From: Darren Hart @ 2016-02-29 23:10 UTC (permalink / raw)
To: Arnd Bergmann
Cc: linux-arm-kernel, Andy Lutomirski, Pali Rohár,
platform-driver-x86, linux-kernel
On Mon, Feb 29, 2016 at 01:13:31PM +0100, Arnd Bergmann wrote:
> CONFIG_DMI is a user-selectable Kconfig symbol that some drivers
> depend on. As part of a recent patch, other drivers started
> adding a 'select' for the same symbol, which now causes
> a recursive dependency:
>
> drivers/gpio/Kconfig:34:error: recursive dependency detected!
> subsection "Kconfig recursive dependency limitations"
> drivers/gpio/Kconfig:34: symbol GPIOLIB is selected by GEOS
> For a resolution refer to Documentation/kbuild/kconfig-language.txt
> subsection "Kconfig recursive dependency limitations"
> arch/x86/Kconfig:2591: symbol GEOS depends on DMI
> For a resolution refer to Documentation/kbuild/kconfig-language.txt
> subsection "Kconfig recursive dependency limitations"
> arch/x86/Kconfig:815: symbol DMI is selected by DELL_LAPTOP
> For a resolution refer to Documentation/kbuild/kconfig-language.txt
> subsection "Kconfig recursive dependency limitations"
> drivers/platform/x86/Kconfig:104: symbol DELL_LAPTOP depends on BACKLIGHT_CLASS_DEVICE
> For a resolution refer to Documentation/kbuild/kconfig-language.txt
> subsection "Kconfig recursive dependency limitations"
> drivers/video/backlight/Kconfig:158: symbol BACKLIGHT_CLASS_DEVICE is selected by FB_BACKLIGHT
> For a resolution refer to Documentation/kbuild/kconfig-language.txt
> subsection "Kconfig recursive dependency limitations"
> drivers/video/fbdev/Kconfig:192: symbol FB_BACKLIGHT is selected by FB_SSD1307
> For a resolution refer to Documentation/kbuild/kconfig-language.txt
> subsection "Kconfig recursive dependency limitations"
> drivers/video/fbdev/Kconfig:2462: symbol FB_SSD1307 depends on GPIOLIB
>
> Basically we should either always use 'depends on' or always use 'select'
> to avoid this kind of loop. Using 'depends on' is more useful here,
> as it still allows users to turn off the symbol of they really
> want to, without having to track down every driver selecting it.
>
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> Fixes: cbd9d95b2b27 ("dell-wmi, dell-laptop: select DMI")
Thanks Arnd, queued to testing.
--
Darren Hart
Intel Open Source Technology Center
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2016-02-29 23:19 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-02-29 12:13 [PATCH] platform/x86: never 'select DMI' from a driver Arnd Bergmann
2016-02-29 15:46 ` Andy Lutomirski
2016-02-29 23:07 ` Darren Hart
2016-02-29 23:13 ` Andy Lutomirski
2016-02-29 23:19 ` Darren Hart
2016-02-29 23:10 ` Darren Hart
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox