On Mon, 9 Mar 2020 at 20:27, Lyude Paul lyude@redhat.com 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 lyude@redhat.com
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.velikov@collabora.com
-Emil