From: "Christian König" <christian.koenig@amd.com>
To: trix@redhat.com, alexander.deucher@amd.com, airlied@linux.ie,
daniel@ffwll.ch, Hawking.Zhang@amd.com, evan.quan@amd.com,
Felix.Kuehling@amd.com, luben.tuikov@amd.com, Dennis.Li@amd.com,
andrey.grodzovsky@amd.com, Joseph.Greathouse@amd.com,
jonathan.kim@amd.com
Cc: amd-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH] drm/amdgpu/display: remove trailing semicolon in macro definition
Date: Mon, 30 Nov 2020 09:00:46 +0100 [thread overview]
Message-ID: <01583ef8-4d70-82b2-a643-754d95d47a0a@amd.com> (raw)
In-Reply-To: <20201127162607.2656353-1-trix@redhat.com>
Am 27.11.20 um 17:26 schrieb trix@redhat.com:
> From: Tom Rix <trix@redhat.com>
>
> The macro use will already have a semicolon.
>
> Signed-off-by: Tom Rix <trix@redhat.com>
Reviewed-by: Christian König <christian.koenig@amd.com>
> ---
> drivers/gpu/drm/amd/amdgpu/amdgpu.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu.h b/drivers/gpu/drm/amd/amdgpu/amdgpu.h
> index f9c81bc21ba4..301e93c9e72a 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu.h
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu.h
> @@ -1213,7 +1213,7 @@ int emu_soc_asic_init(struct amdgpu_device *adev);
> #define amdgpu_asic_update_umd_stable_pstate(adev, enter) \
> ((adev)->asic_funcs->update_umd_stable_pstate ? (adev)->asic_funcs->update_umd_stable_pstate((adev), (enter)) : 0)
>
> -#define amdgpu_inc_vram_lost(adev) atomic_inc(&((adev)->vram_lost_counter));
> +#define amdgpu_inc_vram_lost(adev) atomic_inc(&((adev)->vram_lost_counter))
>
> /* Common functions */
> bool amdgpu_device_has_job_running(struct amdgpu_device *adev);
prev parent reply other threads:[~2020-11-30 8:01 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-11-27 16:26 [PATCH] drm/amdgpu/display: remove trailing semicolon in macro definition trix
2020-11-30 8:00 ` Christian König [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=01583ef8-4d70-82b2-a643-754d95d47a0a@amd.com \
--to=christian.koenig@amd.com \
--cc=Dennis.Li@amd.com \
--cc=Felix.Kuehling@amd.com \
--cc=Hawking.Zhang@amd.com \
--cc=Joseph.Greathouse@amd.com \
--cc=airlied@linux.ie \
--cc=alexander.deucher@amd.com \
--cc=amd-gfx@lists.freedesktop.org \
--cc=andrey.grodzovsky@amd.com \
--cc=daniel@ffwll.ch \
--cc=dri-devel@lists.freedesktop.org \
--cc=evan.quan@amd.com \
--cc=jonathan.kim@amd.com \
--cc=linux-kernel@vger.kernel.org \
--cc=luben.tuikov@amd.com \
--cc=trix@redhat.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).