On Fri, Apr 17, 2020 at 02:33:09PM +0300, Tomi Valkeinen wrote: > There's no need to set 'levels' to NULL. > > Signed-off-by: Tomi Valkeinen <tomi.valkeinen@xxxxxx> Reviewed-by: Daniel Thompson <daniel.thompson@xxxxxxxxxx> > --- > drivers/video/backlight/led_bl.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/video/backlight/led_bl.c b/drivers/video/backlight/led_bl.c > index d4e1ce684366..c46ecdfe8b0a 100644 > --- a/drivers/video/backlight/led_bl.c > +++ b/drivers/video/backlight/led_bl.c > @@ -140,7 +140,7 @@ static int led_bl_parse_levels(struct device *dev, > if (num_levels > 1) { > int i; > unsigned int db; > - u32 *levels = NULL; > + u32 *levels; > > levels = devm_kzalloc(dev, sizeof(u32) * num_levels, > GFP_KERNEL); > -- > Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. > Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki > _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel