On Mon, Jan 27, 2020 at 8:30 PM Rajneesh Bhardwaj <rajneesh.bhardwaj@xxxxxxx> wrote: > > amdgpu_device_suspend might return an error code since it can be called > from both runtime and system suspend flows. Add the missing return code > in case of a failure. > > Signed-off-by: Rajneesh Bhardwaj <rajneesh.bhardwaj@xxxxxxx> Reviewed-by: Alex Deucher <alexander.deucher@xxxxxxx> > --- > drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c > index 137e76f0e3db..71466df6dff6 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c > @@ -1236,6 +1236,9 @@ static int amdgpu_pmops_runtime_suspend(struct device *dev) > drm_kms_helper_poll_disable(drm_dev); > > ret = amdgpu_device_suspend(drm_dev, false); > + if (ret) > + return ret; > + > if (amdgpu_device_supports_boco(drm_dev)) { > /* Only need to handle PCI state in the driver for ATPX > * PCI core handles it for _PR3. > -- > 2.17.1 > > _______________________________________________ > amd-gfx mailing list > amd-gfx@xxxxxxxxxxxxxxxxxxxxx > https://lists.freedesktop.org/mailman/listinfo/amd-gfx _______________________________________________ amd-gfx mailing list amd-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/amd-gfx