OSDN Git Service

drm/panel-novatek-nt35510: Emit an error message if power off fails
authorUwe Kleine-König <u.kleine-koenig@pengutronix.de>
Fri, 8 Jul 2022 09:49:21 +0000 (11:49 +0200)
committerSam Ravnborg <sam@ravnborg.org>
Sat, 9 Jul 2022 08:46:01 +0000 (10:46 +0200)
Returning an error code from a mipi_dsi remove callback fails, this is
silently ignored. (mipi_dsi_drv_remove() propagates the return value to
device_remove() which ignores it.) So emit an error code in the driver
remove function and return 0.

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Reviewed-by: Sam Ravnborg <sam@ravnborg.org>
Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
Link: https://patchwork.freedesktop.org/patch/msgid/20220708094922.1408248-3-u.kleine-koenig@pengutronix.de
drivers/gpu/drm/panel/panel-novatek-nt35510.c

index 40ea41b..1b3e015 100644 (file)
@@ -974,9 +974,12 @@ static int nt35510_remove(struct mipi_dsi_device *dsi)
        mipi_dsi_detach(dsi);
        /* Power off */
        ret = nt35510_power_off(nt);
+       if (ret)
+               dev_err(&dsi->dev, "Failed to power off\n");
+
        drm_panel_remove(&nt->panel);
 
-       return ret;
+       return 0;
 }
 
 /*