[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CADnq5_Oz22ahx3q0a3guye_TUvDDStfsdJBQjg6EfDpctVTRGQ@mail.gmail.com>
Date: Thu, 24 Aug 2023 10:39:18 -0400
From: Alex Deucher <alexdeucher@...il.com>
To: "Wang, Yang(Kevin)" <KevinYang.Wang@....com>
Cc: Colin Ian King <colin.i.king@...il.com>,
"Deucher, Alexander" <Alexander.Deucher@....com>,
"Koenig, Christian" <Christian.Koenig@....com>,
"Pan, Xinhui" <Xinhui.Pan@....com>,
David Airlie <airlied@...il.com>,
Daniel Vetter <daniel@...ll.ch>,
"Lazar, Lijo" <Lijo.Lazar@....com>,
"amd-gfx@...ts.freedesktop.org" <amd-gfx@...ts.freedesktop.org>,
"dri-devel@...ts.freedesktop.org" <dri-devel@...ts.freedesktop.org>,
"kernel-janitors@...r.kernel.org" <kernel-janitors@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH][next] drm/amd: Fix spelling mistake "throtting" -> "throttling"
Applied. Thanks!
Alex
On Wed, Aug 23, 2023 at 5:50 AM Wang, Yang(Kevin)
<KevinYang.Wang@....com> wrote:
>
> [AMD Official Use Only - General]
>
> Reviewed-by: Yang Wang <kevinyang.wang@....com>
>
> Best Regards,
> Kevin
>
> -----Original Message-----
> From: dri-devel <dri-devel-bounces@...ts.freedesktop.org> On Behalf Of Colin Ian King
> Sent: Wednesday, August 23, 2023 5:03 PM
> To: Deucher, Alexander <Alexander.Deucher@....com>; Koenig, Christian <Christian.Koenig@....com>; Pan, Xinhui <Xinhui.Pan@....com>; David Airlie <airlied@...il.com>; Daniel Vetter <daniel@...ll.ch>; Lazar, Lijo <Lijo.Lazar@....com>; amd-gfx@...ts.freedesktop.org; dri-devel@...ts.freedesktop.org
> Cc: kernel-janitors@...r.kernel.org; linux-kernel@...r.kernel.org
> Subject: [PATCH][next] drm/amd: Fix spelling mistake "throtting" -> "throttling"
>
> There is a spelling mistake in variable throtting_events, rename it to throttling_events.
>
> Signed-off-by: Colin Ian King <colin.i.king@...il.com>
> ---
> drivers/gpu/drm/amd/pm/swsmu/smu11/arcturus_ppt.c | 6 +++---
> drivers/gpu/drm/amd/pm/swsmu/smu13/aldebaran_ppt.c | 6 +++---
> drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0_6_ppt.c | 6 +++---
> 3 files changed, 9 insertions(+), 9 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/pm/swsmu/smu11/arcturus_ppt.c b/drivers/gpu/drm/amd/pm/swsmu/smu11/arcturus_ppt.c
> index 704a2b577a0e..fbcff154b1d0 100644
> --- a/drivers/gpu/drm/amd/pm/swsmu/smu11/arcturus_ppt.c
> +++ b/drivers/gpu/drm/amd/pm/swsmu/smu11/arcturus_ppt.c
> @@ -2313,7 +2313,7 @@ static const struct throttling_logging_label { static void arcturus_log_thermal_throttling_event(struct smu_context *smu) {
> int ret;
> - int throttler_idx, throtting_events = 0, buf_idx = 0;
> + int throttler_idx, throttling_events = 0, buf_idx = 0;
> struct amdgpu_device *adev = smu->adev;
> uint32_t throttler_status;
> char log_buf[256];
> @@ -2328,11 +2328,11 @@ static void arcturus_log_thermal_throttling_event(struct smu_context *smu)
> for (throttler_idx = 0; throttler_idx < ARRAY_SIZE(logging_label);
> throttler_idx++) {
> if (throttler_status & logging_label[throttler_idx].feature_mask) {
> - throtting_events++;
> + throttling_events++;
> buf_idx += snprintf(log_buf + buf_idx,
> sizeof(log_buf) - buf_idx,
> "%s%s",
> - throtting_events > 1 ? " and " : "",
> + throttling_events > 1 ? " and " : "",
> logging_label[throttler_idx].label);
> if (buf_idx >= sizeof(log_buf)) {
> dev_err(adev->dev, "buffer overflow!\n"); diff --git a/drivers/gpu/drm/amd/pm/swsmu/smu13/aldebaran_ppt.c b/drivers/gpu/drm/amd/pm/swsmu/smu13/aldebaran_ppt.c
> index cc3169400c9b..bed5a9df1c06 100644
> --- a/drivers/gpu/drm/amd/pm/swsmu/smu13/aldebaran_ppt.c
> +++ b/drivers/gpu/drm/amd/pm/swsmu/smu13/aldebaran_ppt.c
> @@ -1674,7 +1674,7 @@ static const struct throttling_logging_label { static void aldebaran_log_thermal_throttling_event(struct smu_context *smu) {
> int ret;
> - int throttler_idx, throtting_events = 0, buf_idx = 0;
> + int throttler_idx, throttling_events = 0, buf_idx = 0;
> struct amdgpu_device *adev = smu->adev;
> uint32_t throttler_status;
> char log_buf[256];
> @@ -1689,11 +1689,11 @@ static void aldebaran_log_thermal_throttling_event(struct smu_context *smu)
> for (throttler_idx = 0; throttler_idx < ARRAY_SIZE(logging_label);
> throttler_idx++) {
> if (throttler_status & logging_label[throttler_idx].feature_mask) {
> - throtting_events++;
> + throttling_events++;
> buf_idx += snprintf(log_buf + buf_idx,
> sizeof(log_buf) - buf_idx,
> "%s%s",
> - throtting_events > 1 ? " and " : "",
> + throttling_events > 1 ? " and " : "",
> logging_label[throttler_idx].label);
> if (buf_idx >= sizeof(log_buf)) {
> dev_err(adev->dev, "buffer overflow!\n"); diff --git a/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0_6_ppt.c b/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0_6_ppt.c
> index d3b578e6bc2a..fa4ad08099ef 100644
> --- a/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0_6_ppt.c
> +++ b/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0_6_ppt.c
> @@ -1888,7 +1888,7 @@ static const char *const throttling_logging_label[] = {
>
> static void smu_v13_0_6_log_thermal_throttling_event(struct smu_context *smu) {
> - int throttler_idx, throtting_events = 0, buf_idx = 0;
> + int throttler_idx, throttling_events = 0, buf_idx = 0;
> struct amdgpu_device *adev = smu->adev;
> uint32_t throttler_status;
> char log_buf[256];
> @@ -1902,10 +1902,10 @@ static void smu_v13_0_6_log_thermal_throttling_event(struct smu_context *smu)
> throttler_idx < ARRAY_SIZE(throttling_logging_label);
> throttler_idx++) {
> if (throttler_status & (1U << throttler_idx)) {
> - throtting_events++;
> + throttling_events++;
> buf_idx += snprintf(
> log_buf + buf_idx, sizeof(log_buf) - buf_idx,
> - "%s%s", throtting_events > 1 ? " and " : "",
> + "%s%s", throttling_events > 1 ? " and " : "",
> throttling_logging_label[throttler_idx]);
> if (buf_idx >= sizeof(log_buf)) {
> dev_err(adev->dev, "buffer overflow!\n");
> --
> 2.39.2
>
Powered by blists - more mailing lists