> Subject: [PATCH 8/8] drm/i915/dsc: Allow DSC only with fractional bpp when
> forced from debugfs
>
> From: Swati Sharma <swati2.sharma@xxxxxxxxx>
>
> If force_dsc_fractional_bpp_en is set through debugfs allow DSC iff
> compressed bpp is fractional. Continue if the computed compressed bpp turns
> out to be a integer.
>
> v2:
> -Use helpers for fractional, integral bits of bits_per_pixel. (Suraj) -Fix comment
> (Suraj)
>
LGTM.
Reviewed-by: Suraj Kandpal <suraj.kandpal@xxxxxxxxx>
> Signed-off-by: Swati Sharma <swati2.sharma@xxxxxxxxx>
> ---
> drivers/gpu/drm/i915/display/intel_dp.c | 7 +++++++
> 1 file changed, 7 insertions(+)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_dp.c
> b/drivers/gpu/drm/i915/display/intel_dp.c
> index 90c92f3dc62a..f6bfbd89bc3f 100644
> --- a/drivers/gpu/drm/i915/display/intel_dp.c
> +++ b/drivers/gpu/drm/i915/display/intel_dp.c
> @@ -1904,6 +1904,9 @@ xelpd_dsc_compute_link_config(struct intel_dp
> *intel_dp,
> for (compressed_bppx16 = dsc_max_bpp;
> compressed_bppx16 >= dsc_min_bpp;
> compressed_bppx16 -= bppx16_step) {
> + if (intel_dp->force_dsc_fractional_bpp_en &&
> + !to_bpp_frac_dec(compressed_bppx16))
> + continue;
> ret = dsc_compute_link_config(intel_dp,
> pipe_config,
> limits,
> @@ -1911,6 +1914,10 @@ xelpd_dsc_compute_link_config(struct intel_dp
> *intel_dp,
> timeslots);
> if (ret == 0) {
> pipe_config->dsc.compressed_bpp_x16 =
> compressed_bppx16;
> + if (intel_dp->force_dsc_fractional_bpp_en &&
> + to_bpp_frac_dec(compressed_bppx16))
> + drm_dbg_kms(&i915->drm, "Forcing DSC
> fractional bpp\n");
> +
> return 0;
> }
> }
> --
> 2.25.1
