mbox series

[v2,0/4] drm: drop drmP in tda998x, tegra, arm, armada

Message ID 20190804094132.29463-1-sam@ravnborg.org (mailing list archive)
Headers show
Series drm: drop drmP in tda998x, tegra, arm, armada | expand

Message

Sam Ravnborg Aug. 4, 2019, 9:41 a.m. UTC
This set of patches is one of the final steps before
we have succeeded to stop using drmP.h in all of drm/.

There is a few patches in flight through other trees
and the plan is that all users shall be gone in the
upstream kernel after next merge window.

The patches has seen build test with various configs
with various architectures.

The patches has been sent before, but to my best knowledge
they have not been applied anywhere.
All four patches are based on drm-misc-next,
but I checked that the tda998x patch can be applied to
the tda998x tree.

There are no dependencies between the patches.

v2:
- rebase on top of drm-misc-next

To maintainers: (Assuming the patch are OK)
Please let me know if you take the patch, or request
me to apply it to drm-misc-next.
Or let me me know if the patch should be based on another tree.

	Sam


Sam Ravnborg (4):
      drm/i2c/tda998x: drop use of drmP.h
      drm/tegra: drop use of drmP.h
      drm/armada: drop use of drmP.h
      drm/arm: drop use of drmP.h

 drivers/gpu/drm/arm/hdlcd_crtc.c        | 12 +++++++-----
 drivers/gpu/drm/arm/hdlcd_drv.c         |  7 ++++++-
 drivers/gpu/drm/arm/malidp_crtc.c       | 11 +++++++----
 drivers/gpu/drm/arm/malidp_drv.c        |  8 +++++---
 drivers/gpu/drm/arm/malidp_drv.h        |  7 ++++---
 drivers/gpu/drm/arm/malidp_hw.c         |  7 ++++++-
 drivers/gpu/drm/arm/malidp_mw.c         |  5 +++--
 drivers/gpu/drm/arm/malidp_planes.c     |  4 +++-
 drivers/gpu/drm/armada/armada_crtc.c    | 10 +++++++---
 drivers/gpu/drm/armada/armada_debugfs.c |  8 ++++++--
 drivers/gpu/drm/armada/armada_drm.h     |  5 ++++-
 drivers/gpu/drm/armada/armada_drv.c     |  8 ++++++++
 drivers/gpu/drm/armada/armada_fb.c      |  3 +++
 drivers/gpu/drm/armada/armada_fbdev.c   |  3 +++
 drivers/gpu/drm/armada/armada_gem.c     |  7 ++++++-
 drivers/gpu/drm/armada/armada_overlay.c |  8 +++++---
 drivers/gpu/drm/armada/armada_plane.c   |  4 +++-
 drivers/gpu/drm/armada/armada_trace.h   |  5 ++++-
 drivers/gpu/drm/i2c/tda998x_drv.c       |  2 +-
 drivers/gpu/drm/tegra/dc.c              | 13 +++++++++----
 drivers/gpu/drm/tegra/dpaux.c           |  5 +++--
 drivers/gpu/drm/tegra/drm.c             |  8 ++++++++
 drivers/gpu/drm/tegra/drm.h             |  3 +--
 drivers/gpu/drm/tegra/dsi.c             |  8 +++++---
 drivers/gpu/drm/tegra/fb.c              |  6 ++++--
 drivers/gpu/drm/tegra/gem.c             |  3 +++
 drivers/gpu/drm/tegra/gem.h             |  1 -
 drivers/gpu/drm/tegra/gr2d.c            |  1 +
 drivers/gpu/drm/tegra/hdmi.c            |  5 +++++
 drivers/gpu/drm/tegra/hub.c             |  3 ++-
 drivers/gpu/drm/tegra/hub.h             |  1 -
 drivers/gpu/drm/tegra/plane.c           |  1 +
 drivers/gpu/drm/tegra/sor.c             |  3 +++
 drivers/gpu/drm/tegra/vic.c             |  1 +
 34 files changed, 137 insertions(+), 49 deletions(-)

Comments

Sam Ravnborg Aug. 12, 2019, 8:38 p.m. UTC | #1
Hi all.

On Sun, Aug 04, 2019 at 11:41:28AM +0200, Sam Ravnborg wrote:
> This set of patches is one of the final steps before
> we have succeeded to stop using drmP.h in all of drm/.
> 
> There is a few patches in flight through other trees
> and the plan is that all users shall be gone in the
> upstream kernel after next merge window.
> 
> The patches has seen build test with various configs
> with various architectures.
> 
> The patches has been sent before, but to my best knowledge
> they have not been applied anywhere.
> All four patches are based on drm-misc-next,
> but I checked that the tda998x patch can be applied to
> the tda998x tree.
> 
> There are no dependencies between the patches.
> 
> v2:
> - rebase on top of drm-misc-next
> 
> To maintainers: (Assuming the patch are OK)
> Please let me know if you take the patch, or request
> me to apply it to drm-misc-next.
> Or let me me know if the patch should be based on another tree.

ping...

This patchset is one of the last steps to get rid of drmP.h.
Other patches are applied to various sub-system trees.

The idea is that after next merge window can drop drmP.h.
As long as we keep drmP.h around new users will sneak in.

	Sam

> 
> Sam Ravnborg (4):
>       drm/i2c/tda998x: drop use of drmP.h
>       drm/tegra: drop use of drmP.h
>       drm/armada: drop use of drmP.h
>       drm/arm: drop use of drmP.h
> 
>  drivers/gpu/drm/arm/hdlcd_crtc.c        | 12 +++++++-----
>  drivers/gpu/drm/arm/hdlcd_drv.c         |  7 ++++++-
>  drivers/gpu/drm/arm/malidp_crtc.c       | 11 +++++++----
>  drivers/gpu/drm/arm/malidp_drv.c        |  8 +++++---
>  drivers/gpu/drm/arm/malidp_drv.h        |  7 ++++---
>  drivers/gpu/drm/arm/malidp_hw.c         |  7 ++++++-
>  drivers/gpu/drm/arm/malidp_mw.c         |  5 +++--
>  drivers/gpu/drm/arm/malidp_planes.c     |  4 +++-
>  drivers/gpu/drm/armada/armada_crtc.c    | 10 +++++++---
>  drivers/gpu/drm/armada/armada_debugfs.c |  8 ++++++--
>  drivers/gpu/drm/armada/armada_drm.h     |  5 ++++-
>  drivers/gpu/drm/armada/armada_drv.c     |  8 ++++++++
>  drivers/gpu/drm/armada/armada_fb.c      |  3 +++
>  drivers/gpu/drm/armada/armada_fbdev.c   |  3 +++
>  drivers/gpu/drm/armada/armada_gem.c     |  7 ++++++-
>  drivers/gpu/drm/armada/armada_overlay.c |  8 +++++---
>  drivers/gpu/drm/armada/armada_plane.c   |  4 +++-
>  drivers/gpu/drm/armada/armada_trace.h   |  5 ++++-
>  drivers/gpu/drm/i2c/tda998x_drv.c       |  2 +-
>  drivers/gpu/drm/tegra/dc.c              | 13 +++++++++----
>  drivers/gpu/drm/tegra/dpaux.c           |  5 +++--
>  drivers/gpu/drm/tegra/drm.c             |  8 ++++++++
>  drivers/gpu/drm/tegra/drm.h             |  3 +--
>  drivers/gpu/drm/tegra/dsi.c             |  8 +++++---
>  drivers/gpu/drm/tegra/fb.c              |  6 ++++--
>  drivers/gpu/drm/tegra/gem.c             |  3 +++
>  drivers/gpu/drm/tegra/gem.h             |  1 -
>  drivers/gpu/drm/tegra/gr2d.c            |  1 +
>  drivers/gpu/drm/tegra/hdmi.c            |  5 +++++
>  drivers/gpu/drm/tegra/hub.c             |  3 ++-
>  drivers/gpu/drm/tegra/hub.h             |  1 -
>  drivers/gpu/drm/tegra/plane.c           |  1 +
>  drivers/gpu/drm/tegra/sor.c             |  3 +++
>  drivers/gpu/drm/tegra/vic.c             |  1 +
>  34 files changed, 137 insertions(+), 49 deletions(-)
> 
> 
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
Sam Ravnborg Aug. 14, 2019, 4:36 p.m. UTC | #2
> > This set of patches is one of the final steps before
> > we have succeeded to stop using drmP.h in all of drm/.
> > 
> > There is a few patches in flight through other trees
> > and the plan is that all users shall be gone in the
> > upstream kernel after next merge window.
> > 
> > The patches has seen build test with various configs
> > with various architectures.
> > 
> > The patches has been sent before, but to my best knowledge
> > they have not been applied anywhere.
> > All four patches are based on drm-misc-next,
> > but I checked that the tda998x patch can be applied to
> > the tda998x tree.
> > 
> > There are no dependencies between the patches.
> > 
> > v2:
> > - rebase on top of drm-misc-next
> > 
> > To maintainers: (Assuming the patch are OK)
> > Please let me know if you take the patch, or request
> > me to apply it to drm-misc-next.
> > Or let me me know if the patch should be based on another tree.
> 
> ping...
> 
> This patchset is one of the last steps to get rid of drmP.h.
> Other patches are applied to various sub-system trees.
> 
> The idea is that after next merge window can drop drmP.h.
> As long as we keep drmP.h around new users will sneak in.

Thierry reviewed all patches - thanks!

Applied to drm-misc-next and pushed out.

	Sam