Since we cannot do DSC with this output format currently, can this check be
added as part of the intel_dp_supports_dsc() ?

Regards
Manasi


On Tue, Mar 14, 2023 at 4:07 AM Ankit Nautiyal <[email protected]>
wrote:

> Currently, DSC with YCBCR420 is not supported.
> Return -EINVAL when trying with DSC with output_format as YCBCR420.
>
> Signed-off-by: Ankit Nautiyal <[email protected]>
> ---
>  drivers/gpu/drm/i915/display/intel_dp.c | 4 ++++
>  1 file changed, 4 insertions(+)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_dp.c
> b/drivers/gpu/drm/i915/display/intel_dp.c
> index b1431ed175bc..99a5cd370c1f 100644
> --- a/drivers/gpu/drm/i915/display/intel_dp.c
> +++ b/drivers/gpu/drm/i915/display/intel_dp.c
> @@ -1602,6 +1602,10 @@ int intel_dp_dsc_compute_config(struct intel_dp
> *intel_dp,
>         if (!intel_dp_supports_dsc(intel_dp, pipe_config))
>                 return -EINVAL;
>
> +       /* Currently DSC with YCBCR420 format is not supported */
> +       if (pipe_config->output_format == INTEL_OUTPUT_FORMAT_YCBCR420)
> +               return -EINVAL;
> +
>         if (compute_pipe_bpp)
>                 pipe_bpp = intel_dp_dsc_compute_bpp(intel_dp,
> pipe_config->output_format,
>
> conn_state->max_requested_bpc);
> --
> 2.25.1
>
>

Reply via email to