diff options
| author |   Daniel Thompson <daniel.thompson@linaro.org> | 2024-02-20 15:35:26 +0000 |
|---|---|---|
| committer |   Lee Jones <lee@kernel.org> | 2024-03-07 09:03:29 +0000 |
| commit | 392346827fbe8a7fd573dfb145170d7949f639a6 (patch) | |
| tree | 3df57e46ca93a802175c8433bf2c8dd32d87da61 | |
| parent | backlight: lm3639: Fully initialize backlight_properties during probe (diff) | |
| download | wireguard-linux-392346827fbe8a7fd573dfb145170d7949f639a6.tar.xz wireguard-linux-392346827fbe8a7fd573dfb145170d7949f639a6.zip | |
backlight: lp8788: Fully initialize backlight_properties during probe
props is stack allocated and the fields that are not explcitly set
by the probe function need to be zeroed or we'll get undefined behaviour
(especially so power/blank states)!
Fixes: c5a51053cf3b ("backlight: add new lp8788 backlight driver")
Signed-off-by: Daniel Thompson <daniel.thompson@linaro.org>
Link: https://lore.kernel.org/r/20240220153532.76613-4-daniel.thompson@linaro.org
Signed-off-by: Lee Jones <lee@kernel.org>
| -rw-r--r-- | drivers/video/backlight/lp8788_bl.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/video/backlight/lp8788_bl.c b/drivers/video/backlight/lp8788_bl.c index d1a14b0db265..31f97230ee50 100644 --- a/drivers/video/backlight/lp8788_bl.c +++ b/drivers/video/backlight/lp8788_bl.c @@ -191,6 +191,7 @@ static int lp8788_backlight_register(struct lp8788_bl *bl) int init_brt; char *name; + memset(&props, 0, sizeof(struct backlight_properties)); props.type = BACKLIGHT_PLATFORM; props.max_brightness = MAX_BRIGHTNESS; |