Hi Alexander,
On Wed, May 15, 2019 at 12:34 PM Alexander Shiyan <shc_work@xxxxxxx> wrote:
> Looks like you're right.
> Can you create a patch to fix this?
Thank you, done.
> >Среда, 15 мая 2019, 11:57 +03:00 от Geert Uytterhoeven <geert@xxxxxxxxxxxxxx>:
> >
> >Hi Alexander,
> >
> >On Thu, Jan 17, 2019 at 2:39 PM Alexander Shiyan < shc_work@xxxxxxx > wrote:
> >> We have two *_CLASS_DEVICE kernel config options (LCD_CLASS_DEVICE
> >> and BACKLIGHT_LCD_DEVICE) that do the same job.
> >> The patch removes useless BACKLIGHT_LCD_SUPPORT option
> >> and converts LCD_CLASS_DEVICE into a menu.
> >>
> >> Signed-off-by: Alexander Shiyan < shc_work@xxxxxxx >
> >This is now commit 8c5dc8d9f19c7992 ("video: backlight: Remove useless
> >BACKLIGHT_LCD_SUPPORT kernel symbol").
> >
> >> --- a/drivers/video/backlight/Kconfig
> >> +++ b/drivers/video/backlight/Kconfig
> >> @@ -2,13 +2,7 @@
> >> # Backlight & LCD drivers configuration
> >> #
> >>
> >> -menuconfig BACKLIGHT_LCD_SUPPORT
> >> - bool "Backlight & LCD device support"
> >> - help
> >> - Enable this to be able to choose the drivers for controlling the
> >> - backlight and the LCD panel on some platforms, for example on PDAs.
> >> -
> >> -if BACKLIGHT_LCD_SUPPORT
> >> +menu "Backlight & LCD device support"
> >>
> >> #
> >> # LCD
> >
> >Below, we have:
> >
> > config LCD_CLASS_DEVICE
> > tristate "Lowlevel LCD controls"
> > default m
> > help
> > This framework adds support for low-level control of LCD.
> > Some framebuffer devices connect to platform-specific LCD modules
> > in order to have a platform-specific way to control the flat panel
> > (contrast and applying power to the LCD (not to the backlight!)).
> >
> >and:
> >
> > config BACKLIGHT_CLASS_DEVICE
> > tristate "Lowlevel Backlight controls"
> > default m
> > help
> > This framework adds support for low-level control of the LCD
> > backlight. This includes support for brightness and power.
> >
> >Hence running "make oldconfig" on a .config where
> >CONFIG_BACKLIGHT_LCD_SUPPORT was not set leads two to new
> >questions, where the answers default to "m".
> >
> >Perhaps the "default m" statements should be removed?
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel
