From: Alex Deucher <alexdeucher@gmail.com>
To: YueHaibing <yuehaibing@huawei.com>
Cc: "Wentland, Harry" <harry.wentland@amd.com>,
"Leo (Sunpeng) Li" <sunpeng.li@amd.com>,
"Deucher, Alexander" <alexander.deucher@amd.com>,
Christian Koenig <christian.koenig@amd.com>,
Chunming Zhou <David1.Zhou@amd.com>,
Dave Airlie <airlied@linux.ie>, Daniel Vetter <daniel@ffwll.ch>,
Maling list - DRI developers <dri-devel@lists.freedesktop.org>,
amd-gfx list <amd-gfx@lists.freedesktop.org>,
LKML <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH -next] drm/amd/dc: remove unused variable 'video_optimized_pixel_rates'
Date: Fri, 17 Apr 2020 09:59:12 -0400 [thread overview]
Message-ID: <CADnq5_OCqKM3EYnjti0djh5hEPbRgO3qgJFeAU5tpqWc9PZiyA@mail.gmail.com> (raw)
In-Reply-To: <20200417101247.45616-1-yuehaibing@huawei.com>
On Fri, Apr 17, 2020 at 9:16 AM YueHaibing <yuehaibing@huawei.com> wrote:
>
> drivers/gpu/drm/amd/amdgpu/../display/dc/dce/dce_clock_source.c:1017:50:
> warning: ‘video_optimized_pixel_rates’ defined but not used [-Wunused-const-variable=]
> static const struct pixel_rate_range_table_entry video_optimized_pixel_rates[] = {
> ^~~~~~~~~~~~~~~~~~~~~~~~~~~
>
> commit d8cd587d2bfd ("drm/amd/display: removing MODULO change for dcn2")
> left behind this unused vairable, remove it.
>
> Signed-off-by: YueHaibing <yuehaibing@huawei.com>
Applied. Thanks!
Alex
> ---
> .../drm/amd/display/dc/dce/dce_clock_source.c | 33 -------------------
> 1 file changed, 33 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/display/dc/dce/dce_clock_source.c b/drivers/gpu/drm/amd/display/dc/dce/dce_clock_source.c
> index 2e992fbc0d71..d2ad0504b0de 100644
> --- a/drivers/gpu/drm/amd/display/dc/dce/dce_clock_source.c
> +++ b/drivers/gpu/drm/amd/display/dc/dce/dce_clock_source.c
> @@ -1014,39 +1014,6 @@ struct pixel_rate_range_table_entry {
> unsigned short div_factor;
> };
>
> -static const struct pixel_rate_range_table_entry video_optimized_pixel_rates[] = {
> - // /1.001 rates
> - {25170, 25180, 25200, 1000, 1001}, //25.2MHz -> 25.17
> - {59340, 59350, 59400, 1000, 1001}, //59.4Mhz -> 59.340
> - {74170, 74180, 74250, 1000, 1001}, //74.25Mhz -> 74.1758
> - {125870, 125880, 126000, 1000, 1001}, //126Mhz -> 125.87
> - {148350, 148360, 148500, 1000, 1001}, //148.5Mhz -> 148.3516
> - {167830, 167840, 168000, 1000, 1001}, //168Mhz -> 167.83
> - {222520, 222530, 222750, 1000, 1001}, //222.75Mhz -> 222.527
> - {257140, 257150, 257400, 1000, 1001}, //257.4Mhz -> 257.1429
> - {296700, 296710, 297000, 1000, 1001}, //297Mhz -> 296.7033
> - {342850, 342860, 343200, 1000, 1001}, //343.2Mhz -> 342.857
> - {395600, 395610, 396000, 1000, 1001}, //396Mhz -> 395.6
> - {409090, 409100, 409500, 1000, 1001}, //409.5Mhz -> 409.091
> - {445050, 445060, 445500, 1000, 1001}, //445.5Mhz -> 445.055
> - {467530, 467540, 468000, 1000, 1001}, //468Mhz -> 467.5325
> - {519230, 519240, 519750, 1000, 1001}, //519.75Mhz -> 519.231
> - {525970, 525980, 526500, 1000, 1001}, //526.5Mhz -> 525.974
> - {545450, 545460, 546000, 1000, 1001}, //546Mhz -> 545.455
> - {593400, 593410, 594000, 1000, 1001}, //594Mhz -> 593.4066
> - {623370, 623380, 624000, 1000, 1001}, //624Mhz -> 623.377
> - {692300, 692310, 693000, 1000, 1001}, //693Mhz -> 692.308
> - {701290, 701300, 702000, 1000, 1001}, //702Mhz -> 701.2987
> - {791200, 791210, 792000, 1000, 1001}, //792Mhz -> 791.209
> - {890100, 890110, 891000, 1000, 1001}, //891Mhz -> 890.1099
> - {1186810, 1186820, 1188000, 1000, 1001},//1188Mhz -> 1186.8131
> -
> - // *1.001 rates
> - {27020, 27030, 27000, 1001, 1000}, //27Mhz
> - {54050, 54060, 54000, 1001, 1000}, //54Mhz
> - {108100, 108110, 108000, 1001, 1000},//108Mhz
> -};
> -
> static bool dcn20_program_pix_clk(
> struct clock_source *clock_source,
> struct pixel_clk_params *pix_clk_params,
> --
> 2.17.1
>
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
prev parent reply other threads:[~2020-04-17 13:59 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-04-17 10:12 [PATCH -next] drm/amd/dc: remove unused variable 'video_optimized_pixel_rates' YueHaibing
2020-04-17 13:59 ` Alex Deucher [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=CADnq5_OCqKM3EYnjti0djh5hEPbRgO3qgJFeAU5tpqWc9PZiyA@mail.gmail.com \
--to=alexdeucher@gmail.com \
--cc=David1.Zhou@amd.com \
--cc=airlied@linux.ie \
--cc=alexander.deucher@amd.com \
--cc=amd-gfx@lists.freedesktop.org \
--cc=christian.koenig@amd.com \
--cc=daniel@ffwll.ch \
--cc=dri-devel@lists.freedesktop.org \
--cc=harry.wentland@amd.com \
--cc=linux-kernel@vger.kernel.org \
--cc=sunpeng.li@amd.com \
--cc=yuehaibing@huawei.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).