mbox series

[v2,0/9] drm/i915: Dump DSC state to dmesg/debugfs

Message ID 20240628164451.1177612-1-imre.deak@intel.com (mailing list archive)
Headers show
Series drm/i915: Dump DSC state to dmesg/debugfs | expand

Message

Imre Deak June 28, 2024, 4:44 p.m. UTC
This is v2 of [1], renaming the helpers from drm_x16 to fxp_q4 as
suggested by Jani.

[1] https://lore.kernel.org/all/20240614173911.3743172-1-imre.deak@intel.com

Cc: Jani Nikula <jani.nikula@intel.com>

Imre Deak (9):
  drm: Add helpers for q4 fixed point values
  drm/display/dsc: Add a helper to dump the DSC configuration
  drm/i915: Replace to_bpp_x16() with fxp_q4_from_int()
  drm/i915: Replace to_bpp_int() with fxp_q4_to_int()
  drm/i915: Replace to_bpp_int_roundup() with fxp_q4_to_int_roundup()
  drm/i915: Replace to_bpp_frac() with fxp_q4_to_frac()
  drm/i915: Replace BPP_X16_FMT()/ARGS() with FXP_Q4_FMT()/ARGS()
  drm/i915: Dump DSC state to dmesg and debugfs/i915_display_info
  drm/i915: Remove DSC register dump

 drivers/gpu/drm/display/drm_dp_helper.c       |  5 +-
 drivers/gpu/drm/display/drm_dsc_helper.c      | 91 +++++++++++++++++++
 drivers/gpu/drm/i915/display/icl_dsi.c        |  9 +-
 drivers/gpu/drm/i915/display/intel_audio.c    |  5 +-
 drivers/gpu/drm/i915/display/intel_bios.c     |  5 +-
 drivers/gpu/drm/i915/display/intel_cdclk.c    |  5 +-
 .../drm/i915/display/intel_crtc_state_dump.c  |  3 +
 drivers/gpu/drm/i915/display/intel_display.c  |  7 +-
 .../drm/i915/display/intel_display_debugfs.c  |  4 +
 .../drm/i915/display/intel_display_types.h    | 23 -----
 drivers/gpu/drm/i915/display/intel_dp.c       | 51 ++++++-----
 drivers/gpu/drm/i915/display/intel_dp_mst.c   | 26 +++---
 drivers/gpu/drm/i915/display/intel_fdi.c      |  6 +-
 drivers/gpu/drm/i915/display/intel_link_bw.c  |  4 +-
 drivers/gpu/drm/i915/display/intel_vdsc.c     | 51 ++++++-----
 drivers/gpu/drm/i915/display/intel_vdsc.h     |  4 +
 include/drm/display/drm_dsc_helper.h          |  3 +
 include/drm/drm_fixed.h                       | 23 +++++
 18 files changed, 222 insertions(+), 103 deletions(-)

Comments

Jani Nikula July 1, 2024, 11:28 a.m. UTC | #1
On Fri, 28 Jun 2024, Imre Deak <imre.deak@intel.com> wrote:
> This is v2 of [1], renaming the helpers from drm_x16 to fxp_q4 as
> suggested by Jani.
>
> [1] https://lore.kernel.org/all/20240614173911.3743172-1-imre.deak@intel.com
>
> Cc: Jani Nikula <jani.nikula@intel.com>

No detailed review, but on the approach and naming,

Acked-by: Jani Nikula <jani.nikula@intel.com>



>
> Imre Deak (9):
>   drm: Add helpers for q4 fixed point values
>   drm/display/dsc: Add a helper to dump the DSC configuration
>   drm/i915: Replace to_bpp_x16() with fxp_q4_from_int()
>   drm/i915: Replace to_bpp_int() with fxp_q4_to_int()
>   drm/i915: Replace to_bpp_int_roundup() with fxp_q4_to_int_roundup()
>   drm/i915: Replace to_bpp_frac() with fxp_q4_to_frac()
>   drm/i915: Replace BPP_X16_FMT()/ARGS() with FXP_Q4_FMT()/ARGS()
>   drm/i915: Dump DSC state to dmesg and debugfs/i915_display_info
>   drm/i915: Remove DSC register dump
>
>  drivers/gpu/drm/display/drm_dp_helper.c       |  5 +-
>  drivers/gpu/drm/display/drm_dsc_helper.c      | 91 +++++++++++++++++++
>  drivers/gpu/drm/i915/display/icl_dsi.c        |  9 +-
>  drivers/gpu/drm/i915/display/intel_audio.c    |  5 +-
>  drivers/gpu/drm/i915/display/intel_bios.c     |  5 +-
>  drivers/gpu/drm/i915/display/intel_cdclk.c    |  5 +-
>  .../drm/i915/display/intel_crtc_state_dump.c  |  3 +
>  drivers/gpu/drm/i915/display/intel_display.c  |  7 +-
>  .../drm/i915/display/intel_display_debugfs.c  |  4 +
>  .../drm/i915/display/intel_display_types.h    | 23 -----
>  drivers/gpu/drm/i915/display/intel_dp.c       | 51 ++++++-----
>  drivers/gpu/drm/i915/display/intel_dp_mst.c   | 26 +++---
>  drivers/gpu/drm/i915/display/intel_fdi.c      |  6 +-
>  drivers/gpu/drm/i915/display/intel_link_bw.c  |  4 +-
>  drivers/gpu/drm/i915/display/intel_vdsc.c     | 51 ++++++-----
>  drivers/gpu/drm/i915/display/intel_vdsc.h     |  4 +
>  include/drm/display/drm_dsc_helper.h          |  3 +
>  include/drm/drm_fixed.h                       | 23 +++++
>  18 files changed, 222 insertions(+), 103 deletions(-)
Imre Deak July 3, 2024, 3:10 p.m. UTC | #2
On Mon, Jul 01, 2024 at 02:28:31PM +0300, Jani Nikula wrote:
> On Fri, 28 Jun 2024, Imre Deak <imre.deak@intel.com> wrote:
> > This is v2 of [1], renaming the helpers from drm_x16 to fxp_q4 as
> > suggested by Jani.
> >
> > [1] https://lore.kernel.org/all/20240614173911.3743172-1-imre.deak@intel.com
> >
> > Cc: Jani Nikula <jani.nikula@intel.com>
> 
> No detailed review, but on the approach and naming,
> 
> Acked-by: Jani Nikula <jani.nikula@intel.com>

Testing the patchset in CI shards is still pending, however based on the
available BAT result, I pushed already the first two patches with your
ack to drm-misc-next, thanks.

> > Imre Deak (9):
> >   drm: Add helpers for q4 fixed point values
> >   drm/display/dsc: Add a helper to dump the DSC configuration
> >   drm/i915: Replace to_bpp_x16() with fxp_q4_from_int()
> >   drm/i915: Replace to_bpp_int() with fxp_q4_to_int()
> >   drm/i915: Replace to_bpp_int_roundup() with fxp_q4_to_int_roundup()
> >   drm/i915: Replace to_bpp_frac() with fxp_q4_to_frac()
> >   drm/i915: Replace BPP_X16_FMT()/ARGS() with FXP_Q4_FMT()/ARGS()
> >   drm/i915: Dump DSC state to dmesg and debugfs/i915_display_info
> >   drm/i915: Remove DSC register dump
> >
> >  drivers/gpu/drm/display/drm_dp_helper.c       |  5 +-
> >  drivers/gpu/drm/display/drm_dsc_helper.c      | 91 +++++++++++++++++++
> >  drivers/gpu/drm/i915/display/icl_dsi.c        |  9 +-
> >  drivers/gpu/drm/i915/display/intel_audio.c    |  5 +-
> >  drivers/gpu/drm/i915/display/intel_bios.c     |  5 +-
> >  drivers/gpu/drm/i915/display/intel_cdclk.c    |  5 +-
> >  .../drm/i915/display/intel_crtc_state_dump.c  |  3 +
> >  drivers/gpu/drm/i915/display/intel_display.c  |  7 +-
> >  .../drm/i915/display/intel_display_debugfs.c  |  4 +
> >  .../drm/i915/display/intel_display_types.h    | 23 -----
> >  drivers/gpu/drm/i915/display/intel_dp.c       | 51 ++++++-----
> >  drivers/gpu/drm/i915/display/intel_dp_mst.c   | 26 +++---
> >  drivers/gpu/drm/i915/display/intel_fdi.c      |  6 +-
> >  drivers/gpu/drm/i915/display/intel_link_bw.c  |  4 +-
> >  drivers/gpu/drm/i915/display/intel_vdsc.c     | 51 ++++++-----
> >  drivers/gpu/drm/i915/display/intel_vdsc.h     |  4 +
> >  include/drm/display/drm_dsc_helper.h          |  3 +
> >  include/drm/drm_fixed.h                       | 23 +++++
> >  18 files changed, 222 insertions(+), 103 deletions(-)
> 
> -- 
> Jani Nikula, Intel