On Wed, Nov 29, 2023 at 05:45:14PM +0100, Flavio Suligoi wrote: > In the function "pm3309c_parse_dt_node", when the dimming analog control > mode (by I2C messages) is enabled, the local variable "prop_levels" is > tested without any initialization, as indicated by the following smatch > warning: > > drivers/video/backlight/mp3309c.c:279 pm3309c_parse_dt_node() error: > uninitialized symbol 'prop_levels'. > > To avoid any problem in case of undefined behavior, we need to initialize > it to "NULL". > > Reported-by: Dan Carpenter <dan.carpen...@linaro.org> > Closes: > https://lore.kernel.org/dri-devel/af0a1870-693b-442f-9b11-0503cfcd944a@moroto.mountain/ > Fixes: 2e914516a58c ("backlight: mp3309c: Add support for MPS MP3309C") > Signed-off-by: Flavio Suligoi <f.suli...@asem.it>
Reviewed-by: Daniel Thompson <daniel.thomp...@linaro.org> Daniel.
