mbox series

[0/9] drm: drm_fb_helper cleanup.

Message ID 20200302125649.61443-1-pankaj.laxminarayan.bharadiya@intel.com (mailing list archive)
Headers show
Series drm: drm_fb_helper cleanup. | expand

Message

Pankaj Bharadiya March 2, 2020, 12:56 p.m. UTC
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().

Pankaj Bharadiya (9):
  drm: Remove unused arg from drm_fb_helper_init
  drm/radeon: remove radeon_fb_{add,remove}_connector functions
  drm/amdgpu: Remove drm_fb_helper_{add,remove}_one_connector calls
  drm/i915/display: Remove drm_fb_helper_{add,remove}_one_connector calls
  drm: Remove drm_fb_helper add, add all and remove connector calls
  drm/nouveau: Fix unused variable warning
  drm/bridge: remove unused variable warning in tc358764_detach
  drm/fb-helper: Remove drm_fb_helper add, add_all and remove connector functions
  drm/todo: Update drm_fb_helper tasks

 Documentation/gpu/todo.rst                    | 15 +++--------
 drivers/gpu/drm/amd/amdgpu/amdgpu_fb.c        |  5 +---
 .../display/amdgpu_dm/amdgpu_dm_mst_types.c   | 13 ---------
 drivers/gpu/drm/armada/armada_fbdev.c         |  8 +-----
 drivers/gpu/drm/bridge/tc358764.c             |  3 ---
 drivers/gpu/drm/drm_fb_helper.c               |  6 ++---
 drivers/gpu/drm/exynos/exynos_drm_dsi.c       |  1 -
 drivers/gpu/drm/exynos/exynos_drm_fbdev.c     | 10 +------
 drivers/gpu/drm/gma500/framebuffer.c          |  6 +----
 drivers/gpu/drm/i915/display/intel_dp_mst.c   | 12 ---------
 drivers/gpu/drm/i915/display/intel_fbdev.c    |  4 +--
 drivers/gpu/drm/msm/msm_fbdev.c               |  6 +----
 drivers/gpu/drm/nouveau/dispnv50/disp.c       |  7 -----
 drivers/gpu/drm/nouveau/nouveau_fbcon.c       |  6 +----
 drivers/gpu/drm/omapdrm/omap_fbdev.c          |  6 +----
 drivers/gpu/drm/radeon/radeon_dp_mst.c        | 10 -------
 drivers/gpu/drm/radeon/radeon_fb.c            | 19 +------------
 drivers/gpu/drm/radeon/radeon_mode.h          |  3 ---
 drivers/gpu/drm/rockchip/rockchip_drm_fbdev.c |  9 +------
 drivers/gpu/drm/tegra/fb.c                    |  8 +-----
 include/drm/drm_fb_helper.h                   | 27 ++-----------------
 21 files changed, 19 insertions(+), 165 deletions(-)

Comments

Emil Velikov March 2, 2020, 3:58 p.m. UTC | #1
On Mon, 2 Mar 2020 at 13:08, Pankaj Bharadiya
<pankaj.laxminarayan.bharadiya@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().
>
> Pankaj Bharadiya (9):
>   drm: Remove unused arg from drm_fb_helper_init
>   drm/radeon: remove radeon_fb_{add,remove}_connector functions
>   drm/amdgpu: Remove drm_fb_helper_{add,remove}_one_connector calls
>   drm/i915/display: Remove drm_fb_helper_{add,remove}_one_connector calls
>   drm: Remove drm_fb_helper add, add all and remove connector calls
>   drm/nouveau: Fix unused variable warning
>   drm/bridge: remove unused variable warning in tc358764_detach
>   drm/fb-helper: Remove drm_fb_helper add, add_all and remove connector functions
>   drm/todo: Update drm_fb_helper tasks
>
With 6/9 and 7/9 squashed into 1/9, as suggested by Laurent + the
wrapping retained as-is, the series is:
Reviewed-by: Emil Velikov <emil.velikov@collabora.com>

-Emil
P.S. I'm loving the diff stat :-)
Emil Velikov March 2, 2020, 4:17 p.m. UTC | #2
On Mon, 2 Mar 2020 at 15:58, Emil Velikov <emil.l.velikov@gmail.com> wrote:
>
> On Mon, 2 Mar 2020 at 13:08, Pankaj Bharadiya
> <pankaj.laxminarayan.bharadiya@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().
> >
> > Pankaj Bharadiya (9):
> >   drm: Remove unused arg from drm_fb_helper_init
> >   drm/radeon: remove radeon_fb_{add,remove}_connector functions
> >   drm/amdgpu: Remove drm_fb_helper_{add,remove}_one_connector calls
> >   drm/i915/display: Remove drm_fb_helper_{add,remove}_one_connector calls
> >   drm: Remove drm_fb_helper add, add all and remove connector calls
> >   drm/nouveau: Fix unused variable warning
> >   drm/bridge: remove unused variable warning in tc358764_detach
> >   drm/fb-helper: Remove drm_fb_helper add, add_all and remove connector functions
> >   drm/todo: Update drm_fb_helper tasks
> >
> With 6/9 and 7/9 squashed into 1/9, as suggested by Laurent + the
Oops s;into 1/9;respective the patches;g

-Emil