Hi Dan, > The "num_levels" variable is used to store error codes from > device_property_count_u32() so it needs to be signed. This doesn't cause an > issue at runtime because devm_kcalloc() won't allocate negative sizes. > However, it's still worth fixing. > > Fixes: b54c828bdba9 ("backlight: mp3309c: Make use of device properties") > Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> I've just tested on my board with the mp3309c chip, all is ok. Thanks! Tested-by: Flavio Suligoi <f.suligoi@xxxxxxx>