Applied. Thanks!
Alex
On Wed, Aug 23, 2023 at 5:50 AM Wang, Yang(Kevin)
<kevinyang.w...@amd.com> wrote:
>
> [AMD Official Use Only - General]
>
> Reviewed-by: Yang Wang <kevinyang.w...@amd.com>
>
> Best Regards,
> Kevin
>
> -----Original Message-----
> From: dri-devel <dri-devel-boun...@lists.freedesktop.org> On Behalf Of Colin
> Ian King
> Sent: Wednesday, August 23, 2023 5:03 PM
> To: Deucher, Alexander <alexander.deuc...@amd.com>; Koenig, Christian
> <christian.koe...@amd.com>; Pan, Xinhui <xinhui....@amd.com>; David Airlie
> <airl...@gmail.com>; Daniel Vetter <dan...@ffwll.ch>; Lazar, Lijo
> <lijo.la...@amd.com>; amd-...@lists.freedesktop.org;
> dri-devel@lists.freedesktop.org
> Cc: kernel-janit...@vger.kernel.org; linux-ker...@vger.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.k...@gmail.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
>
