Hi Marek, Thank you for the patch.
On Fri, Dec 31, 2021 at 05:00:56PM +0100, Marek Vasut wrote: > When the device is unbound from the driver, the display may be active. > Make sure it gets shut down. > > Signed-off-by: Marek Vasut <ma...@denx.de> > Cc: Daniel Vetter <daniel.vet...@ffwll.ch> > Cc: Laurent Pinchart <laurent.pinch...@ideasonboard.com> > Cc: Lucas Stach <l.st...@pengutronix.de> > Cc: Sam Ravnborg <s...@ravnborg.org> > Cc: Stefan Agner <ste...@agner.ch> > Cc: Thomas Zimmermann <tzimmerm...@suse.de> > --- > drivers/gpu/drm/mxsfb/mxsfb_drv.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/gpu/drm/mxsfb/mxsfb_drv.c > b/drivers/gpu/drm/mxsfb/mxsfb_drv.c > index 86d78634a9799..6d7a3aeff50b0 100644 > --- a/drivers/gpu/drm/mxsfb/mxsfb_drv.c > +++ b/drivers/gpu/drm/mxsfb/mxsfb_drv.c > @@ -374,6 +374,7 @@ static int mxsfb_remove(struct platform_device *pdev) > struct drm_device *drm = platform_get_drvdata(pdev); > > drm_dev_unregister(drm); > + drm_atomic_helper_shutdown(drm); That looks reasonable. While at it, should you also implement the .shutdown driver operation ? Reviewed-by: Laurent Pinchart <laurent.pinch...@ideasonboard.com> > mxsfb_unload(drm); > drm_dev_put(drm); > -- Regards, Laurent Pinchart
