> -----Original Message----- > From: dri-devel <dri-devel-bounces@xxxxxxxxxxxxxxxxxxxxx> On Behalf Of Jani > Nikula > Sent: Wednesday, March 8, 2023 5:00 PM > To: Kandpal, Suraj <suraj.kandpal@xxxxxxxxx>; dri- > devel@xxxxxxxxxxxxxxxxxxxxx; intel-gfx@xxxxxxxxxxxxxxxxxxxxx > Cc: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx>; Nautiyal, Ankit K > <ankit.k.nautiyal@xxxxxxxxx>; Shankar, Uma <uma.shankar@xxxxxxxxx>; > Kandpal, Suraj <suraj.kandpal@xxxxxxxxx> > Subject: Re: [PATCH 0/7] Enable YCbCr420 format for VDSC > > On Wed, 22 Feb 2023, Suraj Kandpal <suraj.kandpal@xxxxxxxxx> wrote: > > This patch series aims to enable the YCbCr420 format for DSC. Changes > > are mostly compute params related for hdmi,dp and dsi along with the > > addition of new rc_tables for native_420 and corresponding changes to > > macros used to fetch them. > > There have been discussions prior to this series in which some patches > > have gotten rb and can be found in the below link > > https://patchwork.freedesktop.org/series/113729 > > I think it would be useful to get [1] from Dmitry merged to drm-misc-next > first, have that in drm-next, and again backmerged to drm-intel-next before > this. At least patches 1-5. > > There's not much point in all drivers duplicating the parameters, and we > need to move towards common code. Dmitry has been helpful in > contributing this to us. > > BR, > Jani. > > Hi Jani, Maarten has acked the patch series to be merged through drm-intel and in the meantime I will work with Dmitry to pull the common code to avoid duplication Regards, Suraj Kandpal > [1] https://patchwork.freedesktop.org/series/114473/ > > > > > Ankit Nautiyal (2): > > drm/dp_helper: Add helper to check DSC support with given o/p format > > drm/i915/dp: Check if DSC supports the given output_format > > > > Suraj Kandpal (4): > > drm/i915: Adding the new registers for DSC > > drm/i915: Enable YCbCr420 for VDSC > > drm/i915/display: Fill in native_420 field > > drm/i915/vdsc: Check slice design requirement > > > > Swati Sharma (1): > > drm/i915/dsc: Add debugfs entry to validate DSC output formats > > > > drivers/gpu/drm/i915/display/icl_dsi.c | 2 - > > .../drm/i915/display/intel_crtc_state_dump.c | 4 +- > > .../drm/i915/display/intel_crtc_state_dump.h | 2 + > > .../drm/i915/display/intel_display_debugfs.c | 78 ++++++++ > > .../drm/i915/display/intel_display_types.h | 1 + > > drivers/gpu/drm/i915/display/intel_dp.c | 39 +++- > > .../gpu/drm/i915/display/intel_qp_tables.c | 187 ++++++++++++++++-- > > .../gpu/drm/i915/display/intel_qp_tables.h | 4 +- > > drivers/gpu/drm/i915/display/intel_vdsc.c | 108 +++++++++- > > drivers/gpu/drm/i915/i915_reg.h | 28 +++ > > include/drm/display/drm_dp_helper.h | 13 ++ > > 11 files changed, 442 insertions(+), 24 deletions(-) > > -- > Jani Nikula, Intel Open Source Graphics Center
