Re: [PATCH 00/20] DSC misc fixes

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On 8/17/2023 3:19 PM, Jani Nikula wrote:
On Thu, 10 Aug 2023, Ankit Nautiyal <ankit.k.nautiyal@xxxxxxxxx> wrote:
This series is an attempt to address multiple issues with DSC,
scattered in separate existing series.
I think it's a good idea to have one person manage the series, and
combine it all together, because it touches the same areas.

However, once you have smaller batches of patches that are all reviewed,
send them out as smaller series, and get them merged. Re-sending and
rebasing ready patches as part of a 19-patch series where some patches
aren't ready has become counter-productive.

Getting patches upstream is like working on a machine that has a certain
peak throughput. Send patches one by one, and it's inefficient. Send too
many at once, and it bogs down. Personally, I think 5-10 non-trivial
patches at a time is about right, get them reviewed and merged, rebase
the rest locally, and again send the next 5-10.

Or you can initially send a longer series, but once the first 5-10 have
been reviewed, send them separately.

Yeah this indeed become bloated with assortment of fixes. Should have separated series as they got reviewed. Will take care going forward.

For this series, I will merge the reviewed i915 patches and send a separate series for the drm patch#8 (drm/display/dp: Fix the DP DSC Receiver cap size).

Regards,

Ankit




BR,
Jani.


Patches 1-4 are DSC fixes from series to Handle BPC for HDMI2.1 PCON
https://patchwork.freedesktop.org/series/107550/

Patches 5-6 are from series DSC fixes for Bigjoiner:
https://patchwork.freedesktop.org/series/115773/

Patches 7-17 are based on series to add DSC fractional BPP support:
https://patchwork.freedesktop.org/series/111391/

Patch 20 is to fix compressed bpc for MST DSC, from Stan's series :
https://patchwork.freedesktop.org/series/116179/

Rev2: Addressed review comments from Stan, Ville.

Rev3: Split larger patches. Separate out common helpers.

Rev4: Rebased, fixed checkpatch warnings.

Rev5: Addressed review comments from Stan.
Added a patch to check if forced dsc format can be used before forcing.

Rev6: Addressed review comments from Stan.

Rev7: Reordered and rebased.

Ankit Nautiyal (19):
   drm/i915/dp: Consider output_format while computing dsc bpp
   drm/i915/dp: Move compressed bpp check with 420 format inside the
     helper
   drm/i915/dp_mst: Use output_format to get the final link bpp
   drm/i915/dp: Use consistent name for link bpp and compressed bpp
   drm/i915/dp: Update Bigjoiner interface bits for computing compressed
     bpp
   drm/i915/intel_cdclk: Add vdsc with bigjoiner constraints on min_cdlck
   drm/i915/dp: Remove extra logs for printing DSC info
   drm/display/dp: Fix the DP DSC Receiver cap size
   drm/i915/dp: Avoid forcing DSC BPC for MST case
   drm/i915/dp: Add functions to get min/max src input bpc with DSC
   drm/i915/dp: Check min bpc DSC limits for dsc_force_bpc also
   drm/i915/dp: Avoid left shift of DSC output bpp by 4
   drm/i915/dp: Rename helper to get DSC max pipe_bpp
   drm/i915/dp: Separate out functions for edp/DP for computing DSC bpp
   drm/i915/dp: Add DSC BPC/BPP constraints while selecting pipe bpp with
     DSC
   drm/i915/dp: Separate out function to get compressed bpp with joiner
   drm/i915/dp: Get optimal link config to have best compressed bpp
   drm/i915/dp: Check src/sink compressed bpp limit for edp
   drm/i915/dp: Check if force_dsc_output_format is possible

Stanislav Lisovskiy (1):
   drm/i915: Query compressed bpp properly using correct DPCD and DP Spec
     info

  drivers/gpu/drm/i915/display/intel_cdclk.c  |  59 +-
  drivers/gpu/drm/i915/display/intel_dp.c     | 655 ++++++++++++++++----
  drivers/gpu/drm/i915/display/intel_dp.h     |  20 +-
  drivers/gpu/drm/i915/display/intel_dp_mst.c |  80 +--
  include/drm/display/drm_dp.h                |   2 +-
  5 files changed, 625 insertions(+), 191 deletions(-)



[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux