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

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

[Alexander Shiyan]

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@mail.ru>
---
 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 b/drivers/gpu/drm/tilcdc/Kconfig
index 5259804..cb7df20 100644
--- a/drivers/gpu/drm/tilcdc/Kconfig
+++ b/drivers/gpu/drm/tilcdc/Kconfig
@@ -8,7 +8,6 @@ config DRM_TILCDC
 	select DRM_PANEL_BRIDGE
 	select VIDEOMODE_HELPERS
 	select BACKLIGHT_CLASS_DEVICE
-	select BACKLIGHT_LCD_SUPPORT
 	help
 	  Choose this option if you have an TI SoC with LCDC display
 	  controller, for example AM33xx in beagle-bone, DA8xx, or
diff --git a/drivers/staging/olpc_dcon/Kconfig b/drivers/staging/olpc_dcon/Kconfig
index 192cc8d..c91a56f 100644
--- a/drivers/staging/olpc_dcon/Kconfig
+++ b/drivers/staging/olpc_dcon/Kconfig
@@ -2,7 +2,6 @@ config FB_OLPC_DCON
 	tristate "One Laptop Per Child Display CONtroller support"
 	depends on OLPC && FB
 	depends on I2C
-	depends on BACKLIGHT_LCD_SUPPORT
 	depends on (GPIO_CS5535 || GPIO_CS5535=n)
 	select BACKLIGHT_CLASS_DEVICE
 	help
diff --git a/drivers/usb/misc/Kconfig b/drivers/usb/misc/Kconfig
index 68d2f2c..a86a80b 100644
--- a/drivers/usb/misc/Kconfig
+++ b/drivers/usb/misc/Kconfig
@@ -141,7 +141,6 @@ config USB_FTDI_ELAN
 
 config USB_APPLEDISPLAY
 	tristate "Apple Cinema Display support"
-	select BACKLIGHT_LCD_SUPPORT
 	select BACKLIGHT_CLASS_DEVICE
 	help
 	  Say Y here if you want to control the backlight of Apple Cinema
diff --git a/drivers/video/backlight/Kconfig b/drivers/video/backlight/Kconfig
index 71ee978..3fdc18e 100644
--- 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
@@ -466,4 +460,4 @@ config BACKLIGHT_RAVE_SP
 
 endif # BACKLIGHT_CLASS_DEVICE
 
-endif # BACKLIGHT_LCD_SUPPORT
+endmenu
diff --git a/drivers/video/fbdev/Kconfig b/drivers/video/fbdev/Kconfig
index ae7712c..7b5c404 100644
--- a/drivers/video/fbdev/Kconfig
+++ b/drivers/video/fbdev/Kconfig
@@ -186,7 +186,6 @@ config FB_MACMODES
 config FB_BACKLIGHT
 	tristate
 	depends on FB
-	select BACKLIGHT_LCD_SUPPORT
 	select BACKLIGHT_CLASS_DEVICE
 
 config FB_MODE_HELPERS
@@ -281,7 +280,6 @@ config FB_ARMCLCD
 	select FB_CFB_IMAGEBLIT
 	select FB_MODE_HELPERS if OF
 	select VIDEOMODE_HELPERS if OF
-	select BACKLIGHT_LCD_SUPPORT if OF
 	select BACKLIGHT_CLASS_DEVICE if OF
 	help
 	  This framebuffer device driver is for the ARM PrimeCell PL110
@@ -315,7 +313,6 @@ config FB_ACORN
 config FB_CLPS711X
 	tristate "CLPS711X LCD support"
 	depends on FB && (ARCH_CLPS711X || COMPILE_TEST)
-	select BACKLIGHT_LCD_SUPPORT
 	select FB_MODE_HELPERS
 	select FB_SYS_FILLRECT
 	select FB_SYS_COPYAREA
@@ -343,7 +340,6 @@ config FB_SA1100
 config FB_IMX
 	tristate "Freescale i.MX1/21/25/27 LCD support"
 	depends on FB && ARCH_MXC
-	select BACKLIGHT_LCD_SUPPORT
 	select LCD_CLASS_DEVICE
 	select FB_CFB_FILLRECT
 	select FB_CFB_COPYAREA
@@ -2192,7 +2188,6 @@ config FB_MX3
 	tristate "MX3 Framebuffer support"
 	depends on FB && MX3_IPU
 	select BACKLIGHT_CLASS_DEVICE
-	select BACKLIGHT_LCD_SUPPORT
 	select FB_CFB_FILLRECT
 	select FB_CFB_COPYAREA
 	select FB_CFB_IMAGEBLIT
-- 
2.10.2

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

[Alexander Shiyan]

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@mail.ru>
---
 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 b/drivers/gpu/drm/tilcdc/Kconfig
index 5259804..cb7df20 100644
--- a/drivers/gpu/drm/tilcdc/Kconfig
+++ b/drivers/gpu/drm/tilcdc/Kconfig
@@ -8,7 +8,6 @@ config DRM_TILCDC
 	select DRM_PANEL_BRIDGE
 	select VIDEOMODE_HELPERS
 	select BACKLIGHT_CLASS_DEVICE
-	select BACKLIGHT_LCD_SUPPORT
 	help
 	  Choose this option if you have an TI SoC with LCDC display
 	  controller, for example AM33xx in beagle-bone, DA8xx, or
diff --git a/drivers/staging/olpc_dcon/Kconfig b/drivers/staging/olpc_dcon/Kconfig
index 192cc8d..c91a56f 100644
--- a/drivers/staging/olpc_dcon/Kconfig
+++ b/drivers/staging/olpc_dcon/Kconfig
@@ -2,7 +2,6 @@ config FB_OLPC_DCON
 	tristate "One Laptop Per Child Display CONtroller support"
 	depends on OLPC && FB
 	depends on I2C
-	depends on BACKLIGHT_LCD_SUPPORT
 	depends on (GPIO_CS5535 || GPIO_CS5535=n)
 	select BACKLIGHT_CLASS_DEVICE
 	help
diff --git a/drivers/usb/misc/Kconfig b/drivers/usb/misc/Kconfig
index 68d2f2c..a86a80b 100644
--- a/drivers/usb/misc/Kconfig
+++ b/drivers/usb/misc/Kconfig
@@ -141,7 +141,6 @@ config USB_FTDI_ELAN
 
 config USB_APPLEDISPLAY
 	tristate "Apple Cinema Display support"
-	select BACKLIGHT_LCD_SUPPORT
 	select BACKLIGHT_CLASS_DEVICE
 	help
 	  Say Y here if you want to control the backlight of Apple Cinema
diff --git a/drivers/video/backlight/Kconfig b/drivers/video/backlight/Kconfig
index 71ee978..3fdc18e 100644
--- 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
@@ -466,4 +460,4 @@ config BACKLIGHT_RAVE_SP
 
 endif # BACKLIGHT_CLASS_DEVICE
 
-endif # BACKLIGHT_LCD_SUPPORT
+endmenu
diff --git a/drivers/video/fbdev/Kconfig b/drivers/video/fbdev/Kconfig
index ae7712c..7b5c404 100644
--- a/drivers/video/fbdev/Kconfig
+++ b/drivers/video/fbdev/Kconfig
@@ -186,7 +186,6 @@ config FB_MACMODES
 config FB_BACKLIGHT
 	tristate
 	depends on FB
-	select BACKLIGHT_LCD_SUPPORT
 	select BACKLIGHT_CLASS_DEVICE
 
 config FB_MODE_HELPERS
@@ -281,7 +280,6 @@ config FB_ARMCLCD
 	select FB_CFB_IMAGEBLIT
 	select FB_MODE_HELPERS if OF
 	select VIDEOMODE_HELPERS if OF
-	select BACKLIGHT_LCD_SUPPORT if OF
 	select BACKLIGHT_CLASS_DEVICE if OF
 	help
 	  This framebuffer device driver is for the ARM PrimeCell PL110
@@ -315,7 +313,6 @@ config FB_ACORN
 config FB_CLPS711X
 	tristate "CLPS711X LCD support"
 	depends on FB && (ARCH_CLPS711X || COMPILE_TEST)
-	select BACKLIGHT_LCD_SUPPORT
 	select FB_MODE_HELPERS
 	select FB_SYS_FILLRECT
 	select FB_SYS_COPYAREA
@@ -343,7 +340,6 @@ config FB_SA1100
 config FB_IMX
 	tristate "Freescale i.MX1/21/25/27 LCD support"
 	depends on FB && ARCH_MXC
-	select BACKLIGHT_LCD_SUPPORT
 	select LCD_CLASS_DEVICE
 	select FB_CFB_FILLRECT
 	select FB_CFB_COPYAREA
@@ -2192,7 +2188,6 @@ config FB_MX3
 	tristate "MX3 Framebuffer support"
 	depends on FB && MX3_IPU
 	select BACKLIGHT_CLASS_DEVICE
-	select BACKLIGHT_LCD_SUPPORT
 	select FB_CFB_FILLRECT
 	select FB_CFB_COPYAREA
 	select FB_CFB_IMAGEBLIT
-- 
2.10.2

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

[PATCH v2 2/2] video: lcd: Remove useless BACKLIGHT_CLASS_DEVICE dependencies

[Alexander Shiyan]

This patch removes dependencies on BACKLIGHT_CLASS_DEVICE for items
that are already placed under #if BACKLIGHT_CLASS_DEVICE.

Signed-off-by: Alexander Shiyan <shc_work@mail.ru>
---
 drivers/video/backlight/Kconfig | 25 ++++++++++++-------------
 1 file changed, 12 insertions(+), 13 deletions(-)

diff --git a/drivers/video/backlight/Kconfig b/drivers/video/backlight/Kconfig
index 3fdc18e..3ed1d90 100644
--- a/drivers/video/backlight/Kconfig
+++ b/drivers/video/backlight/Kconfig
@@ -193,7 +193,6 @@ config BACKLIGHT_IPAQ_MICRO
 
 config BACKLIGHT_LM3533
 	tristate "Backlight Driver for LM3533"
-	depends on BACKLIGHT_CLASS_DEVICE
 	depends on MFD_LM3533
 	help
 	  Say Y to enable the backlight driver for National Semiconductor / TI
@@ -317,7 +316,7 @@ config BACKLIGHT_ADP5520
 
 config BACKLIGHT_ADP8860
 	tristate "Backlight Driver for ADP8860/ADP8861/ADP8863 using WLED"
-	depends on BACKLIGHT_CLASS_DEVICE && I2C
+	depends on I2C
 	select NEW_LEDS
 	select LEDS_CLASS
 	help
@@ -329,7 +328,7 @@ config BACKLIGHT_ADP8860
 
 config BACKLIGHT_ADP8870
 	tristate "Backlight Driver for ADP8870 using WLED"
-	depends on BACKLIGHT_CLASS_DEVICE && I2C
+	depends on I2C
 	select NEW_LEDS
 	select LEDS_CLASS
 	help
@@ -347,28 +346,28 @@ config BACKLIGHT_88PM860X
 
 config BACKLIGHT_PCF50633
 	tristate "Backlight driver for NXP PCF50633 MFD"
-	depends on BACKLIGHT_CLASS_DEVICE && MFD_PCF50633
+	depends on MFD_PCF50633
 	help
 	  If you have a backlight driven by a NXP PCF50633 MFD, say Y here to
 	  enable its driver.
 
 config BACKLIGHT_AAT2870
 	tristate "AnalogicTech AAT2870 Backlight"
-	depends on BACKLIGHT_CLASS_DEVICE && MFD_AAT2870_CORE
+	depends on MFD_AAT2870_CORE
 	help
 	  If you have a AnalogicTech AAT2870 say Y to enable the
 	  backlight driver.
 
 config BACKLIGHT_LM3630A
 	tristate "Backlight Driver for LM3630A"
-	depends on BACKLIGHT_CLASS_DEVICE && I2C && PWM
+	depends on I2C && PWM
 	select REGMAP_I2C
 	help
 	  This supports TI LM3630A Backlight Driver
 
 config BACKLIGHT_LM3639
 	tristate "Backlight Driver for LM3639"
-	depends on BACKLIGHT_CLASS_DEVICE && I2C
+	depends on I2C
 	select REGMAP_I2C
 	select NEW_LEDS
 	select LEDS_CLASS
@@ -377,20 +376,20 @@ config BACKLIGHT_LM3639
 
 config BACKLIGHT_LP855X
 	tristate "Backlight driver for TI LP855X"
-	depends on BACKLIGHT_CLASS_DEVICE && I2C && PWM
+	depends on I2C && PWM
 	help
 	  This supports TI LP8550, LP8551, LP8552, LP8553, LP8555, LP8556 and
 	  LP8557 backlight driver.
 
 config BACKLIGHT_LP8788
 	tristate "Backlight driver for TI LP8788 MFD"
-	depends on BACKLIGHT_CLASS_DEVICE && MFD_LP8788 && PWM
+	depends on MFD_LP8788 && PWM
 	help
 	  This supports TI LP8788 backlight driver.
 
 config BACKLIGHT_OT200
 	tristate "Backlight driver for ot200 visualisation device"
-	depends on BACKLIGHT_CLASS_DEVICE && CS5535_MFGPT && GPIO_CS5535
+	depends on CS5535_MFGPT && GPIO_CS5535
 	help
 	  To compile this driver as a module, choose M here: the module will be
 	  called ot200_bl.
@@ -404,7 +403,7 @@ config BACKLIGHT_PANDORA
 
 config BACKLIGHT_SKY81452
 	tristate "Backlight driver for SKY81452"
-	depends on BACKLIGHT_CLASS_DEVICE && MFD_SKY81452
+	depends on MFD_SKY81452
 	help
 	  If you have a Skyworks SKY81452, say Y to enable the
 	  backlight driver.
@@ -414,14 +413,14 @@ config BACKLIGHT_SKY81452
 
 config BACKLIGHT_TPS65217
 	tristate "TPS65217 Backlight"
-	depends on BACKLIGHT_CLASS_DEVICE && MFD_TPS65217
+	depends on MFD_TPS65217
 	help
 	  If you have a Texas Instruments TPS65217 say Y to enable the
 	  backlight driver.
 
 config BACKLIGHT_AS3711
 	tristate "AS3711 Backlight"
-	depends on BACKLIGHT_CLASS_DEVICE && MFD_AS3711
+	depends on MFD_AS3711
 	help
 	  If you have an Austrian Microsystems AS3711 say Y to enable the
 	  backlight driver.
-- 
2.10.2

[PATCH v2 2/2] video: lcd: Remove useless BACKLIGHT_CLASS_DEVICE dependencies

[Alexander Shiyan]

This patch removes dependencies on BACKLIGHT_CLASS_DEVICE for items
that are already placed under #if BACKLIGHT_CLASS_DEVICE.

Signed-off-by: Alexander Shiyan <shc_work@mail.ru>
---
 drivers/video/backlight/Kconfig | 25 ++++++++++++-------------
 1 file changed, 12 insertions(+), 13 deletions(-)

diff --git a/drivers/video/backlight/Kconfig b/drivers/video/backlight/Kconfig
index 3fdc18e..3ed1d90 100644
--- a/drivers/video/backlight/Kconfig
+++ b/drivers/video/backlight/Kconfig
@@ -193,7 +193,6 @@ config BACKLIGHT_IPAQ_MICRO
 
 config BACKLIGHT_LM3533
 	tristate "Backlight Driver for LM3533"
-	depends on BACKLIGHT_CLASS_DEVICE
 	depends on MFD_LM3533
 	help
 	  Say Y to enable the backlight driver for National Semiconductor / TI
@@ -317,7 +316,7 @@ config BACKLIGHT_ADP5520
 
 config BACKLIGHT_ADP8860
 	tristate "Backlight Driver for ADP8860/ADP8861/ADP8863 using WLED"
-	depends on BACKLIGHT_CLASS_DEVICE && I2C
+	depends on I2C
 	select NEW_LEDS
 	select LEDS_CLASS
 	help
@@ -329,7 +328,7 @@ config BACKLIGHT_ADP8860
 
 config BACKLIGHT_ADP8870
 	tristate "Backlight Driver for ADP8870 using WLED"
-	depends on BACKLIGHT_CLASS_DEVICE && I2C
+	depends on I2C
 	select NEW_LEDS
 	select LEDS_CLASS
 	help
@@ -347,28 +346,28 @@ config BACKLIGHT_88PM860X
 
 config BACKLIGHT_PCF50633
 	tristate "Backlight driver for NXP PCF50633 MFD"
-	depends on BACKLIGHT_CLASS_DEVICE && MFD_PCF50633
+	depends on MFD_PCF50633
 	help
 	  If you have a backlight driven by a NXP PCF50633 MFD, say Y here to
 	  enable its driver.
 
 config BACKLIGHT_AAT2870
 	tristate "AnalogicTech AAT2870 Backlight"
-	depends on BACKLIGHT_CLASS_DEVICE && MFD_AAT2870_CORE
+	depends on MFD_AAT2870_CORE
 	help
 	  If you have a AnalogicTech AAT2870 say Y to enable the
 	  backlight driver.
 
 config BACKLIGHT_LM3630A
 	tristate "Backlight Driver for LM3630A"
-	depends on BACKLIGHT_CLASS_DEVICE && I2C && PWM
+	depends on I2C && PWM
 	select REGMAP_I2C
 	help
 	  This supports TI LM3630A Backlight Driver
 
 config BACKLIGHT_LM3639
 	tristate "Backlight Driver for LM3639"
-	depends on BACKLIGHT_CLASS_DEVICE && I2C
+	depends on I2C
 	select REGMAP_I2C
 	select NEW_LEDS
 	select LEDS_CLASS
@@ -377,20 +376,20 @@ config BACKLIGHT_LM3639
 
 config BACKLIGHT_LP855X
 	tristate "Backlight driver for TI LP855X"
-	depends on BACKLIGHT_CLASS_DEVICE && I2C && PWM
+	depends on I2C && PWM
 	help
 	  This supports TI LP8550, LP8551, LP8552, LP8553, LP8555, LP8556 and
 	  LP8557 backlight driver.
 
 config BACKLIGHT_LP8788
 	tristate "Backlight driver for TI LP8788 MFD"
-	depends on BACKLIGHT_CLASS_DEVICE && MFD_LP8788 && PWM
+	depends on MFD_LP8788 && PWM
 	help
 	  This supports TI LP8788 backlight driver.
 
 config BACKLIGHT_OT200
 	tristate "Backlight driver for ot200 visualisation device"
-	depends on BACKLIGHT_CLASS_DEVICE && CS5535_MFGPT && GPIO_CS5535
+	depends on CS5535_MFGPT && GPIO_CS5535
 	help
 	  To compile this driver as a module, choose M here: the module will be
 	  called ot200_bl.
@@ -404,7 +403,7 @@ config BACKLIGHT_PANDORA
 
 config BACKLIGHT_SKY81452
 	tristate "Backlight driver for SKY81452"
-	depends on BACKLIGHT_CLASS_DEVICE && MFD_SKY81452
+	depends on MFD_SKY81452
 	help
 	  If you have a Skyworks SKY81452, say Y to enable the
 	  backlight driver.
@@ -414,14 +413,14 @@ config BACKLIGHT_SKY81452
 
 config BACKLIGHT_TPS65217
 	tristate "TPS65217 Backlight"
-	depends on BACKLIGHT_CLASS_DEVICE && MFD_TPS65217
+	depends on MFD_TPS65217
 	help
 	  If you have a Texas Instruments TPS65217 say Y to enable the
 	  backlight driver.
 
 config BACKLIGHT_AS3711
 	tristate "AS3711 Backlight"
-	depends on BACKLIGHT_CLASS_DEVICE && MFD_AS3711
+	depends on MFD_AS3711
 	help
 	  If you have an Austrian Microsystems AS3711 say Y to enable the
 	  backlight driver.
-- 
2.10.2

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: [PATCH v2 2/2] video: lcd: Remove useless BACKLIGHT_CLASS_DEVICE dependencies

[Daniel Thompson]

On Thu, Jan 17, 2019 at 04:33:36PM +0300, Alexander Shiyan wrote:
> This patch removes dependencies on BACKLIGHT_CLASS_DEVICE for items
> that are already placed under #if BACKLIGHT_CLASS_DEVICE.

Why the # before the if (in Kconfig its just "if" right?).

> 
> Signed-off-by: Alexander Shiyan <shc_work@mail.ru>

Any didn't I ack this already?


Daniel.


> ---
>  drivers/video/backlight/Kconfig | 25 ++++++++++++-------------
>  1 file changed, 12 insertions(+), 13 deletions(-)
> 
> diff --git a/drivers/video/backlight/Kconfig b/drivers/video/backlight/Kconfig
> index 3fdc18e..3ed1d90 100644
> --- a/drivers/video/backlight/Kconfig
> +++ b/drivers/video/backlight/Kconfig
> @@ -193,7 +193,6 @@ config BACKLIGHT_IPAQ_MICRO
>  
>  config BACKLIGHT_LM3533
>  	tristate "Backlight Driver for LM3533"
> -	depends on BACKLIGHT_CLASS_DEVICE
>  	depends on MFD_LM3533
>  	help
>  	  Say Y to enable the backlight driver for National Semiconductor / TI
> @@ -317,7 +316,7 @@ config BACKLIGHT_ADP5520
>  
>  config BACKLIGHT_ADP8860
>  	tristate "Backlight Driver for ADP8860/ADP8861/ADP8863 using WLED"
> -	depends on BACKLIGHT_CLASS_DEVICE && I2C
> +	depends on I2C
>  	select NEW_LEDS
>  	select LEDS_CLASS
>  	help
> @@ -329,7 +328,7 @@ config BACKLIGHT_ADP8860
>  
>  config BACKLIGHT_ADP8870
>  	tristate "Backlight Driver for ADP8870 using WLED"
> -	depends on BACKLIGHT_CLASS_DEVICE && I2C
> +	depends on I2C
>  	select NEW_LEDS
>  	select LEDS_CLASS
>  	help
> @@ -347,28 +346,28 @@ config BACKLIGHT_88PM860X
>  
>  config BACKLIGHT_PCF50633
>  	tristate "Backlight driver for NXP PCF50633 MFD"
> -	depends on BACKLIGHT_CLASS_DEVICE && MFD_PCF50633
> +	depends on MFD_PCF50633
>  	help
>  	  If you have a backlight driven by a NXP PCF50633 MFD, say Y here to
>  	  enable its driver.
>  
>  config BACKLIGHT_AAT2870
>  	tristate "AnalogicTech AAT2870 Backlight"
> -	depends on BACKLIGHT_CLASS_DEVICE && MFD_AAT2870_CORE
> +	depends on MFD_AAT2870_CORE
>  	help
>  	  If you have a AnalogicTech AAT2870 say Y to enable the
>  	  backlight driver.
>  
>  config BACKLIGHT_LM3630A
>  	tristate "Backlight Driver for LM3630A"
> -	depends on BACKLIGHT_CLASS_DEVICE && I2C && PWM
> +	depends on I2C && PWM
>  	select REGMAP_I2C
>  	help
>  	  This supports TI LM3630A Backlight Driver
>  
>  config BACKLIGHT_LM3639
>  	tristate "Backlight Driver for LM3639"
> -	depends on BACKLIGHT_CLASS_DEVICE && I2C
> +	depends on I2C
>  	select REGMAP_I2C
>  	select NEW_LEDS
>  	select LEDS_CLASS
> @@ -377,20 +376,20 @@ config BACKLIGHT_LM3639
>  
>  config BACKLIGHT_LP855X
>  	tristate "Backlight driver for TI LP855X"
> -	depends on BACKLIGHT_CLASS_DEVICE && I2C && PWM
> +	depends on I2C && PWM
>  	help
>  	  This supports TI LP8550, LP8551, LP8552, LP8553, LP8555, LP8556 and
>  	  LP8557 backlight driver.
>  
>  config BACKLIGHT_LP8788
>  	tristate "Backlight driver for TI LP8788 MFD"
> -	depends on BACKLIGHT_CLASS_DEVICE && MFD_LP8788 && PWM
> +	depends on MFD_LP8788 && PWM
>  	help
>  	  This supports TI LP8788 backlight driver.
>  
>  config BACKLIGHT_OT200
>  	tristate "Backlight driver for ot200 visualisation device"
> -	depends on BACKLIGHT_CLASS_DEVICE && CS5535_MFGPT && GPIO_CS5535
> +	depends on CS5535_MFGPT && GPIO_CS5535
>  	help
>  	  To compile this driver as a module, choose M here: the module will be
>  	  called ot200_bl.
> @@ -404,7 +403,7 @@ config BACKLIGHT_PANDORA
>  
>  config BACKLIGHT_SKY81452
>  	tristate "Backlight driver for SKY81452"
> -	depends on BACKLIGHT_CLASS_DEVICE && MFD_SKY81452
> +	depends on MFD_SKY81452
>  	help
>  	  If you have a Skyworks SKY81452, say Y to enable the
>  	  backlight driver.
> @@ -414,14 +413,14 @@ config BACKLIGHT_SKY81452
>  
>  config BACKLIGHT_TPS65217
>  	tristate "TPS65217 Backlight"
> -	depends on BACKLIGHT_CLASS_DEVICE && MFD_TPS65217
> +	depends on MFD_TPS65217
>  	help
>  	  If you have a Texas Instruments TPS65217 say Y to enable the
>  	  backlight driver.
>  
>  config BACKLIGHT_AS3711
>  	tristate "AS3711 Backlight"
> -	depends on BACKLIGHT_CLASS_DEVICE && MFD_AS3711
> +	depends on MFD_AS3711
>  	help
>  	  If you have an Austrian Microsystems AS3711 say Y to enable the
>  	  backlight driver.
> -- 
> 2.10.2
> 

Re: [PATCH v2 2/2] video: lcd: Remove useless BACKLIGHT_CLASS_DEVICE dependencies

[Daniel Thompson]

On Thu, Jan 17, 2019 at 04:33:36PM +0300, Alexander Shiyan wrote:
> This patch removes dependencies on BACKLIGHT_CLASS_DEVICE for items
> that are already placed under #if BACKLIGHT_CLASS_DEVICE.

Why the # before the if (in Kconfig its just "if" right?).

> 
> Signed-off-by: Alexander Shiyan <shc_work@mail.ru>

Any didn't I ack this already?


Daniel.


> ---
>  drivers/video/backlight/Kconfig | 25 ++++++++++++-------------
>  1 file changed, 12 insertions(+), 13 deletions(-)
> 
> diff --git a/drivers/video/backlight/Kconfig b/drivers/video/backlight/Kconfig
> index 3fdc18e..3ed1d90 100644
> --- a/drivers/video/backlight/Kconfig
> +++ b/drivers/video/backlight/Kconfig
> @@ -193,7 +193,6 @@ config BACKLIGHT_IPAQ_MICRO
>  
>  config BACKLIGHT_LM3533
>  	tristate "Backlight Driver for LM3533"
> -	depends on BACKLIGHT_CLASS_DEVICE
>  	depends on MFD_LM3533
>  	help
>  	  Say Y to enable the backlight driver for National Semiconductor / TI
> @@ -317,7 +316,7 @@ config BACKLIGHT_ADP5520
>  
>  config BACKLIGHT_ADP8860
>  	tristate "Backlight Driver for ADP8860/ADP8861/ADP8863 using WLED"
> -	depends on BACKLIGHT_CLASS_DEVICE && I2C
> +	depends on I2C
>  	select NEW_LEDS
>  	select LEDS_CLASS
>  	help
> @@ -329,7 +328,7 @@ config BACKLIGHT_ADP8860
>  
>  config BACKLIGHT_ADP8870
>  	tristate "Backlight Driver for ADP8870 using WLED"
> -	depends on BACKLIGHT_CLASS_DEVICE && I2C
> +	depends on I2C
>  	select NEW_LEDS
>  	select LEDS_CLASS
>  	help
> @@ -347,28 +346,28 @@ config BACKLIGHT_88PM860X
>  
>  config BACKLIGHT_PCF50633
>  	tristate "Backlight driver for NXP PCF50633 MFD"
> -	depends on BACKLIGHT_CLASS_DEVICE && MFD_PCF50633
> +	depends on MFD_PCF50633
>  	help
>  	  If you have a backlight driven by a NXP PCF50633 MFD, say Y here to
>  	  enable its driver.
>  
>  config BACKLIGHT_AAT2870
>  	tristate "AnalogicTech AAT2870 Backlight"
> -	depends on BACKLIGHT_CLASS_DEVICE && MFD_AAT2870_CORE
> +	depends on MFD_AAT2870_CORE
>  	help
>  	  If you have a AnalogicTech AAT2870 say Y to enable the
>  	  backlight driver.
>  
>  config BACKLIGHT_LM3630A
>  	tristate "Backlight Driver for LM3630A"
> -	depends on BACKLIGHT_CLASS_DEVICE && I2C && PWM
> +	depends on I2C && PWM
>  	select REGMAP_I2C
>  	help
>  	  This supports TI LM3630A Backlight Driver
>  
>  config BACKLIGHT_LM3639
>  	tristate "Backlight Driver for LM3639"
> -	depends on BACKLIGHT_CLASS_DEVICE && I2C
> +	depends on I2C
>  	select REGMAP_I2C
>  	select NEW_LEDS
>  	select LEDS_CLASS
> @@ -377,20 +376,20 @@ config BACKLIGHT_LM3639
>  
>  config BACKLIGHT_LP855X
>  	tristate "Backlight driver for TI LP855X"
> -	depends on BACKLIGHT_CLASS_DEVICE && I2C && PWM
> +	depends on I2C && PWM
>  	help
>  	  This supports TI LP8550, LP8551, LP8552, LP8553, LP8555, LP8556 and
>  	  LP8557 backlight driver.
>  
>  config BACKLIGHT_LP8788
>  	tristate "Backlight driver for TI LP8788 MFD"
> -	depends on BACKLIGHT_CLASS_DEVICE && MFD_LP8788 && PWM
> +	depends on MFD_LP8788 && PWM
>  	help
>  	  This supports TI LP8788 backlight driver.
>  
>  config BACKLIGHT_OT200
>  	tristate "Backlight driver for ot200 visualisation device"
> -	depends on BACKLIGHT_CLASS_DEVICE && CS5535_MFGPT && GPIO_CS5535
> +	depends on CS5535_MFGPT && GPIO_CS5535
>  	help
>  	  To compile this driver as a module, choose M here: the module will be
>  	  called ot200_bl.
> @@ -404,7 +403,7 @@ config BACKLIGHT_PANDORA
>  
>  config BACKLIGHT_SKY81452
>  	tristate "Backlight driver for SKY81452"
> -	depends on BACKLIGHT_CLASS_DEVICE && MFD_SKY81452
> +	depends on MFD_SKY81452
>  	help
>  	  If you have a Skyworks SKY81452, say Y to enable the
>  	  backlight driver.
> @@ -414,14 +413,14 @@ config BACKLIGHT_SKY81452
>  
>  config BACKLIGHT_TPS65217
>  	tristate "TPS65217 Backlight"
> -	depends on BACKLIGHT_CLASS_DEVICE && MFD_TPS65217
> +	depends on MFD_TPS65217
>  	help
>  	  If you have a Texas Instruments TPS65217 say Y to enable the
>  	  backlight driver.
>  
>  config BACKLIGHT_AS3711
>  	tristate "AS3711 Backlight"
> -	depends on BACKLIGHT_CLASS_DEVICE && MFD_AS3711
> +	depends on MFD_AS3711
>  	help
>  	  If you have an Austrian Microsystems AS3711 say Y to enable the
>  	  backlight driver.
> -- 
> 2.10.2
> 
_______________________________________________
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

[Daniel Thompson]

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 <shc_work@mail.ru>

A cover letter with the v1 -> v2 changelog would be nice... but
nevertheless:
Acked-by: Daniel Thompson <daniel.thompson@linaro.org>

> ---
>  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 b/drivers/gpu/drm/tilcdc/Kconfig
> index 5259804..cb7df20 100644
> --- a/drivers/gpu/drm/tilcdc/Kconfig
> +++ b/drivers/gpu/drm/tilcdc/Kconfig
> @@ -8,7 +8,6 @@ config DRM_TILCDC
>  	select DRM_PANEL_BRIDGE
>  	select VIDEOMODE_HELPERS
>  	select BACKLIGHT_CLASS_DEVICE
> -	select BACKLIGHT_LCD_SUPPORT
>  	help
>  	  Choose this option if you have an TI SoC with LCDC display
>  	  controller, for example AM33xx in beagle-bone, DA8xx, or
> diff --git a/drivers/staging/olpc_dcon/Kconfig b/drivers/staging/olpc_dcon/Kconfig
> index 192cc8d..c91a56f 100644
> --- a/drivers/staging/olpc_dcon/Kconfig
> +++ b/drivers/staging/olpc_dcon/Kconfig
> @@ -2,7 +2,6 @@ config FB_OLPC_DCON
>  	tristate "One Laptop Per Child Display CONtroller support"
>  	depends on OLPC && FB
>  	depends on I2C
> -	depends on BACKLIGHT_LCD_SUPPORT
>  	depends on (GPIO_CS5535 || GPIO_CS5535=n)
>  	select BACKLIGHT_CLASS_DEVICE
>  	help
> diff --git a/drivers/usb/misc/Kconfig b/drivers/usb/misc/Kconfig
> index 68d2f2c..a86a80b 100644
> --- a/drivers/usb/misc/Kconfig
> +++ b/drivers/usb/misc/Kconfig
> @@ -141,7 +141,6 @@ config USB_FTDI_ELAN
>  
>  config USB_APPLEDISPLAY
>  	tristate "Apple Cinema Display support"
> -	select BACKLIGHT_LCD_SUPPORT
>  	select BACKLIGHT_CLASS_DEVICE
>  	help
>  	  Say Y here if you want to control the backlight of Apple Cinema
> diff --git a/drivers/video/backlight/Kconfig b/drivers/video/backlight/Kconfig
> index 71ee978..3fdc18e 100644
> --- 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
> @@ -466,4 +460,4 @@ config BACKLIGHT_RAVE_SP
>  
>  endif # BACKLIGHT_CLASS_DEVICE
>  
> -endif # BACKLIGHT_LCD_SUPPORT
> +endmenu
> diff --git a/drivers/video/fbdev/Kconfig b/drivers/video/fbdev/Kconfig
> index ae7712c..7b5c404 100644
> --- a/drivers/video/fbdev/Kconfig
> +++ b/drivers/video/fbdev/Kconfig
> @@ -186,7 +186,6 @@ config FB_MACMODES
>  config FB_BACKLIGHT
>  	tristate
>  	depends on FB
> -	select BACKLIGHT_LCD_SUPPORT
>  	select BACKLIGHT_CLASS_DEVICE
>  
>  config FB_MODE_HELPERS
> @@ -281,7 +280,6 @@ config FB_ARMCLCD
>  	select FB_CFB_IMAGEBLIT
>  	select FB_MODE_HELPERS if OF
>  	select VIDEOMODE_HELPERS if OF
> -	select BACKLIGHT_LCD_SUPPORT if OF
>  	select BACKLIGHT_CLASS_DEVICE if OF
>  	help
>  	  This framebuffer device driver is for the ARM PrimeCell PL110
> @@ -315,7 +313,6 @@ config FB_ACORN
>  config FB_CLPS711X
>  	tristate "CLPS711X LCD support"
>  	depends on FB && (ARCH_CLPS711X || COMPILE_TEST)
> -	select BACKLIGHT_LCD_SUPPORT
>  	select FB_MODE_HELPERS
>  	select FB_SYS_FILLRECT
>  	select FB_SYS_COPYAREA
> @@ -343,7 +340,6 @@ config FB_SA1100
>  config FB_IMX
>  	tristate "Freescale i.MX1/21/25/27 LCD support"
>  	depends on FB && ARCH_MXC
> -	select BACKLIGHT_LCD_SUPPORT
>  	select LCD_CLASS_DEVICE
>  	select FB_CFB_FILLRECT
>  	select FB_CFB_COPYAREA
> @@ -2192,7 +2188,6 @@ config FB_MX3
>  	tristate "MX3 Framebuffer support"
>  	depends on FB && MX3_IPU
>  	select BACKLIGHT_CLASS_DEVICE
> -	select BACKLIGHT_LCD_SUPPORT
>  	select FB_CFB_FILLRECT
>  	select FB_CFB_COPYAREA
>  	select FB_CFB_IMAGEBLIT
> -- 
> 2.10.2
> 

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

[Daniel Thompson]

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 <shc_work@mail.ru>

A cover letter with the v1 -> v2 changelog would be nice... but
nevertheless:
Acked-by: Daniel Thompson <daniel.thompson@linaro.org>

> ---
>  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 b/drivers/gpu/drm/tilcdc/Kconfig
> index 5259804..cb7df20 100644
> --- a/drivers/gpu/drm/tilcdc/Kconfig
> +++ b/drivers/gpu/drm/tilcdc/Kconfig
> @@ -8,7 +8,6 @@ config DRM_TILCDC
>  	select DRM_PANEL_BRIDGE
>  	select VIDEOMODE_HELPERS
>  	select BACKLIGHT_CLASS_DEVICE
> -	select BACKLIGHT_LCD_SUPPORT
>  	help
>  	  Choose this option if you have an TI SoC with LCDC display
>  	  controller, for example AM33xx in beagle-bone, DA8xx, or
> diff --git a/drivers/staging/olpc_dcon/Kconfig b/drivers/staging/olpc_dcon/Kconfig
> index 192cc8d..c91a56f 100644
> --- a/drivers/staging/olpc_dcon/Kconfig
> +++ b/drivers/staging/olpc_dcon/Kconfig
> @@ -2,7 +2,6 @@ config FB_OLPC_DCON
>  	tristate "One Laptop Per Child Display CONtroller support"
>  	depends on OLPC && FB
>  	depends on I2C
> -	depends on BACKLIGHT_LCD_SUPPORT
>  	depends on (GPIO_CS5535 || GPIO_CS5535=n)
>  	select BACKLIGHT_CLASS_DEVICE
>  	help
> diff --git a/drivers/usb/misc/Kconfig b/drivers/usb/misc/Kconfig
> index 68d2f2c..a86a80b 100644
> --- a/drivers/usb/misc/Kconfig
> +++ b/drivers/usb/misc/Kconfig
> @@ -141,7 +141,6 @@ config USB_FTDI_ELAN
>  
>  config USB_APPLEDISPLAY
>  	tristate "Apple Cinema Display support"
> -	select BACKLIGHT_LCD_SUPPORT
>  	select BACKLIGHT_CLASS_DEVICE
>  	help
>  	  Say Y here if you want to control the backlight of Apple Cinema
> diff --git a/drivers/video/backlight/Kconfig b/drivers/video/backlight/Kconfig
> index 71ee978..3fdc18e 100644
> --- 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
> @@ -466,4 +460,4 @@ config BACKLIGHT_RAVE_SP
>  
>  endif # BACKLIGHT_CLASS_DEVICE
>  
> -endif # BACKLIGHT_LCD_SUPPORT
> +endmenu
> diff --git a/drivers/video/fbdev/Kconfig b/drivers/video/fbdev/Kconfig
> index ae7712c..7b5c404 100644
> --- a/drivers/video/fbdev/Kconfig
> +++ b/drivers/video/fbdev/Kconfig
> @@ -186,7 +186,6 @@ config FB_MACMODES
>  config FB_BACKLIGHT
>  	tristate
>  	depends on FB
> -	select BACKLIGHT_LCD_SUPPORT
>  	select BACKLIGHT_CLASS_DEVICE
>  
>  config FB_MODE_HELPERS
> @@ -281,7 +280,6 @@ config FB_ARMCLCD
>  	select FB_CFB_IMAGEBLIT
>  	select FB_MODE_HELPERS if OF
>  	select VIDEOMODE_HELPERS if OF
> -	select BACKLIGHT_LCD_SUPPORT if OF
>  	select BACKLIGHT_CLASS_DEVICE if OF
>  	help
>  	  This framebuffer device driver is for the ARM PrimeCell PL110
> @@ -315,7 +313,6 @@ config FB_ACORN
>  config FB_CLPS711X
>  	tristate "CLPS711X LCD support"
>  	depends on FB && (ARCH_CLPS711X || COMPILE_TEST)
> -	select BACKLIGHT_LCD_SUPPORT
>  	select FB_MODE_HELPERS
>  	select FB_SYS_FILLRECT
>  	select FB_SYS_COPYAREA
> @@ -343,7 +340,6 @@ config FB_SA1100
>  config FB_IMX
>  	tristate "Freescale i.MX1/21/25/27 LCD support"
>  	depends on FB && ARCH_MXC
> -	select BACKLIGHT_LCD_SUPPORT
>  	select LCD_CLASS_DEVICE
>  	select FB_CFB_FILLRECT
>  	select FB_CFB_COPYAREA
> @@ -2192,7 +2188,6 @@ config FB_MX3
>  	tristate "MX3 Framebuffer support"
>  	depends on FB && MX3_IPU
>  	select BACKLIGHT_CLASS_DEVICE
> -	select BACKLIGHT_LCD_SUPPORT
>  	select FB_CFB_FILLRECT
>  	select FB_CFB_COPYAREA
>  	select FB_CFB_IMAGEBLIT
> -- 
> 2.10.2
> 
_______________________________________________
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

[Bartlomiej Zolnierkiewicz]

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 <shc_work@mail.ru>
> 
> A cover letter with the v1 -> v2 changelog would be nice... but
> nevertheless:
> Acked-by: Daniel Thompson <daniel.thompson@linaro.org>

Also:

Acked-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>

Lee, will you pick this up or do you want me to do it?

Best regards,
--
Bartlomiej Zolnierkiewicz
Samsung R&D Institute Poland
Samsung Electronics

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

[Bartlomiej Zolnierkiewicz]

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 <shc_work@mail.ru>
> 
> A cover letter with the v1 -> v2 changelog would be nice... but
> nevertheless:
> Acked-by: Daniel Thompson <daniel.thompson@linaro.org>

Also:

Acked-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>

Lee, will you pick this up or do you want me to do it?

Best regards,
--
Bartlomiej Zolnierkiewicz
Samsung R&D Institute Poland
Samsung Electronics
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: [PATCH v2 2/2] video: lcd: Remove useless BACKLIGHT_CLASS_DEVICE dependencies

[Bartlomiej Zolnierkiewicz]

On 01/17/2019 02:33 PM, Alexander Shiyan wrote:
> This patch removes dependencies on BACKLIGHT_CLASS_DEVICE for items
> that are already placed under #if BACKLIGHT_CLASS_DEVICE.
> 
> Signed-off-by: Alexander Shiyan <shc_work@mail.ru>

Acked-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>

Best regards,
--
Bartlomiej Zolnierkiewicz
Samsung R&D Institute Poland
Samsung Electronics

Re: [PATCH v2 2/2] video: lcd: Remove useless BACKLIGHT_CLASS_DEVICE dependencies

[Bartlomiej Zolnierkiewicz]

On 01/17/2019 02:33 PM, Alexander Shiyan wrote:
> This patch removes dependencies on BACKLIGHT_CLASS_DEVICE for items
> that are already placed under #if BACKLIGHT_CLASS_DEVICE.
> 
> Signed-off-by: Alexander Shiyan <shc_work@mail.ru>

Acked-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>

Best regards,
--
Bartlomiej Zolnierkiewicz
Samsung R&D Institute Poland
Samsung Electronics
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: [PATCH v2 2/2] video: lcd: Remove useless BACKLIGHT_CLASS_DEVICE dependencies

[Daniel Thompson]

On Fri, Feb 08, 2019 at 06:20:14PM +0100, Bartlomiej Zolnierkiewicz wrote:
> 
> On 01/17/2019 02:33 PM, Alexander Shiyan wrote:
> > This patch removes dependencies on BACKLIGHT_CLASS_DEVICE for items
> > that are already placed under #if BACKLIGHT_CLASS_DEVICE.
> > 
> > Signed-off-by: Alexander Shiyan <shc_work@mail.ru>
> 
> Acked-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>

Sorry... looks like I sent neither an ack nor a nitpick about the #if
(#if is no more part of Kconfig syntax than #ifdef was).

However patch is perfectly clear so:
Acked-by: Daniel Thompson <daniel.thompson@linaro.org>


Daniel.


> 
> Best regards,
> --
> Bartlomiej Zolnierkiewicz
> Samsung R&D Institute Poland
> Samsung Electronics

Re: [PATCH v2 2/2] video: lcd: Remove useless BACKLIGHT_CLASS_DEVICE dependencies

[Daniel Thompson]

On Fri, Feb 08, 2019 at 06:20:14PM +0100, Bartlomiej Zolnierkiewicz wrote:
> 
> On 01/17/2019 02:33 PM, Alexander Shiyan wrote:
> > This patch removes dependencies on BACKLIGHT_CLASS_DEVICE for items
> > that are already placed under #if BACKLIGHT_CLASS_DEVICE.
> > 
> > Signed-off-by: Alexander Shiyan <shc_work@mail.ru>
> 
> Acked-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>

Sorry... looks like I sent neither an ack nor a nitpick about the #if
(#if is no more part of Kconfig syntax than #ifdef was).

However patch is perfectly clear so:
Acked-by: Daniel Thompson <daniel.thompson@linaro.org>


Daniel.


> 
> Best regards,
> --
> Bartlomiej Zolnierkiewicz
> Samsung R&D 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

[Lee Jones]

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 <shc_work@mail.ru>
> > 
> > A cover letter with the v1 -> v2 changelog would be nice... but
> > nevertheless:
> > Acked-by: Daniel Thompson <daniel.thompson@linaro.org>
> 
> Also:
> 
> Acked-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
> 
> 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

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

[Lee Jones via dri-devel]

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 <shc_work@mail.ru>
> > 
> > A cover letter with the v1 -> v2 changelog would be nice... but
> > nevertheless:
> > Acked-by: Daniel Thompson <daniel.thompson@linaro.org>
> 
> Also:
> 
> Acked-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
> 
> 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

[Bartlomiej Zolnierkiewicz]

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 <shc_work@mail.ru>
>>>
>>> A cover letter with the v1 -> v2 changelog would be nice... but
>>> nevertheless:
>>> Acked-by: Daniel Thompson <daniel.thompson@linaro.org>
>>
>> Also:
>>
>> Acked-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
>>
>> 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&D Institute Poland
Samsung Electronics

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

[Bartlomiej Zolnierkiewicz via dri-devel]

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 <shc_work@mail.ru>
>>>
>>> A cover letter with the v1 -> v2 changelog would be nice... but
>>> nevertheless:
>>> Acked-by: Daniel Thompson <daniel.thompson@linaro.org>
>>
>> Also:
>>
>> Acked-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
>>
>> 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&D 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

[Daniel Vetter]

On Wed, Feb 13, 2019 at 4:04 PM Bartlomiej Zolnierkiewicz via
dri-devel <dri-devel@lists.freedesktop.org> 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 <shc_work@mail.ru>
> >>>
> >>> A cover letter with the v1 -> v2 changelog would be nice... but
> >>> nevertheless:
> >>> Acked-by: Daniel Thompson <daniel.thompson@linaro.org>
> >>
> >> Also:
> >>
> >> Acked-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
> >>
> >> 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

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

[Daniel Vetter]

On Wed, Feb 13, 2019 at 4:04 PM Bartlomiej Zolnierkiewicz via
dri-devel <dri-devel@lists.freedesktop.org> 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 <shc_work@mail.ru>
> >>>
> >>> A cover letter with the v1 -> v2 changelog would be nice... but
> >>> nevertheless:
> >>> Acked-by: Daniel Thompson <daniel.thompson@linaro.org>
> >>
> >> Also:
> >>
> >> Acked-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
> >>
> >> 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

[Bartlomiej Zolnierkiewicz]

On 02/13/2019 04:19 PM, Daniel Vetter wrote:
> On Wed, Feb 13, 2019 at 4:04 PM Bartlomiej Zolnierkiewicz via
> dri-devel <dri-devel@lists.freedesktop.org> 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 <shc_work@mail.ru>
>>>>>
>>>>> A cover letter with the v1 -> v2 changelog would be nice... but
>>>>> nevertheless:
>>>>> Acked-by: Daniel Thompson <daniel.thompson@linaro.org>
>>>>
>>>> Also:
>>>>
>>>> Acked-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
>>>>
>>>> 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&D Institute Poland
Samsung Electronics

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

[Bartlomiej Zolnierkiewicz via dri-devel]

On 02/13/2019 04:19 PM, Daniel Vetter wrote:
> On Wed, Feb 13, 2019 at 4:04 PM Bartlomiej Zolnierkiewicz via
> dri-devel <dri-devel@lists.freedesktop.org> 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 <shc_work@mail.ru>
>>>>>
>>>>> A cover letter with the v1 -> v2 changelog would be nice... but
>>>>> nevertheless:
>>>>> Acked-by: Daniel Thompson <daniel.thompson@linaro.org>
>>>>
>>>> Also:
>>>>
>>>> Acked-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
>>>>
>>>> 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&D 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

[Lee Jones]

On Wed, 13 Feb 2019, Daniel Vetter wrote:

> On Wed, Feb 13, 2019 at 4:04 PM Bartlomiej Zolnierkiewicz via
> dri-devel <dri-devel@lists.freedesktop.org> 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 <shc_work@mail.ru>
> > >>>
> > >>> A cover letter with the v1 -> v2 changelog would be nice... but
> > >>> nevertheless:
> > >>> Acked-by: Daniel Thompson <daniel.thompson@linaro.org>
> > >>
> > >> Also:
> > >>
> > >> Acked-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
> > >>
> > >> 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

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

[Lee Jones via dri-devel]

On Wed, 13 Feb 2019, Daniel Vetter wrote:

> On Wed, Feb 13, 2019 at 4:04 PM Bartlomiej Zolnierkiewicz via
> dri-devel <dri-devel@lists.freedesktop.org> 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 <shc_work@mail.ru>
> > >>>
> > >>> A cover letter with the v1 -> v2 changelog would be nice... but
> > >>> nevertheless:
> > >>> Acked-by: Daniel Thompson <daniel.thompson@linaro.org>
> > >>
> > >> Also:
> > >>
> > >> Acked-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
> > >>
> > >> 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

[Lee Jones]

> >> 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 <shc_work@mail.ru>
> >>>>>
> >>>>> A cover letter with the v1 -> v2 changelog would be nice... but
> >>>>> nevertheless:
> >>>>> Acked-by: Daniel Thompson <daniel.thompson@linaro.org>
> >>>>
> >>>> Also:
> >>>>
> >>>> Acked-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
> >>>>
> >>>> 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

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

[Lee Jones]

> >> 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 <shc_work@mail.ru>
> >>>>>
> >>>>> A cover letter with the v1 -> v2 changelog would be nice... but
> >>>>> nevertheless:
> >>>>> Acked-by: Daniel Thompson <daniel.thompson@linaro.org>
> >>>>
> >>>> Also:
> >>>>
> >>>> Acked-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
> >>>>
> >>>> 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

[Bartlomiej Zolnierkiewicz]

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 <shc_work@mail.ru>
>>>>>>>
>>>>>>> A cover letter with the v1 -> v2 changelog would be nice... but
>>>>>>> nevertheless:
>>>>>>> Acked-by: Daniel Thompson <daniel.thompson@linaro.org>
>>>>>>
>>>>>> Also:
>>>>>>
>>>>>> Acked-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
>>>>>>
>>>>>> 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&D Institute Poland
Samsung Electronics

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

[Bartlomiej Zolnierkiewicz]

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 <shc_work@mail.ru>
>>>>>>>
>>>>>>> A cover letter with the v1 -> v2 changelog would be nice... but
>>>>>>> nevertheless:
>>>>>>> Acked-by: Daniel Thompson <daniel.thompson@linaro.org>
>>>>>>
>>>>>> Also:
>>>>>>
>>>>>> Acked-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
>>>>>>
>>>>>> 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&D Institute Poland
Samsung Electronics
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: [PATCH v2 2/2] video: lcd: Remove useless BACKLIGHT_CLASS_DEVICE dependencies

[Lee Jones]

On Thu, 17 Jan 2019, Alexander Shiyan wrote:

> This patch removes dependencies on BACKLIGHT_CLASS_DEVICE for items
> that are already placed under #if BACKLIGHT_CLASS_DEVICE.
> 
> Signed-off-by: Alexander Shiyan <shc_work@mail.ru>
> ---
>  drivers/video/backlight/Kconfig | 25 ++++++++++++-------------
>  1 file changed, 12 insertions(+), 13 deletions(-)

Will fix up Daniel's nit about the #if.

Applied, thanks.

-- 
Lee Jones [李琼斯]
Linaro Services Technical Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog

Re: [PATCH v2 2/2] video: lcd: Remove useless BACKLIGHT_CLASS_DEVICE dependencies

[Lee Jones]

On Thu, 17 Jan 2019, Alexander Shiyan wrote:

> This patch removes dependencies on BACKLIGHT_CLASS_DEVICE for items
> that are already placed under #if BACKLIGHT_CLASS_DEVICE.
> 
> Signed-off-by: Alexander Shiyan <shc_work@mail.ru>
> ---
>  drivers/video/backlight/Kconfig | 25 ++++++++++++-------------
>  1 file changed, 12 insertions(+), 13 deletions(-)

Will fix up Daniel's nit about the #if.

Applied, thanks.

-- 
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

[GIT PULL] Immutable branch between MFD and DRM due for the v5.2 merge window

[Lee Jones]

To whom it may concern,

Enjoy!

The following changes since commit 9e98c678c2d6ae3a17cb2de55d17f69dddaa231b:

  Linux 5.1-rc1 (2019-03-17 14:22:26 -0700)

are available in the Git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/lee/backlight.git ib-backlight-drm-v5.2

for you to fetch changes up to 8c5dc8d9f19c7992b5ed557b865127a80149041b:

  video: backlight: Remove useless BACKLIGHT_LCD_SUPPORT kernel symbol (2019-04-03 11:15:57 +0100)

----------------------------------------------------------------
Immutable branch between MFD and DRM due for the v5.2 merge window

----------------------------------------------------------------
Alexander Shiyan (1):
      video: backlight: Remove useless BACKLIGHT_LCD_SUPPORT kernel symbol

 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(-)

-- 
Lee Jones [李琼斯]
Linaro Services Technical Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog

[GIT PULL] Immutable branch between MFD and DRM due for the v5.2 merge window

[Lee Jones]

To whom it may concern,

Enjoy!

The following changes since commit 9e98c678c2d6ae3a17cb2de55d17f69dddaa231b:

  Linux 5.1-rc1 (2019-03-17 14:22:26 -0700)

are available in the Git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/lee/backlight.git ib-backlight-drm-v5.2

for you to fetch changes up to 8c5dc8d9f19c7992b5ed557b865127a80149041b:

  video: backlight: Remove useless BACKLIGHT_LCD_SUPPORT kernel symbol (2019-04-03 11:15:57 +0100)

----------------------------------------------------------------
Immutable branch between MFD and DRM due for the v5.2 merge window

----------------------------------------------------------------
Alexander Shiyan (1):
      video: backlight: Remove useless BACKLIGHT_LCD_SUPPORT kernel symbol

 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(-)

-- 
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: [GIT PULL] Immutable branch between MFD and DRM due for the v5.2 merge window

[Daniel Vetter]

On Wed, Apr 03, 2019 at 11:19:52AM +0100, Lee Jones wrote:
> To whom it may concern,
> 
> Enjoy!
> 
> The following changes since commit 9e98c678c2d6ae3a17cb2de55d17f69dddaa231b:
> 
>   Linux 5.1-rc1 (2019-03-17 14:22:26 -0700)
> 
> are available in the Git repository at:
> 
>   git://git.kernel.org/pub/scm/linux/kernel/git/lee/backlight.git ib-backlight-drm-v5.2
> 
> for you to fetch changes up to 8c5dc8d9f19c7992b5ed557b865127a80149041b:
> 
>   video: backlight: Remove useless BACKLIGHT_LCD_SUPPORT kernel symbol (2019-04-03 11:15:57 +0100)
> 
> ----------------------------------------------------------------
> Immutable branch between MFD and DRM due for the v5.2 merge window

I pinged Jani (who's been looking at backlight dependencies in the past in
drm), seems like no need to pull this into drm.

Thanks, Daniel

> 
> ----------------------------------------------------------------
> Alexander Shiyan (1):
>       video: backlight: Remove useless BACKLIGHT_LCD_SUPPORT kernel symbol
> 
>  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(-)
> 
> -- 
> 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

-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch

Re: [GIT PULL] Immutable branch between MFD and DRM due for the v5.2 merge window

[Daniel Vetter]

On Wed, Apr 03, 2019 at 11:19:52AM +0100, Lee Jones wrote:
> To whom it may concern,
> 
> Enjoy!
> 
> The following changes since commit 9e98c678c2d6ae3a17cb2de55d17f69dddaa231b:
> 
>   Linux 5.1-rc1 (2019-03-17 14:22:26 -0700)
> 
> are available in the Git repository at:
> 
>   git://git.kernel.org/pub/scm/linux/kernel/git/lee/backlight.git ib-backlight-drm-v5.2
> 
> for you to fetch changes up to 8c5dc8d9f19c7992b5ed557b865127a80149041b:
> 
>   video: backlight: Remove useless BACKLIGHT_LCD_SUPPORT kernel symbol (2019-04-03 11:15:57 +0100)
> 
> ----------------------------------------------------------------
> Immutable branch between MFD and DRM due for the v5.2 merge window

I pinged Jani (who's been looking at backlight dependencies in the past in
drm), seems like no need to pull this into drm.

Thanks, Daniel

> 
> ----------------------------------------------------------------
> Alexander Shiyan (1):
>       video: backlight: Remove useless BACKLIGHT_LCD_SUPPORT kernel symbol
> 
>  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(-)
> 
> -- 
> 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

-- 
Daniel Vetter
Software Engineer, Intel Corporation
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

[Geert Uytterhoeven]

Hi Alexander,

On Thu, Jan 17, 2019 at 2:39 PM Alexander Shiyan <shc_work@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_work@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 -- geert@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

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

[Geert Uytterhoeven]

Hi Alexander,

On Thu, Jan 17, 2019 at 2:39 PM Alexander Shiyan <shc_work@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_work@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 -- geert@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 =?UTF-8?B?IEJBQ0tMSUdIVF9MQ

[Alexander Shiyan]

SGVsbG8uCgpMb29rcyBsaWtlIHlvdSdyZSByaWdodC4KQ2FuIHlvdSBjcmVhdGUgYSBwYXRjaCB0
byBmaXggdGhpcz8KClRoYW5rcy4KCgo+0KHRgNC10LTQsCwgMTUg0LzQsNGPIDIwMTksIDExOjU3
ICswMzowMCDQvtGCIEdlZXJ0IFV5dHRlcmhvZXZlbiA8Z2VlcnRAbGludXgtbTY4ay5vcmc+Ogo+
Cj5IaSBBbGV4YW5kZXIsCj4KPk9uIFRodSwgSmFuIDE3LCAyMDE5IGF0IDI6MzkgUE0gQWxleGFu
ZGVyIFNoaXlhbiA8IHNoY193b3JrQG1haWwucnUgPiB3cm90ZToKPj4gV2UgaGF2ZSB0d28gKl9D
TEFTU19ERVZJQ0Uga2VybmVsIGNvbmZpZyBvcHRpb25zIChMQ0RfQ0xBU1NfREVWSUNFCj4+IGFu
ZCBCQUNLTElHSFRfTENEX0RFVklDRSkgdGhhdCBkbyB0aGUgc2FtZSBqb2IuCj4+IFRoZSBwYXRj
aCByZW1vdmVzIHVzZWxlc3MgQkFDS0xJR0hUX0xDRF9TVVBQT1JUIG9wdGlvbgo+PiBhbmQgY29u
dmVydHMgTENEX0NMQVNTX0RFVklDRSBpbnRvIGEgbWVudS4KPj4KPj4gU2lnbmVkLW9mZi1ieTog
QWxleGFuZGVyIFNoaXlhbiA8IHNoY193b3JrQG1haWwucnUgPgo+VGhpcyBpcyBub3cgY29tbWl0
IDhjNWRjOGQ5ZjE5Yzc5OTIgKCJ2aWRlbzogYmFja2xpZ2h0OiBSZW1vdmUgdXNlbGVzcwo+QkFD
S0xJR0hUX0xDRF9TVVBQT1JUIGtlcm5lbCBzeW1ib2wiKS4KPgo+PiAtLS0gYS9kcml2ZXJzL3Zp
ZGVvL2JhY2tsaWdodC9LY29uZmlnCj4+ICsrKyBiL2RyaXZlcnMvdmlkZW8vYmFja2xpZ2h0L0tj
b25maWcKPj4gQEAgLTIsMTMgKzIsNyBAQAo+PiAgIyBCYWNrbGlnaHQgJiBMQ0QgZHJpdmVycyBj
b25maWd1cmF0aW9uCj4+ICAjCj4+Cj4+IC1tZW51Y29uZmlnIEJBQ0tMSUdIVF9MQ0RfU1VQUE9S
VAo+PiAtICAgICAgIGJvb2wgIkJhY2tsaWdodCAmIExDRCBkZXZpY2Ugc3VwcG9ydCIKPj4gLSAg
ICAgICBoZWxwCj4+IC0gICAgICAgICBFbmFibGUgdGhpcyB0byBiZSBhYmxlIHRvIGNob29zZSB0
aGUgZHJpdmVycyBmb3IgY29udHJvbGxpbmcgdGhlCj4+IC0gICAgICAgICBiYWNrbGlnaHQgYW5k
IHRoZSBMQ0QgcGFuZWwgb24gc29tZSBwbGF0Zm9ybXMsIGZvciBleGFtcGxlIG9uIFBEQXMuCj4+
IC0KPj4gLWlmIEJBQ0tMSUdIVF9MQ0RfU1VQUE9SVAo+PiArbWVudSAiQmFja2xpZ2h0ICYgTENE
IGRldmljZSBzdXBwb3J0Igo+Pgo+PiAgIwo+PiAgIyBMQ0QKPgo+QmVsb3csIHdlIGhhdmU6Cj4K
PsKgwqDCoMKgY29uZmlnIExDRF9DTEFTU19ERVZJQ0UKPsKgwqDCoMKgwqDCoMKgwqDCoMKgwqDC
oHRyaXN0YXRlICJMb3dsZXZlbCBMQ0QgY29udHJvbHMiCj7CoMKgwqDCoMKgwqDCoMKgwqDCoMKg
wqBkZWZhdWx0IG0KPsKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoGhlbHAKPsKgwqDCoMKgwqDCoMKg
wqDCoMKgwqDCoMKgwqBUaGlzIGZyYW1ld29yayBhZGRzIHN1cHBvcnQgZm9yIGxvdy1sZXZlbCBj
b250cm9sIG9mIExDRC4KPsKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqBTb21lIGZyYW1lYnVm
ZmVyIGRldmljZXMgY29ubmVjdCB0byBwbGF0Zm9ybS1zcGVjaWZpYyBMQ0QgbW9kdWxlcwo+wqDC
oMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoGluIG9yZGVyIHRvIGhhdmUgYSBwbGF0Zm9ybS1zcGVj
aWZpYyB3YXkgdG8gY29udHJvbCB0aGUgZmxhdCBwYW5lbAo+wqDCoMKgwqDCoMKgwqDCoMKgwqDC
oMKgwqDCoChjb250cmFzdCBhbmQgYXBwbHlpbmcgcG93ZXIgdG8gdGhlIExDRCAobm90IHRvIHRo
ZSBiYWNrbGlnaHQhKSkuCj4KPmFuZDoKPgo+wqDCoMKgwqBjb25maWcgQkFDS0xJR0hUX0NMQVNT
X0RFVklDRQo+wqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgdHJpc3RhdGUgIkxvd2xldmVsIEJhY2ts
aWdodCBjb250cm9scyIKPsKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoGRlZmF1bHQgbQo+wqDCoMKg
wqDCoMKgwqDCoMKgwqDCoMKgaGVscAo+wqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoFRoaXMg
ZnJhbWV3b3JrIGFkZHMgc3VwcG9ydCBmb3IgbG93LWxldmVsIGNvbnRyb2wgb2YgdGhlIExDRAo+
wqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoGJhY2tsaWdodC4gVGhpcyBpbmNsdWRlcyBzdXBw
b3J0IGZvciBicmlnaHRuZXNzIGFuZCBwb3dlci4KPgo+SGVuY2UgcnVubmluZyAibWFrZSBvbGRj
b25maWciIG9uIGEgLmNvbmZpZyB3aGVyZQo+Q09ORklHX0JBQ0tMSUdIVF9MQ0RfU1VQUE9SVCB3
YXMgbm90IHNldCBsZWFkcyB0d28gdG8gbmV3Cj5xdWVzdGlvbnMsIHdoZXJlIHRoZSBhbnN3ZXJz
IGRlZmF1bHQgdG8gIm0iLgo+Cj5QZXJoYXBzIHRoZSAiZGVmYXVsdCBtIiBzdGF0ZW1lbnRzIHNo
b3VsZCBiZSByZW1vdmVkPwoKLS0tCg=

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

[Alexander Shiyan]

Hello.

Looks like you're right.
Can you create a patch to fix this?

Thanks.


>Среда, 15 мая 2019, 11:57 +03:00 от Geert Uytterhoeven <geert@linux-m68k.org>:
>
>Hi Alexander,
>
>On Thu, Jan 17, 2019 at 2:39 PM Alexander Shiyan < shc_work@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_work@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

[Geert Uytterhoeven]

Hi Alexander,

On Wed, May 15, 2019 at 12:34 PM Alexander Shiyan <shc_work@mail.ru> 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@linux-m68k.org>:
> >
> >Hi Alexander,
> >
> >On Thu, Jan 17, 2019 at 2:39 PM Alexander Shiyan < shc_work@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_work@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 -- geert@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

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

[Geert Uytterhoeven]

Hi Alexander,

On Wed, May 15, 2019 at 12:34 PM Alexander Shiyan <shc_work@mail.ru> 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@linux-m68k.org>:
> >
> >Hi Alexander,
> >
> >On Thu, Jan 17, 2019 at 2:39 PM Alexander Shiyan < shc_work@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_work@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 -- geert@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