Re: [PATCH] drivers: depend on instead of select BACKLIGHT_CLASS_DEVICE and ACPI_VIDEO

On Wed, 29 Oct 2014, Michael Ellerman mpe@ellerman.id.au wrote:

On Tue, 2014-10-28 at 13:29 -0700, Randy Dunlap wrote:

...

On 10/27/14 06:13, Tomi Valkeinen wrote:

...

I also think the 'depends on BACKLIGHT_CLASS_DEVICE || BACKLIGHT_CLASS_DEVICE=n' pattern is quite... interesting (i.e. sounds like a hack to me =).

It does exactly what is needed and it is used in many places in kernel Kconfig files.

Is there any reason you can't do:

depends on BACKLIGHT_CLASS_DEVICE != m

That's not the same thing. The FOO || FOO=n allows for all options, but forbids it being a module when the option depending on it is built-in. Obviously something that's built-in can't depend on something built as a module.

BR, Jani.