Re: [PATCH] fbdev: make FB_BACKLIGHT a tristate

From: Arnd Bergmann
Date: Wed Oct 10 2018 - 11:35:34 EST


On 10/10/18, Rob Clark <robdclark@xxxxxxxxx> wrote:
> BACKLIGHT_CLASS_DEVICE is already tristate, but a dependency
> FB_BACKLIGHT prevents it from being built as a module. There
> doesn't seem to be any particularly good reason for this, so
> switch FB_BACKLIGHT over to tristate.
>
> Signed-off-by: Rob Clark <robdclark@xxxxxxxxx>

I don't see anything immediately wrong, but anything related to
BACKLIGHT_CLASS_DEVICE, BACKLIGHT_LCD_SUPPORT
and FB_BACKLIGHT is really fragile in Kconfig, because of the
way those interact with other options.

I've applied your patch to my randconfig build tree for testing,
let's see what happens there before you apply it.

Arnd