Intel-GFX Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Intel-gfx] [PATCH v9 0/8] Add DSC PPS readout
@ 2023-08-22  6:02 Suraj Kandpal
  2023-08-22  6:02 ` [Intel-gfx] [PATCH v9 1/8] drm/i915/vdsc: Refactor dsc register field macro Suraj Kandpal
                   ` (16 more replies)
  0 siblings, 17 replies; 21+ messages in thread
From: Suraj Kandpal @ 2023-08-22  6:02 UTC (permalink / raw)
  To: intel-gfx; +Cc: Jani Nikula

Up until now we only verified one or two of the dsc pps
params like bits_per_component and bits_per_pixel this
patch series aim to readout almost all PPS param and get
them compared.
Along with that some work on making a common function to
read and write PPS param regiters is also done.

--v2
-Remove duplicated code and create function that fetches register
and reuse that. [Jani]
-move WARN_ON one abstraction layer up. [Jani]
-Split patch so that refactor and a new functionality is not added
in the same patch. [Jani]
-Add a new refactor patch so that bit shifting can be done in a
clean way. [Jani]

--v3
-Fix the typo in patch 2 [Jani]
-Get both dsc_reg regardless of dsc_eng_no. [Jani]
-Remove usage of num_vdsc_instances. [Ankit]
-Add macro to further optimize intel_dsc_get_pps_reg

--v4
-Manipulate register addresses rather than creating a macro to
manipulate variable name based on pps [Ankit]

--v5
-Use an array to fill the dsc registers instead of individual register
[Ankit]
-Get only dsc register that are required.

--v6
-make intel_get_no_vdsc_inst_per_pipe func static

--v7
-shorten func name for getting vdsc instances per pipe [Ankit]
-send no vdsc instances to dsc read and write reg funcs [Ankit]

--v8
-remove array dynamic allocation for dsc_reg [Jani]
-no need for an array to just verify dsc engine values [Jani]
-remove no_ from no_vdsc_per_pipe and wherever else it is
applicable [Ankit]

--v9
-change variable name to dsc_reg_size in intel_get_pps_reg [Ankit]
-send ARRAY_SIZE(dsc_reg) in intel_dsc_write_pps_reg [Ankit]

Signed-off-by: Suraj Kandpal <suraj.kandpal@intel.com>
Acked-by: Jani Nikula <jani.nikula@intel.com>

Suraj Kandpal (8):
  drm/i915/vdsc: Refactor dsc register field macro
  drm/i915/vdsc: Add a check for dsc split cases
  drm/i915/vdsc: Add func to get no. of vdsc instances per pipe
  drm/i915/vdsc: Add function to read any PPS register
  drm/i915/vdsc: Add function to write in PPS register
  drm/i915/vdsc: Remove unused dsc registers
  drm/i915/vdsc: Fill the intel_dsc_get_pps_config function
  drm/i915/display: Compare the readout dsc pps params

 drivers/gpu/drm/i915/display/intel_display.c  |  31 ++
 drivers/gpu/drm/i915/display/intel_vdsc.c     | 479 ++++++++----------
 .../gpu/drm/i915/display/intel_vdsc_regs.h    | 365 ++++---------
 3 files changed, 346 insertions(+), 529 deletions(-)

-- 
2.25.1


^ permalink raw reply	[flat|nested] 21+ messages in thread

end of thread, other threads:[~2023-08-23 10:17 UTC | newest]

Thread overview: 21+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-08-22  6:02 [Intel-gfx] [PATCH v9 0/8] Add DSC PPS readout Suraj Kandpal
2023-08-22  6:02 ` [Intel-gfx] [PATCH v9 1/8] drm/i915/vdsc: Refactor dsc register field macro Suraj Kandpal
2023-08-22  6:02 ` [Intel-gfx] [PATCH v9 2/8] drm/i915/vdsc: Add a check for dsc split cases Suraj Kandpal
2023-08-22  6:02 ` [Intel-gfx] [PATCH v9 3/8] drm/i915/vdsc: Add func to get no. of vdsc instances per pipe Suraj Kandpal
2023-08-22  6:02 ` [Intel-gfx] [PATCH v9 4/8] drm/i915/vdsc: Add function to read any PPS register Suraj Kandpal
2023-08-23  7:14   ` Nautiyal, Ankit K
2023-08-23  7:59   ` [Intel-gfx] [PATCH v10 " Suraj Kandpal
2023-08-23 10:17   ` [Intel-gfx] [PATCH v9 " Jani Nikula
2023-08-22  6:02 ` [Intel-gfx] [PATCH v9 5/8] drm/i915/vdsc: Add function to write in " Suraj Kandpal
2023-08-22  6:02 ` [Intel-gfx] [PATCH v9 6/8] drm/i915/vdsc: Remove unused dsc registers Suraj Kandpal
2023-08-22  6:02 ` [Intel-gfx] [PATCH v9 7/8] drm/i915/vdsc: Fill the intel_dsc_get_pps_config function Suraj Kandpal
2023-08-22  6:02 ` [Intel-gfx] [PATCH v9 8/8] drm/i915/display: Compare the readout dsc pps params Suraj Kandpal
2023-08-22  6:54 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for Add DSC PPS readout (rev9) Patchwork
2023-08-22  6:54 ` [Intel-gfx] ✗ Fi.CI.SPARSE: " Patchwork
2023-08-22  7:13 ` [Intel-gfx] ✗ Fi.CI.BAT: failure " Patchwork
2023-08-22 11:42 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork
2023-08-22 13:03 ` [Intel-gfx] ✗ Fi.CI.IGT: failure " Patchwork
2023-08-23  7:20 ` [Intel-gfx] ✓ Fi.CI.IGT: success " Patchwork
2023-08-23  9:35 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for Add DSC PPS readout (rev10) Patchwork
2023-08-23  9:35 ` [Intel-gfx] ✗ Fi.CI.SPARSE: " Patchwork
2023-08-23  9:57 ` [Intel-gfx] ✗ Fi.CI.BAT: failure " Patchwork

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox