All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Maíra Canal" <maira.canal@usp.br>
To: alexander.deucher@amd.com, christian.koenig@amd.com,
	Xinhui.Pan@amd.com, Hawking.Zhang@amd.com, john.clements@amd.com,
	tao.zhou1@amd.com, YiPeng.Chai@amd.com, luben.tuikov@amd.com,
	Stanley.Yang@amd.com, Dennis.Li@amd.com, mukul.joshi@amd.com,
	harry.wentland@amd.com, sunpeng.li@amd.com,
	Rodrigo.Siqueira@amd.com, Anthony.Koo@amd.com
Cc: amd-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org,
	linux-kernel@vger.kernel.org, magalilemes00@gmail.com,
	mwen@igalia.com, isabbasso@riseup.net,
	"Maíra Canal" <maira.canal@usp.br>
Subject: [PATCH 04/10] drm/amd/display: Remove unused temp variable
Date: Tue, 22 Feb 2022 10:16:55 -0300	[thread overview]
Message-ID: <20220222131701.356117-5-maira.canal@usp.br> (raw)
In-Reply-To: <20220222131701.356117-1-maira.canal@usp.br>

Remove unused temp variable from the dmub_rb_flush_pending function by
using arithmetic to remove the loop.

The -Wunused-but-set-variable warning was pointed out by Clang with the
following warning:

drivers/gpu/drm/amd/amdgpu/../display/dmub/inc/dmub_cmd.h:2921:12: warning:
variable 'temp' set but not used [-Wunused-but-set-variable]
    uint64_t temp;
             ^

Signed-off-by: Maíra Canal <maira.canal@usp.br>
---
 drivers/gpu/drm/amd/display/dmub/inc/dmub_cmd.h | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/drivers/gpu/drm/amd/display/dmub/inc/dmub_cmd.h b/drivers/gpu/drm/amd/display/dmub/inc/dmub_cmd.h
index fb01ff49e655..d3088836d4e4 100644
--- a/drivers/gpu/drm/amd/display/dmub/inc/dmub_cmd.h
+++ b/drivers/gpu/drm/amd/display/dmub/inc/dmub_cmd.h
@@ -2918,11 +2918,8 @@ static inline void dmub_rb_flush_pending(const struct dmub_rb *rb)
 	while (rptr != wptr) {
 		uint64_t volatile *data = (uint64_t volatile *)((uint8_t *)(rb->base_address) + rptr);
 		//uint64_t volatile *p = (uint64_t volatile *)data;
-		uint64_t temp;
-		uint8_t i;
 
-		for (i = 0; i < DMUB_RB_CMD_SIZE / sizeof(uint64_t); i++)
-			temp = *data++;
+		*data += DMUB_RB_CMD_SIZE / sizeof(uint64_t);
 
 		rptr += DMUB_RB_CMD_SIZE;
 		if (rptr >= rb->capacity)
-- 
2.35.1


WARNING: multiple messages have this Message-ID (diff)
From: "Maíra Canal" <maira.canal@usp.br>
To: alexander.deucher@amd.com, christian.koenig@amd.com,
	Xinhui.Pan@amd.com, Hawking.Zhang@amd.com, john.clements@amd.com,
	tao.zhou1@amd.com, YiPeng.Chai@amd.com, luben.tuikov@amd.com,
	Stanley.Yang@amd.com, Dennis.Li@amd.com, mukul.joshi@amd.com,
	harry.wentland@amd.com, sunpeng.li@amd.com,
	Rodrigo.Siqueira@amd.com, Anthony.Koo@amd.com
Cc: magalilemes00@gmail.com, "Maíra Canal" <maira.canal@usp.br>,
	linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org,
	mwen@igalia.com, amd-gfx@lists.freedesktop.org,
	isabbasso@riseup.net
Subject: [PATCH 04/10] drm/amd/display: Remove unused temp variable
Date: Tue, 22 Feb 2022 10:16:55 -0300	[thread overview]
Message-ID: <20220222131701.356117-5-maira.canal@usp.br> (raw)
In-Reply-To: <20220222131701.356117-1-maira.canal@usp.br>

Remove unused temp variable from the dmub_rb_flush_pending function by
using arithmetic to remove the loop.

The -Wunused-but-set-variable warning was pointed out by Clang with the
following warning:

drivers/gpu/drm/amd/amdgpu/../display/dmub/inc/dmub_cmd.h:2921:12: warning:
variable 'temp' set but not used [-Wunused-but-set-variable]
    uint64_t temp;
             ^

Signed-off-by: Maíra Canal <maira.canal@usp.br>
---
 drivers/gpu/drm/amd/display/dmub/inc/dmub_cmd.h | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/drivers/gpu/drm/amd/display/dmub/inc/dmub_cmd.h b/drivers/gpu/drm/amd/display/dmub/inc/dmub_cmd.h
index fb01ff49e655..d3088836d4e4 100644
--- a/drivers/gpu/drm/amd/display/dmub/inc/dmub_cmd.h
+++ b/drivers/gpu/drm/amd/display/dmub/inc/dmub_cmd.h
@@ -2918,11 +2918,8 @@ static inline void dmub_rb_flush_pending(const struct dmub_rb *rb)
 	while (rptr != wptr) {
 		uint64_t volatile *data = (uint64_t volatile *)((uint8_t *)(rb->base_address) + rptr);
 		//uint64_t volatile *p = (uint64_t volatile *)data;
-		uint64_t temp;
-		uint8_t i;
 
-		for (i = 0; i < DMUB_RB_CMD_SIZE / sizeof(uint64_t); i++)
-			temp = *data++;
+		*data += DMUB_RB_CMD_SIZE / sizeof(uint64_t);
 
 		rptr += DMUB_RB_CMD_SIZE;
 		if (rptr >= rb->capacity)
-- 
2.35.1


  parent reply	other threads:[~2022-02-22 13:18 UTC|newest]

Thread overview: 52+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-02-22 13:16 [PATCH 00/10] Fix multiple compilation warnings Maíra Canal
2022-02-22 13:16 ` Maíra Canal
2022-02-22 13:16 ` [PATCH 01/10] drm/amdgpu: Change amdgpu_ras_block_late_init_default function scope Maíra Canal
2022-02-22 13:16   ` Maíra Canal
2022-02-22 16:27   ` Alex Deucher
2022-02-22 16:27     ` Alex Deucher
2022-02-22 16:27     ` Alex Deucher
2022-02-22 13:16 ` [PATCH 02/10] drm/amdgpu: Remove tmp unused variable Maíra Canal
2022-02-22 13:16   ` Maíra Canal
2022-02-22 16:29   ` Alex Deucher
2022-02-22 16:29     ` Alex Deucher
2022-02-22 16:29     ` Alex Deucher
2022-02-22 13:16 ` [PATCH 03/10] drm/amdgpu: Remove unused get_umc_v8_7_channel_index function Maíra Canal
2022-02-22 13:16   ` Maíra Canal
2022-02-22 16:29   ` Alex Deucher
2022-02-22 16:29     ` Alex Deucher
2022-02-22 16:29     ` Alex Deucher
2022-02-22 13:16 ` Maíra Canal [this message]
2022-02-22 13:16   ` [PATCH 04/10] drm/amd/display: Remove unused temp variable Maíra Canal
2022-02-22 16:31   ` Alex Deucher
2022-02-22 16:31     ` Alex Deucher
2022-02-22 16:31     ` Alex Deucher
2022-02-22 13:16 ` [PATCH 05/10] drm/amd/display: Remove unused dcn316_smu_set_voltage_via_phyclk function Maíra Canal
2022-02-22 13:16   ` Maíra Canal
2022-02-22 16:32   ` Alex Deucher
2022-02-22 16:32     ` Alex Deucher
2022-02-22 16:32     ` Alex Deucher
2022-02-22 13:16 ` [PATCH 06/10] drm/amd/display: Remove vupdate_int_entry definition Maíra Canal
2022-02-22 13:16   ` Maíra Canal
2022-02-22 16:38   ` Alex Deucher
2022-02-22 16:38     ` Alex Deucher
2022-02-22 16:38     ` Alex Deucher
2022-02-22 13:16 ` [PATCH 07/10] drm/amd/display: Remove unused dmub_outbox_irq_info_funcs variable Maíra Canal
2022-02-22 13:16   ` Maíra Canal
2022-02-22 16:38   ` Alex Deucher
2022-02-22 16:38     ` Alex Deucher
2022-02-22 16:38     ` Alex Deucher
2022-02-22 13:16 ` [PATCH 08/10] drm/amd/display: Remove unused variable Maíra Canal
2022-02-22 13:16   ` Maíra Canal
2022-02-22 16:40   ` Alex Deucher
2022-02-22 16:40     ` Alex Deucher
2022-02-22 16:40     ` Alex Deucher
2022-02-22 13:17 ` [PATCH 09/10] drm/amd/display: Add missing prototypes to dcn201_init Maíra Canal
2022-02-22 13:17   ` Maíra Canal
2022-02-22 16:41   ` Alex Deucher
2022-02-22 16:41     ` Alex Deucher
2022-02-22 16:41     ` Alex Deucher
2022-02-22 13:17 ` [PATCH 10/10] drm/amd/display: Turn global functions into static functions Maíra Canal
2022-02-22 13:17   ` Maíra Canal
2022-02-22 16:45   ` Alex Deucher
2022-02-22 16:45     ` Alex Deucher
2022-02-22 16:45     ` Alex Deucher

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=20220222131701.356117-5-maira.canal@usp.br \
    --to=maira.canal@usp.br \
    --cc=Anthony.Koo@amd.com \
    --cc=Dennis.Li@amd.com \
    --cc=Hawking.Zhang@amd.com \
    --cc=Rodrigo.Siqueira@amd.com \
    --cc=Stanley.Yang@amd.com \
    --cc=Xinhui.Pan@amd.com \
    --cc=YiPeng.Chai@amd.com \
    --cc=alexander.deucher@amd.com \
    --cc=amd-gfx@lists.freedesktop.org \
    --cc=christian.koenig@amd.com \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=harry.wentland@amd.com \
    --cc=isabbasso@riseup.net \
    --cc=john.clements@amd.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=luben.tuikov@amd.com \
    --cc=magalilemes00@gmail.com \
    --cc=mukul.joshi@amd.com \
    --cc=mwen@igalia.com \
    --cc=sunpeng.li@amd.com \
    --cc=tao.zhou1@amd.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.