Re: [PATCH v2 0/3] drm/amdgpu: Fix suspend/resume issues with MST
On Tue, 2019-01-08 at 16:17 -0500, Alex Deucher wrote: > On Tue, Jan 8, 2019 at 4:11 PM Lyude Paul wrote: > > Fix the suspend/issues that Jerry Zuo found in amdgpu, and add some > > compiler warnings for drivers ignoring the return code of > > drm_dp_mst_topology_mgr_resume() to help ensure we don't need to fix > > this again in the future for someone else's driver. > > > > Cc: Jerry Zuo > > > > Lyude Paul (3): > > drm/amdgpu: Don't ignore rc from drm_dp_mst_topology_mgr_resume() > > drm/amdgpu: Don't fail resume process if resuming atomic state fails > > drm/dp_mst: Add __must_check to drm_dp_mst_topology_mgr_resume() > > I can take these through the amdgpu tree or you can commit them to > drm-misc (assuming you have commit rights). Either way works for me. I don't mind just pushing these to drm-misc since you seem OK with it. Thanks! > > Thanks, > > Alex > > > .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 37 +-- > > include/drm/drm_dp_mst_helper.h | 3 +- > > 2 files changed, 27 insertions(+), 13 deletions(-) > > > > -- > > 2.20.1 > > > > ___ > > amd-gfx mailing list > > amd-gfx@lists.freedesktop.org > > https://lists.freedesktop.org/mailman/listinfo/amd-gfx -- Cheers, Lyude Paul ___ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx
Re: [PATCH v2 0/3] drm/amdgpu: Fix suspend/resume issues with MST
On Tue, Jan 8, 2019 at 4:11 PM Lyude Paul wrote: > > Fix the suspend/issues that Jerry Zuo found in amdgpu, and add some > compiler warnings for drivers ignoring the return code of > drm_dp_mst_topology_mgr_resume() to help ensure we don't need to fix > this again in the future for someone else's driver. > > Cc: Jerry Zuo > > Lyude Paul (3): > drm/amdgpu: Don't ignore rc from drm_dp_mst_topology_mgr_resume() > drm/amdgpu: Don't fail resume process if resuming atomic state fails > drm/dp_mst: Add __must_check to drm_dp_mst_topology_mgr_resume() I can take these through the amdgpu tree or you can commit them to drm-misc (assuming you have commit rights). Either way works for me. Thanks, Alex > > .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 37 +-- > include/drm/drm_dp_mst_helper.h | 3 +- > 2 files changed, 27 insertions(+), 13 deletions(-) > > -- > 2.20.1 > > ___ > amd-gfx mailing list > amd-gfx@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/amd-gfx ___ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx
[PATCH v2 0/3] drm/amdgpu: Fix suspend/resume issues with MST
Fix the suspend/issues that Jerry Zuo found in amdgpu, and add some compiler warnings for drivers ignoring the return code of drm_dp_mst_topology_mgr_resume() to help ensure we don't need to fix this again in the future for someone else's driver. Cc: Jerry Zuo Lyude Paul (3): drm/amdgpu: Don't ignore rc from drm_dp_mst_topology_mgr_resume() drm/amdgpu: Don't fail resume process if resuming atomic state fails drm/dp_mst: Add __must_check to drm_dp_mst_topology_mgr_resume() .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 37 +-- include/drm/drm_dp_mst_helper.h | 3 +- 2 files changed, 27 insertions(+), 13 deletions(-) -- 2.20.1 ___ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx