Re: [PATCH v2 0/3] drm/amdgpu: Fix suspend/resume issues with MST

2019-01-08 Thread Lyude Paul
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

2019-01-08 Thread Alex Deucher
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

2019-01-08 Thread Lyude Paul
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