On Wed, 19 Apr 2023 at 13:17, Svyatoslav Ryhel <clamo...@gmail.com> wrote:
>
> This is basic lm3533 driver only with bank A and
> backlight cell support.
>
> Tested-by: Andreas Westman Dorcsak <hed...@yahoo.com> # LG P880 T30
> Tested-by: Svyatoslav Ryhel <clamo...@gmail.com> # LG P895 T30
> Signed-off-by: Svyatoslav Ryhel <clamo...@gmail.com>
> ---
>  drivers/video/Kconfig            |   8 ++
>  drivers/video/Makefile           |   1 +
>  drivers/video/lm3533_backlight.c | 135 +++++++++++++++++++++++++++++++
>  3 files changed, 144 insertions(+)
>  create mode 100644 drivers/video/lm3533_backlight.c
>

Reviewed-by: Simon Glass <s...@chromium.org>

> diff --git a/drivers/video/Kconfig b/drivers/video/Kconfig
> index 334d64c948..1695d17c8d 100644
> --- a/drivers/video/Kconfig
> +++ b/drivers/video/Kconfig
> @@ -606,6 +606,14 @@ config ATMEL_HLCD
>         help
>            HLCDC supports video output to an attached LCD panel.
>
> +config BACKLIGHT_LM3533
> +       bool "Backlight Driver for LM3533"
> +       depends on BACKLIGHT
> +       select DM_I2C
> +       help
> +         Say Y to enable the backlight driver for National Semiconductor / TI
> +         LM3533 Lighting Power chips.

Would be good to mention that it supports a programmable brightness
level. We are supposed to have at least 3 lines of help, so think of
useful info to add.

Regards,
Simon

Reply via email to