Re: [PATCH v1 0/7] Remove debugfs from sti display driver

[Benjamin Gaignard]

Gentle ping on this serie,

Thanks

Benjamin

2018-06-05 15:54 GMT+02:00 Benjamin Gaignard <benjamin.gaignard@xxxxxxxxxx>:
> Thanks to the various atomic_print_state hooks in drm structure all
> custom debugfs code could be remove from sti driver (~ -330 lines).
>
> This patchset does two addtion in drm core:
> - printing normalized zpos of each plane
> - add "atomic_print" hook in encoder structure to be able to dump encoders
>   at the same time than the others elements
>
> All other patches are implemeting the various hooks in sti driver.
>
> Benjamin Gaignard (7):
>   drm: print plane state normalized zpos value
>   drm: add hook to print encoder status
>   drm: sti: make planes use atomic_print_state instead of debugfs
>   drm: sti: make connectors use atomic_print_state instead of debugfs
>   drm: sti: make crtc use atomic_print_state instead of debugfs
>   drm: sti: make encoders use atomic_print_state instead of debugfs
>   drm: sti: remove the last call to debugfs
>
>  drivers/gpu/drm/drm_atomic.c         |  16 +++
>  drivers/gpu/drm/sti/sti_compositor.c |  16 ---
>  drivers/gpu/drm/sti/sti_compositor.h |   3 -
>  drivers/gpu/drm/sti/sti_crtc.c       |  17 +--
>  drivers/gpu/drm/sti/sti_cursor.c     |  65 ++++--------
>  drivers/gpu/drm/sti/sti_drv.c        |  50 ---------
>  drivers/gpu/drm/sti/sti_dvo.c        |  60 +++--------
>  drivers/gpu/drm/sti/sti_gdp.c        | 196 +++++++++++------------------------
>  drivers/gpu/drm/sti/sti_hda.c        |  75 ++++----------
>  drivers/gpu/drm/sti/sti_hdmi.c       | 132 ++++++++++-------------
>  drivers/gpu/drm/sti/sti_hqvdp.c      | 149 +++++++++++---------------
>  drivers/gpu/drm/sti/sti_mixer.c      |  89 +++++-----------
>  drivers/gpu/drm/sti/sti_mixer.h      |   3 +-
>  drivers/gpu/drm/sti/sti_tvout.c      | 162 +++++++++++------------------
>  drivers/gpu/drm/sti/sti_vid.c        |  60 ++++-------
>  drivers/gpu/drm/sti/sti_vid.h        |   2 +-
>  include/drm/drm_encoder.h            |  12 +++
>  17 files changed, 386 insertions(+), 721 deletions(-)
>
> --
> 2.15.0
>



-- 
Benjamin Gaignard

Graphic Study Group

Linaro.org â Open source software for ARM SoCs

Follow Linaro: Facebook | Twitter | Blog