Re: [PATCH 0/5] Cleanup drm_dp_mst_topology_cbs hooks
On Mon, 9 Mar 2020 at 20:27, Lyude Paul wrote: > > On Sat, 2020-03-07 at 14:00 +0530, Pankaj Bharadiya wrote: > > drm_dp_mst_topology_mgr_cbs.register_connector callbacks are identical > > amongst every driver and don't do anything other than calling > > drm_connector_register(). > > drm_dp_mst_topology_mgr_cbs.destroy_connector callbacks are identical > > amongst every driver and don't do anything other than cleaning up the > > connector((drm_connector_unregister()/drm_connector_put())) except for > > amdgpu_dm driver where some amdgpu_dm specific code in there. > > Yeah that amdgpu destruction code kinda stinks a little bit :\. I think we can > just drop some of it and move the rest into their connector destruction > callbacks. > > For the whole series: > Reviewed-by: Lyude Paul > > I'm going to go ahead and let the maintainers know I'm going to push this > (since there's some minor changes here outside of drm-misc), and push this to > drm-misc-next. Then I'll go and write some patches to remove the leftover amd > bits and drop the callback for good (I'll cc it to you as well). > Thanks for following on these Pankaj. For the series: Reviewed-by: Emil Velikov -Emil ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
Re: [PATCH 0/5] Cleanup drm_dp_mst_topology_cbs hooks
On Mon, Mar 9, 2020 at 4:27 PM Lyude Paul wrote: > > On Sat, 2020-03-07 at 14:00 +0530, Pankaj Bharadiya wrote: > > drm_dp_mst_topology_mgr_cbs.register_connector callbacks are identical > > amongst every driver and don't do anything other than calling > > drm_connector_register(). > > drm_dp_mst_topology_mgr_cbs.destroy_connector callbacks are identical > > amongst every driver and don't do anything other than cleaning up the > > connector((drm_connector_unregister()/drm_connector_put())) except for > > amdgpu_dm driver where some amdgpu_dm specific code in there. > > Yeah that amdgpu destruction code kinda stinks a little bit :\. I think we can > just drop some of it and move the rest into their connector destruction > callbacks. > > For the whole series: > Reviewed-by: Lyude Paul > > I'm going to go ahead and let the maintainers know I'm going to push this > (since there's some minor changes here outside of drm-misc), and push this to > drm-misc-next. Then I'll go and write some patches to remove the leftover amd > bits and drop the callback for good (I'll cc it to you as well). Series is: Reviewed-by: Alex Deucher > > > > > This series aims to cleaup these drm_dp_mst_topology_mgr_cbs hooks. > > > > Pankaj Bharadiya (5): > > drm: Register connector instead of calling register_connector callback > > drm: Remove dp mst register connector callbacks > > drm/dp_mst: Remove register_connector callback > > drm: Add drm_dp_destroy_connector helper and use it > > drm: Remove drm dp mst destroy_connector callbacks > > > > .../display/amdgpu_dm/amdgpu_dm_mst_types.c | 6 -- > > drivers/gpu/drm/drm_dp_mst_topology.c | 18 +++--- > > drivers/gpu/drm/i915/display/intel_dp_mst.c | 16 > > drivers/gpu/drm/nouveau/dispnv50/disp.c | 19 --- > > drivers/gpu/drm/radeon/radeon_dp_mst.c| 17 - > > include/drm/drm_dp_mst_helper.h | 1 - > > 6 files changed, 15 insertions(+), 62 deletions(-) > > > -- > Cheers, > Lyude Paul (she/her) > Associate Software Engineer at Red Hat > > ___ > dri-devel mailing list > dri-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
Re: [PATCH 0/5] Cleanup drm_dp_mst_topology_cbs hooks
On Sat, 2020-03-07 at 14:00 +0530, Pankaj Bharadiya wrote: > drm_dp_mst_topology_mgr_cbs.register_connector callbacks are identical > amongst every driver and don't do anything other than calling > drm_connector_register(). > drm_dp_mst_topology_mgr_cbs.destroy_connector callbacks are identical > amongst every driver and don't do anything other than cleaning up the > connector((drm_connector_unregister()/drm_connector_put())) except for > amdgpu_dm driver where some amdgpu_dm specific code in there. Yeah that amdgpu destruction code kinda stinks a little bit :\. I think we can just drop some of it and move the rest into their connector destruction callbacks. For the whole series: Reviewed-by: Lyude Paul I'm going to go ahead and let the maintainers know I'm going to push this (since there's some minor changes here outside of drm-misc), and push this to drm-misc-next. Then I'll go and write some patches to remove the leftover amd bits and drop the callback for good (I'll cc it to you as well). > > This series aims to cleaup these drm_dp_mst_topology_mgr_cbs hooks. > > Pankaj Bharadiya (5): > drm: Register connector instead of calling register_connector callback > drm: Remove dp mst register connector callbacks > drm/dp_mst: Remove register_connector callback > drm: Add drm_dp_destroy_connector helper and use it > drm: Remove drm dp mst destroy_connector callbacks > > .../display/amdgpu_dm/amdgpu_dm_mst_types.c | 6 -- > drivers/gpu/drm/drm_dp_mst_topology.c | 18 +++--- > drivers/gpu/drm/i915/display/intel_dp_mst.c | 16 > drivers/gpu/drm/nouveau/dispnv50/disp.c | 19 --- > drivers/gpu/drm/radeon/radeon_dp_mst.c| 17 - > include/drm/drm_dp_mst_helper.h | 1 - > 6 files changed, 15 insertions(+), 62 deletions(-) > -- Cheers, Lyude Paul (she/her) Associate Software Engineer at Red Hat ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
