Re: [PATCH v2 1/2] video: lcd: Remove useless BACKLIGHT_LCD_SUPPORT kernel symbol
Hello. Looks like you're right. Can you create a patch to fix this? Thanks. >Среда, 15 мая 2019, 11:57 +03:00 от Geert Uytterhoeven : > >Hi Alexander, > >On Thu, Jan 17, 2019 at 2:39 PM Alexander Shiyan < shc_w...@mail.ru > 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_w...@mail.ru > >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? --- ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
Re: [PATCH v2 1/2] video: lcd: Remove useless BACKLIGHT_LCD_SUPPORT kernel symbol
Hi Alexander, On Wed, May 15, 2019 at 12:34 PM Alexander Shiyan 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 > >: > > > >Hi Alexander, > > > >On Thu, Jan 17, 2019 at 2:39 PM Alexander Shiyan < shc_w...@mail.ru > 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_w...@mail.ru > > >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 -- ge...@linux-m68k.org 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@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
Re: [PATCH v2 1/2] video: lcd: Remove useless BACKLIGHT_LCD_SUPPORT kernel symbol
Hi Alexander, On Thu, Jan 17, 2019 at 2:39 PM Alexander Shiyan 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 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 -- ge...@linux-m68k.org 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@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
Re: [PATCH v2 1/2] video: lcd: Remove useless BACKLIGHT_LCD_SUPPORT kernel symbol
On 03/20/2019 11:07 AM, Lee Jones wrote: On 02/12/2019 09:42 AM, Lee Jones wrote: > On Fri, 08 Feb 2019, Bartlomiej Zolnierkiewicz wrote: > >> >> On 01/17/2019 05:19 PM, Daniel Thompson wrote: >>> On Thu, Jan 17, 2019 at 04:33:35PM +0300, Alexander Shiyan 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 >>> >>> A cover letter with the v1 -> v2 changelog would be nice... but >>> nevertheless: >>> Acked-by: Daniel Thompson >> >> Also: >> >> Acked-by: Bartlomiej Zolnierkiewicz >> >> Lee, will you pick this up or do you want me to do it? > > You can take them if you want. I will require a pull-request from an > independent immutable branch though please. OK, immutable branch will also allow other trees (drm) to sync with these changes if desired. I will do it for v5.2 if you are fine with it (too late for v5.1 IMHO and I also won't have time too fix any potential issues next week). > >>> Topic branch for these 2 simple patches seems like serious overkill. > > [...] > >>> Would be good to get it into 5.1 still though, just to avoid too >>> many conflicts. > > I think you just answered your own statement. :) > >> Well, if so I'm also fine with this going in 5.1 w/o topic branch. >> >> Lee, do you want to merge this directly into backlight tree (you >> have now both fbdev and drm acks)? > > It was too late for v5.1. > > Do you still want to take this for v5.2, or should I take them and > provide you with an immutable pull-request? Please take them and provide immutable branch, thank you. Best regards, -- Bartlomiej Zolnierkiewicz Samsung R Institute Poland Samsung Electronics ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
Re: [PATCH v2 1/2] video: lcd: Remove useless BACKLIGHT_LCD_SUPPORT kernel symbol
> >> On 02/12/2019 09:42 AM, Lee Jones wrote: > >>> On Fri, 08 Feb 2019, Bartlomiej Zolnierkiewicz wrote: > >>> > > On 01/17/2019 05:19 PM, Daniel Thompson wrote: > > On Thu, Jan 17, 2019 at 04:33:35PM +0300, Alexander Shiyan 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 > > > > A cover letter with the v1 -> v2 changelog would be nice... but > > nevertheless: > > Acked-by: Daniel Thompson > > Also: > > Acked-by: Bartlomiej Zolnierkiewicz > > Lee, will you pick this up or do you want me to do it? > >>> > >>> You can take them if you want. I will require a pull-request from an > >>> independent immutable branch though please. > >> > >> OK, immutable branch will also allow other trees (drm) to sync with > >> these changes if desired. > >> > >> I will do it for v5.2 if you are fine with it (too late for v5.1 > >> IMHO and I also won't have time too fix any potential issues next > >> week). > > Topic branch for these 2 simple patches seems like serious overkill. [...] > > Would be good to get it into 5.1 still though, just to avoid too > > many conflicts. I think you just answered your own statement. :) > Well, if so I'm also fine with this going in 5.1 w/o topic branch. > > Lee, do you want to merge this directly into backlight tree (you > have now both fbdev and drm acks)? It was too late for v5.1. Do you still want to take this for v5.2, or should I take them and provide you with an immutable pull-request? -- Lee Jones [李琼斯] Linaro Services Technical Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
Re: [PATCH v2 1/2] video: lcd: Remove useless BACKLIGHT_LCD_SUPPORT kernel symbol
On Wed, 13 Feb 2019, Daniel Vetter wrote: > On Wed, Feb 13, 2019 at 4:04 PM Bartlomiej Zolnierkiewicz via > dri-devel wrote: > > > > > > On 02/12/2019 09:42 AM, Lee Jones wrote: > > > On Fri, 08 Feb 2019, Bartlomiej Zolnierkiewicz wrote: > > > > > >> > > >> On 01/17/2019 05:19 PM, Daniel Thompson wrote: > > >>> On Thu, Jan 17, 2019 at 04:33:35PM +0300, Alexander Shiyan 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 > > >>> > > >>> A cover letter with the v1 -> v2 changelog would be nice... but > > >>> nevertheless: > > >>> Acked-by: Daniel Thompson > > >> > > >> Also: > > >> > > >> Acked-by: Bartlomiej Zolnierkiewicz > > >> > > >> Lee, will you pick this up or do you want me to do it? > > > > > > You can take them if you want. I will require a pull-request from an > > > independent immutable branch though please. > > > > OK, immutable branch will also allow other trees (drm) to sync with > > these changes if desired. > > > > I will do it for v5.2 if you are fine with it (too late for v5.1 > > IMHO and I also won't have time too fix any potential issues next > > week). > > Topic branch for these 2 simple patches seems like serious overkill. > Especially for backlight stuff, which doesn't move fast. Anyway, up to > you, ack for all the drm patches to go through whatever tree you > people can agree on. Would be good to get it into 5.1 still though, > just to avoid too many conflicts. It takes me approx 1 min to set-up an immutable branch. If you think it's too much work, I can set it up for you. -- Lee Jones [李琼斯] Linaro Services Technical Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
Re: [PATCH v2 1/2] video: lcd: Remove useless BACKLIGHT_LCD_SUPPORT kernel symbol
On 02/13/2019 04:19 PM, Daniel Vetter wrote: > On Wed, Feb 13, 2019 at 4:04 PM Bartlomiej Zolnierkiewicz via > dri-devel wrote: >> >> >> On 02/12/2019 09:42 AM, Lee Jones wrote: >>> On Fri, 08 Feb 2019, Bartlomiej Zolnierkiewicz wrote: >>> On 01/17/2019 05:19 PM, Daniel Thompson wrote: > On Thu, Jan 17, 2019 at 04:33:35PM +0300, Alexander Shiyan 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 > > A cover letter with the v1 -> v2 changelog would be nice... but > nevertheless: > Acked-by: Daniel Thompson Also: Acked-by: Bartlomiej Zolnierkiewicz Lee, will you pick this up or do you want me to do it? >>> >>> You can take them if you want. I will require a pull-request from an >>> independent immutable branch though please. >> >> OK, immutable branch will also allow other trees (drm) to sync with >> these changes if desired. >> >> I will do it for v5.2 if you are fine with it (too late for v5.1 >> IMHO and I also won't have time too fix any potential issues next >> week). > > Topic branch for these 2 simple patches seems like serious overkill. > Especially for backlight stuff, which doesn't move fast. Anyway, up to This depends on the development stage we are in [*]. Right now things are calm so there is low probability of conflicts and we might just merge these patches without topic branch. [*] In case of fbdev/Kconfig FB_CLPS711X config option was modified in v4.20 and both FB_IMX and FB_MX3 config options still need to be converted to support COMPILE_TEST so at least in theory there is a potential for conflicts with fbdev if merged for 5.2. > you, ack for all the drm patches to go through whatever tree you > people can agree on. Would be good to get it into 5.1 still though, > just to avoid too many conflicts. Well, if so I'm also fine with this going in 5.1 w/o topic branch. Lee, do you want to merge this directly into backlight tree (you have now both fbdev and drm acks)? Best regards, -- Bartlomiej Zolnierkiewicz Samsung R Institute Poland Samsung Electronics ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
Re: [PATCH v2 1/2] video: lcd: Remove useless BACKLIGHT_LCD_SUPPORT kernel symbol
On Wed, Feb 13, 2019 at 4:04 PM Bartlomiej Zolnierkiewicz via dri-devel wrote: > > > On 02/12/2019 09:42 AM, Lee Jones wrote: > > On Fri, 08 Feb 2019, Bartlomiej Zolnierkiewicz wrote: > > > >> > >> On 01/17/2019 05:19 PM, Daniel Thompson wrote: > >>> On Thu, Jan 17, 2019 at 04:33:35PM +0300, Alexander Shiyan 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 > >>> > >>> A cover letter with the v1 -> v2 changelog would be nice... but > >>> nevertheless: > >>> Acked-by: Daniel Thompson > >> > >> Also: > >> > >> Acked-by: Bartlomiej Zolnierkiewicz > >> > >> Lee, will you pick this up or do you want me to do it? > > > > You can take them if you want. I will require a pull-request from an > > independent immutable branch though please. > > OK, immutable branch will also allow other trees (drm) to sync with > these changes if desired. > > I will do it for v5.2 if you are fine with it (too late for v5.1 > IMHO and I also won't have time too fix any potential issues next > week). Topic branch for these 2 simple patches seems like serious overkill. Especially for backlight stuff, which doesn't move fast. Anyway, up to you, ack for all the drm patches to go through whatever tree you people can agree on. Would be good to get it into 5.1 still though, just to avoid too many conflicts. Cheers, Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
Re: [PATCH v2 1/2] video: lcd: Remove useless BACKLIGHT_LCD_SUPPORT kernel symbol
On 02/12/2019 09:42 AM, Lee Jones wrote: > On Fri, 08 Feb 2019, Bartlomiej Zolnierkiewicz wrote: > >> >> On 01/17/2019 05:19 PM, Daniel Thompson wrote: >>> On Thu, Jan 17, 2019 at 04:33:35PM +0300, Alexander Shiyan 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 >>> >>> A cover letter with the v1 -> v2 changelog would be nice... but >>> nevertheless: >>> Acked-by: Daniel Thompson >> >> Also: >> >> Acked-by: Bartlomiej Zolnierkiewicz >> >> Lee, will you pick this up or do you want me to do it? > > You can take them if you want. I will require a pull-request from an > independent immutable branch though please. OK, immutable branch will also allow other trees (drm) to sync with these changes if desired. I will do it for v5.2 if you are fine with it (too late for v5.1 IMHO and I also won't have time too fix any potential issues next week). Best regards, -- Bartlomiej Zolnierkiewicz Samsung R Institute Poland Samsung Electronics ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
Re: [PATCH v2 1/2] video: lcd: Remove useless BACKLIGHT_LCD_SUPPORT kernel symbol
On Fri, 08 Feb 2019, Bartlomiej Zolnierkiewicz wrote: > > On 01/17/2019 05:19 PM, Daniel Thompson wrote: > > On Thu, Jan 17, 2019 at 04:33:35PM +0300, Alexander Shiyan 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 > > > > A cover letter with the v1 -> v2 changelog would be nice... but > > nevertheless: > > Acked-by: Daniel Thompson > > Also: > > Acked-by: Bartlomiej Zolnierkiewicz > > Lee, will you pick this up or do you want me to do it? You can take them if you want. I will require a pull-request from an independent immutable branch though please. -- Lee Jones [李琼斯] Linaro Services Technical Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
Re: [PATCH v2 1/2] video: lcd: Remove useless BACKLIGHT_LCD_SUPPORT kernel symbol
On 01/17/2019 05:19 PM, Daniel Thompson wrote: > On Thu, Jan 17, 2019 at 04:33:35PM +0300, Alexander Shiyan 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 > > A cover letter with the v1 -> v2 changelog would be nice... but > nevertheless: > Acked-by: Daniel Thompson Also: Acked-by: Bartlomiej Zolnierkiewicz Lee, will you pick this up or do you want me to do it? Best regards, -- Bartlomiej Zolnierkiewicz Samsung R Institute Poland Samsung Electronics ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
Re: [PATCH v2 1/2] video: lcd: Remove useless BACKLIGHT_LCD_SUPPORT kernel symbol
On Thu, Jan 17, 2019 at 04:33:35PM +0300, Alexander Shiyan 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 A cover letter with the v1 -> v2 changelog would be nice... but nevertheless: Acked-by: Daniel Thompson > --- > arch/unicore32/Kconfig| 1 - > drivers/gpu/drm/Kconfig | 2 -- > drivers/gpu/drm/bridge/Kconfig| 1 - > drivers/gpu/drm/fsl-dcu/Kconfig | 1 - > drivers/gpu/drm/i915/Kconfig | 1 - > drivers/gpu/drm/nouveau/Kconfig | 2 -- > drivers/gpu/drm/shmobile/Kconfig | 1 - > drivers/gpu/drm/tilcdc/Kconfig| 1 - > drivers/staging/olpc_dcon/Kconfig | 1 - > drivers/usb/misc/Kconfig | 1 - > drivers/video/backlight/Kconfig | 10 ++ > drivers/video/fbdev/Kconfig | 5 - > 12 files changed, 2 insertions(+), 25 deletions(-) > > diff --git a/arch/unicore32/Kconfig b/arch/unicore32/Kconfig > index c3a41bf..1138334 100644 > --- a/arch/unicore32/Kconfig > +++ b/arch/unicore32/Kconfig > @@ -195,7 +195,6 @@ config I2C_EEPROM_AT24 > > config LCD_BACKLIGHT > tristate "LCD Backlight support" > - select BACKLIGHT_LCD_SUPPORT > select BACKLIGHT_PWM > > endmenu > diff --git a/drivers/gpu/drm/Kconfig b/drivers/gpu/drm/Kconfig > index 4385f00..ef442a7 100644 > --- a/drivers/gpu/drm/Kconfig > +++ b/drivers/gpu/drm/Kconfig > @@ -198,7 +198,6 @@ config DRM_RADEON > select POWER_SUPPLY > select HWMON > select BACKLIGHT_CLASS_DEVICE > - select BACKLIGHT_LCD_SUPPORT > select INTERVAL_TREE > help > Choose this option if you have an ATI Radeon graphics card. There > @@ -219,7 +218,6 @@ config DRM_AMDGPU > select POWER_SUPPLY > select HWMON > select BACKLIGHT_CLASS_DEVICE > - select BACKLIGHT_LCD_SUPPORT > select INTERVAL_TREE > select CHASH > help > diff --git a/drivers/gpu/drm/bridge/Kconfig b/drivers/gpu/drm/bridge/Kconfig > index 2fee47b..5b5a8e5 100644 > --- a/drivers/gpu/drm/bridge/Kconfig > +++ b/drivers/gpu/drm/bridge/Kconfig > @@ -75,7 +75,6 @@ config DRM_PARADE_PS8622 > depends on OF > select DRM_PANEL > select DRM_KMS_HELPER > - select BACKLIGHT_LCD_SUPPORT > select BACKLIGHT_CLASS_DEVICE > ---help--- > Parade eDP-LVDS bridge chip driver. > diff --git a/drivers/gpu/drm/fsl-dcu/Kconfig b/drivers/gpu/drm/fsl-dcu/Kconfig > index 14a72c4..dc82588 100644 > --- a/drivers/gpu/drm/fsl-dcu/Kconfig > +++ b/drivers/gpu/drm/fsl-dcu/Kconfig > @@ -2,7 +2,6 @@ config DRM_FSL_DCU > tristate "DRM Support for Freescale DCU" > depends on DRM && OF && ARM && COMMON_CLK > select BACKLIGHT_CLASS_DEVICE > - select BACKLIGHT_LCD_SUPPORT > select DRM_KMS_HELPER > select DRM_KMS_CMA_HELPER > select DRM_PANEL > diff --git a/drivers/gpu/drm/i915/Kconfig b/drivers/gpu/drm/i915/Kconfig > index 148be8e..3d5f1cb 100644 > --- a/drivers/gpu/drm/i915/Kconfig > +++ b/drivers/gpu/drm/i915/Kconfig > @@ -15,7 +15,6 @@ config DRM_I915 > select IRQ_WORK > # i915 depends on ACPI_VIDEO when ACPI is enabled > # but for select to work, need to select ACPI_VIDEO's dependencies, ick > - select BACKLIGHT_LCD_SUPPORT if ACPI > select BACKLIGHT_CLASS_DEVICE if ACPI > select INPUT if ACPI > select ACPI_VIDEO if ACPI > diff --git a/drivers/gpu/drm/nouveau/Kconfig b/drivers/gpu/drm/nouveau/Kconfig > index 432c440..fd7b869 100644 > --- a/drivers/gpu/drm/nouveau/Kconfig > +++ b/drivers/gpu/drm/nouveau/Kconfig > @@ -5,14 +5,12 @@ config DRM_NOUVEAU > select DRM_KMS_HELPER > select DRM_TTM > select BACKLIGHT_CLASS_DEVICE if DRM_NOUVEAU_BACKLIGHT > - select BACKLIGHT_LCD_SUPPORT if DRM_NOUVEAU_BACKLIGHT > select ACPI_VIDEO if ACPI && X86 && BACKLIGHT_CLASS_DEVICE && INPUT > select X86_PLATFORM_DEVICES if ACPI && X86 > select ACPI_WMI if ACPI && X86 > select MXM_WMI if ACPI && X86 > select POWER_SUPPLY > # Similar to i915, we need to select ACPI_VIDEO and it's dependencies > - select BACKLIGHT_LCD_SUPPORT if ACPI && X86 > select BACKLIGHT_CLASS_DEVICE if ACPI && X86 > select INPUT if ACPI && X86 > select THERMAL if ACPI && X86 > diff --git a/drivers/gpu/drm/shmobile/Kconfig > b/drivers/gpu/drm/shmobile/Kconfig > index 61bbe8e..e2a6c82 100644 > --- a/drivers/gpu/drm/shmobile/Kconfig > +++ b/drivers/gpu/drm/shmobile/Kconfig > @@ -4,7 +4,6 @@ config DRM_SHMOBILE > depends on DRM && ARM > depends on ARCH_SHMOBILE || COMPILE_TEST > select BACKLIGHT_CLASS_DEVICE > - select BACKLIGHT_LCD_SUPPORT > select DRM_KMS_HELPER > select DRM_KMS_CMA_HELPER > select DRM_GEM_CMA_HELPER > diff --git a/drivers/gpu/drm/tilcdc/Kconfig