On 09/10/2026 13:33, Adrián Larumbe wrote: > Rather than just failing silently, let's warn the user of device removal > not being able to take a PM reference, because it should be seen as fatal. > > Reviewed-by: Boris Brezillon <[email protected]> > Signed-off-by: Adrián Larumbe <[email protected]>
Reviewed-by: Steven Price <[email protected]> > --- > drivers/gpu/drm/panfrost/panfrost_device.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/panfrost/panfrost_device.c > b/drivers/gpu/drm/panfrost/panfrost_device.c > index a6deabb39fd2..687230225743 100644 > --- a/drivers/gpu/drm/panfrost/panfrost_device.c > +++ b/drivers/gpu/drm/panfrost/panfrost_device.c > @@ -9,6 +9,7 @@ > #include <linux/pm_runtime.h> > #include <linux/regulator/consumer.h> > #include <drm/drm_drv.h> > +#include <drm/drm_print.h> > > #include "panfrost_device.h" > #include "panfrost_devfreq.h" > @@ -385,7 +386,7 @@ int panfrost_device_init(struct panfrost_device *pfdev) > > void panfrost_device_fini(struct panfrost_device *pfdev) > { > - pm_runtime_get_sync(pfdev->base.dev); > + drm_WARN_ON(&pfdev->base, pm_runtime_get_sync(pfdev->base.dev) < 0); > > pm_runtime_dont_use_autosuspend(pfdev->base.dev); > pm_runtime_disable(pfdev->base.dev); >
