Re: [PATCH] drm/amd/display: remove assert around dpp_base replacement

[Alex Hung <alex.hung@amd.com>]

Reviewed-by: Alex Hung <alex.hung@amd.com>

On 1/16/26 08:50, Melissa Wen wrote:
> There is nothing wrong if in_shaper_func type is DISTRIBUTED POINTS.
> Remove the assert placed for a TODO to avoid misinterpretations.
> 
> Signed-off-by: Melissa Wen <mwen@igalia.com>
> ---
>   drivers/gpu/drm/amd/display/dc/hwss/dcn32/dcn32_hwseq.c | 1 -
>   1 file changed, 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/amd/display/dc/hwss/dcn32/dcn32_hwseq.c b/drivers/gpu/drm/amd/display/dc/hwss/dcn32/dcn32_hwseq.c
> index 3cd44c6602b3..4dfd22bc00c4 100644
> --- a/drivers/gpu/drm/amd/display/dc/hwss/dcn32/dcn32_hwseq.c
> +++ b/drivers/gpu/drm/amd/display/dc/hwss/dcn32/dcn32_hwseq.c
> @@ -507,7 +507,6 @@ bool dcn32_set_mcm_luts(
>   		lut_params = &plane_state->in_shaper_func.pwl;
>   	else if (plane_state->in_shaper_func.type == TF_TYPE_DISTRIBUTED_POINTS) {
>   		// TODO: dpp_base replace
> -		ASSERT(false);
>   		cm3_helper_translate_curve_to_hw_format(&plane_state->in_shaper_func,
>   				&dpp_base->shaper_params, true);
>   		lut_params = &dpp_base->shaper_params;