Re: [PATCH v2 1/2] video: lcd: Remove useless BACKLIGHT_LCD_SUPPORT kernel symbol

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux