lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CADnq5_OLPqHCGV2YziVipDcWRdPQbpR12g+Vch9tgo171gyzkQ@mail.gmail.com>
Date: Tue, 29 Oct 2024 09:57:32 -0400
From: Alex Deucher <alexdeucher@...il.com>
To: linux@...blig.org
Cc: harry.wentland@....com, sunpeng.li@....com, Rodrigo.Siqueira@....com, 
	alexander.deucher@....com, christian.koenig@....com, Xinhui.Pan@....com, 
	airlied@...il.com, simona@...ll.ch, amd-gfx@...ts.freedesktop.org, 
	dri-devel@...ts.freedesktop.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] drm/amd/display: Remove unused cm3_helper_translate_curve_to_degamma_hw_format

Applied.  Thanks!

Alex

On Tue, Oct 8, 2024 at 8:43 PM <linux@...blig.org> wrote:
>
> From: "Dr. David Alan Gilbert" <linux@...blig.org>
>
> cm3_helper_translate_curve_to_degamma_hw_format() since it was added in
> 2020's commit
> 03f54d7d3448 ("drm/amd/display: Add DCN3 DPP")
>
> Remove it.
>
> Signed-off-by: Dr. David Alan Gilbert <linux@...blig.org>
> ---
>  .../amd/display/dc/dcn30/dcn30_cm_common.c    | 151 ------------------
>  .../display/dc/dwb/dcn30/dcn30_cm_common.h    |   4 -
>  2 files changed, 155 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/display/dc/dcn30/dcn30_cm_common.c b/drivers/gpu/drm/amd/display/dc/dcn30/dcn30_cm_common.c
> index f31f0e3abfc0..9118a0883f03 100644
> --- a/drivers/gpu/drm/amd/display/dc/dcn30/dcn30_cm_common.c
> +++ b/drivers/gpu/drm/amd/display/dc/dcn30/dcn30_cm_common.c
> @@ -285,157 +285,6 @@ bool cm3_helper_translate_curve_to_hw_format(
>         return true;
>  }
>
> -#define NUM_DEGAMMA_REGIONS    12
> -
> -
> -bool cm3_helper_translate_curve_to_degamma_hw_format(
> -                               const struct dc_transfer_func *output_tf,
> -                               struct pwl_params *lut_params)
> -{
> -       struct curve_points3 *corner_points;
> -       struct pwl_result_data *rgb_resulted;
> -       struct pwl_result_data *rgb;
> -       struct pwl_result_data *rgb_plus_1;
> -
> -       int32_t region_start, region_end;
> -       int32_t i;
> -       uint32_t j, k, seg_distr[MAX_REGIONS_NUMBER], increment, start_index, hw_points;
> -
> -       if (output_tf == NULL || lut_params == NULL || output_tf->type == TF_TYPE_BYPASS)
> -               return false;
> -
> -       corner_points = lut_params->corner_points;
> -       rgb_resulted = lut_params->rgb_resulted;
> -       hw_points = 0;
> -
> -       memset(lut_params, 0, sizeof(struct pwl_params));
> -       memset(seg_distr, 0, sizeof(seg_distr));
> -
> -       region_start = -NUM_DEGAMMA_REGIONS;
> -       region_end   = 0;
> -
> -
> -       for (i = region_end - region_start; i < MAX_REGIONS_NUMBER ; i++)
> -               seg_distr[i] = -1;
> -       /* 12 segments
> -        * segments are from 2^-12 to 0
> -        */
> -       for (i = 0; i < NUM_DEGAMMA_REGIONS ; i++)
> -               seg_distr[i] = 4;
> -
> -       for (k = 0; k < MAX_REGIONS_NUMBER; k++) {
> -               if (seg_distr[k] != -1)
> -                       hw_points += (1 << seg_distr[k]);
> -       }
> -
> -       j = 0;
> -       for (k = 0; k < (region_end - region_start); k++) {
> -               increment = NUMBER_SW_SEGMENTS / (1 << seg_distr[k]);
> -               start_index = (region_start + k + MAX_LOW_POINT) *
> -                               NUMBER_SW_SEGMENTS;
> -               for (i = start_index; i < start_index + NUMBER_SW_SEGMENTS;
> -                               i += increment) {
> -                       if (j == hw_points - 1)
> -                               break;
> -                       if (i >= TRANSFER_FUNC_POINTS)
> -                               return false;
> -                       rgb_resulted[j].red = output_tf->tf_pts.red[i];
> -                       rgb_resulted[j].green = output_tf->tf_pts.green[i];
> -                       rgb_resulted[j].blue = output_tf->tf_pts.blue[i];
> -                       j++;
> -               }
> -       }
> -
> -       /* last point */
> -       start_index = (region_end + MAX_LOW_POINT) * NUMBER_SW_SEGMENTS;
> -       rgb_resulted[hw_points - 1].red = output_tf->tf_pts.red[start_index];
> -       rgb_resulted[hw_points - 1].green = output_tf->tf_pts.green[start_index];
> -       rgb_resulted[hw_points - 1].blue = output_tf->tf_pts.blue[start_index];
> -
> -       corner_points[0].red.x = dc_fixpt_pow(dc_fixpt_from_int(2),
> -                                            dc_fixpt_from_int(region_start));
> -       corner_points[0].green.x = corner_points[0].red.x;
> -       corner_points[0].blue.x = corner_points[0].red.x;
> -       corner_points[1].red.x = dc_fixpt_pow(dc_fixpt_from_int(2),
> -                                            dc_fixpt_from_int(region_end));
> -       corner_points[1].green.x = corner_points[1].red.x;
> -       corner_points[1].blue.x = corner_points[1].red.x;
> -
> -       corner_points[0].red.y = rgb_resulted[0].red;
> -       corner_points[0].green.y = rgb_resulted[0].green;
> -       corner_points[0].blue.y = rgb_resulted[0].blue;
> -
> -       /* see comment above, m_arrPoints[1].y should be the Y value for the
> -        * region end (m_numOfHwPoints), not last HW point(m_numOfHwPoints - 1)
> -        */
> -       corner_points[1].red.y = rgb_resulted[hw_points - 1].red;
> -       corner_points[1].green.y = rgb_resulted[hw_points - 1].green;
> -       corner_points[1].blue.y = rgb_resulted[hw_points - 1].blue;
> -       corner_points[1].red.slope = dc_fixpt_zero;
> -       corner_points[1].green.slope = dc_fixpt_zero;
> -       corner_points[1].blue.slope = dc_fixpt_zero;
> -
> -       if (output_tf->tf == TRANSFER_FUNCTION_PQ) {
> -               /* for PQ, we want to have a straight line from last HW X point,
> -                * and the slope to be such that we hit 1.0 at 10000 nits.
> -                */
> -               const struct fixed31_32 end_value =
> -                               dc_fixpt_from_int(125);
> -
> -               corner_points[1].red.slope = dc_fixpt_div(
> -                       dc_fixpt_sub(dc_fixpt_one, corner_points[1].red.y),
> -                       dc_fixpt_sub(end_value, corner_points[1].red.x));
> -               corner_points[1].green.slope = dc_fixpt_div(
> -                       dc_fixpt_sub(dc_fixpt_one, corner_points[1].green.y),
> -                       dc_fixpt_sub(end_value, corner_points[1].green.x));
> -               corner_points[1].blue.slope = dc_fixpt_div(
> -                       dc_fixpt_sub(dc_fixpt_one, corner_points[1].blue.y),
> -                       dc_fixpt_sub(end_value, corner_points[1].blue.x));
> -       }
> -
> -       lut_params->hw_points_num = hw_points;
> -
> -       k = 0;
> -       for (i = 1; i < MAX_REGIONS_NUMBER; i++) {
> -               if (seg_distr[k] != -1) {
> -                       lut_params->arr_curve_points[k].segments_num =
> -                                       seg_distr[k];
> -                       lut_params->arr_curve_points[i].offset =
> -                                       lut_params->arr_curve_points[k].offset + (1 << seg_distr[k]);
> -               }
> -               k++;
> -       }
> -
> -       if (seg_distr[k] != -1)
> -               lut_params->arr_curve_points[k].segments_num = seg_distr[k];
> -
> -       rgb = rgb_resulted;
> -       rgb_plus_1 = rgb_resulted + 1;
> -
> -       i = 1;
> -       while (i != hw_points + 1) {
> -               if (dc_fixpt_lt(rgb_plus_1->red, rgb->red))
> -                       rgb_plus_1->red = rgb->red;
> -               if (dc_fixpt_lt(rgb_plus_1->green, rgb->green))
> -                       rgb_plus_1->green = rgb->green;
> -               if (dc_fixpt_lt(rgb_plus_1->blue, rgb->blue))
> -                       rgb_plus_1->blue = rgb->blue;
> -
> -               rgb->delta_red   = dc_fixpt_sub(rgb_plus_1->red,   rgb->red);
> -               rgb->delta_green = dc_fixpt_sub(rgb_plus_1->green, rgb->green);
> -               rgb->delta_blue  = dc_fixpt_sub(rgb_plus_1->blue,  rgb->blue);
> -
> -               ++rgb_plus_1;
> -               ++rgb;
> -               ++i;
> -       }
> -       cm3_helper_convert_to_custom_float(rgb_resulted,
> -                                               lut_params->corner_points,
> -                                               hw_points, false);
> -
> -       return true;
> -}
> -
>  bool cm3_helper_convert_to_custom_float(
>                 struct pwl_result_data *rgb_resulted,
>                 struct curve_points3 *corner_points,
> diff --git a/drivers/gpu/drm/amd/display/dc/dwb/dcn30/dcn30_cm_common.h b/drivers/gpu/drm/amd/display/dc/dwb/dcn30/dcn30_cm_common.h
> index bd98b327a6c7..b86347c9b038 100644
> --- a/drivers/gpu/drm/amd/display/dc/dwb/dcn30/dcn30_cm_common.h
> +++ b/drivers/gpu/drm/amd/display/dc/dwb/dcn30/dcn30_cm_common.h
> @@ -63,10 +63,6 @@ bool cm3_helper_translate_curve_to_hw_format(
>         const struct dc_transfer_func *output_tf,
>         struct pwl_params *lut_params, bool fixpoint);
>
> -bool cm3_helper_translate_curve_to_degamma_hw_format(
> -                               const struct dc_transfer_func *output_tf,
> -                               struct pwl_params *lut_params);
> -
>  bool cm3_helper_convert_to_custom_float(
>                 struct pwl_result_data *rgb_resulted,
>                 struct curve_points3 *corner_points,
> --
> 2.46.2
>

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ