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

2019-05-16 Thread 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 :
>
>Hi Alexander,
>
>On Thu, Jan 17, 2019 at 2:39 PM Alexander Shiyan < shc_w...@mail.ru > wrote:
>> We have two *_CLASS_DEVICE kernel config options (LCD_CLASS_DEVICE
>> and BACKLIGHT_LCD_DEVICE) that do the same job.
>> The patch removes useless BACKLIGHT_LCD_SUPPORT option
>> and converts LCD_CLASS_DEVICE into a menu.
>>
>> Signed-off-by: Alexander Shiyan < shc_w...@mail.ru >
>This is now commit 8c5dc8d9f19c7992 ("video: backlight: Remove useless
>BACKLIGHT_LCD_SUPPORT kernel symbol").
>
>> --- a/drivers/video/backlight/Kconfig
>> +++ b/drivers/video/backlight/Kconfig
>> @@ -2,13 +2,7 @@
>>  # Backlight & LCD drivers configuration
>>  #
>>
>> -menuconfig BACKLIGHT_LCD_SUPPORT
>> -   bool "Backlight & LCD device support"
>> -   help
>> - Enable this to be able to choose the drivers for controlling the
>> - backlight and the LCD panel on some platforms, for example on PDAs.
>> -
>> -if BACKLIGHT_LCD_SUPPORT
>> +menu "Backlight & LCD device support"
>>
>>  #
>>  # LCD
>
>Below, we have:
>
>config LCD_CLASS_DEVICE
>tristate "Lowlevel LCD controls"
>default m
>help
>  This framework adds support for low-level control of LCD.
>  Some framebuffer devices connect to platform-specific LCD modules
>  in order to have a platform-specific way to control the flat 
>panel
>  (contrast and applying power to the LCD (not to the backlight!)).
>
>and:
>
>config BACKLIGHT_CLASS_DEVICE
>tristate "Lowlevel Backlight controls"
>default m
>help
>  This framework adds support for low-level control of the LCD
>  backlight. This includes support for brightness and power.
>
>Hence running "make oldconfig" on a .config where
>CONFIG_BACKLIGHT_LCD_SUPPORT was not set leads two to new
>questions, where the answers default to "m".
>
>Perhaps the "default m" statements should be removed?

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

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

2019-05-15 Thread Geert Uytterhoeven
Hi Alexander,

On Wed, May 15, 2019 at 12:34 PM Alexander Shiyan  wrote:
> Looks like you're right.
> Can you create a patch to fix this?

Thank you, done.

> >Среда, 15 мая 2019, 11:57 +03:00 от Geert Uytterhoeven 
> >:
> >
> >Hi Alexander,
> >
> >On Thu, Jan 17, 2019 at 2:39 PM Alexander Shiyan < shc_w...@mail.ru > wrote:
> >> We have two *_CLASS_DEVICE kernel config options (LCD_CLASS_DEVICE
> >> and BACKLIGHT_LCD_DEVICE) that do the same job.
> >> The patch removes useless BACKLIGHT_LCD_SUPPORT option
> >> and converts LCD_CLASS_DEVICE into a menu.
> >>
> >> Signed-off-by: Alexander Shiyan < shc_w...@mail.ru >
> >This is now commit 8c5dc8d9f19c7992 ("video: backlight: Remove useless
> >BACKLIGHT_LCD_SUPPORT kernel symbol").
> >
> >> --- a/drivers/video/backlight/Kconfig
> >> +++ b/drivers/video/backlight/Kconfig
> >> @@ -2,13 +2,7 @@
> >>  # Backlight & LCD drivers configuration
> >>  #
> >>
> >> -menuconfig BACKLIGHT_LCD_SUPPORT
> >> -   bool "Backlight & LCD device support"
> >> -   help
> >> - Enable this to be able to choose the drivers for controlling the
> >> - backlight and the LCD panel on some platforms, for example on 
> >> PDAs.
> >> -
> >> -if BACKLIGHT_LCD_SUPPORT
> >> +menu "Backlight & LCD device support"
> >>
> >>  #
> >>  # LCD
> >
> >Below, we have:
> >
> >config LCD_CLASS_DEVICE
> >tristate "Lowlevel LCD controls"
> >default m
> >help
> >  This framework adds support for low-level control of LCD.
> >  Some framebuffer devices connect to platform-specific LCD 
> > modules
> >  in order to have a platform-specific way to control the flat 
> > panel
> >  (contrast and applying power to the LCD (not to the 
> > backlight!)).
> >
> >and:
> >
> >config BACKLIGHT_CLASS_DEVICE
> >tristate "Lowlevel Backlight controls"
> >default m
> >help
> >  This framework adds support for low-level control of the LCD
> >  backlight. This includes support for brightness and power.
> >
> >Hence running "make oldconfig" on a .config where
> >CONFIG_BACKLIGHT_LCD_SUPPORT was not set leads two to new
> >questions, where the answers default to "m".
> >
> >Perhaps the "default m" statements should be removed?

Gr{oetje,eeting}s,

Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

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

2019-05-15 Thread Geert Uytterhoeven
Hi Alexander,

On Thu, Jan 17, 2019 at 2:39 PM Alexander Shiyan  wrote:
> We have two *_CLASS_DEVICE kernel config options (LCD_CLASS_DEVICE
> and BACKLIGHT_LCD_DEVICE) that do the same job.
> The patch removes useless BACKLIGHT_LCD_SUPPORT option
> and converts LCD_CLASS_DEVICE into a menu.
>
> Signed-off-by: Alexander Shiyan 

This is now commit 8c5dc8d9f19c7992 ("video: backlight: Remove useless
BACKLIGHT_LCD_SUPPORT kernel symbol").

> --- a/drivers/video/backlight/Kconfig
> +++ b/drivers/video/backlight/Kconfig
> @@ -2,13 +2,7 @@
>  # Backlight & LCD drivers configuration
>  #
>
> -menuconfig BACKLIGHT_LCD_SUPPORT
> -   bool "Backlight & LCD device support"
> -   help
> - Enable this to be able to choose the drivers for controlling the
> - backlight and the LCD panel on some platforms, for example on PDAs.
> -
> -if BACKLIGHT_LCD_SUPPORT
> +menu "Backlight & LCD device support"
>
>  #
>  # LCD

Below, we have:

config LCD_CLASS_DEVICE
tristate "Lowlevel LCD controls"
default m
help
  This framework adds support for low-level control of LCD.
  Some framebuffer devices connect to platform-specific LCD modules
  in order to have a platform-specific way to control the flat panel
  (contrast and applying power to the LCD (not to the backlight!)).

and:

config BACKLIGHT_CLASS_DEVICE
tristate "Lowlevel Backlight controls"
default m
help
  This framework adds support for low-level control of the LCD
  backlight. This includes support for brightness and power.

Hence running "make oldconfig" on a .config where
CONFIG_BACKLIGHT_LCD_SUPPORT was not set leads two to new
questions, where the answers default to "m".

Perhaps the "default m" statements should be removed?

Gr{oetje,eeting}s,

Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

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

2019-04-01 Thread 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 
>>>
>>> A cover letter with the v1 -> v2 changelog would be nice... but
>>> nevertheless:
>>> Acked-by: Daniel Thompson 
>>
>> Also:
>>
>> Acked-by: Bartlomiej Zolnierkiewicz 
>>
>> Lee, will you pick this up or do you want me to do it?
>
> You can take them if you want.  I will require a pull-request from an
> independent immutable branch though please.

 OK, immutable branch will also allow other trees (drm) to sync with
 these changes if desired.

 I will do it for v5.2 if you are fine with it (too late for v5.1
 IMHO and I also won't have time too fix any potential issues next
 week).
> 
>>> Topic branch for these 2 simple patches seems like serious overkill.
> 
> [...]
> 
>>> Would be good to get it into 5.1 still though, just to avoid too
>>> many conflicts. 
> 
> I think you just answered your own statement. :)
> 
>> Well, if so I'm also fine with this going in 5.1 w/o topic branch.
>>
>> Lee, do you want to merge this directly into backlight tree (you
>> have now both fbdev and drm acks)?
> 
> It was too late for v5.1.
> 
> Do you still want to take this for v5.2, or should I take them and
> provide you with an immutable pull-request?

Please take them and provide immutable branch, thank you.

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

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

2019-03-20 Thread 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 
> >
> > A cover letter with the v1 -> v2 changelog would be nice... but
> > nevertheless:
> > Acked-by: Daniel Thompson 
> 
>  Also:
> 
>  Acked-by: Bartlomiej Zolnierkiewicz 
> 
>  Lee, will you pick this up or do you want me to do it?
> >>>
> >>> You can take them if you want.  I will require a pull-request from an
> >>> independent immutable branch though please.
> >>
> >> OK, immutable branch will also allow other trees (drm) to sync with
> >> these changes if desired.
> >>
> >> I will do it for v5.2 if you are fine with it (too late for v5.1
> >> IMHO and I also won't have time too fix any potential issues next
> >> week).

> > Topic branch for these 2 simple patches seems like serious overkill.

[...]

> > Would be good to get it into 5.1 still though, just to avoid too
> > many conflicts. 

I think you just answered your own statement. :)

> Well, if so I'm also fine with this going in 5.1 w/o topic branch.
> 
> Lee, do you want to merge this directly into backlight tree (you
> have now both fbdev and drm acks)?

It was too late for v5.1.

Do you still want to take this for v5.2, or should I take them and
provide you with an immutable pull-request?

-- 
Lee Jones [李琼斯]
Linaro Services Technical Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

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

2019-02-14 Thread 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  wrote:
> >
> >
> > On 02/12/2019 09:42 AM, Lee Jones wrote:
> > > On Fri, 08 Feb 2019, Bartlomiej Zolnierkiewicz wrote:
> > >
> > >>
> > >> On 01/17/2019 05:19 PM, Daniel Thompson wrote:
> > >>> On Thu, Jan 17, 2019 at 04:33:35PM +0300, Alexander Shiyan wrote:
> >  We have two *_CLASS_DEVICE kernel config options (LCD_CLASS_DEVICE
> >  and BACKLIGHT_LCD_DEVICE) that do the same job.
> >  The patch removes useless BACKLIGHT_LCD_SUPPORT option
> >  and converts LCD_CLASS_DEVICE into a menu.
> > 
> >  Signed-off-by: Alexander Shiyan 
> > >>>
> > >>> A cover letter with the v1 -> v2 changelog would be nice... but
> > >>> nevertheless:
> > >>> Acked-by: Daniel Thompson 
> > >>
> > >> Also:
> > >>
> > >> Acked-by: Bartlomiej Zolnierkiewicz 
> > >>
> > >> Lee, will you pick this up or do you want me to do it?
> > >
> > > You can take them if you want.  I will require a pull-request from an
> > > independent immutable branch though please.
> >
> > OK, immutable branch will also allow other trees (drm) to sync with
> > these changes if desired.
> >
> > I will do it for v5.2 if you are fine with it (too late for v5.1
> > IMHO and I also won't have time too fix any potential issues next
> > week).
> 
> Topic branch for these 2 simple patches seems like serious overkill.
> Especially for backlight stuff, which doesn't move fast. Anyway, up to
> you, ack for all the drm patches to go through whatever tree you
> people can agree on. Would be good to get it into 5.1 still though,
> just to avoid too many conflicts.

It takes me approx 1 min to set-up an immutable branch.

If you think it's too much work, I can set it up for you.

-- 
Lee Jones [李琼斯]
Linaro Services Technical Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

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

2019-02-13 Thread 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  wrote:
>>
>>
>> On 02/12/2019 09:42 AM, Lee Jones wrote:
>>> On Fri, 08 Feb 2019, Bartlomiej Zolnierkiewicz wrote:
>>>

 On 01/17/2019 05:19 PM, Daniel Thompson wrote:
> On Thu, Jan 17, 2019 at 04:33:35PM +0300, Alexander Shiyan wrote:
>> We have two *_CLASS_DEVICE kernel config options (LCD_CLASS_DEVICE
>> and BACKLIGHT_LCD_DEVICE) that do the same job.
>> The patch removes useless BACKLIGHT_LCD_SUPPORT option
>> and converts LCD_CLASS_DEVICE into a menu.
>>
>> Signed-off-by: Alexander Shiyan 
>
> A cover letter with the v1 -> v2 changelog would be nice... but
> nevertheless:
> Acked-by: Daniel Thompson 

 Also:

 Acked-by: Bartlomiej Zolnierkiewicz 

 Lee, will you pick this up or do you want me to do it?
>>>
>>> You can take them if you want.  I will require a pull-request from an
>>> independent immutable branch though please.
>>
>> OK, immutable branch will also allow other trees (drm) to sync with
>> these changes if desired.
>>
>> I will do it for v5.2 if you are fine with it (too late for v5.1
>> IMHO and I also won't have time too fix any potential issues next
>> week).
> 
> Topic branch for these 2 simple patches seems like serious overkill.
> Especially for backlight stuff, which doesn't move fast. Anyway, up to

This depends on the development stage we are in [*]. Right now things
are calm so there is low probability of conflicts and we might just
merge these patches without topic branch.

[*] In case of fbdev/Kconfig FB_CLPS711X config option was modified
in v4.20 and both FB_IMX and FB_MX3 config options still need to be
converted to support COMPILE_TEST so at least in theory there is
a potential for conflicts with fbdev if merged for 5.2.

> you, ack for all the drm patches to go through whatever tree you
> people can agree on. Would be good to get it into 5.1 still though,
> just to avoid too many conflicts.

Well, if so I'm also fine with this going in 5.1 w/o topic branch.

Lee, do you want to merge this directly into backlight tree (you
have now both fbdev and drm acks)?

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

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

2019-02-13 Thread Daniel Vetter
On Wed, Feb 13, 2019 at 4:04 PM Bartlomiej Zolnierkiewicz via
dri-devel  wrote:
>
>
> On 02/12/2019 09:42 AM, Lee Jones wrote:
> > On Fri, 08 Feb 2019, Bartlomiej Zolnierkiewicz wrote:
> >
> >>
> >> On 01/17/2019 05:19 PM, Daniel Thompson wrote:
> >>> On Thu, Jan 17, 2019 at 04:33:35PM +0300, Alexander Shiyan wrote:
>  We have two *_CLASS_DEVICE kernel config options (LCD_CLASS_DEVICE
>  and BACKLIGHT_LCD_DEVICE) that do the same job.
>  The patch removes useless BACKLIGHT_LCD_SUPPORT option
>  and converts LCD_CLASS_DEVICE into a menu.
> 
>  Signed-off-by: Alexander Shiyan 
> >>>
> >>> A cover letter with the v1 -> v2 changelog would be nice... but
> >>> nevertheless:
> >>> Acked-by: Daniel Thompson 
> >>
> >> Also:
> >>
> >> Acked-by: Bartlomiej Zolnierkiewicz 
> >>
> >> Lee, will you pick this up or do you want me to do it?
> >
> > You can take them if you want.  I will require a pull-request from an
> > independent immutable branch though please.
>
> OK, immutable branch will also allow other trees (drm) to sync with
> these changes if desired.
>
> I will do it for v5.2 if you are fine with it (too late for v5.1
> IMHO and I also won't have time too fix any potential issues next
> week).

Topic branch for these 2 simple patches seems like serious overkill.
Especially for backlight stuff, which doesn't move fast. Anyway, up to
you, ack for all the drm patches to go through whatever tree you
people can agree on. Would be good to get it into 5.1 still though,
just to avoid too many conflicts.

Cheers, Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

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

2019-02-13 Thread 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 
>>>
>>> A cover letter with the v1 -> v2 changelog would be nice... but
>>> nevertheless:
>>> Acked-by: Daniel Thompson 
>>
>> Also:
>>
>> Acked-by: Bartlomiej Zolnierkiewicz 
>>
>> Lee, will you pick this up or do you want me to do it?
> 
> You can take them if you want.  I will require a pull-request from an
> independent immutable branch though please.

OK, immutable branch will also allow other trees (drm) to sync with
these changes if desired.

I will do it for v5.2 if you are fine with it (too late for v5.1
IMHO and I also won't have time too fix any potential issues next
week).

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

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

2019-02-12 Thread 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 
> > 
> > A cover letter with the v1 -> v2 changelog would be nice... but
> > nevertheless:
> > Acked-by: Daniel Thompson 
> 
> Also:
> 
> Acked-by: Bartlomiej Zolnierkiewicz 
> 
> Lee, will you pick this up or do you want me to do it?

You can take them if you want.  I will require a pull-request from an
independent immutable branch though please.

-- 
Lee Jones [李琼斯]
Linaro Services Technical Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

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

2019-02-08 Thread 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 
> 
> A cover letter with the v1 -> v2 changelog would be nice... but
> nevertheless:
> Acked-by: Daniel Thompson 

Also:

Acked-by: Bartlomiej Zolnierkiewicz 

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

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


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

2019-01-17 Thread 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 

A cover letter with the v1 -> v2 changelog would be nice... but
nevertheless:
Acked-by: Daniel Thompson 

> ---
>  arch/unicore32/Kconfig|  1 -
>  drivers/gpu/drm/Kconfig   |  2 --
>  drivers/gpu/drm/bridge/Kconfig|  1 -
>  drivers/gpu/drm/fsl-dcu/Kconfig   |  1 -
>  drivers/gpu/drm/i915/Kconfig  |  1 -
>  drivers/gpu/drm/nouveau/Kconfig   |  2 --
>  drivers/gpu/drm/shmobile/Kconfig  |  1 -
>  drivers/gpu/drm/tilcdc/Kconfig|  1 -
>  drivers/staging/olpc_dcon/Kconfig |  1 -
>  drivers/usb/misc/Kconfig  |  1 -
>  drivers/video/backlight/Kconfig   | 10 ++
>  drivers/video/fbdev/Kconfig   |  5 -
>  12 files changed, 2 insertions(+), 25 deletions(-)
> 
> diff --git a/arch/unicore32/Kconfig b/arch/unicore32/Kconfig
> index c3a41bf..1138334 100644
> --- a/arch/unicore32/Kconfig
> +++ b/arch/unicore32/Kconfig
> @@ -195,7 +195,6 @@ config I2C_EEPROM_AT24
>  
>  config LCD_BACKLIGHT
>   tristate "LCD Backlight support"
> - select BACKLIGHT_LCD_SUPPORT
>   select BACKLIGHT_PWM
>  
>  endmenu
> diff --git a/drivers/gpu/drm/Kconfig b/drivers/gpu/drm/Kconfig
> index 4385f00..ef442a7 100644
> --- a/drivers/gpu/drm/Kconfig
> +++ b/drivers/gpu/drm/Kconfig
> @@ -198,7 +198,6 @@ config DRM_RADEON
>   select POWER_SUPPLY
>   select HWMON
>   select BACKLIGHT_CLASS_DEVICE
> - select BACKLIGHT_LCD_SUPPORT
>   select INTERVAL_TREE
>   help
> Choose this option if you have an ATI Radeon graphics card.  There
> @@ -219,7 +218,6 @@ config DRM_AMDGPU
>   select POWER_SUPPLY
>   select HWMON
>   select BACKLIGHT_CLASS_DEVICE
> - select BACKLIGHT_LCD_SUPPORT
>   select INTERVAL_TREE
>   select CHASH
>   help
> diff --git a/drivers/gpu/drm/bridge/Kconfig b/drivers/gpu/drm/bridge/Kconfig
> index 2fee47b..5b5a8e5 100644
> --- a/drivers/gpu/drm/bridge/Kconfig
> +++ b/drivers/gpu/drm/bridge/Kconfig
> @@ -75,7 +75,6 @@ config DRM_PARADE_PS8622
>   depends on OF
>   select DRM_PANEL
>   select DRM_KMS_HELPER
> - select BACKLIGHT_LCD_SUPPORT
>   select BACKLIGHT_CLASS_DEVICE
>   ---help---
> Parade eDP-LVDS bridge chip driver.
> diff --git a/drivers/gpu/drm/fsl-dcu/Kconfig b/drivers/gpu/drm/fsl-dcu/Kconfig
> index 14a72c4..dc82588 100644
> --- a/drivers/gpu/drm/fsl-dcu/Kconfig
> +++ b/drivers/gpu/drm/fsl-dcu/Kconfig
> @@ -2,7 +2,6 @@ config DRM_FSL_DCU
>   tristate "DRM Support for Freescale DCU"
>   depends on DRM && OF && ARM && COMMON_CLK
>   select BACKLIGHT_CLASS_DEVICE
> - select BACKLIGHT_LCD_SUPPORT
>   select DRM_KMS_HELPER
>   select DRM_KMS_CMA_HELPER
>   select DRM_PANEL
> diff --git a/drivers/gpu/drm/i915/Kconfig b/drivers/gpu/drm/i915/Kconfig
> index 148be8e..3d5f1cb 100644
> --- a/drivers/gpu/drm/i915/Kconfig
> +++ b/drivers/gpu/drm/i915/Kconfig
> @@ -15,7 +15,6 @@ config DRM_I915
>   select IRQ_WORK
>   # i915 depends on ACPI_VIDEO when ACPI is enabled
>   # but for select to work, need to select ACPI_VIDEO's dependencies, ick
> - select BACKLIGHT_LCD_SUPPORT if ACPI
>   select BACKLIGHT_CLASS_DEVICE if ACPI
>   select INPUT if ACPI
>   select ACPI_VIDEO if ACPI
> diff --git a/drivers/gpu/drm/nouveau/Kconfig b/drivers/gpu/drm/nouveau/Kconfig
> index 432c440..fd7b869 100644
> --- a/drivers/gpu/drm/nouveau/Kconfig
> +++ b/drivers/gpu/drm/nouveau/Kconfig
> @@ -5,14 +5,12 @@ config DRM_NOUVEAU
>   select DRM_KMS_HELPER
>   select DRM_TTM
>   select BACKLIGHT_CLASS_DEVICE if DRM_NOUVEAU_BACKLIGHT
> - select BACKLIGHT_LCD_SUPPORT if DRM_NOUVEAU_BACKLIGHT
>   select ACPI_VIDEO if ACPI && X86 && BACKLIGHT_CLASS_DEVICE && INPUT
>   select X86_PLATFORM_DEVICES if ACPI && X86
>   select ACPI_WMI if ACPI && X86
>   select MXM_WMI if ACPI && X86
>   select POWER_SUPPLY
>   # Similar to i915, we need to select ACPI_VIDEO and it's dependencies
> - select BACKLIGHT_LCD_SUPPORT if ACPI && X86
>   select BACKLIGHT_CLASS_DEVICE if ACPI && X86
>   select INPUT if ACPI && X86
>   select THERMAL if ACPI && X86
> diff --git a/drivers/gpu/drm/shmobile/Kconfig 
> b/drivers/gpu/drm/shmobile/Kconfig
> index 61bbe8e..e2a6c82 100644
> --- a/drivers/gpu/drm/shmobile/Kconfig
> +++ b/drivers/gpu/drm/shmobile/Kconfig
> @@ -4,7 +4,6 @@ config DRM_SHMOBILE
>   depends on DRM && ARM
>   depends on ARCH_SHMOBILE || COMPILE_TEST
>   select BACKLIGHT_CLASS_DEVICE
> - select BACKLIGHT_LCD_SUPPORT
>   select DRM_KMS_HELPER
>   select DRM_KMS_CMA_HELPER
>   select DRM_GEM_CMA_HELPER
> diff --git a/drivers/gpu/drm/tilcdc/Kconfig