Javier Martinez Canillas <javierm@xxxxxxxxxx> writes: > Dan Carpenter <dan.carpenter@xxxxxxxxxx> writes: > > Hello Dan, > >> The devm_kzalloc() function returns NULL on failure, not error pointers. >> Also printing an error message for kmalloc() failures is against kernel >> style so just return -ENOMEM without printing a message. (Kmalloc >> already prints a message). >> >> Fixes: 4b35f0f41ee2 ("drm/st7571-i2c: add support for Sitronix ST7571 LCD controller") >> Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> >> --- > > Reviewed-by: Javier Martinez Canillas <javierm@xxxxxxxxxx> > Pushed to drm-misc (drm-misc-next). Thanks! -- Best regards, Javier Martinez Canillas Core Platforms Red Hat