> -----Original Message----- > From: Emil Velikov <emil.l.veli...@gmail.com> > Sent: 02 March 2020 23:51 > To: Laxminarayan Bharadiya, Pankaj > <pankaj.laxminarayan.bharad...@intel.com> > Cc: Jani Nikula <jani.nik...@linux.intel.com>; Daniel Vetter > <dan...@ffwll.ch>; Intel Graphics Development <intel- > g...@lists.freedesktop.org>; ML dri-devel <dri-devel@lists.freedesktop.org> > Subject: Re: [Intel-gfx] [PATCH v2 0/7] drm: drm_fb_helper cleanup. > > Hi Pankaj, > > On Mon, 2 Mar 2020 at 16:33, Pankaj Bharadiya > <pankaj.laxminarayan.bharad...@intel.com> wrote: > > > > This series addresses below drm_fb_helper tasks from > > Documentation/gpu/todo.rst. > > > > - The max connector argument for drm_fb_helper_init() isn't used > > anymore and can be removed. > > > > - The helper doesn't keep an array of connectors anymore so these can > > be removed: drm_fb_helper_single_add_all_connectors(), > > drm_fb_helper_add_one_connector() and > > drm_fb_helper_remove_one_connector(). > > > > Changes since v1: > > - Squashed warning fixes into the patch that introduced the > > warnings (into 5/7) (Laurent) > > - Fixed reflow in in 9/9 (Laurent) > > > For the future, include the changelog in the respective patches. This makes it > easier for reviewers... > Plus you're already changing the commit - might as well mention what/why :- > ) > > Also do include the R-B, Acked-by, other tags accumulated up-to that point, > when sending new revision. Noted, Thank you for the feedback. Will send new series with tags accumulated after 1-2 days. > > > That said, if you're interested in further cleaning this up, one can cleanup > the > drm_dp_mst_topology_cbs hooks. > In particular ::register_connector is identical across the board - create a > helper function using it directly in core, killing the hook. > > While for ::destroy_connector - there's some amdgpu specific code in > there... which I'm not sure if it should stay or not. > To be on the save side - create a helper which will be called for drivers > where > the hook is !=NULL (aka everyone but amdgpu). Will take a look. Thanks, Pankaj > > HTH > Emil _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
RE: [Intel-gfx] [PATCH v2 0/7] drm: drm_fb_helper cleanup.
Laxminarayan Bharadiya, Pankaj Mon, 02 Mar 2020 20:40:56 -0800
- [PATCH v2 2/7] drm/radeon: remove radeon_fb... Pankaj Bharadiya
- [PATCH v2 3/7] drm/amdgpu: Remove drm_fb_he... Pankaj Bharadiya
- [PATCH v2 4/7] drm/i915/display: Remove drm... Pankaj Bharadiya
- [PATCH v2 7/7] drm/todo: Update drm_fb_help... Pankaj Bharadiya
- [PATCH v2 5/7] drm: Remove drm_fb_helper ad... Pankaj Bharadiya
- [PATCH v2 6/7] drm/fb-helper: Remove drm_fb... Pankaj Bharadiya
- Re: [Intel-gfx] [PATCH v2 0/7] drm: drm_fb_... Emil Velikov
- Re: [Intel-gfx] [PATCH v2 0/7] drm: dr... Daniel Vetter
- Re: [Intel-gfx] [PATCH v2 0/7] drm... Daniel Vetter
- RE: [Intel-gfx] [PATCH v2 0/7]... Laxminarayan Bharadiya, Pankaj
- RE: [Intel-gfx] [PATCH v2 0/7] drm: dr... Laxminarayan Bharadiya, Pankaj