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> -- Best regards, Javier Martinez Canillas Core Platforms Red Hat