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 <[email protected]> > Cc: Daniel Vetter <[email protected]> > Cc: Laurent Pinchart <[email protected]> > Cc: Lucas Stach <[email protected]> > Cc: Sam Ravnborg <[email protected]> > Cc: Stefan Agner <[email protected]> > Cc: Thomas Zimmermann <[email protected]> > --- > 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 <[email protected]> > mxsfb_unload(drm); > drm_dev_put(drm); > -- Regards, Laurent Pinchart
